← Back to issue list

spread: add a "big" spread test

View original Github issue

Metadata

Project
rockcraft
Number
#62
Type
pull request
State
merged
Author
tigarmo
Labels
Created
Updated
Closed

Current evaluation

Merged a reusable spread test framework to house multiple checks without repeated setup. The initial test validates bug #44 regarding directory permissions. Approved by one reviewer and passed CI checks.

Suggested action:

No scores available.

Issue body

This new test can be used to "house" multiple different checks without having to "setup" and "teardown" the whole machinery multiple times. This initial version tests bug #44, regarding permissions of added directories. - [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? ----- @sergiusens @cmatsuoka this is an attempt at adding a spread test that we can augment over time, as discussed last week. Thanksss

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Merged a reusable spread test framework to house multiple checks without repeated setup. The initial test validates bug #44 regarding directory permissions. Approved by one reviewer and passed CI checks.
qwen3.6-35b-a3b-mtp-q6 Merged a reusable spread test framework to house multiple checks without repeated setup and teardown. The initial version validates bug #44 regarding added directory permissions.
qwen3.6-35b-a3b-mtp-q6 Merged a reusable spread test framework to consolidate checks and eliminate repeated setup and teardown. The initial implementation validates bug #44 regarding directory permissions, designed for future expansion.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#534 test(spread): workaround for flaky test fetch-service merged Merged a workaround for a flaky spread test (issue #533). The change adjusts one file (+2/-1 lines) to skip a no-op name query check while retaining other validations. Approved by two reviewers and passed all CI checks.
80%
#280 ci, spread: add "large spreads" suite rockcraft merged Merged addition of a new large spreads test suite to CI. Configured to run weekly via cron or manually via label. Approved by two reviewers and successfully integrated.
79%
#3662 tests: update spread url snapcraft merged Merged update to spread tarball URLs in test files. Approved by a reviewer, passed CI checks, modified two files with minimal changes, and maintained existing code coverage.
78%
#13 test(spread): add more spread tests snapcraft-rocks merged Successfully merged spread tests covering architectures, chisel slices, craftctl, package-repositories, and patchelf. Approved by two reviewers across 15 files (+206/-5 lines).
77%
#1287 tests: minor cleanups on the spread tests snapcraft merged Merged minor spread test cleanups into master. Modified four files with 16 additions and 6 deletions. Approved by one reviewer with zero unresolved comments, maintaining 95.27% coverage upon integration.
77%
#926 test: address a couple of non-flaky spread failures rockcraft merged Merged after two approvals and passing all CI checks. Resolves non-flaky spread test failures by updating test configurations across three files, eliminating consistent failures.
77%
#573 ci: run spread tests on 'spread-installed' runners rockcraft merged Merged a two-file, four-line CI configuration change to run spread tests on spread-installed runners. Approved by three reviewers and integrated.
76%
#3020 spread tests: default base for local plugin tests snapcraft merged Merged updates to spread tests establishing a default base for local plugin tests. The change follows a prior commit pattern, adding two lines across two files. Approved by one reviewer and passed CI checks.
76%
#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.
76%
#127 Fix spread tests imagecraft merged Merged after receiving approval from two reviewers and passing all CI checks. The changes fix spread tests across 11 files (+49/-38 lines).
76%