-
Notifications
You must be signed in to change notification settings - Fork 6
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
feat: orchestrate #54
Conversation
Added "orchestrate", a tool for deterministically testing concurrent code.
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.
Not a complete review by any means, just writing down a thought that I had yesterday when I initially glanced over the code out of curiosity.
with open(filename, "r") as pyfile: | ||
tokens = tokenize.generate_tokens(pyfile.readline) | ||
for type, value, start, end, line in tokens: | ||
if type == tokenize.COMMENT and "pragma: SYNCPOINT" in value: |
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.
Suggestion: Let's made the check insensitive to the number of whitespace characters between "pragma:" and "SYNCPOINT"?
I can imagine somebody accidentally typing two spaces instead of one and such syncpoint would unexpectedly not work for no obvious reason.
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.
@plamut FWIW, that isn't how other packages use # pragma:
-- coverage, for instance, uses just plain string matching.
🤖 I have created a release \*beep\* \*boop\* --- ## [1.1.0](https://www.github.com/googleapis/python-test-utils/compare/v1.0.0...v1.1.0) (2021-08-30) ### Features * add 'orchestrate' module ([#54](https://www.github.com/googleapis/python-test-utils/issues/54)) ([ae3da1a](https://www.github.com/googleapis/python-test-utils/commit/ae3da1ab4e7cbf268d6dce60cb467ca7ed6c2c89)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
Migrated from this repo in this PR: googleapis/python-test-utils#54
Migrated from this repo in this PR: googleapis/python-test-utils#54
Adds "orchestrate", a tool for deterministically testing concurrent
code.
Closes #53