Qiskit / qiskit / 19929753632

88%

main: 88%

Build Type

push

github

Committed by web-flow

Commit Message

Fix control-flow builders with uncached Python control-flow (#15413)

The `ElseContext` builder previously relied on referential identity of
the `IfElseOp`'s true body, which is no longer stable with the move of
control-flow operations to Rust space.  This sanity check is simpler and
easier to trick if you try, but it should still catch a proper set of
reasonable user errors.

It would be possible to add significantly more complex special handling
between Rust space and Python space to make the test less foolable, but
this commit is a reasonable trade-off for realistic user errors.

Run Details

22 of 23 new or added lines in 2 files covered. (95.65%)

25 existing lines in 4 files now uncovered.

95968 of 108611 relevant lines covered (88.36%)

1207990.58 hits per line