From c5d02f5aa1a68225dafa0bf99226e4a05384a746 Mon Sep 17 00:00:00 2001 From: "deepsource-autofix[bot]" <62050782+deepsource-autofix[bot]@users.noreply.github.com> Date: Fri, 27 Mar 2020 16:18:40 +0000 Subject: [PATCH 1/3] Remove unused imports --- app/api/helpers/ticketing.py | 1 - app/instance.py | 6 ------ app/models/helpers/versioning.py | 3 --- populate_db.py | 1 - tests/hook_main.py | 3 --- 5 files changed, 14 deletions(-) diff --git a/app/api/helpers/ticketing.py b/app/api/helpers/ticketing.py index 2253f50220..f7c78a4c30 100644 --- a/app/api/helpers/ticketing.py +++ b/app/api/helpers/ticketing.py @@ -16,7 +16,6 @@ ) from app.api.helpers.payment import PayPalPaymentsManager, StripePaymentsManager from app.models import db -from app.models.order import Order from app.models.ticket_fee import TicketFees from app.models.ticket_holder import TicketHolder diff --git a/app/instance.py b/app/instance.py index 56d8b6fb72..daf98a3753 100644 --- a/app/instance.py +++ b/app/instance.py @@ -17,25 +17,19 @@ from flask_rest_jsonapi.errors import jsonapi_errors from flask_rest_jsonapi.exceptions import JsonApiException from healthcheck import HealthCheck -from pytz import utc from sentry_sdk.integrations.celery import CeleryIntegration from sentry_sdk.integrations.flask import FlaskIntegration from sentry_sdk.integrations.redis import RedisIntegration from sentry_sdk.integrations.sqlalchemy import SqlalchemyIntegration from werkzeug.middleware.profiler import ProfilerMiddleware - -from app.api import routes from app.api.helpers.auth import AuthManager, is_token_blacklisted from app.api.helpers.cache import cache from app.api.helpers.jwt import jwt_user_loader from app.extensions import limiter, shell from app.models import db -from app.models.event import Event -from app.models.role_invite import RoleInvite from app.models.utils import add_engine_pidguard, sqlite_datetime_fix from app.templates.flask_ext.jinja.filters import init_filters from app.views.blueprints import BlueprintsManager -from app.views.elastic_search import client from app.views.healthcheck import ( check_migrations, health_check_celery, diff --git a/app/models/helpers/versioning.py b/app/models/helpers/versioning.py index e2cfd12787..95eb1d69be 100644 --- a/app/models/helpers/versioning.py +++ b/app/models/helpers/versioning.py @@ -1,9 +1,6 @@ import re -import unicodedata -from itertools import zip_longest import bleach -import diff_match_patch from bleach.callbacks import nofollow, target_blank diff --git a/populate_db.py b/populate_db.py index 9b9f4c8039..38f2b2b17a 100644 --- a/populate_db.py +++ b/populate_db.py @@ -1,4 +1,3 @@ -from envparse import env from app.api.helpers.db import get_or_create diff --git a/tests/hook_main.py b/tests/hook_main.py index b56cff5f49..7074a976e1 100644 --- a/tests/hook_main.py +++ b/tests/hook_main.py @@ -12,9 +12,6 @@ from flask import Flask from app.models import db from app.models.user import OWNER -from app.models.user_token_blacklist import ( - UserTokenBlackListTime, -) # noqa Workaround for registering unimported model from app.api import routes # noqa Workaround for importing all required models # imports from factories From 228ed5ef06a1de78174c0632850e5ce08e5104ac Mon Sep 17 00:00:00 2001 From: Areeb Jamal Date: Fri, 27 Mar 2020 22:02:47 +0530 Subject: [PATCH 2/3] Apply suggestions from code review --- app/instance.py | 2 ++ tests/hook_main.py | 3 +++ 2 files changed, 5 insertions(+) diff --git a/app/instance.py b/app/instance.py index daf98a3753..aca525cbea 100644 --- a/app/instance.py +++ b/app/instance.py @@ -22,6 +22,8 @@ from sentry_sdk.integrations.redis import RedisIntegration from sentry_sdk.integrations.sqlalchemy import SqlalchemyIntegration from werkzeug.middleware.profiler import ProfilerMiddleware + +from app.api import routes from app.api.helpers.auth import AuthManager, is_token_blacklisted from app.api.helpers.cache import cache from app.api.helpers.jwt import jwt_user_loader diff --git a/tests/hook_main.py b/tests/hook_main.py index 7074a976e1..f741678337 100644 --- a/tests/hook_main.py +++ b/tests/hook_main.py @@ -12,6 +12,9 @@ from flask import Flask from app.models import db from app.models.user import OWNER +from app.models.user_token_blacklist import ( + UserTokenBlackListTime, +) # noqa Workaround for registering unimported model from app.api import routes # noqa Workaround for importing all required models # imports from factories From 1463fc22c1db19a22cacdd219af39d95fe465fbf Mon Sep 17 00:00:00 2001 From: Areeb Jamal Date: Fri, 27 Mar 2020 22:03:23 +0530 Subject: [PATCH 3/3] Update tests/hook_main.py --- tests/hook_main.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/hook_main.py b/tests/hook_main.py index f741678337..b56cff5f49 100644 --- a/tests/hook_main.py +++ b/tests/hook_main.py @@ -13,7 +13,7 @@ from app.models import db from app.models.user import OWNER from app.models.user_token_blacklist import ( - UserTokenBlackListTime, + UserTokenBlackListTime, ) # noqa Workaround for registering unimported model from app.api import routes # noqa Workaround for importing all required models