tests: do not break a systems `bzr whoami`
Metadata
Current evaluation
Merged a fix preventing tests from modifying the system's bzr whoami configuration. Changes span 2 files with 14 additions and 23 deletions. Approved by one reviewer and resolved LP #1693839.
Suggested action: —
No scores available.
Issue body
When running tests we shouldn't be affect an existing bzr configuration.
LP: #1693839
Signed-off-by: Sergio Schvezov <sergio.schvezov@canonical.com>
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged a fix preventing tests from modifying the system's bzr whoami configuration. Changes span 2 files with 14 additions and 23 deletions. Approved by one reviewer and resolved LP #1693839. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged a fix to prevent tests from modifying the system's bzr whoami configuration. The change isolates test execution from existing bzr settings, resolving launchpad issue #1693839. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged a fix preventing tests from modifying the system bzr configuration. The change ensures bzr whoami settings remain untouched during test execution, resolving LP: #1693839. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #1693839 unit tests that use bzr affect whoami | snapcraft (launchpad) | closed | Closed without comments or tracked resolution. The bzr test isolation bug was abandoned or resolved internally, leaving the global whoami side effect unaddressed. | |
| #2713 tests: completely mock bzr tests | snapcraft | merged | Merged changes completely mocking bzr tests to improve execution speed. Approved by a reviewer, passed CI checks, and modified two files (+68/-20). Integration tests remain for validation. | |
| #1128 tests: support bzr branches for external tests | snapcraft | merged | Merged update adding bzr branch support to external tests. Approved by two reviewers, it modifies one file (+21/-2 lines) to resolve Launchpad bug #1663294. |