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

add testing for qiskit_ibm_runtime/fake_provider/ #1316

Closed
1ucian0 opened this issue Jan 15, 2024 · 3 comments
Closed

add testing for qiskit_ibm_runtime/fake_provider/ #1316

1ucian0 opened this issue Jan 15, 2024 · 3 comments
Labels
help wanted Extra attention is needed

Comments

@1ucian0
Copy link
Member

1ucian0 commented Jan 15, 2024

With the move of the fake providers from qiskit to qiskit-ibm-runtime, the tests were not moved yet. This issue is to account for that, as the code has currently 0% coverage:

Screenshot 2024-01-15 at 12 58 36

Tagging this as help wanted as a community member with some experience could move and adapt the tests from Qiskit.

@1ucian0 1ucian0 added the help wanted Extra attention is needed label Jan 15, 2024
@ElePT
Copy link
Collaborator

ElePT commented Jan 15, 2024

The tests were moved in #1140, the base class PR (#1270) is a follow-up of this one. As far as I know, there are no base class unit tests to move. I am not sure why the coverage does not pick up the existing tests, I think it's because the main test is in integration and not unit tests.

@jyu00
Copy link
Collaborator

jyu00 commented Apr 10, 2024

We should verify if tests exist

@kt474
Copy link
Member

kt474 commented Jun 26, 2024

Tests already exist for fake_provider

Image

@kt474 kt474 closed this as completed Jun 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

4 participants