Skip to content

Commit

Permalink
TEST: Remove dependency on setup.py test in surround
Browse files Browse the repository at this point in the history
  • Loading branch information
AkshatBajaj committed Dec 16, 2020
1 parent c37b7c9 commit 05e821b
Show file tree
Hide file tree
Showing 5 changed files with 19 additions and 15 deletions.
13 changes: 4 additions & 9 deletions codefresh.yml
Original file line number Diff line number Diff line change
Expand Up @@ -32,16 +32,11 @@ steps:
# Go inside the surround library
- cd surround

# Install surround
- python3 setup.py install
# Install tox
- pip install tox==3.20.1

# Run tests
- python3 setup.py test

# Run pylint tests
- pip install pylint==2.4.3
- pylint setup.py
- find surround/ -iname "*.py" | xargs pylint
# Run tests through tox
- tox

test_cli:
title: "Test Surround CLI"
Expand Down
1 change: 1 addition & 0 deletions surround/MANIFEST.in
Original file line number Diff line number Diff line change
@@ -1,2 +1,3 @@
include VERSION
include requirements.txt
include surround/*.yaml
1 change: 0 additions & 1 deletion surround/requirements-test.txt

This file was deleted.

6 changes: 1 addition & 5 deletions surround/setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -10,9 +10,6 @@
with open("requirements.txt") as f:
INSTALL_REQUIRES = f.read().split("\n")

with open("requirements-test.txt") as f:
TESTS_REQUIRES = f.read().split("\n")

setup(name='surround',
version=VERSION,
description='Surround is a framework for serving machine learning pipelines in Python.',
Expand All @@ -26,5 +23,4 @@
test_suite='surround.tests',
license="BSD-3-Clause License",
zip_safe=False,
install_requires=INSTALL_REQUIRES,
tests_require=TESTS_REQUIRES)
install_requires=INSTALL_REQUIRES)
13 changes: 13 additions & 0 deletions surround/tox.ini
Original file line number Diff line number Diff line change
@@ -0,0 +1,13 @@
[tox]
envlist = py36, py38
skip_missing_interpreters = True

[testenv]
deps =
pylint==2.4.3

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

0 comments on commit 05e821b

Please sign in to comment.