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.

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

[test only] Make sure ActiveSupport plugin ignores class definition #591

Merged

Conversation

nvasilevski
Copy link
Contributor

This is a cosmetic change to the ActiveSupport plugin test to make sure setup is accurate. Since the plugin is intended for test files that inherit from ActiveSupport::TestCase it would be more accurate for FooTest to be a subclass of AS::TestCase
This way we can also assert that the class definition itself is being ignored and not marked as dead.

@nvasilevski nvasilevski requested a review from a team as a code owner July 29, 2024 14:26
@nvasilevski nvasilevski merged commit 962589d into main Jul 29, 2024
8 checks passed
@nvasilevski nvasilevski deleted the test-activesupport-plugin-ignores-class-definition branch July 29, 2024 14:50
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