You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
gazebosim/sdformat#796 (review) points out that USD has a known bug (PixarAnimationStudios/OpenUSD#86) where fatal errors do not abort cleanly. This could be an issue in unit tests if an error occurs mid-test from something like trying to access an invalid stage - USD will "gracefully exit" without signifying that the stage was invalid, which could give a false positive of tests succeeding.
Steps to reproduce
The following program can be tested (you need to install USD). You'll notice that the program exits cleanly (exit code 0 instead of 1) without throwing an error:
It should be worth knowing that in the unit tests that are being written for the USD component, we are not intentionally doing things like trying to dereference an invalid stage pointer. However, in order to protect ourselves from bad exits we are unaware of and false positive test results, we should address this issue to make sure that all unit tests run to completion. gazebosim/sdformat#796 (review) lists a few ideas for solving this, and in particular, I think the third idea given could work: "make a custom test runner which either captures and check stdout of the tests, or have each test write some artifacts and have the runner check them at the end."
The text was updated successfully, but these errors were encountered:
Description
gazebosim/sdformat#796 (review) points out that USD has a known bug (PixarAnimationStudios/OpenUSD#86) where fatal errors do not abort cleanly. This could be an issue in unit tests if an error occurs mid-test from something like trying to access an invalid stage - USD will "gracefully exit" without signifying that the stage was invalid, which could give a false positive of tests succeeding.
Steps to reproduce
The following program can be tested (you need to install USD). You'll notice that the program exits cleanly (exit code 0 instead of 1) without throwing an error:
It should be worth knowing that in the unit tests that are being written for the USD component, we are not intentionally doing things like trying to dereference an invalid stage pointer. However, in order to protect ourselves from bad exits we are unaware of and false positive test results, we should address this issue to make sure that all unit tests run to completion. gazebosim/sdformat#796 (review) lists a few ideas for solving this, and in particular, I think the third idea given could work: "make a custom test runner which either captures and check stdout of the tests, or have each test write some artifacts and have the runner check them at the end."
The text was updated successfully, but these errors were encountered: