Skip to content

Commit

Permalink
TEST: Remove dependency on setup.py test from surround cli
Browse files Browse the repository at this point in the history
  • Loading branch information
AkshatBajaj committed Dec 16, 2020
1 parent e811736 commit 58f5da3
Show file tree
Hide file tree
Showing 4 changed files with 21 additions and 8 deletions.
16 changes: 9 additions & 7 deletions codefresh.yml
Original file line number Diff line number Diff line change
Expand Up @@ -50,15 +50,14 @@ steps:
# Go inside the cli
- cd surround_cli

# Install surround
- python3 setup.py install
# Remove generated tox directory
- rm -r .tox

# Run tests
- python3 setup.py test
# Install tox
- pip install tox==3.20.1 pylint==2.4.3

# Run pylint tests
- pylint setup.py
- find surround_cli/ -iname "*.py" | xargs pylint
# Run tests through tox
- tox

# Run examples
- cd ..
Expand Down Expand Up @@ -108,6 +107,9 @@ steps:
commands:
- cd surround_cli

# Remove generated files from last release
- rm -r dist

# Install required packages
- python3 -m pip install --user --upgrade setuptools wheel twine

Expand Down
1 change: 0 additions & 1 deletion surround/setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,6 @@
packages=[
'surround'
],
test_suite='surround.tests',
license="BSD-3-Clause License",
zip_safe=False,
install_requires=INSTALL_REQUIRES)
1 change: 1 addition & 0 deletions surround_cli/MANIFEST.in
Original file line number Diff line number Diff line change
@@ -1,4 +1,5 @@
include VERSION
include requirements.txt
include surround/visualise/*.html.txt
include surround/experiment/web/*.html
include surround/experiment/web/*.ico
Expand Down
11 changes: 11 additions & 0 deletions surround_cli/tox.ini
Original file line number Diff line number Diff line change
@@ -0,0 +1,11 @@
[tox]
envlist = py36, py38
skip_missing_interpreters = True

[testenv]

# run the tests
commands =
python -m unittest discover -p "*_test.py"
pylint setup.py
pylint surround

0 comments on commit 58f5da3

Please sign in to comment.