From 2a4dd5b2e224db4a34870e836dc47989c830d837 Mon Sep 17 00:00:00 2001 From: Xiaohua Zhang Date: Tue, 27 Nov 2018 10:18:21 +0000 Subject: Fix pep8 errors in windriver plugin Fix pep8 errors in swagger,sample and extension Change-Id: I255c08565b7eaf175b9d7506596957ecb2887452 Issue-ID: MULTICLOUD-420 Signed-off-by: Xiaohua Zhang --- windriver/titanium_cloud/swagger/__init__.py | 1 - windriver/titanium_cloud/swagger/tests.py | 32 ----------------------- windriver/titanium_cloud/swagger/tests_swagger.py | 31 ++++++++++++++++++++++ windriver/titanium_cloud/swagger/urls.py | 2 +- windriver/titanium_cloud/swagger/views.py | 29 ++++++++++---------- 5 files changed, 46 insertions(+), 49 deletions(-) delete mode 100644 windriver/titanium_cloud/swagger/tests.py create mode 100644 windriver/titanium_cloud/swagger/tests_swagger.py (limited to 'windriver/titanium_cloud/swagger') diff --git a/windriver/titanium_cloud/swagger/__init__.py b/windriver/titanium_cloud/swagger/__init__.py index afa702d3..ae1ce9db 100644 --- a/windriver/titanium_cloud/swagger/__init__.py +++ b/windriver/titanium_cloud/swagger/__init__.py @@ -11,4 +11,3 @@ # WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. # See the License for the specific language governing permissions and # limitations under the License. - diff --git a/windriver/titanium_cloud/swagger/tests.py b/windriver/titanium_cloud/swagger/tests.py deleted file mode 100644 index fde26217..00000000 --- a/windriver/titanium_cloud/swagger/tests.py +++ /dev/null @@ -1,32 +0,0 @@ -# Copyright (c) 2017-2018 Wind River Systems, Inc. -# -# Licensed under the Apache License, Version 2.0 (the "License"); -# you may not use this file except in compliance with the License. -# You may obtain a copy of the License at -# -# http://www.apache.org/licenses/LICENSE-2.0 -# -# Unless required by applicable law or agreed to in writing, software -# distributed under the License is distributed on an "AS IS" BASIS, -# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. -# See the License for the specific language governing permissions and -# limitations under the License. - -import unittest -import json -from django.test import Client -from rest_framework import status - - -class SampleViewTest(unittest.TestCase): - def setUp(self): - self.client = Client() - - def tearDown(self): - pass - - def test_sample(self): - response = self.client.get("/api/multicloud-titaniumcloud/v0/swagger.json") - self.assertEqual(status.HTTP_200_OK, response.status_code, response.content) -# resp_data = response.json() -# self.assertEqual({"status": "active"}, resp_data) diff --git a/windriver/titanium_cloud/swagger/tests_swagger.py b/windriver/titanium_cloud/swagger/tests_swagger.py new file mode 100644 index 00000000..eba67c06 --- /dev/null +++ b/windriver/titanium_cloud/swagger/tests_swagger.py @@ -0,0 +1,31 @@ +# Copyright (c) 2017-2018 Wind River Systems, Inc. +# +# Licensed under the Apache License, Version 2.0 (the "License"); +# you may not use this file except in compliance with the License. +# You may obtain a copy of the License at +# +# http://www.apache.org/licenses/LICENSE-2.0 +# +# Unless required by applicable law or agreed to in writing, software +# distributed under the License is distributed on an "AS IS" BASIS, +# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. +# See the License for the specific language governing permissions and +# limitations under the License. + +import unittest +from django.test import Client +from rest_framework import status + + +class SampleViewTest(unittest.TestCase): + def setUp(self): + self.client = Client() + + def tearDown(self): + pass + + def test_sample(self): + response = self.client.get("/api/multicloud-titaniumcloud/v0/swagger.json") + self.assertEqual(status.HTTP_200_OK, response.status_code, response.content) +# resp_data = response.json() +# self.assertEqual({"status": "active"}, resp_data) diff --git a/windriver/titanium_cloud/swagger/urls.py b/windriver/titanium_cloud/swagger/urls.py index 38573d58..a690d9cf 100644 --- a/windriver/titanium_cloud/swagger/urls.py +++ b/windriver/titanium_cloud/swagger/urls.py @@ -12,7 +12,7 @@ # See the License for the specific language governing permissions and # limitations under the License. -from django.conf.urls import patterns, url +from django.conf.urls import url from rest_framework.urlpatterns import format_suffix_patterns from titanium_cloud.swagger.views import SwaggerJsonViewDepreciated diff --git a/windriver/titanium_cloud/swagger/views.py b/windriver/titanium_cloud/swagger/views.py index 6829714b..bbf897e1 100644 --- a/windriver/titanium_cloud/swagger/views.py +++ b/windriver/titanium_cloud/swagger/views.py @@ -12,16 +12,16 @@ # See the License for the specific language governing permissions and # limitations under the License. -import json +# import json import logging -import os -import traceback +# import os +# import traceback -from rest_framework import status +# from rest_framework import status from rest_framework.response import Response -from rest_framework.views import APIView +# from rest_framework.views import APIView -from common.exceptions import VimDriverNewtonException +# from common.exceptions import VimDriverNewtonException from newton_base.swagger import views as newton_json_view logger = logging.getLogger(__name__) @@ -36,15 +36,14 @@ class SwaggerJsonViewDepreciated(newton_json_view.SwaggerJsonView): :return: ''' - resp = super(SwaggerJsonViewDepreciated,self).get(request) + resp = super(SwaggerJsonViewDepreciated, self).get(request) json_data = resp.data if resp else None if json_data: json_data["basePath"] = "/api/multicloud-titanium_cloud/v0/" json_data["info"]["title"] = "Depreciated Service NBI of MultiCloud plugin for Titanium Cloud" return Response(data=json_data, status=200) else: - return Response(data={'error':'internal error'}, status=500) - + return Response(data={'error': 'internal error'}, status=500) class APIv1SwaggerJsonViewDepreciated(newton_json_view.SwaggerJsonView): @@ -56,14 +55,14 @@ class APIv1SwaggerJsonViewDepreciated(newton_json_view.SwaggerJsonView): :return: ''' - resp = super(APIv1SwaggerJsonViewDepreciated,self).get(request) + resp = super(APIv1SwaggerJsonViewDepreciated, self).get(request) json_data = resp.data if resp else None if json_data: json_data["basePath"] = "/api/multicloud-titanium_cloud/v1/" json_data["info"]["title"] = "Depreciated Service NBI v1 of MultiCloud plugin for Titanium Cloud" return Response(data=json_data, status=200) else: - return Response(data={'error':'internal error'}, status=500) + return Response(data={'error': 'internal error'}, status=500) class SwaggerJsonView(newton_json_view.SwaggerJsonView): @@ -75,14 +74,14 @@ class SwaggerJsonView(newton_json_view.SwaggerJsonView): :return: ''' - resp = super(SwaggerJsonView,self).get(request) + resp = super(SwaggerJsonView, self).get(request) json_data = resp.data if resp else None if json_data: json_data["basePath"] = "/api/multicloud-titaniumcloud/v0/" json_data["info"]["title"] = "Service NBI v0 of MultiCloud plugin for Titanium Cloud" return Response(data=json_data, status=200) else: - return Response(data={'error':'internal error'}, status=500) + return Response(data={'error': 'internal error'}, status=500) class APIv1SwaggerJsonView(newton_json_view.SwaggerJsonView): @@ -94,11 +93,11 @@ class APIv1SwaggerJsonView(newton_json_view.SwaggerJsonView): :return: ''' - resp = super(APIv1SwaggerJsonView,self).get(request) + resp = super(APIv1SwaggerJsonView, self).get(request) json_data = resp.data if resp else None if json_data: json_data["basePath"] = "/api/multicloud-titaniumcloud/v1/" json_data["info"]["title"] = "Service NBI v1 of MultiCloud plugin for Titanium Cloud" return Response(data=json_data, status=200) else: - return Response(data={'error':'internal error'}, status=500) + return Response(data={'error': 'internal error'}, status=500) -- cgit 1.2.3-korg