Skip to content
This repository has been archived by the owner on Aug 21, 2024. It is now read-only.

Commit

Permalink
fix: test/ci environment fixes (#1170)
Browse files Browse the repository at this point in the history
* feat: remove libsass pin

* fix: tox/travis env match python/django versions
  • Loading branch information
zacharis278 authored Aug 31, 2021
1 parent e128dde commit 8399d7f
Show file tree
Hide file tree
Showing 11 changed files with 28 additions and 24 deletions.
3 changes: 2 additions & 1 deletion .travis.yml
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,8 @@ install:
script:
- docker exec -t -e TRAVIS=1 insights_testing bash -c "
cd /edx/app/insights/edx_analytics_dashboard/ &&
PATH=\$PATH:/edx/app/insights/nodeenvs/insights/bin:/snap/bin &&
source /edx/app/insights/insights_env &&
PATH=\$PATH:/snap/bin &&
make $TARGETS "

matrix:
Expand Down
2 changes: 1 addition & 1 deletion Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ ROOT = $(shell echo "$$PWD")
COVERAGE_DIR = $(ROOT)/build/coverage
NODE_BIN=./node_modules/.bin
PYTHON_ENV=py38
DJANGO_VERSION=django22
DJANGO_VERSION=django32

DJANGO_SETTINGS_MODULE ?= "analytics_dashboard.settings.local"

Expand Down
2 changes: 1 addition & 1 deletion requirements/base.in
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ edx-analytics-data-api-client
edx-i18n-tools
edx-rest-api-client # Apache
edx-toggles
libsass==0.11.1 # MIT
libsass # MIT
logutils # BSD
requests # Apache 2.0
stevedore
Expand Down
4 changes: 2 additions & 2 deletions requirements/base.txt
Original file line number Diff line number Diff line change
Expand Up @@ -131,7 +131,7 @@ idna==3.2
# via requests
jinja2==3.0.1
# via code-annotations
libsass==0.11.1
libsass==0.21.0
# via -r requirements/base.in
logutils==0.3.5
# via -r requirements/base.in
Expand Down Expand Up @@ -189,7 +189,7 @@ pyyaml==5.4.1
# code-annotations
# edx-django-release-util
# edx-i18n-tools
regex==2021.8.21
regex==2021.8.28
# via awesome-slugify
requests==2.26.0
# via
Expand Down
4 changes: 2 additions & 2 deletions requirements/doc.txt
Original file line number Diff line number Diff line change
Expand Up @@ -173,7 +173,7 @@ jinja2==3.0.1
# -r requirements/base.txt
# code-annotations
# sphinx
libsass==0.11.1
libsass==0.21.0
# via -r requirements/base.txt
logutils==0.3.5
# via -r requirements/base.txt
Expand Down Expand Up @@ -271,7 +271,7 @@ pyyaml==5.4.1
# code-annotations
# edx-django-release-util
# edx-i18n-tools
regex==2021.8.21
regex==2021.8.28
# via
# -r requirements/base.txt
# awesome-slugify
Expand Down
4 changes: 2 additions & 2 deletions requirements/local.txt
Original file line number Diff line number Diff line change
Expand Up @@ -201,7 +201,7 @@ lazy-object-proxy==1.4.3
# via
# -r requirements/test.txt
# astroid
libsass==0.11.1
libsass==0.21.0
# via -r requirements/test.txt
logutils==0.3.5
# via -r requirements/test.txt
Expand Down Expand Up @@ -340,7 +340,7 @@ pyyaml==5.4.1
# code-annotations
# edx-django-release-util
# edx-i18n-tools
regex==2021.8.21
regex==2021.8.28
# via
# -r requirements/test.txt
# awesome-slugify
Expand Down
4 changes: 2 additions & 2 deletions requirements/production.txt
Original file line number Diff line number Diff line change
Expand Up @@ -164,7 +164,7 @@ jinja2==3.0.1
# via
# -r requirements/base.txt
# code-annotations
libsass==0.11.1
libsass==0.21.0
# via -r requirements/base.txt
logutils==0.3.5
# via -r requirements/base.txt
Expand Down Expand Up @@ -263,7 +263,7 @@ pyyaml==5.4.1
# code-annotations
# edx-django-release-util
# edx-i18n-tools
regex==2021.8.21
regex==2021.8.28
# via
# -r requirements/base.txt
# awesome-slugify
Expand Down
4 changes: 2 additions & 2 deletions requirements/test.txt
Original file line number Diff line number Diff line change
Expand Up @@ -187,7 +187,7 @@ lazy==1.4
# via bok-choy
lazy-object-proxy==1.4.3
# via astroid
libsass==0.11.1
libsass==0.21.0
# via -r requirements/base.txt
logutils==0.3.5
# via -r requirements/base.txt
Expand Down Expand Up @@ -308,7 +308,7 @@ pyyaml==5.4.1
# code-annotations
# edx-django-release-util
# edx-i18n-tools
regex==2021.8.21
regex==2021.8.28
# via
# -r requirements/base.txt
# awesome-slugify
Expand Down
14 changes: 8 additions & 6 deletions requirements/tox.txt
Original file line number Diff line number Diff line change
@@ -1,19 +1,21 @@
#
# This file is autogenerated by pip-compile
# This file is autogenerated by pip-compile with python 3.8
# To update, run:
#
# make upgrade
#
appdirs==1.4.4
backports.entry-points-selectable==1.1.0
# via virtualenv
distlib==0.3.2
# via virtualenv
filelock==3.0.12
# via
# tox
# virtualenv
packaging==20.9
packaging==21.0
# via tox
platformdirs==2.3.0
# via virtualenv
pluggy==0.13.1
# via tox
py==1.10.0
Expand All @@ -26,12 +28,12 @@ six==1.16.0
# virtualenv
toml==0.10.2
# via tox
tox-battery==0.6.1
# via -r requirements/tox.in
tox==3.14.6
# via
# -c requirements/constraints.txt
# -r requirements/tox.in
# tox-battery
virtualenv==20.4.7
tox-battery==0.6.1
# via -r requirements/tox.in
virtualenv==20.7.2
# via tox
8 changes: 4 additions & 4 deletions requirements/travis.txt
Original file line number Diff line number Diff line change
Expand Up @@ -10,9 +10,9 @@ backports.entry-points-selectable==1.1.0
# virtualenv
certifi==2021.5.30
# via requests
charset-normalizer==2.0.3
charset-normalizer==2.0.4
# via requests
codecov==2.1.11
codecov==2.1.12
# via -r requirements/travis.in
coverage==5.5
# via codecov
Expand All @@ -31,7 +31,7 @@ packaging==21.0
# via
# -r requirements/tox.txt
# tox
platformdirs==2.2.0
platformdirs==2.3.0
# via
# -r requirements/tox.txt
# virtualenv
Expand Down Expand Up @@ -67,7 +67,7 @@ tox-battery==0.6.1
# via -r requirements/tox.txt
urllib3==1.26.6
# via requests
virtualenv==20.6.0
virtualenv==20.7.2
# via
# -r requirements/tox.txt
# tox
3 changes: 2 additions & 1 deletion tox.ini
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
[tox]
skipsdist = True
envlist = py38-django{22,30,31}-{a11y,accept,check_isort,create_acceptance_test_soapbox_messages,delete_acceptance_test_soapbox_messages,detect_changed_translations,dummy_translations,extract_translations,pycodestyle,pylint,tests,validate_translations,waffle_learner_analytics,waffle_course_filters,waffle_course_passing}
envlist = py38-django{22,30,31,32}-{a11y,accept,check_isort,create_acceptance_test_soapbox_messages,delete_acceptance_test_soapbox_messages,detect_changed_translations,dummy_translations,extract_translations,pycodestyle,pylint,tests,validate_translations,waffle_learner_analytics,waffle_course_filters,waffle_course_passing}

[pytest]
DJANGO_SETTINGS_MODULE = analytics_dashboard.settings.test
Expand Down Expand Up @@ -34,6 +34,7 @@ deps =
django22: Django>=2.2,<2.3
django30: Django>=3.0,<3.1
django31: Django>=3.1,<3.2
django32: -r requirements/django.txt
-r {toxinidir}/requirements/test.txt
changedir =
detect_changed_translations,dummy_translations,validate_translations: analytics_dashboard
Expand Down

0 comments on commit 8399d7f

Please sign in to comment.