Remove duplicate getting started tests now that we have doctests

unitaryfund / mitiq

Mitiq is an open source toolkit for implementing error mitigation techniques on most current intermediate-scale quantum computers.

https://mitiq.readthedocs.io

GNU General Public License v3.0

363 stars 160 forks source link

Remove duplicate getting started tests now that we have doctests #118

Closed willzeng closed 4 years ago

willzeng commented 4 years ago

If I understand #105 correctly, we now have automated testing of the code in the docs.

Does this mean that the tests in:

tests/test_zne.py (https://github.com/unitaryfund/mitiq/blob/master/mitiq/tests/test_zne.py)
and in mitiq_qiskit/tests/test_qiskit_getting_started.py on the PR (https://github.com/unitaryfund/mitiq/pull/88/files#diff-2dd3520468986648509132b80ab4d8e9) are redundant and can be removed?

@nathanshammah ?

willzeng commented 4 years ago

The new updates to #88 do now remove the duplicate testing code. I'm stoked about doc tests meaning we don't have to duplicate this. Nice find for setting this up @nathanshammah