Skip to content

build: leave the plugin tests out of the code package - #178

Merged
marc-n-dream merged 1 commit into
masterfrom
marc/remove-tests-from-package
Oct 12, 2026
Merged

marc-n-dream merged 1 commit into
masterfrom
marc/remove-tests-from-package

Conversation

@marc-n-dream

@marc-n-dream marc-n-dream commented Oct 9, 2026 •

Copy link
Copy Markdown
Collaborator

The code package no longer ships Assets/AirConsole/scripts/Tests. The tests change PlayerSettings, so they must not run in game projects.

  • Packager exports every folder under scripts/ except Tests, plus unity-webview and examples.
  • ProjectCodeUpdater deletes the Tests folder that 2.6.3 and earlier installed. The check finds that folder by AirConsole.EditMode.Tests.asmdef. Without it, a stale copy that compiles against internal members would break compilation after a later release.
  • CHANGELOG entry under Changed.

Testing (Unity 2022.3.62f3, batch mode, package exported from this branch):

Check scripts/Tests on disk AirConsole code assemblies Compile errors
Code package contents vs 2.6.3 10 Tests entries removed, nothing else – –
Fresh project, branch package no Editor, Examples, Runtime 0
Upgrade step 1: install 2.6.3 yes above + EditMode, Editor, PlayMode Tests 0
Upgrade step 2: branch package on top no Editor, Examples, Runtime 0

In every run, ProjectCodeUpdater finished and deleted itself and the code package. The test projects had com.unity.ugui 1.0.0, com.unity.textmeshpro 3.0.9 and com.unity.test-framework 1.4.4 added, because a project made with -createProject has no UI packages and the plugin does not compile without them (2.6.3 also).

The tests change PlayerSettings, so they must not run in game projects.
The upgrade deletes the Tests folder that 2.6.3 and earlier installed.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_015uK9hhtA5vxmCSveT2uQm3
@marc-n-dream
marc-n-dream merged commit 6b3b024 into master Oct 12, 2026
3 checks passed
@marc-n-dream
marc-n-dream deleted the marc/remove-tests-from-package branch October 12, 2026 14:52
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