Skip to content
This repository has been archived by the owner on Feb 10, 2022. It is now read-only.

Meaningful unit test coverage - Layout #114

Open
ryanbellison opened this issue Oct 6, 2019 · 2 comments
Open

Meaningful unit test coverage - Layout #114

ryanbellison opened this issue Oct 6, 2019 · 2 comments
Labels
enhancement New feature or request good first issue Good for newcomers hacktoberfest Great for a hacktoberfest contribution!

Comments

@ryanbellison
Copy link
Collaborator

Problem

As part of issue #59, all snapshots tests where removed. We would now like to replace these snapshots with a more meaningful unit test.

Acceptance Criteria

Using @testing-library/react create a test for Layout.js found in src/components/Layout/

@ryanbellison ryanbellison added enhancement New feature or request good first issue Good for newcomers hacktoberfest Great for a hacktoberfest contribution! labels Oct 6, 2019
@bagshik
Copy link

bagshik commented Oct 8, 2019

Hey there, I can take this one. Which cases need to be covered?

@Cooperbuilt
Copy link
Contributor

Hey @bagshik, we are leaving the coverage choices up to the contributor - whatever you think is useful coverage feel free to do that.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request good first issue Good for newcomers hacktoberfest Great for a hacktoberfest contribution!
Projects
None yet
Development

No branches or pull requests

3 participants