MixedRealityToolkit-Unity icon indicating copy to clipboard operation
MixedRealityToolkit-Unity copied to clipboard

[Tests fixed!] Process DeviceCommands in simulated devices to ensure sync requests are correctly processed

Open Zee2 opened this issue 2 years ago • 2 comments

Overview

This PR will fix play-mode tests in batch mode (i.e., CI)!

image

  • Made SimulatedControllers implement ExecuteCommand to receive Sync commands and report success, fixing the focus-lost-device-reset bug
  • [x] Make SimulatedCamera implement the above ExecuteCommand method to report success on sync commands
  • Changed our test harness to use the InputTestFixture
  • Added com.unity.inputsystem to testables
  • Manually registered XRI custom interactions/fallbackcomposites that were not registered in the custom input backend that the TestFixture uses, which was preventing us from using the InputTestFixture
  • Fixed edge case in cursor magnetism script that was referencing colliders after they had been destroyed
  • [x] Find out why roughly half of the tests continue to fail despite the input devices + interactors working correctly

Breaking: All references to "Simulated Camera" replaced with Simulated HMD for accuracy.

Zee2 avatar Aug 09 '22 00:08 Zee2

/azp run

Zee2 avatar Aug 10 '22 19:08 Zee2

Azure Pipelines failed to run 1 pipeline(s).

azure-pipelines[bot] avatar Aug 10 '22 19:08 azure-pipelines[bot]