← Back to issue list

list-plugins in one column for integration tests

View original Github issue

Metadata

Project
snapcraft
Number
#606
Type
pull request
State
closed
Author
sergiusens
Labels
Created
Updated
Closed

Current evaluation

Closed without merging. The change failed CI due to multi-column output discrepancies. It was superseded by PR #607, which replaces the formatting approach with multiple assertIn checks to verify plugin presence.

Suggested action:

No scores available.

Issue body

This minimizes discrepancies in expectations depending on where this test is run. LP: #1596114 Signed-off-by: Sergio Schvezov sergio.schvezov@ubuntu.com

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Closed without merging. The change failed CI due to multi-column output discrepancies. It was superseded by PR #607, which replaces the formatting approach with multiple assertIn checks to verify plugin presence.
qwen3.6-35b-a3b-mtp-q6 Superseded by PR #607. The single-column formatting attempt failed in CI. Reviewers recommended using multiple assertIn checks instead, so a new branch was proposed to resolve the test discrepancy without relying on output formatting.
qwen3.6-35b-a3b-mtp-q6 Superseded by pull request #607. The single-column formatting attempt failed integration tests due to multi-column output. Reviewers recommended using multiple assertIn checks, prompting an alternative branch that replaced this closed pull request.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#2315 tests: use mocked plugins for list-plugins snapcraft merged Merged without review after being extracted from PR #2308. Updates tests to use mocked plugins for the list-plugins command.
70%