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

Create a simple ListGroup test #98

Closed
daniel-norris opened this issue Oct 27, 2020 · 1 comment · Fixed by #103
Closed

Create a simple ListGroup test #98

daniel-norris opened this issue Oct 27, 2020 · 1 comment · Fixed by #103
Assignees
Labels
good first issue Good for newcomers hacktoberfest Open for Hacktoberfest 2020 tests Add or improve existing tests
Milestone

Comments

@daniel-norris
Copy link
Owner

Is your feature request related to a problem? Please describe.
Test coverage across the application is low. We need to build confidence that the components have the expected behaviour that we want and to help mitigate any regression in the future.

Describe the solution you'd like
We need to implement better test coverage across the library. Ideally each component should be accompanied by a test case written using Jest. At a minimum the test should check whether the component successfully shows any child props. If you are able to include tests for any additional functionality then that would be appreciated!

We should have a test case covering the ListGroup component implemented using Jest. More info on Jest can be found here.

For examples of how this is done, take a look at existing test cases in the library. An example would be the CardHeader or Input components.

You can run your tests using npm run test or to see test coverage across the library npm run test:cov.

This is part of epic #19.

@daniel-norris daniel-norris added good first issue Good for newcomers tests Add or improve existing tests hacktoberfest Open for Hacktoberfest 2020 labels Oct 27, 2020
@daniel-norris daniel-norris added this to the v1.0.0. milestone Oct 27, 2020
@lyonsbp
Copy link
Contributor

lyonsbp commented Oct 27, 2020

I can take this one as well, was easy after doing ListItem

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers hacktoberfest Open for Hacktoberfest 2020 tests Add or improve existing tests
Projects
Release v1.0.0.
  
In progress
Development

Successfully merging a pull request may close this issue.

2 participants