tests(spread): requirements dependencies
Metadata
Current evaluation
Merged following approval by two reviewers. Introduces regression tests for requirements file and dependency combinations, resolving issue #1179. The change modifies five files, adding 149 lines and removing one.
Suggested action: —
No scores available.
Issue body
This tests various combinations of requirements files and dependencies for regressions.
Fixes #1179
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged following approval by two reviewers. Introduces regression tests for requirements file and dependency combinations, resolving issue #1179. The change modifies five files, adding 149 lines and removing one. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged. Adds regression tests for requirements file and dependency combinations, resolving issue #1179 by validating dependency resolution across configurations. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged regression tests for requirements file and dependency combinations. Resolves #1179 by validating various configuration scenarios to prevent future dependency resolution issues. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #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. | |
| #1366 tests(spread): make release test less flaky | charmcraft | merged | Merged changes that reduce flakiness in the spread release test, resolving issue #1362. Approved by two reviewers, the update modifies one file (+17/-7) after addressing feedback. | |
| #1322 fix(test): better matching for strict dependencies spread test | charmcraft | merged | Merged a single-line test fix improving matching logic for the strict dependencies spread test. Approved by two reviewers with zero unresolved comments. | |
| #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). | |
| #1223 tests: fix resource spread test | charmcraft | merged | Merged to fix the resource spread test, advancing progress toward issue #1193. The update modified two files (+44/-23 lines) and was approved by two reviewers. Related library and store test corrections are addressed in separate pull requests. | |
| #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). | |
| #2379 test(spread): per-base destructive mode tests | charmcraft | merged | Merged following approval by two reviewers. Added per-base destructive mode tests for spread and removed a redundant uv test. Changes were integrated despite mixed CI results. | |
| #4768 tests(spread): regression test for grammar and non-grammar lists | snapcraft | merged | Merged regression tests for grammar and non-grammar lists. Requires craft-application 2.6.2. Author merged despite unrelated spread test failures, confirming they do not impact the changes. Approved by reviewers. | |
| #1290 test(spread): clean up store tests | charmcraft | merged | Merged cleanup of spread store tests, removing 28 lines across four files. The underlying library test failure is fixed in PR #1292. Approved by two reviewers. | |
| #2763 fix(tests/spread): fix reactive spread tests | charmcraft | merged | Merged after resolving failing reactive spread tests. The changes split the test suite into legacy and resolute variants, switched to binary wheels to fix dependency resolution, and removed obsolete flags. Approved by reviewers and integrated. |