Skip to content

fix: qcodes error from fixture (SE-755)

Description

Fix an error on python 3.13 where pytest would fail on test_device_info with the error KeyError: 'Another instrument has the name: cluster'.

Somehow, swapping the order of test_device_info and test_build_info fixes it. I actually suspect a bug in pytest, but did not investigate further.

Furthermore, I removed a duplicated fixture and made the original one global for the whole test suite (put it in conftest.py).

Type

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Code refactoring (only refactoring of existing code, without new functionality)
  • Backport (cherry-picking fixes/features from development into a release branch)

Issue(s)

Related MRs

Testing

Notes

Merge checklist

  • Cleaned up code (correct alignment/spacing/code style, no dead code, etc).
  • Merge request targets correct branch (usually development).
  • Added team label to MR (team::fe, team::ese or team::dd).
  • Added MR dependencies and link to feature MR at highest hierarchy level (usually product-level)
  • Selected assignee (you) and reviewers (team-members).
  • If merging a repository with a release package, updated appropriate CHANGELOG.md:
    • CLUSTER_MM, QBLOX_INSTRUMENTS: file in same repo
    • CLUSTER_QxM: file in CLUSTER_MM repo
  • Merge request has been reviewed by at least one team-member.
  • All pipelines pass, including manually triggered ones.
Edited by Thomas Middelburg

Merge request reports

Loading