Skip to content

Commit

Permalink
Fix test commands in READMEs & PR template (#106)
Browse files Browse the repository at this point in the history
Summary:
Pull Request resolved: #106

Some of the test commands were outdated in the READMEs & PR templates, from before we renamed all test files to end in _test.py instead of _tests.py.

Reviewed By: jbitton

Differential Revision: D30281710

fbshipit-source-id: afc02e47bf954b4151bbf9c8ca83299b339e75c1
  • Loading branch information
Zoe Papakipos authored and facebook-github-bot committed Aug 12, 2021
1 parent d27d8e9 commit 053a750
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 7 deletions.
2 changes: 1 addition & 1 deletion .github/pull-request-template.md
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ python -m unittest discover -s augly/tests/audio_tests/ -p "*"

### Image
```bash
python -m unittest discover -s augly/tests/image_tests/ -p "*_tests.py"
python -m unittest discover -s augly/tests/image_tests/ -p "*_test.py"
# Or `python -m unittest discover -s augly/tests/image_tests/ -p "*.py"` to run pytorch test too (must install `torchvision` to run)
```

Expand Down
4 changes: 2 additions & 2 deletions augly/image/README.md
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
# Image

## Installation
## Installation
If you would like to use the image augmentations, please install AugLy using the following command:
```bash
pip install augly
Expand Down Expand Up @@ -79,7 +79,7 @@ np_aug_img = aug_np_wrapper(np_image, overlay_emoji, **{'opacity': 0.5, 'y_pos':

You can run our normal image unit tests if you have cloned `augly` (see [here](../../README.md)) by running the following:
```bash
python -m unittest discover -s augly/tests/image_tests/ -p "*_tests.py"
python -m unittest discover -s augly/tests/image_tests/ -p "*_test.py"
```

Note: If you want to additionally run the pytorch unit test (you must have torchvision installed), you can run:
Expand Down
8 changes: 4 additions & 4 deletions augly/video/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -89,8 +89,8 @@ python -m unittest discover -s augly/tests/video_tests/ -p "*"

Note: some of the video tests take a while to run (up to a few minutes). If you want to run the 4 test suites individually, you can run any of the following commands (listed in order of increasing runtime):
```bash
python -m unittest augly.tests.video_tests.transforms.composite_tests
python -m unittest augly.tests.video_tests.transforms.cv2_tests
python -m unittest augly.tests.video_tests.transforms.ffmpeg_tests
python -m unittest augly.tests.video_tests.transforms.image_based_tests
python -m unittest augly.tests.video_tests.transforms.composite_test
python -m unittest augly.tests.video_tests.transforms.cv2_test
python -m unittest augly.tests.video_tests.transforms.ffmpeg_test
python -m unittest augly.tests.video_tests.transforms.image_based_test
```

0 comments on commit 053a750

Please sign in to comment.