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

Bring back tests for iOS, update to dotnet 6+, ensure works on device/simulator #8099

Closed
wants to merge 68 commits into from

Conversation

Mindfulplays
Copy link
Contributor

Make iOS interactive tests work

  • Update to .net6.0+ (for now using .net7.0 until .net8.0 is widely deployed elsewhere)
  • Ensure code compiles (Color type collision; newer iOS type names that needed updating; remove Exit but still maintain functionality by recreating the main page) etc

Two existing interactive tests made to work (one test is old uses keyboard input - which does not work any longer).

Will add more tests to this project.
This might be a good framework to add more mobile tests centrally in the future? (Android/iOS/WASM targets?)

tomspilman and others added 30 commits August 10, 2020 07:39
In other `Platform` classes (such as `SDLGamePlatform` and `AndroidGamePlatform`, `Threading.Run()` is called during `Tick()` or `RunLoop()` (inconsistently, it may be called before or after `Game.Tick()` unfortunately).

Fixing this for `iOSGamePlatform`. `WinForms` and perhaps other platforms (Switch?) may suffer from the same issue.

Note that texture loading from background thread is discouraged however a lot of computationally intensive operations (reading/decoding the image file streams, allocating CPU buffers) could be done in the background thread and hence is a good practice to move such work alone into the bgthread until we request the actual texture from the main thread.
 - Add/update to netx.0-ios (for now using .net 6 as .net 8 just got released)
   - Using a base Sample project created from iOS VS template
 - Remove non-working/incomplete stuff (TextureColorScaling)
mindfulplay.build added 3 commits November 27, 2023 16:54
 - Update to .net6.0+ (for now using .net7.0 until .net8.0 is widely deployed elsewhere)
 - Ensure code compiles (`Color` type collision; newer iOS type names that needed updating; remove `Exit` but still maintain functionality by recreating the main page) etc

 Two existing interactive tests made to work (one test is old uses keyboard input - which does not work any longer).

 Will add more tests here.
 This might be a good framework to add more mobile tests centrally in the future? (Android/iOS/WASM targets?)
@Mindfulplays
Copy link
Contributor Author

@SimonDarksideJ @mrhelmut @harry-cpp Please take a look.
I am bringing back/updating iOS tests - I am generally interested in a path towards iOS Metal in the future for which tests are paramount. I will send a feature request with my plan for iOS Metal next.

Mindfulplays and others added 9 commits November 30, 2023 15:56
 - Update to .net6.0+ (for now using .net7.0 until .net8.0 is widely deployed elsewhere)
 - Ensure code compiles (`Color` type collision; newer iOS type names that needed updating; remove `Exit` but still maintain functionality by recreating the main page) etc

 Two existing interactive tests made to work (one test is old uses keyboard input - which does not work any longer).

 Will add more tests here.
 This might be a good framework to add more mobile tests centrally in the future? (Android/iOS/WASM targets?)
…g for `isHighFrequency`.

 - Also consistently use `high-frequency` instead of `coalesced` when referring to this new behavior.
@Mindfulplays
Copy link
Contributor Author

Note to self: Once #8092 is in, ensure new interactive high-frequency touch event test is added.

@Mindfulplays
Copy link
Contributor Author

Note to self: Once #8092 is in, ensure new interactive high-frequency touch event test is added.

This is now addressed, will wait to commit after #8092 is in.

@Mindfulplays
Copy link
Contributor Author

@tomspilman @harry-cpp if you don't mind reviewing this, thanks!
I realize there is a plan to run this in the lab, but hopefully a review will help catch any fundamental issues ahead of time.

@Mindfulplays
Copy link
Contributor Author

@SimonDarksideJ too, as they were actually planning on adding the test runs in the lab (excited!)

@Mindfulplays
Copy link
Contributor Author

Superceded by #8110
This makes the interactive tests generic and runnable on Desktop (Mac/Windows) and on iOS (Android coming soon)

@Mindfulplays Mindfulplays deleted the ios-metal branch December 28, 2023 22:42
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.

None yet

3 participants