Lint changes

Thiyagu55 · Thiyagu55 · commit 31336cd7c347 · 2022-06-28T12:05:48.000+05:30
diff --git a/.github/workflows/unit-tests.yaml b/.github/workflows/unit-tests.yaml
@@ -19,3 +19,8 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - run: 'echo "No build required" '
+
+  lint:
+    runs-on: ubuntu-latest
+    steps:
+      - run: 'echo "No build required" '
diff --git a/python/sqlcommenter-python/google/cloud/sqlcommenter/django/middleware.py b/python/sqlcommenter-python/google/cloud/sqlcommenter/django/middleware.py
@@ -19,7 +19,6 @@
 
 import django
 from django.db import connections
-
 from django.db.backends.utils import CursorDebugWrapper
 from google.cloud.sqlcommenter import add_sql_comment
 from google.cloud.sqlcommenter.opencensus import get_opencensus_values
diff --git a/python/sqlcommenter-python/tests/django/tests.py b/python/sqlcommenter-python/tests/django/tests.py
@@ -17,9 +17,11 @@
 import django
 from django.db import connection, connections
 from django.http import HttpRequest
-from django.test import TestCase, override_settings, modify_settings
+from django.test import TestCase, modify_settings, override_settings
 from django.urls import resolve, reverse
-from google.cloud.sqlcommenter.django.middleware import SqlCommenter, QueryWrapper
+from google.cloud.sqlcommenter.django.middleware import (
+    QueryWrapper, SqlCommenter,
+)
 
 from ..compat import mock
 from ..opencensus_mock import mock_opencensus_tracer
@@ -30,6 +32,8 @@
 # Adding the middleware twice in modify_settings
 # doesn't work. The middleware is only used once
 # if used in modify_settings
+
+
 class TestMiddleware:
     def __init__(self, get_response):
         self.get_response = get_response
@@ -38,10 +42,12 @@ def __call__(self, request):
         with connection.execute_wrapper(QueryWrapper(request)):
             return self.get_response(request)
 
+
 # Query log only active if DEBUG=True.
 @override_settings(DEBUG=True)
 class Tests(TestCase):
     databases = '__all__'
+
     @staticmethod
     def get_request(path):
         request = HttpRequest()
diff --git a/python/sqlcommenter-python/tests/fastapi/app.py b/python/sqlcommenter-python/tests/fastapi/app.py
@@ -2,10 +2,11 @@
 
 from fastapi import FastAPI, status
 from fastapi.responses import JSONResponse
+from google.cloud.sqlcommenter.fastapi import (
+    SQLCommenterMiddleware, get_fastapi_info,
+)
 from starlette.exceptions import HTTPException as StarletteHTTPException
 
-from google.cloud.sqlcommenter.fastapi import SQLCommenterMiddleware, get_fastapi_info
-
 app = FastAPI(title="SQLCommenter")
 
 app.add_middleware(SQLCommenterMiddleware)
diff --git a/python/sqlcommenter-python/tests/fastapi/tests.py b/python/sqlcommenter-python/tests/fastapi/tests.py
@@ -18,9 +18,8 @@
 
 import fastapi
 import pytest
-from starlette.testclient import TestClient
-
 from google.cloud.sqlcommenter.fastapi import get_fastapi_info
+from starlette.testclient import TestClient
 
 from .app import app
 
diff --git a/python/sqlcommenter-python/tests/generic/test_add_sql_comment.py b/python/sqlcommenter-python/tests/generic/test_add_sql_comment.py
@@ -27,10 +27,10 @@ def test_end_comment_escaping(self):
         self.assertEqual("Select 1 /*a='%%2A/abc'*/", add_sql_comment("Select 1", a='*/abc'))
 
     def test_bytes(self):
-        self.assertIn("Select 1 /*a='%%2A/abc'*/", add_sql_comment("Select 1",a=b'*/abc'))
+        self.assertIn("Select 1 /*a='%%2A/abc'*/", add_sql_comment("Select 1", a=b'*/abc'))
 
     def test_url_quoting(self):
-        self.assertIn("Select 1 /*foo='bar%%2Cbaz'*/", add_sql_comment("Select 1",foo='bar,baz'))
+        self.assertIn("Select 1 /*foo='bar%%2Cbaz'*/", add_sql_comment("Select 1", foo='bar,baz'))
 
     def test_sql_with_semicolon(self):
-        self.assertIn("Select 1 /*foo='bar%%2Cbaz'*/;", add_sql_comment("Select 1;",foo='bar,baz'))
+        self.assertIn("Select 1 /*foo='bar%%2Cbaz'*/;", add_sql_comment("Select 1;", foo='bar,baz'))
