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

Meaningful unit test coverage - Header #108

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

Meaningful unit test coverage - Header #108

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 Header.js found in src/components/Header/
@ryanbellison ryanbellison added enhancement New feature or request good first issue Good for newcomers hacktoberfest Great for a hacktoberfest contribution! labels Oct 6, 2019
@oliverfb
Copy link
Contributor

oliverfb commented Oct 7, 2019

Would like to take on this one!

@SheaBelsky
Copy link
Contributor

@eshno It's all yours! Thank you for picking this up!

@oliverfb oliverfb mentioned this issue Oct 11, 2019
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
enhancement New feature or request good first issue Good for newcomers hacktoberfest Great for a hacktoberfest contribution!
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@ryanbellison @SheaBelsky @oliverfb