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

Commit

Permalink
chore: update tox.ini
Browse files Browse the repository at this point in the history
  • Loading branch information
mumarkhan999 committed Dec 20, 2023
1 parent ba7b3d4 commit 5c45b3e
Show file tree
Hide file tree
Showing 10 changed files with 26 additions and 28 deletions.
6 changes: 3 additions & 3 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -12,13 +12,13 @@ jobs:
strategy:
matrix:
include:
- django-env: django32
- django-env: django42
testname: quality-and-jobs
targets: PYTHON_ENV=py38 requirements.js check_translations_up_to_date validate_translations clean_static static quality validate_js check_keywords
- django-env: django32
- django-env: django42
testname: test-python
targets: PYTHON_ENV=py38 requirements.js clean_static static validate_python
- django-env: django32
- django-env: django42
testname: acceptance-python
targets: PYTHON_ENV=py38 requirements.js clean_static static acceptance

Expand Down
2 changes: 1 addition & 1 deletion Makefile
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
NODE_BIN=./node_modules/.bin
DIFF_COVER_BASE_BRANCH=master
PYTHON_ENV=py38
DJANGO_ENV_VAR=$(if $(DJANGO_ENV),$(DJANGO_ENV),django32)
DJANGO_ENV_VAR=$(if $(DJANGO_ENV),$(DJANGO_ENV),django42)

help:
@echo ''
Expand Down
4 changes: 2 additions & 2 deletions ecommerce/extensions/partner/apps.py
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@


from oscar.apps.partner import apps
from oscar.apps.partner.apps import PartnerConfig as CorePartnerConfig


class PartnerConfig(apps.PartnerConfig):
class PartnerConfig(CorePartnerConfig):
name = 'ecommerce.extensions.partner'
1 change: 0 additions & 1 deletion ecommerce/extensions/refund/__init__.py
Original file line number Diff line number Diff line change
@@ -1 +0,0 @@
default_app_config = 'ecommerce.extensions.refund.apps.RefundConfig' # pragma: no cover
28 changes: 14 additions & 14 deletions ecommerce/settings/_oscar.py
Original file line number Diff line number Diff line change
Expand Up @@ -14,19 +14,19 @@

# APP CONFIGURATION
OSCAR_APPS = [
'oscar',
'oscar.apps.address',
'oscar.apps.shipping',
'oscar.apps.catalogue.reviews',
'oscar.apps.search',
'oscar.apps.wishlists',
"oscar.config.Shop",
"oscar.apps.address.apps.AddressConfig",
'oscar.apps.shipping.apps.ShippingConfig',
'oscar.apps.catalogue.reviews.apps.CatalogueReviewsConfig',
"oscar.apps.search.apps.SearchConfig",
"oscar.apps.wishlists.apps.WishlistsConfig",

'ecommerce.extensions',
'ecommerce.extensions.iap',
'ecommerce.extensions.api',
'ecommerce.extensions.communication.apps.CommunicationConfig',
'ecommerce.extensions.fulfillment',
'ecommerce.extensions.refund',
'ecommerce.extensions.refund.apps.RefundConfig',
'ecommerce.extensions.analytics',
'ecommerce.extensions.basket',
'ecommerce.extensions.catalogue',
Expand All @@ -41,16 +41,16 @@
# Dashboard applications depend on models declared in the core applications (basket, catalogue, etc).
# To prevent issues with Oscar’s dynamic model loading, overrides of dashboard applications should
# follow overrides of core applications
'oscar.apps.dashboard.reports',
'oscar.apps.dashboard.partners',
'oscar.apps.dashboard.pages',
'oscar.apps.dashboard.ranges',
"oscar.apps.dashboard.reports",
"oscar.apps.dashboard.partners",
"oscar.apps.dashboard.pages",
"oscar.apps.dashboard.ranges",
'oscar.apps.dashboard.reviews',
'oscar.apps.dashboard.vouchers',
'oscar.apps.dashboard.communications',
"oscar.apps.dashboard.vouchers",
"oscar.apps.dashboard.communications",
'oscar.apps.dashboard.shipping',

'ecommerce.extensions.dashboard',
'ecommerce.extensions.dashboard.apps.DashboardConfig',
'ecommerce.extensions.dashboard.catalogue',
'ecommerce.extensions.dashboard.offers',
'ecommerce.extensions.dashboard.refunds.apps.RefundsDashboardConfig', # Providing full path to make the signals work
Expand Down
2 changes: 1 addition & 1 deletion requirements/base.txt
Original file line number Diff line number Diff line change
Expand Up @@ -170,7 +170,7 @@ django-oscar==3.2.2
# via
# -c requirements/constraints.txt
# -r requirements/base.in
django-phonenumber-field==5.0.0
django-phonenumber-field==5.1.0
# via django-oscar
django-simple-history==3.0.0
# via
Expand Down
2 changes: 1 addition & 1 deletion requirements/dev.txt
Original file line number Diff line number Diff line change
Expand Up @@ -254,7 +254,7 @@ django-model-utils==4.3.1
# edx-rbac
django-oscar==3.2.2
# via -r requirements/test.txt
django-phonenumber-field==5.0.0
django-phonenumber-field==5.1.0
# via
# -r requirements/test.txt
# django-oscar
Expand Down
2 changes: 1 addition & 1 deletion requirements/production.txt
Original file line number Diff line number Diff line change
Expand Up @@ -173,7 +173,7 @@ django-oscar==3.2.2
# via
# -c requirements/constraints.txt
# -r requirements/base.in
django-phonenumber-field==5.0.0
django-phonenumber-field==5.1.0
# via django-oscar
django-ses==3.5.0
# via -r requirements/production.in
Expand Down
2 changes: 1 addition & 1 deletion requirements/test.txt
Original file line number Diff line number Diff line change
Expand Up @@ -246,7 +246,7 @@ django-oscar==3.2.2
# via
# -c requirements/constraints.txt
# -r requirements/base.txt
django-phonenumber-field==5.0.0
django-phonenumber-field==5.1.0
# via
# -r requirements/base.txt
# django-oscar
Expand Down
5 changes: 2 additions & 3 deletions tox.ini
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
[tox]
skipsdist=True
envlist = py38-django32-{static,pylint,tests,theme_static,check_keywords},py38-{isort,pycodestyle,extract_translations,dummy_translations,compile_translations, detect_changed_translations,validate_translations},docs

envlist = py38-django42-{static,pylint,tests,theme_static,check_keywords},py38-{isort,pycodestyle,extract_translations,dummy_translations,compile_translations, detect_changed_translations,validate_translations},docs
[pytest]
addopts = --ds=ecommerce.settings.test --cov=ecommerce --cov-report term --cov-config=.coveragerc --no-cov-on-fail -p no:randomly --no-migrations -m "not acceptance"
testpaths = ecommerce
Expand Down Expand Up @@ -47,7 +46,7 @@ setenv =
SELENIUM_BROWSER=firefox
deps =
-r{toxinidir}/requirements/test.txt
django32: Django>=3.2,<3.3
django42: Django>=4.2,<4.3
allowlist_externals =
/bin/bash
changedir =
Expand Down

0 comments on commit 5c45b3e

Please sign in to comment.