Skip to content

Commit

Permalink
chore: explode aliases when looking up owners in .test_patterns.yml (
Browse files Browse the repository at this point in the history
…#12526)

Fixes the issue of reading the anchors rather than the actual values
when looking up owners of tests.
  • Loading branch information
TomAFrench authored Mar 6, 2025
1 parent a69f416 commit 2e0d791
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion ci3/run_test_cmd
Original file line number Diff line number Diff line change
Expand Up @@ -113,7 +113,7 @@ EOF
[ "$CI" -eq 0 ] && fail

# Get list of owners of this failed test.
owners=$(yq e -r '.tests[] | .regex as $pattern | select(strenv(test_cmd) | test($pattern)) | .owners[]' .test_patterns.yml | sort -u)
owners=$(yq e -r 'explode(.) | .tests[] | .regex as $pattern | select(strenv(test_cmd) | test($pattern)) | .owners[]' .test_patterns.yml | sort -u)

# To not fail a test, we at least need an owner to notify.
if [ -z "$owners" ]; then
Expand Down

0 comments on commit 2e0d791

Please sign in to comment.