test: enable sourcecraft inspector tests
Metadata
Current evaluation
Merged changes enabling sourcecraft inspector tests disabled during refactoring. Approved by two reviewers, passed CI, and updated four files. Deb inspector tests require separate handling.
Suggested action: —
No scores available.
Issue body
- [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
Enable tests, that were accidentally disabled during the code refactoring
The same case is for the `deb` inspector, that should be addressed separately.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged changes enabling sourcecraft inspector tests disabled during refactoring. Approved by two reviewers, passed CI, and updated four files. Deb inspector tests require separate handling. |
Update history
| Date | Change |
|---|---|
| created |
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #219 docs: document sourcecraft inspector | fetch-service | merged | Merged documentation for the sourcecraft inspector. Approved by two reviewers, the changes were successfully integrated, resolving CRAFT-3337. | |
| #229 fix: remove inspecting text/html in sourcecraft inspector | fetch-service | merged | Merged to resolve issue #228 by removing text/html inspection in the sourcecraft inspector, preventing accidental processing of non-git requests. Approved by two reviewers and passed CI checks. | |
| #408 test: add git inspector spread test | fetch-service | merged | Merged a test suite addition featuring a git inspector smoke test. Approved by two reviewers, the change introduces 105 lines across three files to validate inspector functionality. | |
| #217 feat: add sourcecraft inspector | fetch-service | merged | Merged after two approvals. Introduces the sourcecraft inspector and resolves repository cloning issues with the fetch-service. Closes #211 and was integrated following #216. | |
| #626 feat: bincraft inspector | fetch-service | merged | Merged the bincraft inspector, adapted from Rockcraft with updated tests and a test data generation script. Approved by two reviewers, passed all CI checks, and added 625 lines across eight files. | |
| #200 docs: document the Maven inspectors | fetch-service | merged | Merged documentation update adding Maven inspector details across three files (+124/-0). Resolves issue #184. Approved by two reviewers. A minor failing unit test was noted but addressed separately, allowing the merge to proceed. |