New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add tests for stop #29

Merged

StavromulaBeta merged 1 commit into master from stop

Aug 31, 2024

+15 −0

Collaborator

hedyhli commented Aug 31, 2024

Opening a PR to draw your attention (sorry) in case you have better ideas for some of these:

Tests had to be split due to the nature of that function
On failure it would print both a PASS and a FAIL since we can't print anything after Stop successfully stops execution - if you can think of a better way I'd be glad!
Begin is tested because both begin and stop are implemented with catching errors in cognatejs, it might be helpful since these tests could be used for alternative cognate implementations


          Add tests for stop

24da5d4

StavromulaBeta merged commit 05c166f into master

8 checks passed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet