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

MoveFieldCloserToUsageInspection should ignore RD's Assert class variable in test modules. #2847

Closed
comintern opened this issue Mar 12, 2017 · 0 comments · Fixed by #2850
Closed

MoveFieldCloserToUsageInspection should ignore RD's Assert class variable in test modules. #2847

comintern opened this issue Mar 12, 2017 · 0 comments · Fixed by #2850
Assignees
@comintern
Labels
enhancement Feature requests, or enhancements to existing features. Ideas. Anything within the project's scope. feature-inspections feature-unit-testing
Milestone
Release 2.1.0

Comments

@comintern
Copy link
Contributor

Title pretty much covers it. We're generating template code, and a '@TestModule with only one test in it will trigger this inspection. RD probably shouldn't suggest moving it into the '@TestMethod.
@comintern comintern added enhancement Feature requests, or enhancements to existing features. Ideas. Anything within the project's scope. feature-inspections feature-unit-testing labels Mar 12, 2017
@comintern comintern self-assigned this Mar 13, 2017
@comintern comintern added this to the Release 2.0.13 milestone Mar 13, 2017
comintern added a commit to comintern/Rubberduck that referenced this issue Mar 14, 2017
@comintern
Ignore RD Assert classes in MoveFieldCloserToUsageInspection closes r…
33f9737 
…ubberduck-vba#2847
@comintern comintern mentioned this issue Mar 14, 2017
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@comintern comintern

Labels
enhancement Feature requests, or enhancements to existing features. Ideas. Anything within the project's scope. feature-inspections feature-unit-testing
Projects
None yet
Milestone
Release 2.1.0
Development

Successfully merging a pull request may close this issue.

Add type checking for Asserts, implement {Not}SequenceEquals comintern/Rubberduck
1 participant
@comintern