Skip to content

Commit 47356bf

Browse files
committed
Testing: Simplify Circle testing and move coveralls to Travis
1 parent a450dae commit 47356bf

File tree

5 files changed

+29
-70
lines changed

5 files changed

+29
-70
lines changed

Diff for: .travis.yml

+4-2
Original file line numberDiff line numberDiff line change
@@ -18,8 +18,7 @@ matrix:
1818
- python: "3.5"
1919
env: TEST_MODULES=true USE_PYQT=pyqt4
2020
os: linux
21-
# This slot test runs our tests only with pip
22-
# packages
21+
# This slot runs our tests only with pip packages
2322
- python: "3.5"
2423
env: TEST_MODULES=false USE_PYQT=pyqt5
2524
os: linux
@@ -42,3 +41,6 @@ script:
4241
./continuous_integration/travis/test-qt4.sh
4342
fi
4443
fi
44+
45+
after_success:
46+
- export PATH="$HOME/miniconda/bin:$PATH" && source activate test && coveralls

Diff for: circle.yml

+3-7
Original file line numberDiff line numberDiff line change
@@ -3,11 +3,7 @@
33
machine:
44
environment:
55
# Python versions to tests (Maximum of 4 different versions)
6-
# The last container is reserved to test with PyQt5 wheels
7-
# because they are more up to date than conda páckages.
8-
PY_VERSIONS: "2.7 3.5 3.6 3.5"
9-
# For Coveralls
10-
COVERALLS_REPO_TOKEN: yt7YToGxRtnAcM6kSULoiCmOPetq10TaU
6+
PY_VERSIONS: "3.5 3.6"
117
# Environment variable used by astropy helpers
128
TRAVIS_OS_NAME: "linux"
139

@@ -17,7 +13,7 @@ dependencies:
1713
# See https://github.com/TestFX/TestFX/issues/158#issuecomment-62421691
1814
- sudo apt-get install matchbox-window-manager
1915
override:
20-
- ./continuous_integration/install.sh
16+
- ./continuous_integration/circle/install.sh
2117
- DISPLAY=:99 /usr/bin/matchbox-window-manager:
2218
background: true
2319
- sleep 5
@@ -28,5 +24,5 @@ test:
2824
#- export PATH="$HOME/miniconda/bin:$PATH" && source activate test && ciocheck spyder: # note the colon
2925
# parallel: true
3026
# PyQt5
31-
- ./continuous_integration/circle/test-qt5.sh || ./continuous_integration/circle/test-qt5.sh || ./continuous_integration/circle/test-qt5.sh: # note the colon
27+
- ./continuous_integration/circle/test-qt5.sh || ./continuous_integration/circle/test-qt5.sh: # note the colon
3228
parallel: true

Diff for: continuous_integration/circle/install.sh

+21
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,21 @@
1+
#!/bin/bash
2+
3+
export CONDA_DEPENDENCIES_FLAGS="--quiet"
4+
export CONDA_DEPENDENCIES="rope pyflakes sphinx pygments pylint psutil nbconvert \
5+
qtawesome pickleshare qtpy pyzmq chardet mock nomkl pandas \
6+
pytest pytest-cov numpydoc scipy cython pillow"
7+
export PIP_DEPENDENCIES="coveralls pytest-qt pytest-xvfb flaky jedi pycodestyle"
8+
9+
# Download and install miniconda and conda/pip dependencies
10+
# with astropy helpers
11+
export PY_VERSIONS=($PY_VERSIONS)
12+
export TRAVIS_PYTHON_VERSION=${PY_VERSIONS[$CIRCLE_NODE_INDEX]}
13+
14+
echo -e "PYTHON = $TRAVIS_PYTHON_VERSION \n============"
15+
git clone git://github.com/astropy/ci-helpers.git > /dev/null
16+
source ci-helpers/travis/setup_conda_$TRAVIS_OS_NAME.sh
17+
export PATH="$HOME/miniconda/bin:$PATH"
18+
source activate test
19+
20+
# Install ciocheck (not working right now)
21+
#conda install -q ciocheck -c spyder-ide --no-update-deps

Diff for: continuous_integration/circle/test-qt5.sh

+1-24
Original file line numberDiff line numberDiff line change
@@ -3,29 +3,6 @@
33
export PATH="$HOME/miniconda/bin:$PATH"
44
source activate test
55

6-
# We use container 3 to test with pip
7-
if [ "$CIRCLE_NODE_INDEX" = "0" ]; then
8-
# Skip Python 2.7 for now because it's failing frequently
9-
# and randomly at different places.
10-
exit 0
11-
elif [ "$CIRCLE_NODE_INDEX" != "3" ]; then
12-
conda install -q qt=5.* pyqt=5.* qtconsole matplotlib
13-
else
14-
pip install -q pyqt5
15-
fi
6+
conda install -q qt=5.* pyqt=5.* qtconsole matplotlib
167

178
python runtests.py
18-
19-
# Force quitting if exit status of runtests.py was not 0
20-
if [ $? -ne 0 ]; then
21-
exit 1
22-
fi
23-
24-
# Run coveralls also here because there are errors if run
25-
# as an independent command
26-
coveralls
27-
28-
# Don't stop if coveralls fails for whatever reason
29-
if [ $? -ne 0 ]; then
30-
exit 0
31-
fi

Diff for: continuous_integration/install.sh

-37
This file was deleted.

0 commit comments

Comments
 (0)