Qiskit / qiskit-ibm-runtime / 36459273913

91%

Build Type

push

github

Committed by web-flow

Commit Message

Merge CloudAccount and Account (#3389)

### Summary

Rename `CloudAccount` to `Account`, removing the "base" class.

We have been using instances of `CloudAccount` for quite a while, with
no real instantiation of a pure `Account`. This PR takes advantage of
that to simplify the hierarchy:
* move all non-abstract methods from `Account`  to `CloudAccount`
* revise the init to match behavior
* additionally, revise the `_assert_valid_*` methods and move them into
`validate()`, as "assert" could be misleading and they are mostly single
`ifs` that are not called separately except from that function
* split the `test_account.py` file into two files, as the file was
having a high line count

### Details and comments

Related to #3385

### AI/LLM disclosure

- [x] I didn't use LLM tooling, or only used it privately.

Coverage Stats

45 of 46 new or added lines in 1 file covered. (97.83%)

2 existing lines in 1 file now uncovered.

8621 of 9488 relevant lines covered (90.86%)

9.93 hits per line