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 a311640
Show file tree
Hide file tree
Showing 10 changed files with 48 additions and 50 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
72 changes: 36 additions & 36 deletions ecommerce/settings/_oscar.py
Original file line number Diff line number Diff line change
Expand Up @@ -14,48 +14,48 @@

# APP CONFIGURATION
OSCAR_APPS = [
'oscar',
'oscar.apps.address',
'oscar.apps.shipping',
'oscar.apps.catalogue.reviews',
'oscar.apps.search',
'oscar.apps.wishlists',

'ecommerce.extensions',
'ecommerce.extensions.iap',
'ecommerce.extensions.api',
"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.config.EdxShop',
'ecommerce.extensions.iap.apps.IapConfig',
'ecommerce.extensions.api', # isn't an oscar app
'ecommerce.extensions.communication.apps.CommunicationConfig',
'ecommerce.extensions.fulfillment',
'ecommerce.extensions.refund',
'ecommerce.extensions.analytics',
'ecommerce.extensions.basket',
'ecommerce.extensions.catalogue',
'ecommerce.extensions.checkout',
'ecommerce.extensions.customer',
'ecommerce.extensions.offer',
'ecommerce.extensions.order',
'ecommerce.extensions.partner',
'ecommerce.extensions.payment',
'ecommerce.extensions.voucher',
'ecommerce.extensions.fulfillment', # isn't an oscar app
'ecommerce.extensions.refund.apps.RefundConfig',
'ecommerce.extensions.analytics.apps.AnalyticsConfig',
'ecommerce.extensions.basket.apps.BasketConfig',
'ecommerce.extensions.catalogue.apps.CatalogueConfig',
'ecommerce.extensions.checkout.apps.CheckoutConfig',
'ecommerce.extensions.customer.apps.CustomerConfig',
'ecommerce.extensions.offer.apps.OfferConfig',
'ecommerce.extensions.order.apps.OrderConfig',
'ecommerce.extensions.partner.apps.PartnerConfig',
'ecommerce.extensions.payment.apps.PaymentConfig',
'ecommerce.extensions.voucher.apps.VoucherConfig',

# 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.reviews',
'oscar.apps.dashboard.vouchers',
'oscar.apps.dashboard.communications',
'oscar.apps.dashboard.shipping',

'ecommerce.extensions.dashboard',
'ecommerce.extensions.dashboard.catalogue',
'ecommerce.extensions.dashboard.offers',
"oscar.apps.dashboard.reports.apps.ReportsDashboardConfig",
"oscar.apps.dashboard.partners.apps.PartnersDashboardConfig",
"oscar.apps.dashboard.pages.apps.PagesDashboardConfig",
"oscar.apps.dashboard.ranges.apps.RangesDashboardConfig",
'oscar.apps.dashboard.reviews.apps.ReviewsDashboardConfig',
"oscar.apps.dashboard.vouchers.apps.VouchersDashboardConfig",
"oscar.apps.dashboard.communications.apps.CommunicationsDashboardConfig",
'oscar.apps.dashboard.shipping.apps.ShippingDashboardConfig',

'ecommerce.extensions.dashboard.apps.DashboardConfig',
'ecommerce.extensions.dashboard.catalogue.apps.CatalogueDashboardConfig',
'ecommerce.extensions.dashboard.offers.apps.OffersDashboardConfig',
'ecommerce.extensions.dashboard.refunds.apps.RefundsDashboardConfig', # Providing full path to make the signals work
'ecommerce.extensions.dashboard.orders',
'ecommerce.extensions.dashboard.users',
'ecommerce.extensions.dashboard.orders.apps.OrdersDashboardConfig',
'ecommerce.extensions.dashboard.users.apps.UsersDashboardConfig',

# 3rd-party apps that oscar depends on
'haystack',
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 a311640

Please sign in to comment.