OCaml test runner v3: Allow to link test to a task_id

Exercism Building Exercism
1

Hello, I submitted a PR to port the ocaml test runner to version 3, discovering task_id(s) by matching test names against a trailing pattern such as test_name @ task 1

Sorry If I missed contributing guidelines, feel free to point me to the correct procedure if necessary :slight_smile:

2 Likes
2

@NyuB I’m not an OCaml person, but the RegExp |^\(.*\)\b @ task \([1-9]+\)$| will not match @ task 10. In your unit tests you have 123 for multiple digits, which does not cover 0 being allowed for other places than the first.

1 Like
3

@mk-mxp you’re completely right ! Sorry fot that, I’ll open a fix asap if nobody does it before :slight_smile:

4

Here is the related fix PR: Fix: task ids with zeros would not be recognized by NyuB · Pull Request #63 · exercism/ocaml-test-runner · GitHub

1 Like