Commit 05e0c35
authored
fix: skipped snapshot test (#919)
## 📜 Description
Fixed skipped test flakiness.
## 💡 Motivation and Context
> One thing that I couldn't solve is KeyboardToolbar snapshot testing
component. For some reasons it serializes entire react tree. I think it
happens because the ecosystem is not ready for React 19 yet and
serializers somehow doesn't work. In order not to block this PR I
temporarily disabled this test. I'll enable it back when serializer
stuff will be sorted out.
It has been fixed in this PR by passing content as pre-rendered property
👀
Follow up for
#812
<!-- Why is this change required? What problem does it solve? -->
<!-- If it fixes an open issue, please link to the issue here. -->
## 📢 Changelog
<!-- High level overview of important changes -->
<!-- For example: fixed status bar manipulation; added new types
declarations; -->
<!-- If your changes don't affect one of platform/language below - then
remove this platform/language -->
### JS
- remove `.skip` from tests;
## 🤔 How Has This Been Tested?
Tested on CI and locally.
## 📸 Screenshots (if appropriate):
<img width="852" alt="image"
src="https://github.com/user-attachments/assets/d3b0b721-f757-4035-bed2-4cbc173a592a"
/>
## 📝 Checklist
- [x] CI successfully passed
- [x] I added new mocks and corresponding unit-tests if library API was
changed1 parent c44e7e7 commit 05e0c35
4 files changed
Lines changed: 10 additions & 1862 deletions
File tree
- FabricExample/__tests__
- __snapshots__
- example/__tests__
- __snapshots__
0 commit comments