-
Notifications
You must be signed in to change notification settings - Fork 7
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
test: validity for gate, job_type, measure, noise, pauli_string and other_instructions #106
Conversation
…/ColibrITD-SAS/mpqp into chore-features-for-all-providers
tests/execution/test_validity.py
Outdated
assert noise_build.to_other_language(language) is not None | ||
|
||
|
||
other_instr = [ |
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.
I would have that as a fixture
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.
done
tests/execution/test_validity.py
Outdated
assert measure.to_other_language(language) is not None | ||
|
||
|
||
pauli_strings = [Ip @ Xp @ Yp @ Zp, Xp + Zp, Yp] |
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.
fixture
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.
done
tests/execution/test_validity.py
Outdated
assert gate_build.to_other_language(language) is not None | ||
|
||
|
||
measures = [ |
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.
fixture
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.
done
tests/execution/test_validity.py
Outdated
+ list(GOOGLEDevice) | ||
+ list(AZUREDevice), | ||
) | ||
def test_validity_run_job_type(device: AvailableDevice): |
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.
maybe this could be a bit more generic ? I could try to rework it if you want me to
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.
tiny tweaks
No description provided.