Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Dec 10, 2025

Addresses feedback from #2442 to remove a misleading comment in test/test_helper.rb.

Changes

  • Removed outdated comment claiming integration tests must declare fixtures explicitly
  • Comment was incorrect: ActionDispatch::IntegrationTest inherits from ActiveSupport::TestCase, so integration tests automatically inherit the fixtures :all declaration
class ActiveSupport::TestCase
  fixtures :all  # Integration tests inherit this
  # ...
end

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI self-assigned this Dec 10, 2025
Co-authored-by: fbacall <503373+fbacall@users.noreply.github.com>
Copilot AI changed the title [WIP] Update loading of all fixtures in test execution Remove outdated comment about integration test fixture inheritance Dec 10, 2025
Copilot AI requested a review from fbacall December 10, 2025 14:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants