Skip to content

AppTest: the example report is in 'report/' inside its artifact directory - #300

Merged
Paebbels merged 1 commit into
devfrom
claude/apptest-report-path
Oct 11, 2026
Merged

Paebbels merged 1 commit into
devfrom
claude/apptest-report-path

Conversation

@pytooling-claude

Copy link
Copy Markdown

Unit Tests

  • DownloadedArtifacts.test_ExampleReport (ApplicationTesting: download artifacts the application tests read #296) reads artifacts/Example-Report/report/Example.txt.
    • The job ExampleReport uploads report/Example.txt; pyTooling/upload-artifact keeps the file's path (it doesn't
      strip a common prefix like actions/upload-artifact), so the file is in report/ inside the artifact's directory.
    • The test expected artifacts/Example-Report/Example.txt. Until pyTooling/download-artifact v1.11.0, it failed for
      another reason first: a single matching artifact was put directly into artifacts/.

Tests

  • With pyTooling/download-artifact v1.11.0 (@v8), the field test of dev (Python 3.15.0, PyPy 8.0) failed only in
    this test: Moving artifact(s) to 'artifacts/Example-Report' ... [OK], extracted Directories: 1, Files: 1.

Related Issues and Pull-Requests

…tory.

pyTooling/upload-artifact keeps the uploaded file's path, 'report/Example.txt'.

Co-Authored-By: Patrick Lehmann <Paebbels@gmail.com>
@pytooling-claude
pytooling-claude Bot requested a review from Paebbels as a code owner October 11, 2026 13:08
@Paebbels
Paebbels merged commit c13a6e3 into dev Oct 11, 2026
71 of 132 checks passed
@codacy-production

Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

🟢 Metrics 0 complexity · 0 duplication

Metric Results
Complexity 0
Duplication 0

View in Codacy

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

@Paebbels
Paebbels deleted the claude/apptest-report-path branch October 11, 2026 13:09
This was referenced Oct 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants