← Back to issue list

tests: add package repository tests

View original Github issue

Metadata

Project
craft-archives
Number
#38
Type
pull request
State
merged
Author
lengau
Labels
—
Created
Updated
Closed

Current evaluation

Merged PR #38 as commit e49a3d9, adding package-repository tests ahead of pinning changes; Codecov reported a 1.41% coverage increase.

Resolution: closed resolved

Reason: PR #38 was approved and merged into main as commit e49a3d9. It added tests for previously uncovered package-repository behavior in preparation for pinning changes.

Evaluated by: mr-cal-bot

No scores available.

Issue body

- [x] Have you followed the guidelines for contributing? - [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? - [x] Have you successfully run `tox`? ----- Added these while I was adding pinning to package repositories. Not a direct requirement of the work item, but I'm going to change some things that weren't fully tested.

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged PR #38 as commit e49a3d9, adding package-repository tests ahead of pinning changes; Codecov reported a 1.41% coverage increase.
stealth/space-bunny-alpha — closed resolved Merged PR #38 added package-repository tests, guarding behavior affected by repository pinning work.
stealth/space-bunny-alpha — closed resolved Merged package-repository tests into main in #38 via commit e49a3d9, increasing reported coverage by 1.41% and supporting repository-pinning changes.
stealth/space-bunny-alpha — closed resolved Merged package repository tests into main via canonical/craft-archives#38 (commit e49a3d9), increasing reported code coverage by 1.41%.
qwen/qwen3.6-35b-a3b — — Added package repository tests to validate pinning functionality. Approved by a reviewer, passed CI checks, and merged into main. The change increased overall code coverage by 1.41 percent.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#19 tests: update test environments for coverage craft-archives merged Merged PR #19 as commit 584f098, updating four test-environment files; Codecov reported 100% diff coverage.
72%
#56 Add a fuller integration test, and multiple minor changes to support it craft-archives merged Merged PR #56 added fuller repo.install() integration coverage for supported repository types, plus minor supporting changes.
72%
#461 test: improve test coverage fetch-service merged Merged PR #461 via commit 76d6134, adding 270 lines of tests across three files; all CI checks passed and two reviewers approved.
70%
#278 tests: additional tests to improve coverage fetch-service merged Merged coverage-focused test additions via #278 (commit 095e41a), changing 10 files (+386/-60) with two approvals and no reported CI failures.
70%