aboutsummaryrefslogtreecommitdiffstats
path: root/genericparser
diff options
context:
space:
mode:
authordyh <dengyuanhong@chinamobile.com>2019-04-24 15:47:22 +0800
committeryangyan <yangyanyj@chinamobile.com>2019-04-25 09:41:37 +0800
commitd91657fce78f76ddb7467651c6e9647c6fcf9e67 (patch)
treeb08753fa04585d48a40e1a8efd361134267c1456 /genericparser
parent1956f96041fb81e46a1fb5973da5acb80349c8ba (diff)
update coverage command
Change-Id: Ibbc6ce866296f61cf5b100c7da4e931c3b177531 Issue-ID: MODELING-159 Signed-off-by: dyh <dengyuanhong@chinamobile.com>
Diffstat (limited to 'genericparser')
-rw-r--r--genericparser/packages/biz/nsdm_subscription.py15
-rw-r--r--genericparser/packages/tests/test_health_check.py6
-rw-r--r--genericparser/packages/tests/test_nspackage.py30
-rw-r--r--genericparser/packages/tests/test_pnf_descriptor.py3
-rw-r--r--genericparser/packages/tests/test_vnfpackage.py14
-rw-r--r--genericparser/pub/database/migrations/0002_auto_20190422_1442.py47
-rw-r--r--genericparser/settings.py4
7 files changed, 78 insertions, 41 deletions
diff --git a/genericparser/packages/biz/nsdm_subscription.py b/genericparser/packages/biz/nsdm_subscription.py
index ba74c70..9bfe5aa 100644
--- a/genericparser/packages/biz/nsdm_subscription.py
+++ b/genericparser/packages/biz/nsdm_subscription.py
@@ -94,11 +94,11 @@ class NsdmSubscription:
response = requests.get(self.callback_uri, timeout=2)
if response.status_code != status.HTTP_204_NO_CONTENT:
raise GenericparserException("callbackUri %s returns %s status "
- "code." % (self.callback_uri,
- response.status_code))
+ "code." % (self.callback_uri,
+ response.status_code))
except Exception:
raise GenericparserException("callbackUri %s didn't return 204 status"
- "code." % self.callback_uri)
+ "code." % self.callback_uri)
def fill_resp_data(self, subscription):
subscription_filter = dict()
@@ -158,8 +158,7 @@ class NsdmSubscription:
if const.BASIC in self.authentication.get("authType", '') and \
"paramsBasic" in self.authentication.keys() and \
not is_filter_type_equal(PARAMSBASICKEYS,
- self.authentication.
- get("paramsBasic").keys()):
+ self.authentication.get("paramsBasic").keys()):
raise NsdmBadRequestException('userName and password needed '
'for ' + const.BASIC)
if const.OAUTH2_CLIENT_CREDENTIALS in \
@@ -167,9 +166,7 @@ class NsdmSubscription:
"paramsOauth2ClientCredentials" in \
self.authentication.keys() and \
not is_filter_type_equal(PARAMSOAUTH2CLIENTCREDENTIALSKEYS,
- self.authentication.
- get("paramsOauth2ClientCredentials")
- .keys()):
+ self.authentication.get("paramsOauth2ClientCredentials").keys()):
raise NsdmBadRequestException('clientId, clientPassword and '
'tokenEndpoint required '
'for ' + const.OAUTH2_CLIENT_CREDENTIALS)
@@ -203,7 +200,7 @@ class NsdmSubscription:
links = {
"self": {
"href":
- const.NSDM_SUBSCRIPTION_ROOT_URI + self.subscription_id
+ const.NSDM_SUBSCRIPTION_ROOT_URI + self.subscription_id
}
}
subscription_save_db = {
diff --git a/genericparser/packages/tests/test_health_check.py b/genericparser/packages/tests/test_health_check.py
index e3befca..e074bba 100644
--- a/genericparser/packages/tests/test_health_check.py
+++ b/genericparser/packages/tests/test_health_check.py
@@ -37,12 +37,6 @@ class TestHealthCheck(TestCase):
resp_data = json.loads(response.content)
self.assertEqual({"status": "active"}, resp_data)
- def test_catalog_health_check(self):
- response = self.client.get("/api/genericparser/v1/health_check")
- self.assertEqual(status.HTTP_200_OK, response.status_code, response.content)
- resp_data = json.loads(response.content)
- self.assertEqual({"status": "active"}, resp_data)
-
def test_parser_health_check(self):
response = self.client.get("/api/parser/v1/health_check")
self.assertEqual(status.HTTP_200_OK, response.status_code, response.content)
diff --git a/genericparser/packages/tests/test_nspackage.py b/genericparser/packages/tests/test_nspackage.py
index f44488e..100ebd5 100644
--- a/genericparser/packages/tests/test_nspackage.py
+++ b/genericparser/packages/tests/test_nspackage.py
@@ -246,10 +246,8 @@ class TestNsPackage(TestCase):
"version": 1,
"name": "VCPE_NS",
"desginer": "ZTE",
- "invariant_id": "vcpe_ns_sff_1"
- }
- }
- }
+ "invariant_id": "vcpe_ns_sff_1"}}
+ }
def tearDown(self):
pass
@@ -257,7 +255,7 @@ class TestNsPackage(TestCase):
def test_ns_pkg_distribute_when_ns_exists(self):
NSPackageModel(nsPackageId="1", nsdId="2").save()
resp = self.client.post(
- "/api/genericparser/v1/nspackages", {"csarId": "1"}, format='json')
+ "/api/parser/v1/nspackages", {"csarId": "1"}, format='json')
self.assertEqual(resp.status_code, status.HTTP_202_ACCEPTED)
self.assertEqual("failed", resp.data["status"])
self.assertEqual(
@@ -268,7 +266,7 @@ class TestNsPackage(TestCase):
def test_ns_pkg_distribute_when_csar_not_exist(self, mock_call_req):
mock_call_req.return_value = [0, "[]", '200']
resp = self.client.post(
- "/api/genericparser/v1/nspackages", {"csarId": "1"}, format='json')
+ "/api/parser/v1/nspackages", {"csarId": "1"}, format='json')
self.assertEqual(resp.status_code, status.HTTP_202_ACCEPTED)
self.assertEqual("failed", resp.data["status"])
self.assertEqual(
@@ -289,7 +287,7 @@ class TestNsPackage(TestCase):
}]), '200']
NSPackageModel(nsPackageId="2", nsdId="VCPE_NS").save()
resp = self.client.post(
- "/api/genericparser/v1/nspackages", {"csarId": "1"}, format='json')
+ "/api/parser/v1/nspackages", {"csarId": "1"}, format='json')
self.assertEqual(resp.status_code, status.HTTP_202_ACCEPTED)
self.assertEqual("failed", resp.data["status"])
self.assertEqual(
@@ -309,7 +307,7 @@ class TestNsPackage(TestCase):
"distributionStatus": "DISTRIBUTED",
}]), '200']
resp = self.client.post(
- "/api/genericparser/v1/nspackages", {"csarId": "1"}, format='json')
+ "/api/parser/v1/nspackages", {"csarId": "1"}, format='json')
self.assertEqual(resp.status_code, status.HTTP_202_ACCEPTED)
self.assertEqual("failed", resp.data["status"])
self.assertEqual(
@@ -331,7 +329,7 @@ class TestNsPackage(TestCase):
VnfPackageModel(vnfPackageId="1", vnfdId="vcpe_vfw_zte_1_0").save()
PnfPackageModel(pnfPackageId="1", pnfdId="m6000_s").save()
resp = self.client.post(
- "/api/genericparser/v1/nspackages", {"csarId": "1"}, format='json')
+ "/api/parser/v1/nspackages", {"csarId": "1"}, format='json')
self.assertEqual(resp.status_code, status.HTTP_202_ACCEPTED)
self.assertEqual("success", resp.data["status"])
self.assertEqual(
@@ -354,7 +352,7 @@ class TestNsPackage(TestCase):
"lastUpdaterUserId": "jh0003"
}]
resp = self.client.post(
- "/api/genericparser/v1/nspackages", {"csarId": "1"}, format='json')
+ "/api/parser/v1/nspackages", {"csarId": "1"}, format='json')
self.assertEqual(resp.status_code, status.HTTP_202_ACCEPTED)
self.assertEqual("failed", resp.data["status"])
self.assertEqual(
@@ -365,7 +363,7 @@ class TestNsPackage(TestCase):
def test_ns_pkg_normal_delete(self):
NSPackageModel(nsPackageId="8", nsdId="2").save()
- resp = self.client.delete("/api/genericparser/v1/nspackages/8")
+ resp = self.client.delete("/api/parser/v1/nspackages/8")
self.assertEqual(resp.status_code, status.HTTP_200_OK)
self.assertEqual("success", resp.data["status"])
self.assertEqual(
@@ -387,7 +385,7 @@ class TestNsPackage(TestCase):
nsdVersion="3",
nsPackageUri="14.csar",
nsdModel="").save()
- resp = self.client.get("/api/genericparser/v1/nspackages")
+ resp = self.client.get("/api/parser/v1/nspackages")
self.assertEqual(resp.status_code, status.HTTP_200_OK)
expect_data = [{"csarId": "13",
"packageInfo": {"csarName": "13.csar",
@@ -418,7 +416,7 @@ class TestNsPackage(TestCase):
nsdVersion="4",
nsPackageUri="14.csar",
nsdModel="").save()
- resp = self.client.get("/api/genericparser/v1/nspackages/14")
+ resp = self.client.get("/api/parser/v1/nspackages/14")
self.assertEqual(resp.status_code, status.HTTP_200_OK)
expect_data = {
"csarId": "14",
@@ -434,7 +432,7 @@ class TestNsPackage(TestCase):
self.assertEqual(expect_data, resp.data)
def test_ns_pkg_get_one_not_found(self):
- resp = self.client.get("/api/genericparser/v1/nspackages/22")
+ resp = self.client.get("/api/parser/v1/nspackages/22")
self.assertEqual(resp.status_code, status.HTTP_500_INTERNAL_SERVER_ERROR)
self.assertEqual(
{"error": "Ns package[22] not Found."},
@@ -448,7 +446,7 @@ class TestNsPackage(TestCase):
mock_parse_nsd.return_value = json.JSONEncoder().encode({"a": "b"})
req_data = {"csarId": "18", "inputs": []}
resp = self.client.post(
- "/api/genericparser/v1/parsernsd",
+ "/api/parser/v1/parsernsd",
req_data,
format='json')
self.assertEqual(resp.status_code, status.HTTP_202_ACCEPTED)
@@ -457,7 +455,7 @@ class TestNsPackage(TestCase):
def test_nsd_parse_when_csar_not_exist(self):
req_data = {"csarId": "1", "inputs": []}
resp = self.client.post(
- "/api/genericparser/v1/parsernsd",
+ "/api/parser/v1/parsernsd",
req_data,
format='json')
self.assertEqual(
diff --git a/genericparser/packages/tests/test_pnf_descriptor.py b/genericparser/packages/tests/test_pnf_descriptor.py
index a6da1db..22615f5 100644
--- a/genericparser/packages/tests/test_pnf_descriptor.py
+++ b/genericparser/packages/tests/test_pnf_descriptor.py
@@ -18,7 +18,6 @@ import json
import mock
import os
-
from django.test import TestCase
from rest_framework import status
from rest_framework.test import APIClient
@@ -277,6 +276,6 @@ class TestPnfDescriptor(TestCase):
PnfPackageModel(pnfPackageId="8", pnfdId="10").save()
mock_parse_pnfd.return_value = json.JSONEncoder().encode({"c": "d"})
req_data = {"csarId": "8", "inputs": []}
- resp = self.client.post("/api/genericparser/v1/parserpnfd", req_data, format='json')
+ resp = self.client.post("/api/parser/v1/parserpnfd", req_data, format='json')
self.assertEqual(resp.status_code, status.HTTP_202_ACCEPTED)
self.assertEqual({"model": '{"c": "d"}'}, resp.data)
diff --git a/genericparser/packages/tests/test_vnfpackage.py b/genericparser/packages/tests/test_vnfpackage.py
index b969b4f..34b9134 100644
--- a/genericparser/packages/tests/test_vnfpackage.py
+++ b/genericparser/packages/tests/test_vnfpackage.py
@@ -257,7 +257,7 @@ class TestNfPackage(TestCase):
@mock.patch.object(NfDistributeThread, 'run')
def test_nf_pkg_distribute_normal(self, mock_run):
- resp = self.client.post("/api/genericparser/v1/vnfpackages", {
+ resp = self.client.post("/api/parser/v1/vnfpackages", {
"csarId": "1",
"vimIds": ["1"]
}, format='json')
@@ -310,7 +310,7 @@ class TestNfPackage(TestCase):
@mock.patch.object(NfPkgDeleteThread, 'run')
def test_nf_pkg_delete_normal(self, mock_run):
- resp = self.client.delete("/api/genericparser/v1/vnfpackages/1")
+ resp = self.client.delete("/api/parser/v1/vnfpackages/1")
self.assertEqual(resp.status_code, status.HTTP_202_ACCEPTED)
def test_nf_pkg_normal_delete(self):
@@ -323,7 +323,7 @@ class TestNfPackage(TestCase):
vnfSoftwareVersion='', vnfPackageUri='', vnfdModel='').save()
VnfPackageModel(vnfPackageId="4", vnfdId="4", vnfVendor='4', vnfdVersion='4',
vnfSoftwareVersion='', vnfPackageUri='', vnfdModel='').save()
- resp = self.client.get("/api/genericparser/v1/vnfpackages")
+ resp = self.client.get("/api/parser/v1/vnfpackages")
self.assertEqual(resp.status_code, status.HTTP_200_OK)
expect_data = [
{
@@ -361,7 +361,7 @@ class TestNfPackage(TestCase):
VnfPackageModel(vnfPackageId="4", vnfdId="4", vnfVendor='4', vnfdVersion='4',
vnfSoftwareVersion='', vnfPackageUri='', vnfdModel='').save()
- resp = self.client.get("/api/genericparser/v1/vnfpackages/4")
+ resp = self.client.get("/api/parser/v1/vnfpackages/4")
self.assertEqual(resp.status_code, status.HTTP_200_OK)
expect_data = {
"imageInfo": [],
@@ -383,7 +383,7 @@ class TestNfPackage(TestCase):
VnfPackageModel(vnfPackageId="4", vnfdId="4", vnfVendor='4', vnfdVersion='4',
vnfSoftwareVersion='', vnfPackageUri='', vnfdModel='').save()
- resp = self.client.get("/api/genericparser/v1/vnfpackages/2")
+ resp = self.client.get("/api/parser/v1/vnfpackages/2")
self.assertEqual(resp.status_code, status.HTTP_500_INTERNAL_SERVER_ERROR)
self.assertEqual({'error': 'Vnf package[2] not Found.'}, resp.data)
@@ -394,12 +394,12 @@ class TestNfPackage(TestCase):
VnfPackageModel(vnfPackageId="8", vnfdId="10").save()
mock_parse_vnfd.return_value = json.JSONEncoder().encode({"c": "d"})
req_data = {"csarId": "8", "inputs": []}
- resp = self.client.post("/api/genericparser/v1/parservnfd", req_data, format='json')
+ resp = self.client.post("/api/parser/v1/parservnfd", req_data, format='json')
self.assertEqual(resp.status_code, status.HTTP_202_ACCEPTED)
self.assertEqual({"model": '{"c": "d"}'}, resp.data)
def test_vnfd_parse_when_csar_not_exist(self):
req_data = {"csarId": "1", "inputs": []}
- resp = self.client.post("/api/genericparser/v1/parservnfd", req_data, format='json')
+ resp = self.client.post("/api/parser/v1/parservnfd", req_data, format='json')
self.assertEqual(resp.status_code, status.HTTP_500_INTERNAL_SERVER_ERROR)
self.assertEqual(resp.data, {"error": "VNF CSAR(1) does not exist."})
diff --git a/genericparser/pub/database/migrations/0002_auto_20190422_1442.py b/genericparser/pub/database/migrations/0002_auto_20190422_1442.py
new file mode 100644
index 0000000..da23582
--- /dev/null
+++ b/genericparser/pub/database/migrations/0002_auto_20190422_1442.py
@@ -0,0 +1,47 @@
+# -*- coding: utf-8 -*-
+# Generated by Django 1.11.9 on 2019-04-22 14:42
+from __future__ import unicode_literals
+
+from django.db import migrations
+
+
+class Migration(migrations.Migration):
+
+ dependencies = [
+ ('database', '0001_initial'),
+ ]
+
+ operations = [
+ migrations.AlterModelTable(
+ name='jobmodel',
+ table='GENERICPARSER_JOB',
+ ),
+ migrations.AlterModelTable(
+ name='jobstatusmodel',
+ table='GENERICPARSER_JOB_STATUS',
+ ),
+ migrations.AlterModelTable(
+ name='nsdmsubscriptionmodel',
+ table='GENERICPARSER_NSDM_SUBSCRIPTION',
+ ),
+ migrations.AlterModelTable(
+ name='nspackagemodel',
+ table='GENERICPARSER_NSPACKAGE',
+ ),
+ migrations.AlterModelTable(
+ name='pnfpackagemodel',
+ table='GENERICPARSER_PNFPACKAGE',
+ ),
+ migrations.AlterModelTable(
+ name='servicepackagemodel',
+ table='GENERICPARSER_SERVICEPACKAGE',
+ ),
+ migrations.AlterModelTable(
+ name='softwareimagemodel',
+ table='GENERICPARSER_SOFTWAREIMAGEMODEL',
+ ),
+ migrations.AlterModelTable(
+ name='vnfpackagemodel',
+ table='GENERICPARSER_VNFPACKAGE',
+ ),
+ ]
diff --git a/genericparser/settings.py b/genericparser/settings.py
index a3310b9..9bdeb1e 100644
--- a/genericparser/settings.py
+++ b/genericparser/settings.py
@@ -23,6 +23,7 @@ from genericparser.pub.config.config import DB_NAME, DB_IP, DB_USER, DB_PASSWD,
from genericparser.pub.config import config as pub_config
from logging import config as log_config
from onaplogging import monkey
+
monkey.patch_all()
# Build paths inside the project like this: os.path.join(BASE_DIR, ...)
@@ -86,7 +87,7 @@ MIDDLEWARE_CLASSES = [
'django.contrib.auth.middleware.SessionAuthenticationMiddleware',
'django.contrib.messages.middleware.MessageMiddleware',
'django.middleware.clickjacking.XFrameOptionsMiddleware',
- 'genericparser.middleware.LogContextMiddleware',
+ 'genericparser.middleware.LogContextMiddleware',
]
ROOT_URLCONF = 'genericparser.urls'
@@ -189,5 +190,6 @@ if 'test' in sys.argv:
import mock
from genericparser.pub.utils import idutil
+
idutil.get_auto_id = mock.Mock()
idutil.get_auto_id.return_value = 1