Skip to content

Commit

Permalink
Merge pull request #508 from DataDog/amaan.qureshi/update-scripts
Browse files Browse the repository at this point in the history
chore(integration-tests): require minimum results to be at least 2
  • Loading branch information
amaanq authored Sep 5, 2024
2 parents 6445f55 + 07b8ebb commit c852de8
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 5 deletions.
2 changes: 1 addition & 1 deletion misc/integration-test-docker.sh
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@ RES=`jq '.runs[0].results | length ' "${REPO_DIR}/results1.json"`

echo "Found $RES errors on first run"

if [ "$RES" -lt "12" ]; then
if [ "$RES" -lt "2" ]; then
echo "not enough errors found without the static-analysis.datadog.yml file"
exit 1
fi
Expand Down
4 changes: 2 additions & 2 deletions misc/integration-test-js-ts.sh
Original file line number Diff line number Diff line change
Expand Up @@ -28,8 +28,8 @@ fi

FINDINGS=`jq '.runs[0].results|length' ${REPO_DIR}/results.json`
echo "Found $FINDINGS violations"
if [ $FINDINGS -lt 10 ]; then
echo "only $FINDINGS found, expected at least 10 findings"
if [ $FINDINGS -lt 2 ]; then
echo "only $FINDINGS found, expected at least 2 findings"
exit 1
fi

Expand Down
4 changes: 2 additions & 2 deletions misc/integration-test-python.sh
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@ RES=`jq '.runs[0].results | length ' "${REPO_DIR}/results1.json"`

echo "Found $RES errors on first run"

if [ "$RES" -lt "18" ]; then
if [ "$RES" -lt "2" ]; then
echo "not enough errors found"
exit 1
fi
Expand All @@ -44,7 +44,7 @@ RES=`jq '.runs[0].results | length ' "${REPO_DIR}/results2.json"`

echo "Found $RES errors on second run"

if [ "$RES" -lt "18" ]; then
if [ "$RES" -lt "2" ]; then
echo "not enough errors found"
exit 1
fi
Expand Down

0 comments on commit c852de8

Please sign in to comment.