Add protein-translation exercise #52
Conversation
pfertyk
force-pushed
the
add-protein-translation-exercise
branch
3 times, most recently
from
8f8ff25
to
7636a9f
Compare
|
@BNAndras since there is an issue with the grains exercise, I decided to implement another one, until we solve that problem ;) |
| description = "Non-existing codon can't translate" | ||
|
|
||
| [9eac93f3-627a-4c90-8653-6d0a0595bc6f] | ||
| description = "Unknown amino acids, not part of a codon, can't translate" |
There was a problem hiding this comment.
We should mark the tests not actually implemented by adding include = false underneath the description.
There was a problem hiding this comment.
@BNAndras done ;) I copied the tests from Python, I'm not sure why these ones were not included ... I've added a task for later to implement these missing tests, but I believe that can wait until after the track is public ;)
There was a problem hiding this comment.
Adding some tests later means any previously submitted solutions will be rerun to see if they pass the full test suite. Depending on when you add those tests, there might not be too many solutions to rerun. On established popular tracks like Python, we could be rerunning thousands of submitted solutions every time we update the tests so I'd be careful about doing that long-term.
Marking tests as include = false means we have seen the tests but we won't implement them so configlet knows to ignore them when we run configlet sync. If you're still planning on implementing those tests, I'd instead remove the tests completely from the tests.toml. Then configlet sync will think we haven't synced to the latest tests and warn us that these additional tests need to be added or marked not included. That way, we won't forget in the future that some tests still need to be added.
pfertyk
force-pushed
the
add-protein-translation-exercise
branch
from
7636a9f
to
cc9e01b
Compare
No description provided.