Qiskit / qiskit-ibm-runtime / 30120351326

88%

main: 88%

Build Type

push

github

Committed by web-flow

Commit Message

Write "high-level" prepare function for executor-based Estimator (#3102)

### Summary
Currently, the `EstimatorV2.run` function takes care of dispatching the
PUB's to the "low-level" `prepare_*` (`prepare`, `prepare-pea`,
`prepare_pec`, ...) functions. This creates some issues:
- Some issues when testing: we cannot test the dispatching without
initializing the Estimator and using mocks
- As well as an asymmetry with `SamplerV2`: in Sampler, we have a single
`prepare` function that takes as input pubs and options and returns pubs
and options, mapping inputs of Sampler to inputs of Executor.

In this PR, we:

- [x] Rename the current `prepare` function as `prepare_vanilla`
- [x] Write a `prepare` function that takes inputs of Estimator and maps
them to inputs of Executor, taking care of dispatching to the low-level
`prepare_*` functions.
- [x] Rewrite tests such as
`test_run_dispatches_to_prepare_pec_when_pec_enabled` in a way that does
not require mocks

Fixes #3090 

### AI/LLM disclosure

- [x] I didn't use LLM tooling, or only used it privately.
- [ ] I used the following tool to help write this PR description:
- [ ] I used the following tool to generate or modify code:

---------

Co-authored-by: Anthony <125415978+apchytr@users.noreply.github.com>

Coverage Stats

68 of 70 new or added lines in 3 files covered. (97.14%)

3 existing lines in 1 file now uncovered.

9019 of 10279 relevant lines covered (87.74%)

6.96 hits per line