Commit 37bb8a6f by Oleg Marshev

Quality fix.

parent 2eff9cdc
...@@ -4,7 +4,7 @@ from django.core.exceptions import ValidationError ...@@ -4,7 +4,7 @@ from django.core.exceptions import ValidationError
from django.conf import settings from django.conf import settings
from django.db.models import signals from django.db.models import signals
from django.dispatch import receiver from django.dispatch import receiver
from elasticutils.contrib.django import Indexable, MappingType, get_es from elasticutils.contrib.django import Indexable, MappingType
class Note(models.Model): class Note(models.Model):
......
...@@ -46,15 +46,13 @@ class HasAccessToken(BasePermission): ...@@ -46,15 +46,13 @@ class HasAccessToken(BasePermission):
if req_user == auth_user: if req_user == auth_user:
return True return True
else: else:
logger.debug("Token user {auth_user} did not match {field} user {req_user}".format( logger.debug("Token user %s did not match %s user %s", auth_user, request_field, req_user)
auth_user=auth_user, field=request_field, req_user=req_user
))
return False return False
logger.info("No user was present to compare in GET, POST or DATA") logger.info("No user was present to compare in GET, POST or DATA")
except jwt.ExpiredSignature: except jwt.ExpiredSignature:
logger.debug("Token was expired: {}".format(token)) logger.debug("Token was expired: %s", token)
except jwt.DecodeError: except jwt.DecodeError:
logger.debug("Could not decode token {}".format(token)) logger.debug("Could not decode token %s", token)
except TokenWrongIssuer: except TokenWrongIssuer:
logger.debug("Token has wrong issuer {}".format(token)) logger.debug("Token has wrong issuer %s", token)
return False return False
...@@ -67,7 +67,7 @@ class BaseAnnotationViewTests(APITestCase): ...@@ -67,7 +67,7 @@ class BaseAnnotationViewTests(APITestCase):
get_es().indices.delete(index=settings.ES_INDEXES['default']) get_es().indices.delete(index=settings.ES_INDEXES['default'])
get_es().indices.refresh() get_es().indices.refresh()
def _create_annotation(self, refresh=True, **kwargs): def _create_annotation(self, **kwargs):
""" """
Create annotation Create annotation
""" """
...@@ -170,7 +170,6 @@ class AnnotationViewTests(BaseAnnotationViewTests): ...@@ -170,7 +170,6 @@ class AnnotationViewTests(BaseAnnotationViewTests):
Test if annotation 'updated' field is not used by API. Test if annotation 'updated' field is not used by API.
""" """
self.payload['updated'] = 'abc' self.payload['updated'] = 'abc'
payload = self.payload
response = self.client.post(reverse('api:v1:annotations'), self.payload, format='json') response = self.client.post(reverse('api:v1:annotations'), self.payload, format='json')
self.assertEqual(response.status_code, status.HTTP_201_CREATED) self.assertEqual(response.status_code, status.HTTP_201_CREATED)
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment