Skip to content

Commit 05e0c35

Browse files
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 changed
1 parent c44e7e7 commit 05e0c35

File tree

4 files changed

+10
-1862
lines changed

4 files changed

+10
-1862
lines changed

0 commit comments

Comments
 (0)