Test files should contain tests JAVA-W1088
Anti-pattern
Minor
6 months ago
—
a year old
This test file does not contain tests
22import org.sonar.api.batch.fs.InputFile;
23import org.sonar.api.batch.sensor.internal.SensorContextTester;
24
25public class CxxFileTester {2627 public InputFile cxxFile;28 public SensorContextTester context;2930 public InputFile asInputFile() {31 return cxxFile;32 }3334}This test file does not contain tests
22import org.sonar.api.batch.fs.InputFile;
23import org.sonar.api.batch.sensor.internal.SensorContextTester;
24
25public class CxxFileTester {2627 public InputFile cxxFile;28 public SensorContextTester context;2930 public InputFile asInputFile() {31 return cxxFile;32 }3334}
Description
Classes that look like test cases should contain tests.
This issue is reported when a file within a test directory looks like a test file (has "Test" in its name, or contains classes with test framework related annotations) but contains no actual test code.
This issue will be reported if no symbols related to any of the following frameworks are found:
- Junit 3
- Junit 4
- Junit 5
- TestNG
- ArchUnit
Bad Practice
Avoid declaring classes that seem like tests but don't contain any test cases.
class SomethingTest {
// no test methods.
}
Recommended
Give the class a better name, or remove it altogether if there is no need for it.