Qiskit / qiskit / 23895846592

88%

main: 88%

Build Type

push

github

Committed by web-flow

Commit Message

Correctly enforce matrix size check in `CommutationChecker` (#15933) (#15936)

* Fixed the matrix size check for non-cachable gates in the commutation checker.

A typo in the matrix size check allowed to sneak into a matrix-based check even if the operations exceeded the maximum matrix size. The operations `first_..` and `second_..` are sorted, but `..1` and `..2` are not! The wrong one was used here.

* Reduce test size

This test now doesn't rely on the internal path panicking, but still tests the right behavior since the results with and without a matrix check differ.

(cherry picked from commit b43a2f33d)

Co-authored-by: Julien Gacon <jules.gacon@googlemail.com>

Coverage Stats

1 of 1 new or added line in 1 file covered. (100.0%)

25 existing lines in 6 files now uncovered.

103944 of 118710 relevant lines covered (87.56%)

997887.7 hits per line