Rule S2187: Test class should contain test methods

[fmallet]

Implements RSPEC-2187

[sm-g]

@fmallet testfixute class may contain only TestCaseSource and Theory or it may be abstract (for NUnit).

Also I have suggestion for another rule: to check that there is no public methods in TestFixture class which is not tests (this may be after forgetting to add [Test] after refactoring of tests).

[Evangelink]

@sm-g Thanks for the feedback. I have created a ticket to improve the rule to handle these cases (#786). Also thanks for the TestCaseSource attribute, never used it so some other rules need to be improved too (#785).

[sm-g]

@Evangelink did you forget to mention that

[TestFixture]
public abstract class TestsBase {
}

and

[TestFixture]
public class Tests{

[Theory]
public void Test(...){}
}
}

also valid?

[Evangelink]

Gosh! Thanks for raising the hand! I have updated the tickets.