← Back to issue list

fix(tests/gradle): explicitly use build output directory

View original Github issue

Metadata

Project
craft-parts
Number
#1320
Type
pull request
State
merged
Author
vpa1977
Labels
—
Created
Updated
Closed

Current evaluation

Merged as commit f7719cb6. PR #1320 changes Gradle integration tests to write test.txt to the Gradle build output dir instead of user.dir, fixing s390x/ppc64el failures in #1304.

Resolution: closed resolved

Reason: PR #1320 was approved by two reviewers with all CI checks passing and merged as commit f7719cb6; the init.gradle testWrite task now writes to the build directory and the test asserts on parts/<part>/build/build/test.txt, closing issue #1304.

Evaluated by: mr-cal-bot

No scores available.

Issue body

Instead of user.dir which may be reset by the Gradle daemon use gradle build output directory to write the test file Fixes #1304 - [x] Have you followed the guidelines for contributing? - [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? - [x] Have you successfully run `make lint && make test`? - [ ] Have you added an entry to the changelog (`docs/reference/changelog.rst`)? ---

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged as commit f7719cb6. PR #1320 changes Gradle integration tests to write test.txt to the Gradle build output dir instead of user.dir, fixing s390x/ppc64el failures in #1304.
stealth/space-bunny-alpha — closed resolved PR #1320 merged as f7719cb6: Gradle tests now write the test file to Gradle's build output directory instead of user.dir, fixing architecture-dependent failures (s390x/ppc64el) reported in #1304.
stealth/space-bunny-alpha — closed resolved Merged as f7719cb6 (PR #1320): Gradle tests now write to the build output directory instead of user.dir, fixing #1304.
qwen/qwen3.6-35b-a3b — — Merged a fix for Gradle tests using the build output directory instead of user.dir to prevent daemon resets from breaking test writes. Resolves #1304. Approved by two reviewers and passed all CI checks.
qwen3.6-35b-a3b-mtp-q6 — — Merged a fix for Gradle tests that explicitly uses the build output directory instead of user.dir, preventing failures when the Gradle daemon resets the working directory. Resolves #1304.

Update history

No update history recorded yet.

Related work

  • Likely Fixed By: canonical/craft-parts#1304 (confidence 95%)

    Issue #1304 (Gradle integration tests failing on s390x/ppc64el) was closed by this PR, cited in its body as 'Fixes #1304'

Related issues

Issue Project State Summary Similarity
#1304 Gradle integration tests are failing on s390x and ppc64el craft-parts closed Gradle tests now write test.txt to Gradle's build directory, avoiding architecture-dependent paths; fixed by merged PR #1320 (f7719cb6).
77%