chore: unity cli testing setup - #2780
Conversation
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
![sentry[bot]](https://avatars.githubusercontent.com/in/12637?s=60&v=4){kind=small}
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 2 potential issues.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit c765751. Configure here.
| $baseUrl = "$($conf.repo)/releases/download/$($conf.version)/sentry-cli-" | ||
| $targetFiles = $platforms | ForEach-Object { | ||
| $name = if ($_.StartsWith('Windows')) { "$_.exe" } else { $_ } | ||
| Join-Path $targetDir "sentry-cli-$name" |
There was a problem hiding this comment.
Windows CLI skip check broken
Medium Severity
The Windows sentry-cli cache check builds expected filenames with "$_.exe", which PowerShell reads as a property on $_, not the platform name plus .exe. The path never matches sentry-cli-Windows-x86_64.exe, so the script treats a complete download as incomplete and re-downloads on every run on Windows.
Reviewed by Cursor Bugbot for commit c765751. Configure here.
| iOSRenderExtraFrameOnPause: 0 | ||
| iosCopyPluginsCodeInsteadOfSymlink: 0 | ||
| appleDeveloperTeamID: | ||
| appleDeveloperTeamID: 97JCY7859U |
There was a problem hiding this comment.
Committed Apple team identifier
Low Severity
This change sets a concrete appleDeveloperTeamID in the shared Unity 2021 sample while the PR moves team setup to APPLE_ID via samples-setup-apple-id.ps1. That pins one developer team in repo history and can confuse iOS signing for other contributors.
Reviewed by Cursor Bugbot for commit c765751. Configure here.
2 participants
This enables and utilizes the new Unity CLI to run the unit tests. Updated the existing script to do basic error handling and logging along with parsing the results.
#skip-changelog