← Back to issue list

test(spread): fix python3.6 test

View original Github issue

Metadata

Project
rockcraft
Number
#1021
Type
pull request
State
merged
Author
tigarmo
Labels
Created
Updated
Closed

Current evaluation

Merged a fix for a broken spread test after python3.6 was removed from the deadsnakes PPA. The test now manually removes the python3 symlink to verify PARTS_PYTHON_INTERPRETER handling. Approved and merged after review.

Suggested action:

No scores available.

Issue body

The actual purpose of the test is to check that a rock that is built using the Python plugin but without a 'python3' symlink works correctly if PARTS_PYTHON_INTERPRETER is correctly set. The test checked this by using python3.6 from the deadsnakes ppa, whose dependency chain did *not* include 'python3-minimal' (I think), which is the package that provides 'python3'. Since python3.6 is gone from deadsnakes, update the test to be more explicit about the intention by using the default Python interpreter for the base but removing 'python3' out "manually". - [ ] Have you followed the guidelines for contributing? - [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? - [ ] Have you successfully run `make lint && make test`? ---

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Merged a fix for a broken spread test after python3.6 was removed from the deadsnakes PPA. The test now manually removes the python3 symlink to verify PARTS_PYTHON_INTERPRETER handling. Approved and merged after review.
qwen3.6-35b-a3b-mtp-q6 Merged a test update replacing the discontinued deadsnakes python3.6 with the default Python interpreter. The test now manually removes the python3 symlink to explicitly verify PARTS_PYTHON_INTERPRETER behavior.
qwen3.6-35b-a3b-mtp-q6 Merged a fix for a spread test relying on the discontinued deadsnakes python3.6 package. The test now uses the default Python interpreter with the python3 symlink manually removed, preserving the original intent to validate PARTS_PYTHON_INTERPRETER behavior.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#6171 test(spread): update python tests snapcraft merged Merged update aligning core26 spread tests and documentation Python package ordering for consistency. Approved by reviewers and passed CI checks.
75%
#3670 spread: fix python-hello snapcraft merged Merged fix for python-hello spread tests broken by setuptools 61.1.0. Updated test configuration to comply with new package discovery rules, resolving metadata generation failures. Approved and passed CI.
72%
#1398 tests: fix issues with python 3.6 snapcraft merged Merged fix for Python 3.6 test compatibility. Approved by one reviewer, modified three files (+34/-7). Resolves Launchpad bug #1699009.
72%
#6159 test: fix core26 python tests snapcraft merged Merged update to core26 Python spread tests, switching to Python 3.14. The change modified four files, passed code review, and completed most CI checks.
72%