testdata group named `x.in` triggers 'missing x.ans' warning

Kattis / problemtools

Tools to manage problem packages using the Kattis problem package format.

MIT License

105 stars 72 forks source link

testdata group named `x.in` triggers 'missing x.ans' warning #169

Closed RagnarGrootKoerkamp closed 3 years ago

RagnarGrootKoerkamp commented 4 years ago

See title. The check for missing .ans files should probably check that the .in files are actually files and not directories.

ghamerly commented 3 years ago

While the code may technically be wrong here, it feels like it would cause more confusion than clarity to have test data directories that are named "x.in" or "y.ans".

RagnarGrootKoerkamp commented 3 years ago

I agree, but ran into this when adding testcases for BAPCtools. What do you think is a reasonable stricter spec for testgroup names? Since currently the spec does allow this.