Search Issue Tracker
Fixed in 1.0.16
Votes
0
Found in [Package]
1.0.14
Issue ID
1126050
Regression
No
Failure in IPreBuildSetup/IPostBuildCleanup results in "PASS" status in UTR
Steps to reproduce:
1) Open the attached project.
2) Open Test Runner window.
3) Run all tests.
- Exception is thrown in the console during IPreBuildSetup; tests are not executed.
Expected behavior: the fact that tests were not run should be reflected in the test runner window.
Actual behavior: the Test Runner window doesn't show the failure to run the tests.
While the behavior described above is somewhat reasonable (tests were not run, so their status is unknown and therefore not set), this issue is more apparent/important when running the tests through UTR console.
When the tests are run through UTR, the final outcome is "PASS", even though the tests were not run at all.
This issue happens because the method that is called after failed IPrebuildSetup/IPostBuildCleanup:
Extensions/TestRunner/UnityEditor.TestRunner/UnityTestProtocol/UnityTestProtocolListener:: RunFinished()
Is empty - implementing this method does set the final status to "FAILED", sample here:
https://ono.unity3d.com/unity/unity/changeset/024dc9d986d1
However, this method only gets a fake test with its result hardcoded to "FAIL" (see CallbacksDelegator::RunFailed() where this fake test is created), when it should have the actual targeted test information.
All about bugs
View bugs we have successfully reproduced, and vote for the bugs you want to see fixed most urgently.
Latest issues
- Mono Windows Builds don't produce full log callstacks when generating logs
- AssetBundles fail to load when running in Built Players for Mobile Devices
- UI elements with text gets bigger and grey when Player window is moved to another screen with different resolution
- System name accepts multiline text but crops it on confirmation, duplicates input, and shrinks the field when empty
- UI element scale and position are wrong in project build when DRS is changed with HDR and Software Dynamic Resolution enabled
Add comment