Skip to content

Compare sorted tags in test_save_all#709

Merged
wshanks merged 4 commits into
qiskit-community:mainfrom
mriedem:708-fix-test-save-all
Mar 4, 2022
Merged

Compare sorted tags in test_save_all#709
wshanks merged 4 commits into
qiskit-community:mainfrom
mriedem:708-fix-test-save-all

Conversation

@mriedem
Copy link
Copy Markdown
Contributor

@mriedem mriedem commented Mar 3, 2022

Summary

qiskit-experiments does not maintain order on experiment
tags since #522 so we need to compare experiment tags in
the test_save_all test with a predictable sort order.

Details and comments

This was originally fixed in qiskit-ibm-provider:

Qiskit/qiskit-ibm-provider#214

Closes #708

mriedem and others added 3 commits March 3, 2022 16:16
qiskit-experiments does not maintain order on experiment
tags since qiskit-community#522 so we need to compare experiment tags in
the `test_save_all` test with a predictable sort order.

This was originally fixed in qiskit-ibm-provider:

Qiskit/qiskit-ibm-provider#214

Closes qiskit-community#708
@wshanks wshanks merged commit be322e3 into qiskit-community:main Mar 4, 2022
@mriedem mriedem deleted the 708-fix-test-save-all branch March 4, 2022 21:01
@chriseclectic chriseclectic added the Changelog: Bugfix Include in the "Fixed" section of the changelog label Apr 25, 2022
paco-ri pushed a commit to paco-ri/qiskit-experiments that referenced this pull request Jul 11, 2022
qiskit-experiments does not maintain order on experiment
tags since qiskit-community#522 so we need to compare experiment tags in
the `test_save_all` test with a predictable sort order.

This was originally fixed in qiskit-ibm-provider:

Qiskit/qiskit-ibm-provider#214

Closes qiskit-community#708

Co-authored-by: Will Shanks <willshanks@us.ibm.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Changelog: Bugfix Include in the "Fixed" section of the changelog

Projects

None yet

Development

Successfully merging this pull request may close these issues.

TestExperimentDataIntegration.test_save_all fails with AssertionError: Lists differ: ['foo', 'bar'] != ['bar', 'foo']

3 participants