store: on prem base client
Metadata
Current evaluation
Merged #3880 as commit 383b844ee, adding a craft-store-derived OnPrem base client and updating the legacy JSON decoder mock.
Resolution: closed resolved
Reason: PR #3880 was merged into main as commit 383b844ee, adding OnPrem store support and adjusting the legacy test mock for requests.JSONDecoder.
Evaluated by: mr-cal-bot
No scores available.
Issue body
Derivation from craft-store's BaseClient to support the OnPrem store.
This implementation is specific enough to Snapcraft to be implemented
here.
This includes an update to the test mock in legacy pertaining to the
change of where JSONDecoder comes from now (requests).
Signed-off-by: Sergio Schvezov <sergio.schvezov@canonical.com>
- [x] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)?
- [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [x] Have you successfully run `make lint`?
- [x] Have you successfully run `pytest tests/unit`?
-----
CRAFT-1243
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged #3880 as commit 383b844ee, adding a craft-store-derived OnPrem base client and updating the legacy JSON decoder mock. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged the Snapcraft-specific OnPrem store BaseClient and legacy JSON decoder mock update in commit 383b844ee via PR #3880. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged in #3880 as commit 383b844ee, adding a craft-store-derived OnPrem base client and updating the legacy JSON decoder test mock. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged into main as commit 383b844ee, adding an OnPrem store base client derived from craft-store and updating the legacy JSONDecoder mock. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged into main. Adds an OnPrem base client derived from craft-store to support Snapcraft's on-premise store. Approved by reviewer, passed CI, and increased unit test coverage by 0.05%. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged, resolving CRAFT-1243. Adds an OnPrem store base client derived from craft-store's BaseClient, tailored for Snapcraft, with updated test mocks for JSONDecoder origin changes. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #3885 store: initial support for an onprem store | snapcraft | merged | Merged initial on-premises store support for CRAFT-1228, landing as commit 260214788. |