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

race condition in max-threads-count.qtest #4387

Closed
davidnich opened this issue Jan 14, 2022 · 0 comments
Closed

race condition in max-threads-count.qtest #4387

davidnich opened this issue Jan 14, 2022 · 0 comments
Assignees
Milestone

Comments

@davidnich
Copy link
Contributor

QUnit Test "Max thread count test" v1.0
FAILURE: Test max thread count: 2 assertions, 1 succeeded
Assertion "number of threads is 1" failure at ./examples/test/qore/threads/max-threads-count.qtest:47 [MaxThreadCountTest::testMaxThreadCount()] <- ./examples/test/qore/threads/max-threads-count.qtest:20 [MaxThreadCountTest::constructor()]
-----
	>> Expected: True, Actual: False
-----
Ran 1 test case, 1 error (2 assertions, 1 succeeded)
@davidnich davidnich added this to the 1.1 milestone Jan 14, 2022
@davidnich davidnich self-assigned this Jan 14, 2022
@davidnich davidnich added this to To do in Qore Development via automation Jan 14, 2022
Qore Development automation moved this from To do to Done Jan 14, 2022
davidnich added a commit that referenced this issue Jan 14, 2022
#4386)

* refs #3409 added warnings for the [] operator for constant values that are not integers

* refs #4387 fixed a race condition in max-threads-count.qtest
davidnich added a commit that referenced this issue Jan 14, 2022
* refs #2096 fixed a crash in the scanner handling EOF in regexes
refs #4379 fixed an error in the last commit related to this fix

* refs #4387 fixed another race condition in the test
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

No branches or pull requests

1 participant