Skip to content

Commit 1c304ac

Browse files
committed
split tests for coverity versions
1 parent 077a2b6 commit 1c304ac

File tree

1 file changed

+10
-6
lines changed

1 file changed

+10
-6
lines changed

plugin/src/test/java/org/owasp/benchmarkutils/score/parsers/CoverityReaderTest.java

Lines changed: 10 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -40,14 +40,16 @@ void setUp() {
4040
}
4141

4242
@Test
43-
public void onlyCoverityReaderReportsCanReadAsTrue() {
43+
public void onlyCoverityReaderReportsCanReadAsTrueForV3() {
4444
assertOnlyMatcherClassIs(this.resultFile_v3, CoverityReader.class);
45+
}
46+
47+
public void onlyCoverityReaderReportsCanReadAsTrueForV10() {
4548
assertOnlyMatcherClassIs(this.resultFile_v10, CoverityReader.class);
4649
}
4750

4851
@Test
49-
void readerHandlesGivenResultFile() throws Exception {
50-
// For Coverity v3.0
52+
void readerHandlesGivenResultFileInV3() throws Exception {
5153
CoverityReader reader = new CoverityReader();
5254
TestSuiteResults result = reader.parse(resultFile_v3);
5355

@@ -59,10 +61,12 @@ void readerHandlesGivenResultFile() throws Exception {
5961

6062
assertEquals(CweNumber.PATH_TRAVERSAL, result.get(1).get(0).getCWE());
6163
assertEquals(CweNumber.SQL_INJECTION, result.get(2).get(0).getCWE());
64+
}
6265

63-
// For Coverity v10.0
64-
reader = new CoverityReader();
65-
result = reader.parse(resultFile_v10);
66+
@Test
67+
void readerHandlesGivenResultFileInV10() throws Exception {
68+
CoverityReader reader = new CoverityReader();
69+
TestSuiteResults result = reader.parse(resultFile_v10);
6670

6771
assertEquals(TestSuiteResults.ToolType.SAST, result.getToolType());
6872
assertTrue(result.isCommercial());

0 commit comments

Comments
 (0)