test: add test for NavbarMobile#339
Conversation
|
Thank you for opening this PR! Before a maintainer takes a look, it would be really helpful if you could walk through your changes using GitHub's review tools. Please take a moment to:
More information on how to conduct a self review: This helps make the review process smoother and gives us a clearer understanding of your thought process. Once you've added your self-review, we'll continue from our side. Thank you! |
| } from '../navbar-utils'; | ||
|
|
||
| jest.mock('../navbar-utils', () => ({ | ||
| jest.mock('@/components/utils/ExportTasks', () => ({ |
There was a problem hiding this comment.
mocked export fnc,s
| expect(handleLogout).toHaveBeenCalled(); | ||
| }); | ||
|
|
||
| test('export task as json and close menu', () => { |
There was a problem hiding this comment.
added test to ensure that export fnc calls and close menu for both json && txt
and added test to check the autosync renders in dialog
Neeraj-gagat
left a comment
Neeraj-gagat
left a comment
There was a problem hiding this comment.
done with the self review nothing breaks and all tests passed
increased test coverage from 79 to 93%
its-me-abhishek
changed the title
2 participants
Description
This PR adds unit and interaction tests for the NavbarMobile component
Added tests for following
these tests increaced coverage from 79% to 93%
- Fixes: #302
image