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

[Proposal] Add ability to run tests per extension #752

Closed
1 of 2 tasks
jsmith-bdai opened this issue Jul 29, 2024 · 0 comments · Fixed by #753
Closed
1 of 2 tasks

[Proposal] Add ability to run tests per extension #752

jsmith-bdai opened this issue Jul 29, 2024 · 0 comments · Fixed by #753
Assignees
Labels
enhancement New feature or request

Comments

@jsmith-bdai
Copy link
Collaborator

jsmith-bdai commented Jul 29, 2024

Proposal

Add a cli arg to run only tests for a selected extension.

Example:

isaaclab -t --extension omni.isaac.lab_assets # Only runs tests under omni.isaac.lab_assets

Motivation

When building out a new extension, sometimes a user only wants to specifically test that extension as the full test suite can take long.

Additional context

Add any other context or screenshots about the feature request here.

Checklist

  • I have checked that there is no similar issue in the repo (required)

Acceptance Criteria

Add the criteria for which this task is considered done. If not known at issue creation time, you can add this once the issue is assigned.

  • Tests can be run per extensions, specified via CLI arg to isaaclab -t
@jsmith-bdai jsmith-bdai self-assigned this Jul 29, 2024
@jsmith-bdai jsmith-bdai added the enhancement New feature or request label Jul 29, 2024
jsmith-bdai added a commit that referenced this issue Aug 2, 2024
…#753)

# Description

Add a cli argument `--extension` to run only unit tests for a selected
extension. As an example:

```bash
isaaclab -t --extension omni.isaac.lab_assets # Only runs tests under omni.isaac.lab_assets
```

Fixes #752 

## Type of change

- New feature (non-breaking change which adds functionality)

## Checklist

- [x] I have run the [`pre-commit` checks](https://pre-commit.com/) with
`./isaaclab.sh --format`
- [x] 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
iamdrfly pushed a commit to iamdrfly/IsaacLab that referenced this issue Nov 21, 2024
…isaac-sim#753)

# Description

Add a cli argument `--extension` to run only unit tests for a selected
extension. As an example:

```bash
isaaclab -t --extension omni.isaac.lab_assets # Only runs tests under omni.isaac.lab_assets
```

Fixes isaac-sim#752 

## Type of change

- New feature (non-breaking change which adds functionality)

## Checklist

- [x] I have run the [`pre-commit` checks](https://pre-commit.com/) with
`./isaaclab.sh --format`
- [x] 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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant