Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Split and decouple models in integration_spec.rb to their separate files #332

Open
2 of 3 tasks
ellnix opened this issue Feb 27, 2024 · 0 comments
Open
2 of 3 tasks

Split and decouple models in integration_spec.rb to their separate files #332

ellnix opened this issue Feb 27, 2024 · 0 comments
Labels
maintenance Anything related to maintenance (CI, tests, refactoring...)
Milestone
Major Refactoring

Comments

@ellnix
Copy link
Collaborator

ellnix commented Feb 27, 2024 •
edited

  • Keep models that are logically independent separate
    • Only create the databases that are required for specific test being run (faster iterative testing on new features)
  • Break out shared behavior into RSpec shared contexts to allow easily validating basic functionality
@ellnix ellnix added this to the Test suite refactor milestone Feb 27, 2024
@ellnix ellnix added the maintenance Anything related to maintenance (CI, tests, refactoring...) label Feb 27, 2024
This was referenced Feb 28, 2024
Merged
Closed
meili-bors bot added a commit that referenced this issue Feb 29, 2024
@meili-bors @ellnix
Merge #342
7459bde 
342: Split `spec/support/active_record_classes.rb` r=ellnix a=ellnix

A first step toward #332 


Co-authored-by: ellnix <[email protected]>
@ellnix ellnix mentioned this issue Feb 29, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
maintenance Anything related to maintenance (CI, tests, refactoring...)
Projects
None yet
Milestone
Major Refactoring
Development

No branches or pull requests
1 participant
@ellnix