← Back to issue list

tests: remove plainbox part from plugin tests

View original Github issue

Metadata

Project
snapcraft
Number
#1889
Type
pull request
State
merged
Author
jocave
Labels
Created
Updated
Closed

Current evaluation

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.

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`? ----- Fixes for current failures of integration tests introduced by changes in external plainbox library. The plainbox library is being merged in to the checkbox application. The plainbox part should therefore no longer be needed in snaps.

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b 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.
qwen3.6-35b-a3b-mtp-q6 Merged removal of the plainbox part from plugin tests. The plainbox library was integrated into the checkbox application, making the plugin obsolete. This update resolves integration test failures caused by upstream library changes.
qwen3.6-35b-a3b-mtp-q6 Merged changes removing the plainbox part from plugin tests. This resolves integration test failures caused by external library updates, as plainbox is now integrated into the checkbox application and no longer needed in snaps.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#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.
77%
#2140 lp 1768233 workaround for plainbox_provider plugin snapcraft merged Merged a workaround for Launchpad bug 1768233 to fix plainbox-provider plugin failures blocking checkbox snap builds. Includes integration tests, passed CI with 100% diff coverage, and was approved before merging into master.
75%
#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.
74%
#609 Create Plainbox Provider plugin snapcraft merged Merged the Plainbox Provider plugin, adding integration tests for standard Checkbox workflows. Fixed a failing demo by adding a missing Python package dependency to the demo snap. Passed CI retests and was successfully integrated.
73%
#1642 tests: move the plainbox test to the integration suite snapcraft merged 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.
73%
#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.
72%
#1095 Plainbox providers run validate snapcraft merged Merged. Adds a build-time validation step to the plainbox-provider plugin to prevent buggy providers from packaging into checkbox snaps. Handles cross-provider dependencies and includes an integration test. Approved after minor test naming adjustments.
72%
#1740 tests: add the home plug to the plainbox snap snapcraft merged Merged changes adding the home plug to the plainbox snap. This resolves permission errors for tests writing to XDG_CACHE_HOME in $HOME/autopkgtest_tmp during autopkgtests. Approved by two reviewers and passed CI.
71%
#1931 No plainbox snapcraft merged Merged. Removed plainbox dependencies and local application components, replacing them with checkbox-cli and checkbox-ng. Static and unit tests passed, review was approved, and changes were integrated.
71%