← Back to issue list

Simplify the handler from uri tests.

View original Github issue

Metadata

Project
snapcraft
Number
#853
Type
pull request
State
merged
Author
sergiusens
Labels
—
Created
Updated
Closed

Current evaluation

Merged PR #853 via commit d53bfbfd9, consolidating URI handler mocks and reorganizing tests into dedicated test case classes.

Resolution: closed resolved

Reason: PR #853 was approved and merged as commit d53bfbfd9. It minimized mock declarations to one location and moved the URI handler tests into proper test case classes.

Evaluated by: mr-cal-bot

No scores available.

Issue body

Minimize mock declarations to one location and move tests to proper test case classes. LP: #1630632 Signed-off-by: Sergio Schvezov sergio.schvezov@ubuntu.com

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged PR #853 via commit d53bfbfd9, consolidating URI handler mocks and reorganizing tests into dedicated test case classes.
stealth/space-bunny-alpha — closed resolved Merged PR #853 as commit d53bfbfd9, consolidating mock declarations and reorganizing handler URI tests into proper test case classes.
stealth/space-bunny-alpha — closed resolved Merged PR #853, simplifying handler URI tests by consolidating mock declarations and reorganizing tests into proper test case classes.
stealth/space-bunny-alpha — closed resolved Merged PR #853, consolidating URI handler mock declarations and reorganizing tests into proper test case classes (+14/-9).
qwen/qwen3.6-35b-a3b — — Merged after two approvals and CI. The update simplified handler URI tests by consolidating mock declarations and reorganizing them into proper test case classes, modifying one file with 14 additions and 9 deletions.
qwen3.6-35b-a3b-mtp-q6 — — Successfully merged. Consolidated mock declarations and reorganized handler URI tests into proper test case classes.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#852 Simplify the parser tests. snapcraft merged Merged PR #852 simplified parser test mocks to ease future refactoring; landed as commit 223743de5.
71%