test: update remote-build artifact name
Metadata
Current evaluation
Merged after two reviewer approvals. Updates the expected filename for a remote-build spread test to fix a minor test mismatch. Passed all required CI checks and involved a single-line change.
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`?
---
Updates the expected file name of an overlooked spread test.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged after two reviewer approvals. Updates the expected filename for a remote-build spread test to fix a minor test mismatch. Passed all required CI checks and involved a single-line change. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged a fix to update the expected filename for a remote-build spread test artifact. The change corrects an overlooked test expectation, ensuring accurate validation during manual spread runs. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged correction to the expected filename for an overlooked remote-build spread test, aligning test assertions with current artifact naming conventions. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #1561 fix(spread): workaround for remote builds getting the same name | charmcraft | merged | Merged a three-line workaround to prevent remote builds from receiving duplicate names. Approved by two reviewers and integrated into the codebase. | |
| #4433 tests(spread): add remote-build spread test | snapcraft | merged | Merged changes adding spread tests for new and legacy remote-build, resolving issue #4398. Approved by two reviewers with passing CI checks. | |
| #1413 build: rename weekly spread test | charmcraft | merged | Merged after two reviewers approved. Renamed the weekly spread test in the build configuration, requiring #1410. Changes were minimal across two files. | |
| #4826 fix(remote-build): fail if no artifacts are created | snapcraft | merged | Merged fix for issue #4783 that ensures remote builds fail when no artifacts are generated. Approved by two reviewers, passed CI, and merged with 214 additions and 4 deletions across two files. | |
| #2681 fix: correctly get project name for remote builds | charmcraft | merged | Merged after three approvals. Corrects project name retrieval for remote builds, resolving #2598. Integrated despite non-blocking type-checking warnings and two unresolved review comments. | |
| #758 fix(testing): don't change the output artifact location | craft-application | merged | Merged fix for #752 that keeps test output artifacts in the standard location instead of moving them to a spread subdirectory. Approved by reviewers and passed CI checks prior to merge. | |
| #4767 tests: fix remote-build tests | snapcraft | merged | Merged fix for failing weekly remote-build tests. All tests pass except Fedora, which remains excluded due to unrelated spread configuration issues. Approved by two reviewers and passed CI checks. | |
| #351 fix(remotebuild): use full project name in log files | craft-application | merged | Merged a fix ensuring remote build log files use the full project name, resolving issue #4781. Approved by three reviewers, passed CI, and updated two files. |