Skip to content

Commit 7c9c554

Browse files
author
Dave Wichers
committed
Fix 2 merge errors in previous commit.
1 parent 9e268e3 commit 7c9c554

File tree

2 files changed

+0
-18
lines changed

2 files changed

+0
-18
lines changed

plugin/src/main/java/org/owasp/benchmarkutils/score/parsers/SemgrepReader.java

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -144,8 +144,6 @@ public static int translate(int cwe) {
144144
return CweNumber.XXE;
145145
case 614:
146146
return CweNumber.INSECURE_COOKIE;
147-
case 1004:
148-
return CweNumber.COOKIE_WITHOUT_HTTPONLY;
149147
case 643:
150148
return CweNumber.XPATH_INJECTION;
151149
case 1004:

plugin/src/test/java/org/owasp/benchmarkutils/score/parsers/sarif/SemgrepSarifReaderTest.java

Lines changed: 0 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -77,20 +77,4 @@ void readerHandlesSemgrepPROResultFile() throws Exception {
7777
assertEquals(CweNumber.COOKIE_WITHOUT_HTTPONLY, result.get("1").get(0).getCWE());
7878
assertEquals(CweNumber.XSS, result.get("2").get(0).getCWE());
7979
}
80-
81-
@Test
82-
void readerHandlesSemgrepPROResultFile() throws Exception {
83-
SemgrepSarifReader reader = new SemgrepSarifReader();
84-
TestSuiteResults result = reader.parse(resultFilePRO);
85-
86-
assertEquals(TestSuiteResults.ToolType.SAST, result.getToolType());
87-
assertFalse(result.isCommercial());
88-
assertEquals("Semgrep PRO", result.getToolName());
89-
assertEquals("1.68.1", result.getToolVersion());
90-
91-
assertEquals(2, result.getTotalResults());
92-
93-
assertEquals(CweNumber.COOKIE_WITHOUT_HTTPONLY, result.get(1).get(0).getCWE());
94-
assertEquals(CweNumber.XSS, result.get(2).get(0).getCWE());
95-
}
9680
}

0 commit comments

Comments
 (0)