tests: move the plainbox test to the integration suite
Metadata
Current evaluation
Merged changes to move the plainbox test to the integration suite. Updated expected namespace results to remove outdated year prefixes, fixing a test failure caused by upstream plainbox changes. Approved and passed CI.
Suggested action: —
No scores available.
Issue body
- [x] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)?
- [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [x] If this is a bugfix. Have you checked that there is a bug report open for the issue you are trying to fix on [bug reports](https://bugs.launchpad.net/snapcraft)?
- [x] If this is a new feature. Have you discussed the design on the [forum](https://forum.snapcraft.io)?
- [x] Have you successfully run `./runtests.sh static`?
- [x] Have you successfully run `./runtests.sh unit`?
-----
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged changes to move the plainbox test to the integration suite. Updated expected namespace results to remove outdated year prefixes, fixing a test failure caused by upstream plainbox changes. Approved and passed CI. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged after moving the plainbox test to the integration suite. A test failure from plainbox dropping year prefixes was fixed by updating expected outputs to remove the 2013 prefix. Tests passed and the change was merged. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged after relocating the plainbox test to the integration suite. Fixed a failing test by removing the '2013.' year prefix from expected namespace strings to align with upstream plainbox changes. Additional year prefixes were cleaned up across test files before merge. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #2799 Plainbox test updates | snapcraft | merged | Merged plainbox test updates involving minor changes across three files. Approved by one reviewer with zero unresolved comments and no CI failures. | |
| #1608 tests: fix the duplicate plainbox test scenarios | snapcraft | merged | Merged a fix resolving duplicate plainbox test scenarios. The update modified a single test file with five additions and two deletions, successfully clearing static and unit test checks without requiring reviewer approval. | |
| #1889 tests: remove plainbox part from plugin tests | snapcraft | merged | Merged changes removing the plainbox component from plugin tests. This resolves integration test failures caused by the plainbox library being integrated into the checkbox application, rendering the separate plainbox part obsolete for snaps. | |
| #2317 tests: add spread suite for plainbox plugin | snapcraft | merged | Merged after approval and passing CI checks. Introduces a Spread test suite for plainbox plugin integration tests and pins the checkbox-ng package in the test environment. | |
| #166 Translated the plainbox tests to python. | snapcraft | closed | Merged after passing CI. Translated plainbox tests to Python, preserving original behavior and temporarily ignoring exit codes. Reviewer approved closure following successful builds. | |
| #2321 tests: add spread suite for plainbox plugin (#2317) | snapcraft | merged | Merged after approval and passing CI. The pull request adds a Spread test suite for the plainbox plugin, pins the checkbox-ng package, and removes a misplaced test from the snapcraft test suite. | |
| #2583 plainbox spread tests: use https for the git source entries | snapcraft | merged | Merged a one-line change updating plainbox spread tests to use HTTPS for git source entries. Approved by two reviewers and passed CI checks. | |
| #2426 tests: use fixed version for idna in plainbox | snapcraft | merged | Merged into main. Updates test dependencies to pin a fixed idna package version in plainbox. Authored by maintainer sergiusens, the change modifies five files with seven added lines and was successfully integrated. | |
| #2428 tests: increase test timeout for plainbox | snapcraft | merged | Merged into master. The pull request increases the test timeout for plainbox, modifying a single configuration value with a negligible coverage decrease of less than 0.01%. | |
| #60 Readded the plainbox examples suite to travis config. | snapcraft | merged | Merged re-adding the plainbox examples suite to Travis CI config. Despite initial comments about failing tests and missing dependencies, the author verified build times and proceeded with the merge. |