search

gildor478 / ounit

Other
88 stars 19 forks source link

Invalid test name can be set. #68

Open gildor478 opened 3 years ago

gildor478 commented 3 years ago

This bug has been migrated from artifact #1455 on forge.ocamlcore.org. It was assigned to user102.

user20074 posted on 2015-02-19 12:37:03:

It is possible to add a TestLabel to a test, but if we have a '/' in our TestLabel, we have this kind of error :

Fatal error: exception Sys_error("/home/gregoire/Work/my_projet/_build/oUnit-Test toto/tata -suze#00.log: No such file or directory")

This error is probably simply due to the fact that it try to create/read in a file that will have the name of the test, and so see this name as directories/file instead of file.

user102 replied on 2015-02-19 14:52:58:

Good point.

bolives-hax commented 1 month ago

i also ran into this unintentionally. Played around with it a little, inserting \ doesn't work like

"testname bla bla bla that does stuff \/"

but i still feel like this is honestly not something that should happen. While I see somehwhat limited potential in abusing that for anything its still misleading. I don't have time to look into it properly and prolly wont just saying I can reproduce this and thought id be the first one to open an issue