Skip to content

Commit 87caedb

Browse files
committed
ci: The tests job no longer excludes the docker wrapper test
dev-container/docker.test.sh passes since the previous commit, so the skip list and its explanation go away. It is this repository's only *.test.sh, which means the tests job was green here because it ran nothing at all. Signed-off-by: Simon Walz <contact@simonwalz.de>
1 parent 0c0e629 commit 87caedb

1 file changed

Lines changed: 0 additions & 13 deletions

File tree

‎.github/workflows/ci.yml‎

Lines changed: 0 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -105,21 +105,8 @@ jobs:
105105
echo "No test scripts in this repository."
106106
exit 0
107107
fi
108-
# Excluded until it passes. It had never run in CI, and against this
109-
# repository's own wrapper (dev-container/docker) it fails 3 of 9: the
110-
# host-path mapping drops the path suffix, producing /HOST where
111-
# /HOST/data/work is expected. Named here rather than hidden behind
112-
# continue-on-error, so the gate stays real for everything else.
113-
# See brain/tasks/fix-failing-tests.md.
114-
skip=( "dev-container/docker.test.sh" )
115108
status=0
116109
for file in "${files[@]}"; do
117-
for s in "${skip[@]}"; do
118-
if [[ "${file}" == "${s}" ]]; then
119-
echo "SKIPPED ${file} - see the comment in this workflow"
120-
continue 2
121-
fi
122-
done
123110
echo "::group::${file}"
124111
if ! bash "${file}"; then
125112
status=1

0 commit comments

Comments
 (0)