Skip to content

Conversation

@JasonXuDeveloper
Copy link
Owner

Summary

  • Fix test artifact paths that were causing "No files found" warnings
  • Explicitly set artifactsPath for game-ci/unity-test-runner
  • Add if-no-files-found: warn to gracefully handle missing artifacts

Problem

The workflow was looking for artifacts at:

  • UnityProject/artifacts/PlayMode
  • UnityProject/artifacts/EditMode

But game-ci/unity-test-runner outputs to the repo root by default, not inside the project directory.

Solution

  • Explicitly set artifactsPath: artifacts/EditMode and artifactsPath: artifacts/PlayMode in the test runner steps
  • Update upload paths to match: artifacts/EditMode and artifacts/PlayMode
  • Add if-no-files-found: warn so missing artifacts show a warning instead of failing

Test plan

  • Run the release workflow again after merging
  • Verify artifacts are uploaded correctly
  • Check that test results appear in the workflow summary

🤖 Generated with Claude Code

- Explicitly set artifactsPath for game-ci/unity-test-runner
- Fix upload paths to match where artifacts are actually created
- Add if-no-files-found: warn to prevent failures on missing artifacts

The test runner outputs to repo root artifacts/ folder, not inside
the project directory.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Signed-off-by: JasonXuDeveloper - 傑 <jason@xgamedev.net>
@JasonXuDeveloper JasonXuDeveloper enabled auto-merge (squash) January 25, 2026 04:58
@claude
Copy link

claude bot commented Jan 25, 2026

Code review

No issues found. Checked for bugs and CLAUDE.md compliance.

@JasonXuDeveloper JasonXuDeveloper merged commit 9685bfd into master Jan 25, 2026
11 checks passed
@JasonXuDeveloper JasonXuDeveloper deleted the fix/unity-tests-artifact-path branch January 25, 2026 05:01
@github-actions
Copy link

Unity Test Results

EditMode: All tests passed
PlayMode: All tests passed

Unity Version: 2022.3.55f1
Project Path: UnityProject

✅ All tests passed! The PR is ready for review.

View workflow run

Click here to view the full workflow run

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