test(hero):add test for hero.tsx#329
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! |
Neeraj-gagat
left a comment
Neeraj-gagat
left a comment
There was a problem hiding this comment.
done with self review added new tests in existing file Hero.test.tsx for rendering uuid by default and for hiding uuid and encryption-secret whwen button to hide them is clicked
ne changes break nothing because no fuctional chnages were made and passes all tests
new tests increased coverage for hero.tsx from 76.92% to 100%
| expect(uuidCode).toBeInTheDocument(); | ||
| }); | ||
|
|
||
| test('Hides UUId when button is clicked', () => { |
There was a problem hiding this comment.
small typo here, 'UUId' should be 'UUID'. please fix, will merge later
2 participants
Description
This PR adds interaction tests for Hero component for hiding uuid and encryption-secret when toggled
- Fixes: #315
image