It seems there's an invariant (the argument of keyword must show up in
keywords) which is not enforced. Let's enforce that dynamically by
calling error on violations: this is triggered during tests often
enough (for each keyword call which is covered, but coverage seems good
enough), so violations are triggered before deployment often enough.
The runtime overhead is not noticeable, since parsing time is overall
minimal.
It seems there's an invariant (the argument of
keywordmust show up inkeywords) which is not enforced. Let's enforce that dynamically by callingerroron violations: this is triggered during tests often enough (for each keyword call which is covered, but coverage seems good enough), so violations are triggered before deployment often enough.The runtime overhead is not noticeable, since parsing time is overall minimal.
See violation of this in: https://github.com/Blaisorblade/pts/commit/b73eb5c1fdc6fab1e11b6349aee2471fd133266f#commitcomment-7509176
Also fix another violation which remained here:
_(was not a keyword, and this was detected during testing.