Skip to content

Fix tests in trunk #6315

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
montplaisir wants to merge 5 commits into
base: develop
Choose a base branch
from
+4 −12
Open

Fix tests in trunk #6315

Changes from all commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
b74e702
ix standalone player build failure with IL2CPP (#6313)
Codengineer Jul 21, 2026
3d1dd85
Use trunk for Yamato tests
montplaisir Jul 22, 2026
12d4de7
use upm-pvp
montplaisir Jul 23, 2026
d702a31
Rebase on develop
montplaisir Jul 23, 2026
3ab1c83
Ensure DevProject tests are run
montplaisir Jul 23, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
16 changes: 4 additions & 12 deletions .yamato/com.unity.ml-agents-test.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,10 +7,8 @@ test_editors:
testProject: DevProject
enableNoDefaultPackages: !!bool true

# Pinning to 6000.5 instead of `trunk`.
# Bump back to `trunk` once the CI image's pinned UTR understands UTF-1.8.
trunk_editor:
- version: 6000.5
- version: trunk
testProject: DevProject

test_platforms:
Expand Down Expand Up @@ -115,17 +113,11 @@ test_{{ package.name }}_{{ platform.name }}_{{ editor.version}}_{{ editor.testPr
name : {{ package.name }} test {{ editor.version }} on {{ platform.name }} in {{ editor.testProject }}
agent:
type: {{ platform.type }}
image: {{ platform.image }}
image: {% if platform.name == "linux" %}package-ci/ubuntu-22.04:v4{% else %}{{ platform.image }}{% endif %}
flavor: {{ platform.flavor}}
commands:
- |
{% if platform.name == "linux" %}
conda activate python3.10
{% endif %}
python -m pip install unity-downloader-cli --index-url https://artifactory.prd.it.unity3d.com/artifactory/api/pypi/pypi/simple --upgrade
unity-downloader-cli -u {{ editor.version }} -c editor --wait --fast
npm install upm-ci-utils@stable -g --registry https://artifactory.prd.cds.internal.unity3d.com/artifactory/api/npm/upm-npm
upm-ci project test -u {{ editor.version }} --project-path {{ editor.testProject }} --package-filter {{ package.name }} --extra-create-project-arg="-upmNoDefaultPackages" --extra-utr-arg "reruncount=2"
- unity-downloader-cli -u {{ editor.version }} -c editor --wait --fast
- UnifiedTestRunner --suite=editor --suite=playmode --editor-location=.Editor --testproject={{ editor.testProject }} --artifacts-path=upm-ci~/test-results --reruncount=2

@u-pr u-pr Bot Jul 23, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1

This job now calls UnifiedTestRunner directly, but unlike every other active direct UTR invocation in this repo it does not bootstrap that tool first. The Wrench-generated jobs all curl/extract wrench-localapv.zip before running UnifiedTestRunner, and the older non-Wrench example fetched utr explicitly; this template now has neither step after removing upm-ci. On a clean agent that means trunk can fail with UnifiedTestRunner: command not found before any tests run. Please either add the same bootstrap/install step used by the existing UTR jobs, or keep using the wrapper that provisions UTR for you.

🤖 Helpful? 👍/👎

artifacts:
logs:
paths:
Expand Down
Loading