search

jmid / mutaml

An OCaml mutation tester
BSD 2-Clause "Simplified" License
65 stars 4 forks source link

rename: instrumentation-tests.t -> instrumentation-tests #17

Closed Alizter closed 1 year ago

Alizter commented 1 year ago

This looks accidental to me.

jmid commented 1 year ago

Indeed, I think this started as something with a single run.t and then grew to multiple .t files while forgetting to rename the surrounding directory...

I remember trying to follow the documentation at https://dune.readthedocs.io/en/stable/tests.html#directory-tests ("The name of the test directory must end with .t [...]") and studying dune's internal test suite back then. Reading that link again it may actually be good to document the "multiple .t file mode" :slightly_smiling_face:

jmid commented 1 year ago

...and thanks for noticing BTW! :smiley:

Alizter commented 1 year ago

Yeah, that chapter in the dune doc could use a rewrite. FWIW cram tests as files are explained above and cram tests where you want "starter files" are the directory tests they mention.