Skip to content

Commit

Permalink
[trivial] Remove src/translate/{tests,regression-tests}.
Browse files Browse the repository at this point in the history
We discussed this in Discord, and there were no objections.
Copying the relevant message:

While working on some translator code, we stumbled over the directories
src/translate/tests and src/translate/regression-tests/.

We haven't touched tests since 2015 except for global changes like
Python 2 => Python 3.

We haven't touched regression-tests since 2013.

As far as I remember, these predate the tests we actually use (and which
live in a different part of the repository), and they don't satisfy the
requirements you'd typically want of tests (not all are meant to be
automatic tests in the usual way; the ones that are don't necessarily
pass, even though there isn't a bug; the one that perhaps passes doesn't
test something we still consider important, such as direct executability
of certain translator source files).

I suggest we delete these directories. If someone is keen on the
functionality they were intended for, I think we would need to invest
some work to make them useful. But I'm not convinced this is good value
for time.
  • Loading branch information
maltehelmert committed Aug 19, 2024
1 parent bff55bb commit 8989cfb
Show file tree
Hide file tree
Showing 20 changed files with 0 additions and 6,582 deletions.
7 changes: 0 additions & 7 deletions src/translate/regression-tests/README

This file was deleted.

375 changes: 0 additions & 375 deletions src/translate/regression-tests/issue34-domain.pddl

This file was deleted.

Loading

0 comments on commit 8989cfb

Please sign in to comment.