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
No DeprecationWarning deduping during testing #4239
No DeprecationWarning deduping during testing #4239
Conversation
…on assertion logic in json serialization tests
Blocked on #4250. |
The CI failure is interesting, there is still some weirdness happening (at least locally): This fails:
These pass:
While they should all fail. |
…/Cirq into fix_deprecation_logic_json_tests
It seems that conftest.py is required by all modules to turn on |
ready for review @tanujkhattar |
|
||
|
||
def _get_testspecs_for_modules(): | ||
modules = [] | ||
for m in TESTED_MODULES: | ||
for m in TESTED_MODULES.keys(): |
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.
Use the default iterator instead of calling .keys()
on the dictionary.
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.
Why? I'm not using the dictionary part...
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.
Using default iterators, whenever supported, is more generic and avoids an unnecessary function call.
https://google.github.io/styleguide/pyguide.html#28-default-iterators-and-operators
ready for re-review @tanujkhattar. |
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.
LGTM! Thanks!
Disable deprecation warning deduping during testing and fix deprecation assertion logic in json serialization tests. Fixes quantumlib#4227. Merge after quantumlib#4250 (it contains a fix that is merged in here).
Extracts cirq-pasqal as a separate module. Merge after quantumlib#4239, a fix for json testing and deprecation warning deduping and quantumlib#4250, a fix for flnyt / minimal pytest dependencies (it contains them currently)
Disable deprecation warning deduping during testing and fix deprecation assertion logic in json serialization tests.
Fixes #4227.
Merge after #4250 (it contains a fix that is merged in here).