Skip to content

Commit 37eaa19

Browse files
author
Mila Votradovec
committed
test: fix tests after reachability update
1 parent c9cf8c4 commit 37eaa19

File tree

2 files changed

+10
-8
lines changed

2 files changed

+10
-8
lines changed

tests/api.spec.ts

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,7 @@ describe('Requests to public API', () => {
4242
if (response.type === 'error') return;
4343
expect(new Set(response.value.configFiles)).toEqual(new Set(['.dcignore', '.gitignore']));
4444
expect(new Set(response.value.extensions)).toEqual(
45-
new Set(['.es', '.es6', '.htm', '.html', '.js', '.jsx', '.py', '.ts', '.tsx', '.vue', '.java']),
45+
new Set(['.es', '.es6', '.htm', '.html', '.js', '.jsx', '.py', '.ts', '.tsx', '.vue', '.java', '.java-dummy']),
4646
);
4747
});
4848

@@ -404,8 +404,9 @@ describe('Requests to public API', () => {
404404
} while (response.value.status !== AnalysisStatus.done);
405405

406406
expect(Object.keys(response.value.analysisResults.suggestions).length).toEqual(4);
407-
expect(new Set(Object.keys(response.value.analysisResults.files)))
408-
.toEqual(new Set(['/GitHubAccessTokenScrambler12.java', '/not/ignored/this_should_not_be_ignored.java']));
407+
expect(new Set(Object.keys(response.value.analysisResults.files))).toEqual(
408+
new Set(['/GitHubAccessTokenScrambler12.java', '/not/ignored/this_should_not_be_ignored.java']),
409+
);
409410

410411
// Get analysis results without linters but with severity 3
411412
do {
@@ -423,8 +424,9 @@ describe('Requests to public API', () => {
423424
} while (response.value.status !== AnalysisStatus.done);
424425

425426
expect(Object.keys(response.value.analysisResults.suggestions).length).toEqual(2);
426-
expect(new Set(Object.keys(response.value.analysisResults.files)))
427-
.toEqual(new Set(['/GitHubAccessTokenScrambler12.java', '/not/ignored/this_should_not_be_ignored.java']));
427+
expect(new Set(Object.keys(response.value.analysisResults.files))).toEqual(
428+
new Set(['/GitHubAccessTokenScrambler12.java', '/not/ignored/this_should_not_be_ignored.java']),
429+
);
428430
},
429431
TEST_TIMEOUT,
430432
);

tests/git.analysis.spec.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -95,7 +95,7 @@ describe('Functional test of analysis', () => {
9595
});
9696

9797
// Test DC JSON format first
98-
expect(Object.keys(bundle.analysisResults.suggestions).length).toEqual(119);
98+
expect(Object.keys(bundle.analysisResults.suggestions).length).toEqual(134);
9999

100100
const cweSuggestion = Object.values(bundle.analysisResults.suggestions).find(
101101
s => s.id === 'java%2Fdc_interfile_project%2FPT',
@@ -105,8 +105,8 @@ describe('Functional test of analysis', () => {
105105
expect(cweSuggestion?.title).toBeTruthy();
106106
expect(cweSuggestion?.text).toBeTruthy();
107107

108-
expect(bundle.sarifResults?.runs[0].results?.length).toEqual(400);
109-
expect(bundle.sarifResults?.runs[0].tool?.driver.rules?.length).toEqual(119);
108+
expect(bundle.sarifResults?.runs[0].results?.length).toEqual(442);
109+
expect(bundle.sarifResults?.runs[0].tool?.driver.rules?.length).toEqual(134);
110110

111111
const cweRule = bundle.sarifResults?.runs[0].tool?.driver.rules?.find(r => r.id === 'java/PT');
112112
expect(cweRule?.properties?.cwe).toContain('CWE-23');

0 commit comments

Comments
 (0)