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

Update S2187: Rule should handle "TheoryAttribute" #786

Closed
Evangelink opened this issue Sep 18, 2017 · 1 comment
Closed

Update S2187: Rule should handle "TheoryAttribute" #786

Evangelink opened this issue Sep 18, 2017 · 1 comment
Assignees
@Evangelink
Milestone
6.7

Comments

@Evangelink
Copy link
Contributor

Evangelink commented Sep 18, 2017 •
edited
Loading

  • Handle Theory attribute
  • Handle TestCaseSource attribute
@Evangelink Evangelink mentioned this issue Sep 18, 2017
Closed
@Evangelink Evangelink self-assigned this Oct 16, 2017
@Evangelink Evangelink changed the title Update S2187: Rule should handle more attributes from NUnit. Update S2187: Rule should handle more attributes from NUnit Nov 13, 2017
@Evangelink
Copy link
Contributor Author

Reading at NUnit documentation it seems like TestCaseSource attribute should always been used in pair with Test attribute so it is probably safe to ignore it.

@Evangelink Evangelink added this to the 6.7 milestone Nov 16, 2017
@Evangelink Evangelink changed the title Update S2187: Rule should handle more attributes from NUnit Update S2187: Rule should handle "TheoryAttribute" Nov 16, 2017
@Evangelink Evangelink mentioned this issue Nov 16, 2017
Merged
@ghost ghost removed the Status: Needs Review label Nov 16, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Evangelink Evangelink

Labels
None yet
Projects
None yet
Milestone
6.7
Development

No branches or pull requests
1 participant
@Evangelink