Qiskit / qiskit / 23893436460

88%

Build Type

push

github

Committed by web-flow

Commit Message

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

* 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.

Coverage Stats

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

18 existing lines in 4 files now uncovered.

104121 of 119029 relevant lines covered (87.48%)

1022571.2 hits per line