Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Commit 394c8af

Browse files
committed
save coverage information for all versions
1 parent 2f151eb commit 394c8af

File tree

2 files changed

+8
-10
lines changed

2 files changed

+8
-10
lines changed

circle.yml

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,4 +28,6 @@ test:
2828
# - chmod 660 ${PLOTLY_CONFIG_DIR} && python -c "import plotly"
2929

3030
# test core things in the general 2.7 version that circle has
31-
- nosetests -xv plotly/tests/test_core
31+
- nosetests -xv plotly/tests --with-coverage --cover-package=plotly
32+
- mkdir "${CIRCLE_ARTIFACTS}/${PYENV_VERSION}"
33+
- coverage html -d "${CIRCLE_ARTIFACTS}/${PYENV_VERSION}"

circle/test.sh

Lines changed: 5 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -40,13 +40,9 @@ for version in ${PLOTLY_PYTHON_VERSIONS[@]}; do
4040
# error_exit "${LINENO}: permissions test 666 on .plotly dir failed"
4141

4242
echo "running tests for Python ${version} as user '$(whoami)'"
43-
if [ ${version:0:3} == '2.7' ]
44-
then
45-
nosetests -xv plotly/tests --with-coverage --cover-package=plotly ||
46-
error_exit "${LINENO}: test suite failed for Python ${version}"
47-
coverage html -d ${CIRCLE_ARTIFACTS}
48-
else
49-
nosetests -xv plotly/tests ||
50-
error_exit "${LINENO}: test suite failed for Python ${version}"
51-
fi
43+
nosetests -xv plotly/tests --with-coverage --cover-package=plotly ||
44+
error_exit "${LINENO}: test suite failed for Python ${version}"
45+
mkdir "${CIRCLE_ARTIFACTS}/${PYENV_VERSION}"
46+
coverage html -d "${CIRCLE_ARTIFACTS}/${PYENV_VERSION}"
47+
5248
done

0 commit comments

Comments
 (0)