Skip to content

Commit 77e842d

Browse files
Check for compilation warnings in test files
1 parent a06d0fc commit 77e842d

1 file changed

Lines changed: 4 additions & 2 deletions

File tree

bin/test_exercises.sh

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -80,7 +80,7 @@ do
8080
mv .meta/example.ex lib/solution.ex
8181
fi
8282

83-
# test compilation with --warnings-as-errors flag as the example and test should not raise any
83+
# test compilation with --warnings-as-errors flag as the example should not raise any
8484
set +e
8585
compiler_results=$(MIX_ENV=test mix compile --force --warnings-as-errors 2>&1)
8686
compile_exit_code="$?"
@@ -98,7 +98,9 @@ do
9898

9999
# perform unit tests
100100
set +e
101-
test_results=$(mix test --color --no-elixir-version-check --include pending 2> /dev/null)
101+
# the earlier mix compile did not catch warnings in *_test.exs files,
102+
# so we also run tests with `--warnings-as-errors`
103+
test_results=$(mix test --warnings-as-errors --color --no-elixir-version-check --include pending 2>&1)
102104
test_exit_code="$?"
103105
set -e
104106
else

0 commit comments

Comments
 (0)