test: substring-rejection tests for unlink, queue, assets schema (#236)

[Zious11]

Summary

Extends handler-level partial_match single-substring rejection coverage to three more call sites from #236:

• issue unlink (binary test) — jr unlink --type block → exit 64, Ambiguous link type, no DELETE fires
• queue resolve (library test on resolve_queue_by_name) — complements existing ExactMultiple test; asserts JrError::UserError via downcast
• assets schema --schema (binary test) — jr assets schema AnyType --schema its → exit 64, Ambiguous schema, no object-type listing fires

Each test uses a genuine single-hit substring (block, escal, its) against a candidate list where only one entry substring-matches, so the test exercises the #193 regression surface directly (single-hit substrings that used to resolve silently now route through Ambiguous).

Scope

This PR covers 3 of the 8 call sites listed in #236. The remaining 5 are deferred to #240:

• helpers.rs team/user/asset resolution — uses bail! (exit 1) and needs a parallel refactor similar to refactor: unify exit codes for ambiguous input — move/link currently exit 1, list exits 64 #237
• assets search --status — needs AQL + ticket-enrichment plumbing
• assets schema <type_name> object-type match — needs schemas + objecttypes/flat mocks

Design choices

• Queue test is library-level, not binary. The full binary path for jr queue view requires project-meta + service-desk discovery + queue list mocks (3 chained mocks). The test precedent at queue.rs:190 (resolve_queue_duplicate_names_error_message) is library-level; this PR mirrors that pattern for the Ambiguous variant.
• Unlink test mounts no DELETE mock — wiremock 404s unmatched requests, which would surface as a different error if a delete were attempted. Combined with the exit 64 + Ambiguous link type assertions this implicitly locks the short-circuit.
• Assets schema test uses .expect(0) on both objecttypes/flat endpoints — explicit short-circuit assertion.

Test plan

• cargo fmt --all -- --check clean
• cargo clippy --all-targets -- -D warnings clean
• cargo test — 806 passed, 0 failed (+3 new over baseline 803)
• Local code review: no critical/important findings

Refs #236

[Copilot]

Pull request overview

Expands regression coverage for the partial_match strict-matching behavior (single-substring inputs now routed through Ambiguous) across additional CLI/library call sites to prevent silent resolution under --no-input.

Changes:

• Add a library-level test ensuring queue name single-substring input yields JrError::UserError with the disambiguation message.
• Add a binary integration test ensuring issue unlink --type <substring> fails with exit code 64 and short-circuits before any DELETE.
• Add a binary integration test ensuring assets schema --schema <substring> fails with exit code 64 and short-circuits before any object-type listing.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File	Description
tests/queue.rs	Adds resolve_queue_by_name single-substring ambiguity test and asserts UserError + message content.
tests/issue_commands.rs	Adds jr issue unlink single-substring rejection test asserting exit 64 and no state-changing HTTP calls.
tests/assets.rs	Adds jr assets schema --schema single-substring rejection test asserting exit 64 and no object-type listing calls.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.