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

Cover strictly 100% of the code #49

Open
3 tasks
ichiriac opened this issue Jan 2, 2017 · 1 comment
Open
3 tasks

Cover strictly 100% of the code #49

ichiriac opened this issue Jan 2, 2017 · 1 comment

Comments

@ichiriac
Copy link
Member

ichiriac commented Jan 2, 2017

Improve the coverage with mocha :

  • 100% of lines of code
  • 100% of statements
  • > 95% of branches
@ichiriac ichiriac added this to the 1.1.0 milestone Jan 2, 2017
@ichiriac ichiriac self-assigned this Jan 2, 2017
@ichiriac ichiriac modified the milestones: 1.2.0, 1.1.0 Jan 21, 2017
@ichiriac ichiriac modified the milestones: 3.0.0, 1.2.0 Mar 6, 2017
@ichiriac ichiriac modified the milestones: 3.0.0, 3.1.0 Mar 6, 2018
@ichiriac ichiriac pinned this issue Jan 18, 2020
@cseufert
Copy link
Collaborator

There are a lot of branches that are not straightforward to test, as the branch will not be exercised unless the mode is changed from error to warning (for the expect() functions, and random AST nodes are removed to trigger all these cases (lots of expected brackets, arrows, etc). I wonder if writing a test driver that drops each symbol in each test, and takes a snapshot of the output would work here?

@czosel czosel unpinned this issue Jan 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants