The implementation looks good and makes sense. A bit hard to read at the first glance but elegant.
However, I would suggest not to add specific test scripts for each feature add. That will make it hard to figure out what went wrong if one of the tests fails on a future feature add. I'd prefer to add test cases to the current test system. Or do I miss something here?
I can add appropriate test cases, but I cannot commit into this PR.
Originally posted by @ekkea in #96 (comment)
The implementation looks good and makes sense. A bit hard to read at the first glance but elegant.
However, I would suggest not to add specific test scripts for each feature add. That will make it hard to figure out what went wrong if one of the tests fails on a future feature add. I'd prefer to add test cases to the current test system. Or do I miss something here?
I can add appropriate test cases, but I cannot commit into this PR.
Originally posted by @ekkea in #96 (comment)