Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Implement toHaveAccessibilityValue matcher #1496

Conversation

tarunrajput
Copy link
Contributor

@tarunrajput tarunrajput commented Sep 11, 2023

Summary

#1485 - Implement toHaveAccessibilityValue matcher

Test plan

Add unit tests and ensure all test passes

@tarunrajput tarunrajput changed the title Implements toHaveAccessibilityValue matcher Implement toHaveAccessibilityValue matcher Sep 12, 2023
@tarunrajput tarunrajput force-pushed the feat/add-toHaveAccessibilityValue-matcher branch from c46061c to 61626bc Compare September 12, 2023 12:19
@codecov
Copy link

codecov bot commented Sep 12, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (328466d) 98.13% compared to head (ebcbab5) 98.15%.
Report is 1 commits behind head on main.

❗ Current head ebcbab5 differs from pull request most recent head 6cecf2b. Consider uploading reports for the commit 6cecf2b to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1496      +/-   ##
==========================================
+ Coverage   98.13%   98.15%   +0.01%     
==========================================
  Files          97       98       +1     
  Lines        5747     5787      +40     
  Branches      890      894       +4     
==========================================
+ Hits         5640     5680      +40     
  Misses        107      107              
Files Coverage Δ
src/matchers/extend-expect.ts 100.00% <100.00%> (ø)
src/matchers/to-have-accessibility-value.tsx 100.00% <100.00%> (ø)

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

Copy link
Collaborator

@pierrezimmermannbam pierrezimmermannbam left a comment

Choose a reason for hiding this comment

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

@tarunrajput Thanks for the pr! Implementation looks good, I just have some feedback on the tests that needs to be addressed before we merge this

@mdjastrzebski mdjastrzebski force-pushed the feat/add-toHaveAccessibilityValue-matcher branch 3 times, most recently from 4c0df36 to 3848c5e Compare October 25, 2023 10:08
@mdjastrzebski mdjastrzebski force-pushed the feat/add-toHaveAccessibilityValue-matcher branch from 3848c5e to ebcbab5 Compare October 25, 2023 10:09
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.

Looks good! Let's merge 🔀 this!

@mdjastrzebski mdjastrzebski merged commit 28c8729 into callstack:main Oct 25, 2023
6 checks passed
@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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants