← Back to issue list

test(spread): update store tests

View original Github issue

Metadata

Project
snapcraft
Number
#5371
Type
pull request
State
merged
Author
mr-cal
Labels
Created
Updated
Closed

Current evaluation

Merged after updating store tests and credentials following a secret re-provisioning that broke existing tests. Adjusted configurations for metrics, base64 credential encoding, and staging store keys to restore test functionality.

Suggested action:

No scores available.

Issue body

- [x] Have you followed the [guidelines for contributing](https://github.com/canonical/snapcraft/blob/main/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 `make test`? --- Only the second commit needs to be reviewed (the tiny one). All of snapcraft's secrets were re-provisioned from Sergio's credentials to mine and it broke some of the store tests. This PR adjusts the tests and the secrets. | failing test | fix | | -------------------------------------------- | ---------------------------------------------------------------------------- | | tests/spread/store/basic-workflow:candid | use the testing snap for `snapcraft metrics` | | tests/spread/store/basic-workflow:legacy | use base64 encoding for the credentials | | tests/spread/store/basic-workflow:ubuntu_one | use the testing snap for `snapcraft metrics` | | tests/spread/store/confdbs | use a new key registered on the staging store | | tests/spread/store/validation-sets | use a new key registered on the staging store | Needs #5356 (CRAFT-4419)

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Merged after updating store tests and credentials following a secret re-provisioning that broke existing tests. Adjusted configurations for metrics, base64 credential encoding, and staging store keys to restore test functionality.
qwen3.6-35b-a3b-mtp-q6 Merged. Updated store spread tests for re-provisioned credentials. Fixed failures by using testing snaps for metrics, base64 encoding credentials, and registering new staging store keys. Depends on PR #5356.
qwen3.6-35b-a3b-mtp-q6 Merged. Updated store spread tests for re-provisioned secrets. Fixed failures by using testing snaps, base64 credential encoding, and new staging store keys. Resolves credential rotation breakage.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#5370 test: fix store tests snapcraft closed Merged to fix store test failures caused by re-provisioned credentials. Updated test configurations and secrets for staging workflows, including base64 encoding and new key registrations.
85%
#5954 tests: fix store tests snapcraft merged Merged after approval. Updated store tests with new staging credentials and switched to wheels for automated spread tests. CI passed, with unrelated spread failures noted by the author.
76%
#86 tests: update store tests for new image craft-store merged Merged after approval. Updates store tests for a new image and adds replication instructions. All tests passed successfully.
71%
#3561 spread tests: store metrics against staging (CRAFT-391) snapcraft merged Merged spread tests to store metrics against staging. Approved, passed CI, and added four lines without affecting coverage.
70%