Skip to content

Conversation

@rigidit
Copy link
Contributor

@rigidit rigidit commented Jan 28, 2026

Delete recordings infrastracture and all related tests as part of recordings removal effort.

We can delete these tests because in earlier PRs:

  • Unit tests that run without recordings were migrated to unit/local folder.
  • Live tests were moved to *-clients repo to run as part of e2e tests pipeline.
  • tox now runs against unit/local/test_qiskit_offline.py

qiskit-ionq helpers and exceptions (commit cea8f9874b992f82a35648582c06958869370c69) - Apache-2.0

Portions of azure.quantum.qiskit.backends._qiskit_ionq.py and azure-quantum/tests/unit/test_qiskit.py
Portions of azure.quantum.qiskit.backends._qiskit_ionq.py and azure-quantum/tests/unit/local/test_qiskit_offline.py
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ScottCarda-MS could you please have a look if we need to include test_qiskit_offline here?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No I don't think so. The live test files were quite large so it is difficult to tell what parts of them came from the community repo, but the test_qiskit_offline tests are considerably simpler than the live tests, so I don't think they are conceptually related to community content.

@rigidit rigidit marked this pull request as ready for review January 28, 2026 22:06
@rigidit rigidit requested a review from a team as a code owner January 28, 2026 22:06
@rigidit rigidit changed the title Delete live tests with recordings infrastructure Delete tests requiring recordings infrastructure Jan 28, 2026
@rigidit rigidit changed the title Delete tests requiring recordings infrastructure Delete recordings infrastructure with related tests Jan 28, 2026
@pavelbraginskiy
Copy link

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@pavelbraginskiy
Copy link

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants