Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Implement test runner #3

Merged
merged 23 commits into from
Jul 22, 2024
Merged

Implement test runner #3

merged 23 commits into from
Jul 22, 2024

Conversation

0xNeshi
Copy link
Contributor

@0xNeshi 0xNeshi commented Jul 20, 2024

No description provided.

Nenad and others added 23 commits July 17, 2024 10:14
Update readme
2cae771
Implement run(-test).sh files
ab21759
Add Dockerfile
e548963
Remove asdf installation
66e0def
Sort test results in run.sh
0537c67
remove the need to sort result contents in run-tests.sh
2647013
Add tests/**/Scarb.lock & target to gitignore
dfd3dfb
Add all-fail tests
5e47366
Update all-file + empty-file not yet working
fcaeeb0
Fix error case in run.sh
3472a66
Move sorting to run-tests.sh
98efd8d
Fix comparison in run-tests.sh
bb3e170
Regenerate empty-file's expected_results.json
b4534ee
generate partial-fail's expected_results.json
82c4678
generate success's & syntax-error's expected_results.json
9ce7ade
Include relative file path in the error message
3c14cdc
UPdate Dockerfile to make run-in-docker work
ce62962
Improve 'done' msgs in run-x.sh scripts
1eea511
Switch Docker (and scarb install) to alpine
69c3cf7
Remove solution_dir prefix after reading errors up to 'could not comp…
ed41902 
…ile'
Add msg if results match
bea0e9d
Remove echo of run-tests done
9f38066
Refactor
2deac8c
@0xNeshi 0xNeshi added x:type/coding Write code that is not student-facing content (e.g. test-runners, generators, but not exercises) x:module/test-runner Work on Test Runners labels Jul 20, 2024
@0xNeshi 0xNeshi requested a review from ErikSchierboom July 20, 2024 10:05
@0xNeshi 0xNeshi self-assigned this Jul 20, 2024
@0xNeshi 0xNeshi requested a review from a team as a code owner July 20, 2024 10:05
@0xNeshi 0xNeshi removed the request for review from ErikSchierboom July 20, 2024 10:05
@ErikSchierboom
Copy link
Member

@misicnenad Once exercism/terraform#117 is merged and the changes deployed (which is usually done shortly after merging), the test runner can be enabled in the track's config.json file.

@0xNeshi 0xNeshi merged commit a2833b9 into main Jul 22, 2024
2 checks passed
@0xNeshi 0xNeshi deleted the build branch July 22, 2024 06:46
@0xNeshi 0xNeshi mentioned this pull request Jul 22, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ErikSchierboom ErikSchierboom ErikSchierboom approved these changes

Assignees

@0xNeshi 0xNeshi

Labels
x:module/test-runner Work on Test Runners x:type/coding Write code that is not student-facing content (e.g. test-runners, generators, but not exercises)
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@0xNeshi @ErikSchierboom