fix: use consistent path for pytest test reports

With #41412c1 pytest reports are generated in "test_results/". Update devtool and release.sh to align with this change. Signed-off-by: Sudan Landge <[email protected]>
firecracker-microvm · Sep 22, 2023 · 42f986e · 42f986e
1 parent 00e234c
commit 42f986e
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/.buildkite/pipeline_pr.py b/.buildkite/pipeline_pr.py
@@ -49,7 +49,7 @@
 
 release_grp = group(
     "📦 Release Sanity Build",
-    "touch ./tests/test-report.json && ./tools/devtool -y sh ./tools/release.sh --libc musl --profile release --make-release",
+    "mkdir -p ./test_results && touch ./test_results/test-report.json && ./tools/devtool -y sh ./tools/release.sh --libc musl --profile release --make-release",
     **defaults_once_per_architecture,
 )
 

diff --git a/tools/devtool b/tools/devtool
@@ -483,7 +483,7 @@ cmd_build() {
 }
 
 function cmd_make_release {
-    cmd_test -- --json-report --json-report-file=test-report.json || die "Tests failed!"
+    cmd_test || die "Tests failed!"
 
     run_devctr \
         --user "$(id -u):$(id -g)" \

diff --git a/tools/release.sh b/tools/release.sh
@@ -167,7 +167,7 @@ cp -v -t "$RELEASE_DIR" LICENSE NOTICE THIRD-PARTY
 check_swagger_artifact src/api_server/swagger/firecracker.yaml "$VERSION"
 cp -v src/api_server/swagger/firecracker.yaml "$RELEASE_DIR/firecracker_spec-$VERSION.yaml"
 
-cp -v tests/test-report.json "$RELEASE_DIR/"
+cp -v test_results/test-report.json "$RELEASE_DIR/"
 
 (
     cd "$RELEASE_DIR"