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
✨ Add fc.scheduler
arbitrary
#479
Conversation
test/e2e/AsyncScheduler.spec.ts
Outdated
const handle = buildGraph(initialPackageName, fetch); | ||
// Or: await s.waitAll(); | ||
while (s.count() !== 0) { | ||
expect(s.count()).toBeLessThanOrEqual(Object.keys(allPackages).length); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We might add a maxThreadPoolSize parameters with a quitehigh initial value or force the users to define one
This pull request is automatically built and testable in CodeSandbox. To see build info of the built libraries, click here or the icon next to each commit SHA. Latest deployment of this branch, based on commit 5121fa6:
|
Why is this PR for?
Race conditions are the most difficult to debug and to detect issues. Property basedtesting might help to detect such issues earlier by resolving promises in a random order to uncover unexpected orderings causing crashes in our apps.
PR is at a very very early stage. Tests are missings, code is in a single file...
For the moment, I need to gather cases that can be used to illustrate the purpose of such arbitrary. I will put some of those examples into
test/e2e/AsyncScheduler.spec.ts
.In a nutshell
✔️ New feature
❌ Fix an issue
❌ Documentation improvement
❌ Other: please explain
(✔️: yes, ❌: no)
Potential impacts
None.