Skip to content

Commit

Permalink
Remove deprecated use of 'setup.py test' (#1275)
Browse files Browse the repository at this point in the history
Since setuptools v41.5.0 (27 Oct 2019), the 'test' command is formally
deprecated and should not be used. Now use unittest as the test entry
point.
  • Loading branch information
jdufresne committed May 8, 2020
1 parent 4041c56 commit 1382eab
Show file tree
Hide file tree
Showing 6 changed files with 4 additions and 5 deletions.
2 changes: 1 addition & 1 deletion .github/ISSUE_TEMPLATE/bug_report.md
Expand Up @@ -29,7 +29,7 @@ assignees: ""
- create a new virtualenv (make sure it's the same Python version);
- clone this repository;
- run `pip install -e .`;
- make sure it's sane by running `python setup.py test`; and
- make sure it's sane by running `python -m unittest`; and
- run `black` like you did last time.

**Additional context** Add any other context about the problem here.
2 changes: 1 addition & 1 deletion .github/workflows/test.yml
Expand Up @@ -27,4 +27,4 @@ jobs:
- name: Unit tests
run: |
coverage run tests/test_black.py
coverage run -m unittest
2 changes: 1 addition & 1 deletion .travis.yml
Expand Up @@ -4,7 +4,7 @@ cache:
directories:
- $HOME/.cache/pre-commit
env:
- TEST_CMD="coverage run tests/test_black.py"
- TEST_CMD="coverage run -m unittest"
install:
- pip install coverage coveralls pre-commit
- pip install -e '.[d]'
Expand Down
2 changes: 1 addition & 1 deletion CONTRIBUTING.md
Expand Up @@ -37,7 +37,7 @@ from the cloned _Black_ repo. It will do the correct thing.
Before submitting pull requests, run tests with:

```
$ python setup.py test
$ python -m unittest
```

## Hygiene
Expand Down
1 change: 0 additions & 1 deletion setup.py
Expand Up @@ -78,7 +78,6 @@ def get_long_description() -> str:
"mypy_extensions>=0.4.3",
],
extras_require={"d": ["aiohttp>=3.3.2", "aiohttp-cors"]},
test_suite="tests.test_black",
classifiers=[
"Development Status :: 4 - Beta",
"Environment :: Console",
Expand Down
Empty file added tests/__init__.py
Empty file.

0 comments on commit 1382eab

Please sign in to comment.