profiler icon indicating copy to clipboard operation
profiler copied to clipboard

Test coverage for src/components/app

Open gregtatum opened this issue 7 years ago • 2 comments

In trying to find our worst offenders for test coverage, src/components/app/ stands out as being particularly bad. We might need to take on enzyme as a dependency and shallow rendering to properly deal with some of these components.

Files ordered by lines uncovered:

  • [ ] Root.js (77 lines uncovered)
  • [ ] ProfileSharing.js (67 lines uncovered)
  • [ ] ProfileViewer.js (40 lines uncovered)
  • [ ] UrlManager.js (32 lines uncovered)
  • [ ] Home.js (27 lines uncovered)
  • [ ] SymbolicationStatusOverlay.js (23 lines uncovered)
  • [ ] TabBar.js (17 lines uncovered)
  • [ ] ProfileFilterNavigator.js (9 lines uncovered)
  • [ ] FooterLinks.js (3 lines uncovered)
  • [ ] Initializing.js (2 lines uncovered)

┆Issue is synchronized with this Jira Task

gregtatum avatar Jan 24 '18 20:01 gregtatum

Can I work on this?

ria-19 avatar Oct 27 '21 05:10 ria-19

Since this bug was filed, this evolved quite a bit, I'd need to look at our current coverage.

Also if you are an outreachy student, please note that this project isn't part of outreachy this time.

If you're still interested, a good component to start with is FooterLinks. We don't use enzyme nowadays but react-testing-library, you'll find plenty of examples in our codebase.

julienw avatar Oct 27 '21 10:10 julienw