Skip to content

bin/verify-exercises-in-docker#504

Merged
glennj merged 3 commits into
exercism:mainfrom
keiravillekode:verify-exercises-in-docker
Apr 14, 2026
Merged

bin/verify-exercises-in-docker#504
glennj merged 3 commits into
exercism:mainfrom
keiravillekode:verify-exercises-in-docker

Conversation

@keiravillekode
Copy link
Copy Markdown
Contributor

This script provides a convenient way to test exercises against a test runner image.

@keiravillekode
Copy link
Copy Markdown
Contributor Author

Adapted from Go bin/verify-exercises-in-docker, which was itself adapted from other tracks.

@keiravillekode keiravillekode force-pushed the verify-exercises-in-docker branch from 6badb7e to 454fb09 Compare April 13, 2026 04:48
Comment thread bin/verify-exercises-in-docker Outdated
@keiravillekode
Copy link
Copy Markdown
Contributor Author

I'm not a groovy maintainer.

@glennj glennj merged commit ac1eda4 into exercism:main Apr 14, 2026
5 checks passed
@keiravillekode keiravillekode deleted the verify-exercises-in-docker branch April 14, 2026 02:57
@keiravillekode
Copy link
Copy Markdown
Contributor Author

I recently found a different way to solve the clean-up problem of root-owned sub-directories:

docker_args+=( --mount "type=volume,dst=/solution/target" )

Then ${tmpdir}/target never exists, so trap 'rm -rf "${tmpdir}"' EXIT succeeds.

With groovy's current test runner we would have noise

rm: cannot remove 'target': Resource busy

because the test runner has rm -rf target immediately before running the tests.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants