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

Fix FLX error scenario tests (compensating writes) #4615

Merged
merged 4 commits into from
Jun 7, 2022

Conversation

kneth
Copy link
Contributor

@kneth kneth commented Jun 3, 2022

What, How & Why?

Compensating writes avoid many fatal errors and our tests need to be adjusted accordingly.

☑️ ToDos

  • 📝 Changelog entry
  • [ ] 📝 Compatibility label is updated or copied from previous entry
  • 🚦 Tests
  • 🔀 Executed flexible sync tests locally if modifying flexible sync
  • [ ] 📱 Check the React Native/other sample apps work if necessary
  • [ ] 📝 Public documentation PR created or is not necessary
  • [ ] 💥 Breaking label has been applied or is not necessary

If this PR adds or changes public API's:

  • [ ] typescript definitions file is updated
  • [ ] jsdoc files updated
  • [ ] Chrome debug API is updated if API is available on React Native

@cla-bot cla-bot bot added the cla: yes label Jun 3, 2022
@kneth kneth requested a review from takameyer June 3, 2022 14:43
@kneth kneth self-assigned this Jun 3, 2022
Copy link
Contributor

@takameyer takameyer left a comment

Choose a reason for hiding this comment

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

Just remove the setTimeout

@takameyer
Copy link
Contributor

Looks like we can also remove expect-sync-error.ts since errors are being produced by throwing.

@kneth kneth merged commit c203614 into master Jun 7, 2022
@kneth kneth deleted the kneth/fix-flx-error-scenarios branch June 7, 2022 10:26
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 15, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants