Skip to content
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.

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

ci: Change Unity UnitTest from Mono to IL2CPP and enabled on CI #603

Merged
merged 15 commits into from
Jul 30, 2024

Conversation

guitarrapc
Copy link
Contributor

@guitarrapc guitarrapc commented Jul 24, 2024

tl;dr;

Previously Unity's Unit Test was commented out due to Unity Healess (DedicatedServer) API change was unstable. I've confirm it's stable now and re-enable on GitHub Actions.

To Run UnitTest IL2CPP.

  • Bump Unity version from 2020.2.1f1 to 2022.3.39f1. (LTS)
  • Change Unity versions running on matrix to latest LTS.
  • Re-enable Unity UnitTest on GitHub Actions
  • Change Unity UnitTest from Mono to IL2CPP
  • Change RuntimeUnitTestToolkit from UnityPackage to UPM
  • Bump RuntimeUnitTestToolkit from 2.3.0 to 2.6.0

For CI re-run usability improvement.

  • Split Unity UnitTest from build, as IL2CPP build takes over 13min (3min for Mono)

@timcassell
Copy link

FYI, unity-test-runner supports standalone test mode now for a while. If you want to, you could just use that instead of doing separate build + test steps. https://game.ci/docs/github/test-runner#testmode

@guitarrapc guitarrapc changed the title ci: Change Unity Unit Test from Mono to IL2CPP ci: Change Unity Unit Test from Mono to IL2CPP and Enable UnitTest on CI Jul 30, 2024
@guitarrapc guitarrapc changed the title ci: Change Unity Unit Test from Mono to IL2CPP and Enable UnitTest on CI ci: Change Unity UnitTest from Mono to IL2CPP and Enable UnitTest on CI Jul 30, 2024
@guitarrapc guitarrapc changed the title ci: Change Unity UnitTest from Mono to IL2CPP and Enable UnitTest on CI ci: Change Unity UnitTest from Mono to IL2CPP and enabled on CI Jul 30, 2024
@guitarrapc
Copy link
Contributor Author

guitarrapc commented Jul 30, 2024

@timcassell Thank you for interesting information. Currently Cysharp is using it's own package RuntimeUnitTestToolkit and run it as binary for Dog-fooding. I've added test-runner to watchlist.

@guitarrapc guitarrapc self-assigned this Jul 30, 2024
@guitarrapc guitarrapc requested a review from neuecc July 30, 2024 06:25
@guitarrapc guitarrapc merged commit 1827be2 into master Jul 30, 2024
8 checks passed
@guitarrapc guitarrapc deleted the feature/test branch July 30, 2024 06:41
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