Skip to content

Conversation

@siepra
Copy link
Contributor

@siepra siepra commented Sep 11, 2023

Summary

#1486 - Implement toContainElement() matcher.

Based on Jest Native's toContainElement()

Test plan

yarn test

@siepra siepra changed the title feat: add toContainElement matcher feat: toContainElement matcher Sep 11, 2023
@codecov
Copy link

codecov bot commented Sep 11, 2023

Codecov Report

Patch coverage: 100.00% and project coverage change: +0.01% 🎉

Comparison is base (f538029) 98.08% compared to head (49ad4e1) 98.09%.
Report is 1 commits behind head on main.

Additional details and impacted files
@@ Coverage Diff @@ ## main #1495 +/- ## ========================================== + Coverage 98.08% 98.09% +0.01%  ========================================== Files 93 94 +1 Lines 5590 5630 +40 Branches 864 873 +9 ========================================== + Hits 5483 5523 +40  Misses 107 107 
Files Changed Coverage Δ
src/matchers/extend-expect.ts 100.00% <100.00%> (ø)
src/matchers/to-contain-element.tsx 100.00% <100.00%> (ø)

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@siepra siepra marked this pull request as ready for review September 12, 2023 16:00
@siepra
Copy link
Contributor Author

siepra commented Sep 18, 2023

@mdjastrzebski could you spare a minute to review this one?

@mdjastrzebski
Copy link
Member

@siepra will do it soon, at the moment the PR queue extends my capacity to zero it in one day, so I'll handling it in the FIFO order.

@siepra
Copy link
Contributor Author

siepra commented Sep 19, 2023

@mdjastrzebski I didn't mean to hurry you up, just thought you might have missed it (as I didn't explicitly linked this PR in the main issue). Good luck with all the reviews!

@mdjastrzebski mdjastrzebski force-pushed the feat/toContainElement-matcher branch from 6177070 to a383a34 Compare September 19, 2023 09:29
Copy link
Member

@mdjastrzebski mdjastrzebski left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All good, let's ship it ✅

@mdjastrzebski
Copy link
Member

🎉 Release in version: v12.4.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants