Skip to content
This repository has been archived by the owner on Sep 16, 2022. It is now read-only.

Commit

Permalink
CSCMETAX-61: [REF] Optimize imports
Browse files Browse the repository at this point in the history
  • Loading branch information
junsk1 committed Nov 22, 2017
1 parent 4602f9b commit 7534acf
Show file tree
Hide file tree
Showing 44 changed files with 94 additions and 79 deletions.
1 change: 0 additions & 1 deletion .githooks/pre-commit
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
#!/usr/bin/env python
import os
import sys

from flake8.main import git
Expand Down
2 changes: 2 additions & 0 deletions src/metax_api/api/base/router.py
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,7 @@
2. Add a URL to urlpatterns: url(r'^blog/', include('blog.urls'))
"""
from rest_framework.routers import DefaultRouter, Route

from .views import (
ContractViewSet,
DataCatalogViewSet,
Expand All @@ -22,6 +23,7 @@
FileViewSet
)


class CustomRouter(DefaultRouter):

"""
Expand Down
4 changes: 2 additions & 2 deletions src/metax_api/api/base/serializers/__init__.py
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
from .contract_serializer import ContractSerializer
from .catalog_record_serializer import CatalogRecordSerializer
from .directory_serializer import DirectorySerializer
from .contract_serializer import ContractSerializer
from .data_catalog_serializer import DataCatalogSerializer
from .directory_serializer import DirectorySerializer
from .file_serializer import FileSerializer
from .xml_metadata_serializer import XmlMetadataSerializer
Original file line number Diff line number Diff line change
@@ -1,15 +1,15 @@
import logging
from os import path

from rest_framework.serializers import ValidationError

from metax_api.models import CatalogRecord, DataCatalog, File, Contract
from metax_api.services import CatalogRecordService as CRS, CommonService
from .data_catalog_serializer import DataCatalogSerializer
from rest_framework.serializers import ValidationError

from .common_serializer import CommonSerializer
from .contract_serializer import ContractSerializer
from .data_catalog_serializer import DataCatalogSerializer
from .serializer_utils import validate_json

import logging
_logger = logging.getLogger(__name__)
d = logging.getLogger(__name__).debug

Expand Down
4 changes: 2 additions & 2 deletions src/metax_api/api/base/serializers/common_serializer.py
Original file line number Diff line number Diff line change
@@ -1,11 +1,11 @@
import logging
from collections import OrderedDict

from rest_framework.serializers import ModelSerializer
from rest_framework.fields import SkipField
from rest_framework.relations import PKOnlyObject
from rest_framework.serializers import ModelSerializer
from rest_framework.serializers import ValidationError

import logging
_logger = logging.getLogger(__name__)
d = _logger.debug

Expand Down
2 changes: 2 additions & 0 deletions src/metax_api/api/base/serializers/contract_serializer.py
Original file line number Diff line number Diff line change
@@ -1,7 +1,9 @@
from metax_api.models import Contract

from .common_serializer import CommonSerializer
from .serializer_utils import validate_json


class ContractSerializer(CommonSerializer):

class Meta:
Expand Down
Original file line number Diff line number Diff line change
@@ -1,10 +1,13 @@
from os import path
from rest_framework.serializers import ValidationError

from metax_api.models import DataCatalog
from metax_api.services import DataCatalogService as DCS
from rest_framework.serializers import ValidationError

from .common_serializer import CommonSerializer
from .serializer_utils import validate_json


class DataCatalogSerializer(CommonSerializer):

class Meta:
Expand Down
5 changes: 3 additions & 2 deletions src/metax_api/api/base/serializers/directory_serializer.py
Original file line number Diff line number Diff line change
@@ -1,9 +1,10 @@
from rest_framework.serializers import ValidationError
import logging

from metax_api.models import Directory
from rest_framework.serializers import ValidationError

from .common_serializer import CommonSerializer

import logging
_logger = logging.getLogger(__name__)
d = _logger.debug

Expand Down
6 changes: 4 additions & 2 deletions src/metax_api/api/base/serializers/file_serializer.py
Original file line number Diff line number Diff line change
@@ -1,11 +1,13 @@
import logging

from metax_api.models import File, FileStorage
from rest_framework import serializers
from rest_framework.serializers import ValidationError

from metax_api.models import File, FileStorage
from .common_serializer import CommonSerializer
from .file_storage_serializer import FileStorageSerializer
from .serializer_utils import validate_json
import logging

_logger = logging.getLogger(__name__)
d = logging.getLogger(__name__).debug

Expand Down
2 changes: 2 additions & 0 deletions src/metax_api/api/base/serializers/file_storage_serializer.py
Original file line number Diff line number Diff line change
@@ -1,6 +1,8 @@
from metax_api.models import FileStorage

from .common_serializer import CommonSerializer


class FileStorageSerializer(CommonSerializer):

class Meta:
Expand Down
2 changes: 2 additions & 0 deletions src/metax_api/api/base/serializers/xml_metadata_serializer.py
Original file line number Diff line number Diff line change
@@ -1,6 +1,8 @@
from metax_api.models import XmlMetadata

from .common_serializer import CommonSerializer


class XmlMetadataSerializer(CommonSerializer):

class Meta:
Expand Down
4 changes: 2 additions & 2 deletions src/metax_api/api/base/views/__init__.py
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
from .contract_view import ContractViewSet
from .common_view import CommonViewSet
from .dataset_view import DatasetViewSet
from .contract_view import ContractViewSet
from .data_catalog_view import DataCatalogViewSet
from .dataset_view import DatasetViewSet
from .directory_view import DirectoryViewSet
from .file_view import FileViewSet
7 changes: 4 additions & 3 deletions src/metax_api/api/base/views/contract_view.py
Original file line number Diff line number Diff line change
@@ -1,13 +1,14 @@
import logging

from metax_api.models import Contract
from metax_api.services import CommonService
from rest_framework import status
from rest_framework.decorators import detail_route
from rest_framework.response import Response

from metax_api.models import Contract
from metax_api.services import CommonService
from .common_view import CommonViewSet
from ..serializers import ContractSerializer, CatalogRecordSerializer

import logging
_logger = logging.getLogger(__name__)
d = logging.getLogger(__name__).debug

Expand Down
12 changes: 7 additions & 5 deletions src/metax_api/api/base/views/data_catalog_view.py
Original file line number Diff line number Diff line change
@@ -1,12 +1,14 @@
from metax_api.models import DataCatalog
import logging

from django.http import Http404
from .common_view import CommonViewSet
from ..serializers import DataCatalogSerializer
from metax_api.models import DataCatalog
from rest_framework import status
from rest_framework.decorators import detail_route
from rest_framework.response import Response
from rest_framework import status

import logging
from .common_view import CommonViewSet
from ..serializers import DataCatalogSerializer

_logger = logging.getLogger(__name__)
d = logging.getLogger(__name__).debug

Expand Down
8 changes: 4 additions & 4 deletions src/metax_api/api/base/views/directory_view.py
Original file line number Diff line number Diff line change
@@ -1,16 +1,16 @@
# from django.http import Http404
# from rest_framework import status
from rest_framework.response import Response

from rest_framework.decorators import detail_route, list_route
import logging

from metax_api.api.base.serializers import DirectorySerializer
from metax_api.exceptions import Http400, Http501
from metax_api.models import Directory
from metax_api.services import FileService
from rest_framework.decorators import detail_route, list_route
from rest_framework.response import Response

from .common_view import CommonViewSet

import logging
_logger = logging.getLogger(__name__)
d = logging.getLogger(__name__).debug

Expand Down
11 changes: 6 additions & 5 deletions src/metax_api/api/base/views/file_view.py
Original file line number Diff line number Diff line change
@@ -1,18 +1,19 @@
import logging

from django.http import Http404
from metax_api.exceptions import Http400
from metax_api.models import File, XmlMetadata
from metax_api.renderers import XMLRenderer
from metax_api.services import CommonService, FileService
from rest_framework import status
from rest_framework.decorators import detail_route
from rest_framework.exceptions import ValidationError
from rest_framework.renderers import JSONRenderer
from rest_framework.response import Response

from metax_api.exceptions import Http400
from metax_api.models import File, XmlMetadata
from metax_api.renderers import XMLRenderer
from metax_api.services import CommonService, FileService
from .common_view import CommonViewSet
from ..serializers import FileSerializer, XmlMetadataSerializer

import logging
_logger = logging.getLogger(__name__)
d = logging.getLogger(__name__).debug

Expand Down
4 changes: 2 additions & 2 deletions src/metax_api/middleware/identifyapicaller.py
Original file line number Diff line number Diff line change
@@ -1,12 +1,12 @@
import logging
from base64 import b64decode
import yaml

import yaml
from django.conf import settings as django_settings
from django.http import HttpResponseForbidden
from metax_api.exceptions import Http403
from metax_api.utils import executing_test_case, executing_travis

import logging
_logger = logging.getLogger(__name__)
d = logging.getLogger(__name__).debug

Expand Down
2 changes: 1 addition & 1 deletion src/metax_api/models/__init__.py
Original file line number Diff line number Diff line change
@@ -1,8 +1,8 @@
from .catalog_record import AlternateRecordSet, CatalogRecord
from .contract import Contract
from .data_catalog import DataCatalog
from .directory import Directory
from .file import File
from .file_storage import FileStorage
from .data_catalog import DataCatalog
from .metax_user import MetaxUser
from .xml_metadata import XmlMetadata
1 change: 1 addition & 0 deletions src/metax_api/models/contract.py
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@

from .common import Common


class Contract(Common):

contract_json = JSONField(blank=True, null=True)
Expand Down
1 change: 1 addition & 0 deletions src/metax_api/models/data_catalog.py
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@

from .common import Common


class DataCatalog(Common):

catalog_json = JSONField()
Expand Down
1 change: 1 addition & 0 deletions src/metax_api/models/file_storage.py
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@

from .common import Common


class FileStorage(Common):

file_storage_json = JSONField(blank=True, null=True)
1 change: 1 addition & 0 deletions src/metax_api/models/xml_metadata.py
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@
from .common import Common
from .file import File


class XmlMetadata(Common):

namespace = models.CharField(max_length=200)
Expand Down
5 changes: 2 additions & 3 deletions src/metax_api/onappstart.py
Original file line number Diff line number Diff line change
@@ -1,12 +1,11 @@
from time import sleep
import logging
import sys
from time import sleep

from django.apps import AppConfig
from django.conf import settings

from metax_api.utils import RedisSentinelCache, executing_test_case, ReferenceDataLoader

import logging
_logger = logging.getLogger(__name__)
d = logging.getLogger(__name__).debug

Expand Down
6 changes: 3 additions & 3 deletions src/metax_api/services/file_service.py
Original file line number Diff line number Diff line change
@@ -1,16 +1,16 @@
import logging
from os import getpid
from os.path import dirname, basename
from time import time
from uuid import uuid3, NAMESPACE_DNS as UUID_NAMESPACE_DNS

from django.http import Http404
from rest_framework.serializers import ValidationError

from metax_api.exceptions import Http400
from metax_api.models import Directory, File
from rest_framework.serializers import ValidationError

from .common_service import CommonService

import logging
_logger = logging.getLogger(__name__)
d = logging.getLogger(__name__).debug

Expand Down
3 changes: 1 addition & 2 deletions src/metax_api/services/reference_data_mixin.py
Original file line number Diff line number Diff line change
@@ -1,11 +1,10 @@
import logging
from time import sleep

from django.conf import settings as django_settings

from metax_api.exceptions import Http503
from metax_api.utils import ReferenceDataLoader

import logging
_logger = logging.getLogger(__name__)
d = _logger.debug

Expand Down
1 change: 1 addition & 0 deletions src/metax_api/settings.py
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,7 @@

import logging.config
import os

import yaml
from metax_api.utils import executing_test_case, executing_travis

Expand Down
2 changes: 1 addition & 1 deletion src/metax_api/tests/api/base/apitests/authnz/authnz.py
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
from metax_api.tests.utils import TestClassUtils
from rest_framework import status
from rest_framework.test import APITestCase
from metax_api.tests.utils import TestClassUtils

FORBIDDEN = status.HTTP_403_FORBIDDEN

Expand Down
5 changes: 2 additions & 3 deletions src/metax_api/tests/api/base/apitests/catalog_records/read.py
Original file line number Diff line number Diff line change
@@ -1,9 +1,8 @@
from django.core.management import call_command
from rest_framework import status
from rest_framework.test import APITestCase

from metax_api.models import CatalogRecord
from metax_api.tests.utils import test_data_file_path, TestClassUtils
from rest_framework import status
from rest_framework.test import APITestCase


class CatalogRecordApiReadCommon(APITestCase, TestClassUtils):
Expand Down
4 changes: 2 additions & 2 deletions src/metax_api/tests/api/base/apitests/contracts/contracts.py
Original file line number Diff line number Diff line change
@@ -1,9 +1,9 @@
from django.core.management import call_command
from metax_api.models import Contract, CatalogRecord
from metax_api.tests.utils import test_data_file_path, TestClassUtils
from rest_framework import status
from rest_framework.test import APITestCase

from metax_api.models import Contract, CatalogRecord
from metax_api.tests.utils import test_data_file_path, TestClassUtils

class ContractApiReadTestV1(APITestCase, TestClassUtils):

Expand Down
5 changes: 2 additions & 3 deletions src/metax_api/tests/api/base/apitests/data_catalogs/read.py
Original file line number Diff line number Diff line change
@@ -1,8 +1,7 @@
from rest_framework import status
from rest_framework.test import APITestCase
from django.core.management import call_command

from metax_api.tests.utils import test_data_file_path, TestClassUtils
from rest_framework import status
from rest_framework.test import APITestCase


class DataCatalogApiReadBasicTests(APITestCase, TestClassUtils):
Expand Down
5 changes: 2 additions & 3 deletions src/metax_api/tests/api/base/apitests/data_catalogs/write.py
Original file line number Diff line number Diff line change
@@ -1,8 +1,7 @@
from rest_framework import status
from rest_framework.test import APITestCase
from django.core.management import call_command

from metax_api.tests.utils import test_data_file_path, TestClassUtils
from rest_framework import status
from rest_framework.test import APITestCase


class DataCatalogApiWriteCommon(APITestCase, TestClassUtils):
Expand Down
Loading

0 comments on commit 7534acf

Please sign in to comment.