Skip to content
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.

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

Adding test harness to current implementation #58

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

hath995
Copy link

@hath995 hath995 commented May 1, 2016

To run the tests:
npm install -g mocha
npm install chai

mocha tests/

@hath995
Copy link
Author

hath995 commented May 1, 2016

Just to be clear with the current implementation of try/catch/finally. The following tests fail:

  1. Should handle the exception in the outer try catch exception, and finish inner finally
  2. Should handle rethrowing the error, and catching in the outer (infinite loop)
  3. Should not catch if returning early in a finally block (infinite loop)
  4. It should not leave undone finally statements for uncaught exceptions
  5. return should respect finally
  6. return should respect catch and finally
  7. try/catch/finally should respect continue statements

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant