Skip to content
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.

Test: add test for DataTag component #2858

Merged
merged 8 commits into from Oct 6, 2022
Merged

Conversation

depimomo
Copy link
Contributor

@depimomo depimomo commented Oct 3, 2022

Description

Added unit tests for DataTag component and new snapshot.
directory :- /src/components/DataTag/

Add total of 3 new unit tests.

Related Issues

Related to #2744

Check List

  • I have read the Contributing Guidelines and made commit messages that follow the guideline.
  • I have run npm run lint:js -- --fix and/or npm run lint:md -- --fix for my JavaScript and/or Markdown changes.
    • This is important as most of the cases your code changes might not be correctly linted
  • I have run npm run test to check if all tests are passing, and/or npm run test -- -u to update snapshots if I created and/or updated React Components.
  • I have checked that the build works locally and that npm run build work fine.
  • I've covered new added functionality with unit tests if necessary.

@codecov-commenter
Copy link

codecov-commenter commented Oct 3, 2022

Codecov Report

Base: 66.02% // Head: 66.84% // Increases project coverage by +0.82% 🎉

Coverage data is based on head (99e1f02) compared to base (fd57b87).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2858      +/-   ##
==========================================
+ Coverage   66.02%   66.84%   +0.82%     
==========================================
  Files         118      124       +6     
  Lines        1351     1460     +109     
  Branches      342      363      +21     
==========================================
+ Hits          892      976      +84     
- Misses        422      444      +22     
- Partials       37       40       +3     
Impacted Files Coverage Δ
src/__fixtures__/hooks.tsx 100.00% <ø> (ø)
src/__fixtures__/page.tsx 94.11% <ø> (-5.89%) ⬇️
...components/ApiComponents/Components/ApiHeading.tsx 100.00% <ø> (+100.00%) ⬆️
...rc/components/ApiComponents/Components/ApiLink.tsx 100.00% <ø> (+100.00%) ⬆️
...ents/ApiComponents/Components/SourceLink/index.tsx 50.00% <ø> (+50.00%) ⬆️
...ApiComponents/Components/VersionSelector/index.tsx 100.00% <ø> (+100.00%) ⬆️
src/components/ApiComponents/index.tsx 100.00% <ø> (+100.00%) ⬆️
src/components/Article/index.tsx 100.00% <ø> (ø)
src/components/Banner/index.tsx 100.00% <ø> (ø)
src/components/Codebox/index.tsx 100.00% <ø> (ø)
... and 86 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@ovflowd ovflowd added the hacktoberfest-accepted Used to label PR's label Oct 3, 2022
@manishprivet manishprivet enabled auto-merge (squash) October 6, 2022 09:32
@ovflowd ovflowd merged commit d5e74eb into nodejs:main Oct 6, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
hacktoberfest-accepted Used to label PR's
Projects
None yet
5 participants