Skip to content

Commit

Permalink
Reorganize the requirements files for Juniper tox tests
Browse files Browse the repository at this point in the history
The purpose of this commit is to clean up the requirements files. This
makes things a bit clearer. We now have a "Juniper community" mode and a
"Juniper multisite" mode.

* We remove Py38 because Juniper needs to run in Py35
* The `juniper_community.txt` and `juniper_multisite.txt` requirements
files read from `juniper_base.txt`. Each of the 'community' and
'multisite' have only the package differences, which at this point is
the version/branch of `edx-organizations`
  • Loading branch information
johnbaldwin committed May 19, 2021
1 parent 5a105a8 commit f602859
Show file tree
Hide file tree
Showing 6 changed files with 24 additions and 18 deletions.
7 changes: 5 additions & 2 deletions devsite/requirements/juniper_base.txt
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ pytz==2020.1
## Django package dependencies
##

Django==2.2.15
Django==2.2.16

djangorestframework==3.9.4
django-countries==5.5
Expand All @@ -35,6 +35,9 @@ django-environ==0.4.5

jsonfield==2.1.1

# For


##
## Documentation (Sphinx) dependencies
##
Expand All @@ -49,7 +52,6 @@ Sphinx==3.1.2
edx-opaque-keys[django]==2.1.0
#edx-drf-extensions==6.0.0

edx-organizations==5.2.0

##
## Devsite
Expand All @@ -75,3 +77,4 @@ pytest-cov==2.8.1
tox==3.15.0
freezegun==0.3.12
edx-lint==1.4.1
mock==3.0.5
5 changes: 5 additions & 0 deletions devsite/requirements/juniper_community.txt
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
# Requirements needed for Juniper community environment

-r juniper_base.txt

edx-organizations==5.2.0
6 changes: 6 additions & 0 deletions devsite/requirements/juniper_multisite.txt
Original file line number Diff line number Diff line change
@@ -0,0 +1,6 @@
# Requirements needed for Juniper multisite environment

-r juniper_base.txt

# Organization/site mapping requires Appsembler's fork
git+https://github.com/appsembler/[email protected]
4 changes: 0 additions & 4 deletions devsite/requirements/py35_juniper.txt

This file was deleted.

4 changes: 0 additions & 4 deletions devsite/requirements/py38_juniper.txt

This file was deleted.

16 changes: 8 additions & 8 deletions tox.ini
Original file line number Diff line number Diff line change
Expand Up @@ -13,8 +13,8 @@ envlist =
py27-ginkgo
py27-hawthorn
py27-hawthorn_multisite
py35-juniper
py38-juniper
py35-juniper_community
py35-juniper_multisite
lint
edx_lint_check

Expand All @@ -27,8 +27,8 @@ deps =
ginkgo: -r{toxinidir}/devsite/requirements/ginkgo.txt
hawthorn: -r{toxinidir}/devsite/requirements/hawthorn.txt
hawthorn_multisite: -r{toxinidir}/devsite/requirements/hawthorn_multisite.txt
py35_juniper: -r{toxinidir}/devsite/requirements/py35_juniper.txt
py38_juniper: -r{toxinidir}/devsite/requirements/py38_juniper.txt
juniper_community: -r{toxinidir}/devsite/requirements/juniper_community.txt
juniper_multisite: -r{toxinidir}/devsite/requirements/juniper_multisite.txt

whitelist_externals =
git
Expand All @@ -40,15 +40,15 @@ setenv =
ginkgo: OPENEDX_RELEASE = GINKGO
hawthorn: OPENEDX_RELEASE = HAWTHORN
hawthorn_multisite: OPENEDX_RELEASE = HAWTHORN
py35_juniper: OPENEDX_RELEASE = JUNIPER
py38_juniper: OPENEDX_RELEASE = JUNIPER
juniper_community: OPENEDX_RELEASE = JUNIPER
juniper_multisite: OPENEDX_RELEASE = JUNIPER

commands =
ginkgo: pytest -c pytest-ginkgo.ini {posargs}
hawthorn: pytest -c pytest-hawthorn.ini {posargs}
hawthorn_multisite: pytest -c pytest-hawthorn.ini {posargs}
py35_juniper: pytest -c pytest-juniper.ini {posargs}
py38_juniper: pytest -c pytest-juniper.ini {posargs}
juniper_community: pytest -c pytest-juniper.ini {posargs}
juniper_multisite: pytest -c pytest-juniper.ini {posargs}

[testenv:lint]
basepython=python2
Expand Down

0 comments on commit f602859

Please sign in to comment.