Conversation
|
This PR touches files which potentially affect the outcome of the tests of an exercise. This will cause all students' solutions to affected exercises to be re-tested. If this PR does not affect the result of the test (or, for example, adds an edge case that is not worth rerunning all tests for), please add the following to the merge-commit message which will stops student's tests from re-running. Please copy-paste to avoid typos. For more information, refer to the documentation. If you are unsure whether to add the message or not, please ping |
|
Yeah, you could pull out the slug from either the test file itself or the containing exercise folder. |
|
Other than the feedback, is this somewhat in the direction you had previously mentioned? It is not pretty at the moment, and I will invest a lot more time into it if it is to move ahead. |
|
I am going through the exercises one by one. Once I reach the exercises which use comparators other than |
|
Yeah, having the assertion functions as actual ReScript code means we can check them. As strings, we don't have that option. |
There was a problem hiding this comment.
I am a bit stuck here. Having the comparator functions as strings is the only way I can see to inject them into the exercise test files. Copying the old Assertions module into each exercise's test directory isn't feasible.
There was a problem hiding this comment.
this is an abstracted version of the comparator function that we can use.
|
All tests converted to use ReScript. |
|
Nice! Looks good to me |
|
Happy to merge if you are. I think it's a good place to build from and continue adding more exercises |
Draft
Before I go any further, I would like some feedback on how we might handle test templates moving forward.
test_templatesdirectory, where test templates are now written in pure ReScript_testpart.)I have re-implemented a few of the exercises using this test generator for your review.