Skip to content

Commit

Permalink
Merge pull request #12295 from woocommerce/unit_tests_merge_improvements
Browse files Browse the repository at this point in the history
Unit tests merge improvements
  • Loading branch information
wzieba authored Aug 12, 2024
2 parents 56bb040 + 67fbe10 commit a30e004
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 43 deletions.
4 changes: 2 additions & 2 deletions .buildkite/commands/run-unit-tests.sh
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ echo "--- 🚦 Report Tests Status"
results_file="WooCommerce/build/test-results/merged-test-results.xml"
# Merge JUnit results into a single file (for performance reasons with reporting)
# See https://github.com/woocommerce/woocommerce-android/pull/12064
./merge_junit.sh -d WooCommerce/build/test-results/testJalapenoDebugUnitTest -o $results_file
merge_junit_reports -d WooCommerce/build/test-results/testJalapenoDebugUnitTest -o "$results_file"

if [[ $BUILDKITE_BRANCH == trunk ]] || [[ $BUILDKITE_BRANCH == release/* ]]; then
annotate_test_failures "$results_file" --slack "build-and-ship"
Expand All @@ -31,6 +31,6 @@ echo "--- ⚒️ Generating and uploading code coverage"
.buildkite/commands/upload-code-coverage.sh

echo "--- 🧪 Copying test logs for test collector"
mkdir WooCommerce/build/buildkite-test-analytics && cp WooCommerce/build/test-results/*.xml WooCommerce/build/buildkite-test-analytics
mkdir WooCommerce/build/buildkite-test-analytics && cp "$results_file" WooCommerce/build/buildkite-test-analytics

exit $TESTS_EXIT_STATUS
2 changes: 1 addition & 1 deletion .buildkite/pipeline.yml
Original file line number Diff line number Diff line change
Expand Up @@ -87,7 +87,7 @@ steps:
<<: *test_collector_common_params
api-token-env-name: "BUILDKITE_ANALYTICS_TOKEN_UNIT_TESTS"
artifact_paths:
- "**/build/test-results/*/*.xml"
- "**/build/test-results/merged-test-results.xml"

- label: "Instrumented tests"
command: .buildkite/commands/run-instrumented-tests.sh
Expand Down
40 changes: 0 additions & 40 deletions merge_junit.sh

This file was deleted.

0 comments on commit a30e004

Please sign in to comment.