• David Hoeller's avatar
    Fixes the unit test success criterion in the CI pipeline (#1251) · 91f760ee
    David Hoeller authored
    # Description
    
    - Fixes the condition for a test to report success in the
    `run_all_tests.py` script. Before, the test could crash and the script
    would still report a success. Now we have an explicit check to verify
    the test reports success.
    - Improved the tests involving environments. Before they could crash
    during initialization without any error message and interrupting the
    test. This is now caught and reported, the subtest failed, and the
    running the other subtests is resumed properly.
    
    ## Type of change
    
    - Bug fix (non-breaking change which fixes an issue)
    
    ## Checklist
    
    - [x] I have run the [`pre-commit` checks](https://pre-commit.com/) with
    `./isaaclab.sh --format`
    - [ ] I have made corresponding changes to the documentation
    - [x] My changes generate no new warnings
    - [ ] I have added tests that prove my fix is effective or that my
    feature works
    - [ ] I have updated the changelog and the corresponding version in the
    extension's `config/extension.toml` file
    - [x] I have added my name to the `CONTRIBUTORS.md` or my name already
    exists there
    91f760ee
Name
Last commit
Last update
..
app Loading commit data...
assets Loading commit data...
controllers Loading commit data...
deps Loading commit data...
devices Loading commit data...
envs Loading commit data...
managers Loading commit data...
markers Loading commit data...
performance Loading commit data...
scene Loading commit data...
sensors Loading commit data...
sim Loading commit data...
terrains Loading commit data...
utils Loading commit data...