Skip to content

Commit

Permalink
fix: tests relying on exact results number
Browse files Browse the repository at this point in the history
  • Loading branch information
michelkaporin committed Sep 26, 2022
1 parent e6052a8 commit 4741e64
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion tests/analysis.spec.ts
Original file line number Diff line number Diff line change
Expand Up @@ -70,7 +70,7 @@ describe('Functional test of analysis', () => {
if (bundle.analysisResults.type !== 'sarif') return;

expect(bundle.analysisResults.sarif.runs[0].tool.driver.rules?.length).toBeGreaterThan(0);
expect(bundle.analysisResults.sarif.runs[0].results?.length).toEqual(17);
expect(bundle.analysisResults.sarif.runs[0].results?.length).toBeGreaterThan(0);
const sampleRes = bundle.analysisResults.sarif.runs[0].results!.find(
res => res.locations?.[0].physicalLocation?.artifactLocation?.uri === `GitHubAccessTokenScrambler12.java`,
);
Expand Down
2 changes: 1 addition & 1 deletion tests/api.spec.ts
Original file line number Diff line number Diff line change
Expand Up @@ -322,7 +322,7 @@ describe('Requests to public API', () => {
expect(response.value.status !== AnalysisStatus.failed).toBeTruthy();

if (response.value.status === AnalysisStatus.complete && response.value.type === 'sarif') {
expect(response.value.sarif.runs[0].results).toBeGreaterThan(0);
expect(response.value.sarif.runs[0].results?.length).toBeGreaterThan(0);

expect(new Set(response.value.coverage)).toEqual(
new Set([
Expand Down

0 comments on commit 4741e64

Please sign in to comment.