Skip to content

Commit 642dece

Browse files
committed
Merge branch 'master' into manifest-file
Conflicts: setup.py
2 parents cb37c6a + a5f5067 commit 642dece

File tree

2 files changed

+8
-7
lines changed

2 files changed

+8
-7
lines changed

circle.yml

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -27,13 +27,14 @@ test:
2727
# run test suite in all our python versions
2828
- bash circle/test.sh
2929

30-
# # test that it imports when you don't have file permissions
31-
# - sudo chmod 400 ${PLOTLY_CONFIG_DIR} && python -c "import plotly"
32-
#
33-
# # test that setting permissions will work for import (and tests)
34-
# - sudo chmod 600 ${PLOTLY_CONFIG_DIR} && python -c "import plotly"
30+
# test that it imports when you don't have write permissions
31+
- sudo chmod -R 444 ${PLOTLY_CONFIG_DIR} && python -c "import plotly"
32+
33+
# test that giving back write permissions works again
34+
# this also has to pass the test suite that follows
35+
- sudo chmod -R 777 ${PLOTLY_CONFIG_DIR} && python -c "import plotly"
3536

3637
# test core things in the general 2.7 version that circle has
37-
- nosetests -xv plotly/tests --with-coverage --cover-package=plotly
38+
- nosetests -xv plotly/tests/test_core --with-coverage --cover-package=plotly
3839
- mkdir "${CIRCLE_ARTIFACTS}/2.7" || true
3940
- coverage html -d "${CIRCLE_ARTIFACTS}/2.7" --title=2.7

setup.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,7 @@ def readme():
3434
packages=find_packages(),
3535
data_files=[(GRAPH_REFERENCE_DIR, GRAPH_REFERENCE_FILES),
3636
(WIDGETS_DIR, WIDGETS_FILES)],
37-
install_requires=['requests', 'six', 'pytz'],
37+
install_requires=['requests[security]', 'six', 'pytz'],
3838
extras_require={"PY2.6": ['simplejson', 'ordereddict',
3939
'requests[security]']},
4040
zip_safe=False)

0 commit comments

Comments
 (0)