Skip to content

Commit 1270e98

Browse files
committed
Exit early if there are no package directories to test
1 parent cf5e10a commit 1270e98

File tree

1 file changed

+10
-8
lines changed

1 file changed

+10
-8
lines changed

.github/workflows/scripts/run_affected_tests

Lines changed: 10 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -114,6 +114,13 @@ main() {
114114
# Find unique package directories:
115115
directories=$(echo "${changed}" | tr ' ' '\n' | sed -E 's/\/(bin|data|etc|include|lib|src|test)\/?$//' | uniq)
116116

117+
if [ -z "${directories}" ]; then
118+
echo 'No packages to test.' >&2
119+
cleanup
120+
print_success
121+
exit 0
122+
fi
123+
117124
# Extract package names from changed package directories (e.g., @stdlib/math/base/special/sin) by removing the leading 'lib/node_modules/':
118125
packages=$(echo "${directories}" | sed -E 's/^lib\/node_modules\///')
119126

@@ -131,15 +138,10 @@ main() {
131138
fi
132139
done
133140

134-
if [[ -z "${directories}" && -z "${required_by}" ]]; then
135-
echo 'No directories to test.' >&2
136-
cleanup
137-
print_success
138-
exit 0
139-
fi
140-
141141
# Concatenate the list of changed package directories and package directories which `require()` one of the changed packages:
142-
directories="${directories} ${required_by}"
142+
if [ -n "${required_by}" ]; then
143+
directories="${directories} ${required_by}"
144+
fi
143145

144146
# Find all test files in package directories:
145147
files=$(find ${directories} -maxdepth 2 -wholename '**/test/test*.js' | grep -v '/fixtures/' | sort -u | tr '\n' ' ')

0 commit comments

Comments
 (0)