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.

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

100% coverage #161

Closed
sindresorhus opened this issue Nov 8, 2015 · 6 comments
Closed

100% coverage #161

sindresorhus opened this issue Nov 8, 2015 · 6 comments
Labels
enhancement new functionality good for beginner help wanted scope:internals scope:tooling

Comments

@sindresorhus
Copy link
Member

sindresorhus commented Nov 8, 2015 •
edited
Loading

We're currently at 96% coverage. Which is amazing. Getting us closer to 100% would be a good beginner task.

Please leave this for first time contributors.

See https://coveralls.io/github/avajs/ava?branch=master for uncovered code.
sindresorhus added a commit that referenced this issue Nov 12, 2015
@sindresorhus
Merge pull request #196 from sohamkamani/master
c149db2 
Improved test coverage for runner.js (issue #161)
sindresorhus pushed a commit that referenced this issue Nov 12, 2015
@sohamkamani @sindresorhus
Close #203 PR: More tests for runner.js (issue #161).
8645916
@sotojuan sotojuan mentioned this issue Dec 13, 2015
Closed
@sotojuan sotojuan mentioned this issue Dec 21, 2015
Merged
@Carnubak Carnubak mentioned this issue Feb 16, 2016
Closed
@vadimdemedes vadimdemedes reopened this Feb 18, 2016
@paulohp paulohp mentioned this issue Mar 29, 2016
Closed
@dananichev dananichev mentioned this issue Jul 16, 2016
Merged
@LyzioOh
Copy link

LyzioOh commented Oct 12, 2016

Fine, i dev beginner and i will be happy to take à look and see how improve code coverage.

@platy11
Copy link
Contributor

platy11 commented Nov 3, 2016

@sindresorhus I was looking at cli.js100 and in my testing (in your cat-names repo) I cannot get the deprecation error to appear - is this just me?

@novemberborn
Copy link
Member

@platy11 that line is in master but not yet in a release.

@platy11
Copy link
Contributor

platy11 commented Nov 6, 2016

@novemberborn Ok, thanks! 😃

@jarlehansen jarlehansen mentioned this issue Jan 4, 2017
Merged
liubinyi added a commit to liubinyi/ava that referenced this issue Mar 11, 2017
@liubinyi
This PR increases code coverage avajs#161
71347f1 
added tests to cover callee$0$0
changed one of the snapshot test to take an optional message, so it hit
one more line
@liubinyi liubinyi mentioned this issue Mar 11, 2017
Closed
@wildaces215
Copy link

Where can I start writing tests for ava it seems like a good place to start contributing to open source software!

@novemberborn
Copy link
Member

@wildaces215 you can browse our current code coverage at https://codecov.io/gh/avajs/ava. Look for a file that has little coverage, try and figure out what it does, and see if you can write a test that extends the coverage.

We'd be interested in covering larger pieces of logic rather than small edge cases.

@ghost ghost mentioned this issue Feb 24, 2019
Merged
@awaisab172 awaisab172 mentioned this issue Oct 12, 2024
Open
@sramam sramam mentioned this issue Oct 13, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement new functionality good for beginner help wanted scope:internals scope:tooling
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
6 participants
@novemberborn @sindresorhus @vadimdemedes @wildaces215 @platy11 @LyzioOh