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

Updated unit tests for SA1402 and SA1649 to verify both file content and file names #2237

Conversation

bjornhellander
Copy link
Contributor

Fixes #352.

Updated unit tests for SA1402 to verify file names, in addition to file content.
Also updated unit tests for SA1649 to verify file content, in addition to file names.

…mes (now using the normal VerifyCSharpFixAsync, instead of the previous specialized check)
@codecov-io
Copy link

codecov-io commented Nov 24, 2016

Current coverage is 96.85% (diff: 100%)

Merging #2237 into master will increase coverage by <.01%

@@             master      #2237   diff @@
==========================================
  Files           598        598          
  Lines         84025      84027     +2   
  Methods        3581       3580     -1   
  Messages          0          0          
  Branches       3196       3199     +3   
==========================================
+ Hits          81384      81386     +2   
+ Misses         1774       1773     -1   
- Partials        867        868     +1   

Powered by Codecov. Last update 0325dd1...a4c20e8

@sharwell sharwell merged commit 4cf3e03 into DotNetAnalyzers:master Dec 24, 2016
@sharwell sharwell added this to the 1.1.0 Beta 2 milestone Dec 24, 2016
@bjornhellander bjornhellander deleted the Issue352_ImproveUnitTestsForSa1402FileMayOnlyContainASingleClass branch March 25, 2017 08:39
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.

4 participants