summaryrefslogtreecommitdiffstats
path: root/src/test/groovy
diff options
context:
space:
mode:
authoremaclee <lee.anjella.macabuhay@est.tech>2022-07-15 12:31:46 +0100
committeremaclee <lee.anjella.macabuhay@est.tech>2022-07-26 10:18:46 +0100
commitf5f5e6be4e7d222a7530b96d2d77ed611934822f (patch)
treee329f206b51163aa70d853bed82010bc820f77bf /src/test/groovy
parentaf8d434068e6f0d2b88c152b069afdcaf162f6c2 (diff)
Groovy clean tests clean-up
- test labels are modified to set standards for writing tests - labels are not expected to be technical - labels need to be in simple language that ANYONE can understand - labels should not have exact method names except for some mocking - variables are modified to give highlight to ONLY important parameter of test - wild cards are preferred when possible i.e. for exception cases Issue-ID: CPS-588 Signed-off-by: emaclee <lee.anjella.macabuhay@est.tech> Change-Id: I3d42471e7aa4bc61f962ad19cc1f237f6985a9a2
Diffstat (limited to 'src/test/groovy')
-rw-r--r--src/test/groovy/org/onap/cps/ncmp/dmi/rest/controller/ControllerSecuritySpec.groovy2
-rw-r--r--src/test/groovy/org/onap/cps/ncmp/dmi/rest/controller/DmiRestControllerSpec.groovy112
-rw-r--r--src/test/groovy/org/onap/cps/ncmp/dmi/service/DmiServiceImplSpec.groovy204
-rw-r--r--src/test/groovy/org/onap/cps/ncmp/dmi/service/client/NcmpRestClientSpec.groovy16
-rw-r--r--src/test/groovy/org/onap/cps/ncmp/dmi/service/client/SdncRestconfClientSpec.groovy38
-rw-r--r--src/test/groovy/org/onap/cps/ncmp/dmi/service/operation/SdncOperationsSpec.groovy37
6 files changed, 205 insertions, 204 deletions
diff --git a/src/test/groovy/org/onap/cps/ncmp/dmi/rest/controller/ControllerSecuritySpec.groovy b/src/test/groovy/org/onap/cps/ncmp/dmi/rest/controller/ControllerSecuritySpec.groovy
index 8b13fc79..67de1b93 100644
--- a/src/test/groovy/org/onap/cps/ncmp/dmi/rest/controller/ControllerSecuritySpec.groovy
+++ b/src/test/groovy/org/onap/cps/ncmp/dmi/rest/controller/ControllerSecuritySpec.groovy
@@ -38,7 +38,7 @@ class ControllerSecuritySpec extends Specification {
def testEndpoint = '/test'
- def 'Get request with authentication'() {
+ def 'Get request with valid authentication'() {
when: 'request is sent with authentication'
def response = mvc.perform(
get(testEndpoint).header("Authorization", 'Basic Y3BzdXNlcjpjcHNyMGNrcyE=')
diff --git a/src/test/groovy/org/onap/cps/ncmp/dmi/rest/controller/DmiRestControllerSpec.groovy b/src/test/groovy/org/onap/cps/ncmp/dmi/rest/controller/DmiRestControllerSpec.groovy
index 5bfbc400..2be0b59c 100644
--- a/src/test/groovy/org/onap/cps/ncmp/dmi/rest/controller/DmiRestControllerSpec.groovy
+++ b/src/test/groovy/org/onap/cps/ncmp/dmi/rest/controller/DmiRestControllerSpec.groovy
@@ -77,20 +77,20 @@ class DmiRestControllerSpec extends Specification {
def basePathV1
def 'Get all modules.'() {
- given: 'REST endpoint for getting all modules'
+ given: 'URL for getting all modules and some request data'
def getModuleUrl = "$basePathV1/ch/node1/modules"
- and: 'get modules for cm-handle returns a json'
- def json = '{"cmHandleProperties" : {}}'
+ def someValidJson = '{}'
+ and: 'DMI service returns some module'
def moduleSetSchema = new ModuleSetSchemas(namespace:'some-namespace',
- moduleName:'some-moduleName',
- revision:'some-revision')
+ moduleName:'some-moduleName',
+ revision:'some-revision')
def moduleSetSchemasList = [moduleSetSchema] as List<ModuleSetSchemas>
def moduleSet = new ModuleSet()
moduleSet.schemas(moduleSetSchemasList)
mockDmiService.getModulesForCmHandle('node1') >> moduleSet
- when: 'post is being called'
+ when: 'the request is posted'
def response = mvc.perform(post(getModuleUrl)
- .contentType(MediaType.APPLICATION_JSON).content(json))
+ .contentType(MediaType.APPLICATION_JSON).content(someValidJson))
.andReturn().response
then: 'status is OK'
response.status == OK.value()
@@ -99,14 +99,14 @@ class DmiRestControllerSpec extends Specification {
}
def 'Get all modules with exception handling of #scenario.'() {
- given: 'REST endpoint for getting all modules'
+ given: 'URL for getting all modules and some request data'
def getModuleUrl = "$basePathV1/ch/node1/modules"
- and: 'given request body and get modules for cm-handle throws #exceptionClass'
- def json = '{"cmHandleProperties" : {}}'
+ def someValidJson = '{}'
+ and: 'a #exception is thrown during the process'
mockDmiService.getModulesForCmHandle('node1') >> { throw exception }
- when: 'post is invoked'
+ when: 'the request is posted'
def response = mvc.perform( post(getModuleUrl)
- .contentType(MediaType.APPLICATION_JSON).content(json))
+ .contentType(MediaType.APPLICATION_JSON).content(someValidJson))
.andReturn().response
then: 'response status is #expectedResponse'
response.status == expectedResponse.value()
@@ -119,10 +119,10 @@ class DmiRestControllerSpec extends Specification {
}
def 'Register given list.'() {
- given: 'register cm handle url and cm handles json'
+ given: 'register cm handle url and cmHandles'
def registerCmhandlesPost = "${basePathV1}/inventory/cmHandles"
def cmHandleJson = '{"cmHandles":["node1", "node2"]}'
- when: 'register cm handles api is invoked with POST'
+ when: 'the request is posted'
def response = mvc.perform(
post(registerCmhandlesPost)
.contentType(MediaType.APPLICATION_JSON)
@@ -138,7 +138,7 @@ class DmiRestControllerSpec extends Specification {
given: 'register cm handle url and empty json'
def registerCmhandlesPost = "${basePathV1}/inventory/cmHandles"
def emptyJson = '{"cmHandles":[]}'
- when: 'register cm handles post api is invoked with no content'
+ when: 'the request is posted'
def response = mvc.perform(
post(registerCmhandlesPost).contentType(MediaType.APPLICATION_JSON)
.content(emptyJson)
@@ -150,9 +150,10 @@ class DmiRestControllerSpec extends Specification {
}
def 'Retrieve module resources.'() {
- given: 'an endpoint and json data'
+ given: 'URL to get module resources'
def getModulesEndpoint = "$basePathV1/ch/some-cm-handle/moduleResources"
- String jsonData = TestUtils.getResourceFileContent('GetModules.json')
+ and: 'request data to get some modules'
+ String jsonData = TestUtils.getResourceFileContent('moduleResources.json')
and: 'the DMI service returns the yang resources'
ModuleReference moduleReference1 = new ModuleReference(name: 'ietf-yang-library', revision: '2016-06-21')
ModuleReference moduleReference2 = new ModuleReference(name: 'nc-notifications', revision: '2008-07-14')
@@ -161,23 +162,24 @@ class DmiRestControllerSpec extends Specification {
def yangResource = new YangResource(yangSource: '"some-data"', moduleName: 'NAME', revision: 'REVISION')
yangResources.add(yangResource)
mockDmiService.getModuleResources('some-cm-handle', moduleReferences) >> yangResources
- when: 'get module resource api is invoked'
+ when: 'the request is posted'
def response = mvc.perform(post(getModulesEndpoint)
.contentType(MediaType.APPLICATION_JSON)
.content(jsonData)).andReturn().response
then: 'a OK status is returned'
response.status == OK.value()
- and: 'the expected response is returned'
+ and: 'the response content matches the result from the DMI service'
response.getContentAsString() == '[{"yangSource":"\\"some-data\\"","moduleName":"NAME","revision":"REVISION"}]'
}
def 'Retrieve module resources with exception handling.'() {
- given: 'an endpoint and json data'
+ given: 'URL to get module resources'
def getModulesEndpoint = "$basePathV1/ch/some-cm-handle/moduleResources"
- String jsonData = TestUtils.getResourceFileContent('GetModules.json')
- and: 'the service method is invoked to get module resources and throws an exception'
+ and: 'request data to get some modules'
+ String jsonData = TestUtils.getResourceFileContent('moduleResources.json')
+ and: 'the system throws a not-found exception (during the processing)'
mockDmiService.getModuleResources('some-cm-handle', _) >> { throw Mock(ModuleResourceNotFoundException.class) }
- when: 'get module resource api is invoked'
+ when: 'the request is posted'
def response = mvc.perform(post(getModulesEndpoint)
.contentType(MediaType.APPLICATION_JSON)
.content(jsonData)).andReturn().response
@@ -186,31 +188,32 @@ class DmiRestControllerSpec extends Specification {
}
def 'Get resource data for pass-through operational.'() {
- given: 'Get resource data url'
+ given: 'Get resource data url and some request data'
def getResourceDataForCmHandleUrl = "${basePathV1}/ch/some-cmHandle/data/ds/ncmp-datastore:passthrough-operational" +
"?resourceIdentifier=parent/child&options=(fields=myfields,depth=5)"
- def json = '{"cmHandleProperties" : { "prop1" : "value1", "prop2" : "value2"}}'
- when: 'get resource data POST api is invoked'
+ def someValidJson = '{}'
+ when: 'the request is posted'
def response = mvc.perform(
- post(getResourceDataForCmHandleUrl).contentType(MediaType.APPLICATION_JSON).content(json)
+ post(getResourceDataForCmHandleUrl).contentType(MediaType.APPLICATION_JSON).content(someValidJson)
).andReturn().response
then: 'response status is ok'
response.status == OK.value()
- and: 'dmi service called with get resource data'
+ and: 'dmi service method to get resource data is invoked once'
1 * mockDmiService.getResourceData('some-cmHandle',
'parent/child',
'(fields=myfields,depth=5)',
'content=all')
}
- def 'Get resource data for pass-through operational with bad request.'() {
+ def 'Get resource data for pass-through operational with write request (invalid).'() {
given: 'Get resource data url'
def getResourceDataForCmHandleUrl = "${basePathV1}/ch/some-cmHandle/data/ds/ncmp-datastore:passthrough-operational" +
- "?resourceIdentifier=parent/child&options=(fields=myfields,depth=5)"
- def jsonData = TestUtils.getResourceFileContent('createDataWithNormalChar.json')
- when: 'get resource data POST api is invoked'
+ "?resourceIdentifier=parent/child&options=(fields=myfields,depth=5)"
+ and: 'an invalid write request data for "create" operation'
+ def jsonData = '{"operation":"create"}'
+ when: 'the request is posted'
def response = mvc.perform(
- post(getResourceDataForCmHandleUrl).contentType(MediaType.APPLICATION_JSON).content(jsonData)
+ post(getResourceDataForCmHandleUrl).contentType(MediaType.APPLICATION_JSON).content(jsonData)
).andReturn().response
then: 'response status is bad request'
response.status == BAD_REQUEST.value()
@@ -219,22 +222,23 @@ class DmiRestControllerSpec extends Specification {
}
def 'data with #scenario operation using passthrough running.'() {
- given: 'write data for passthrough running url and jsonData'
+ given: 'write data for passthrough running url'
def writeDataForPassthroughRunning = "${basePathV1}/ch/some-cmHandle/data/ds/ncmp-datastore:passthrough-running" +
"?resourceIdentifier=some-resourceIdentifier"
+ and: 'request data for #scenario'
def jsonData = TestUtils.getResourceFileContent(requestBodyFile)
and: 'dmi service is called'
mockDmiService.writeData(operationEnum, 'some-cmHandle',
'some-resourceIdentifier', dataType,
'normal request body' ) >> '{some-json}'
- when: 'write data for passthrough running post api is invoked with json data'
+ when: 'the request is posted'
def response = mvc.perform(
post(writeDataForPassthroughRunning).contentType(MediaType.APPLICATION_JSON)
.content(jsonData)
).andReturn().response
- then: 'response status is #expectedResponseStatus'
+ then: 'response status is #expectedResponseStatus'
response.status == expectedResponseStatus
- and: 'the data in the request body is as expected'
+ and: 'the response content matches the result from the DMI service'
response.getContentAsString() == expectedJsonResponse
where: 'given request body and data'
scenario | requestBodyFile | operationEnum | dataType || expectedResponseStatus | expectedJsonResponse
@@ -246,51 +250,53 @@ class DmiRestControllerSpec extends Specification {
}
def 'Create data using passthrough for special characters.'(){
- given: 'create data for cmHandle url and JsonData'
+ given: 'create data for cmHandle url'
def writeDataForCmHandlePassthroughRunning = "${basePathV1}/ch/some-cmHandle/data/ds/ncmp-datastore:passthrough-running" +
- "?resourceIdentifier=some-resourceIdentifier"
+ "?resourceIdentifier=some-resourceIdentifier"
+ and: 'request data with special characters'
def jsonData = TestUtils.getResourceFileContent('createDataWithSpecialChar.json')
- and: 'dmi service is called'
+ and: 'dmi service returns data'
mockDmiService.writeData(CREATE, 'some-cmHandle', 'some-resourceIdentifier', 'application/json',
- 'data with quote \" and new line \n') >> '{some-json}'
- when: 'create cmHandle passthrough running post api is invoked with json data with special chars'
+ 'data with quote \" and new line \n') >> '{some-json}'
+ when: 'the request is posted'
def response = mvc.perform(
- post(writeDataForCmHandlePassthroughRunning).contentType(MediaType.APPLICATION_JSON).content(jsonData)
+ post(writeDataForCmHandlePassthroughRunning).contentType(MediaType.APPLICATION_JSON).content(jsonData)
).andReturn().response
- then: 'response status is CREATED'
+ then: 'response status is CREATED'
response.status == CREATED.value()
- and: 'the data in the request body is as expected'
+ and: 'the response content matches the result from the DMI service'
response.getContentAsString() == '{some-json}'
}
def 'PassThrough Returns OK when topic is used for async'(){
- given: 'an endpoint'
+ given: 'Passthrough read URL and request data with a topic (parameter)'
def readPassThroughUrl ="${basePathV1}/ch/some-cmHandle/data/ds/ncmp-datastore:" +
- resourceIdentifier +
- '?resourceIdentifier=some-resourceIdentifier&topic=test-topic'
- when: 'endpoint is invoked'
+ resourceIdentifier +
+ '?resourceIdentifier=some-resourceIdentifier&topic=test-topic'
def jsonData = TestUtils.getResourceFileContent('readData.json')
+ when: 'the request is posted'
def response = mvc.perform(
- post(readPassThroughUrl).contentType(MediaType.APPLICATION_JSON).content(jsonData)
+ post(readPassThroughUrl).contentType(MediaType.APPLICATION_JSON).content(jsonData)
).andReturn().response
then: 'response status is OK'
assert response.status == HttpStatus.NO_CONTENT.value()
where: 'the following values are used'
- resourceIdentifier << ['passthrough-operational', 'passthrough-running']
+ resourceIdentifier << ['passthrough-operational', 'passthrough-running']
}
def 'Get resource data for pass-through running with #scenario value in resource identifier param.'() {
given: 'Get resource data url'
def getResourceDataForCmHandleUrl = "${basePathV1}/ch/some-cmHandle/data/ds/ncmp-datastore:passthrough-running" +
"?resourceIdentifier="+resourceIdentifier+"&options=(fields=myfields,depth=5)"
+ and: 'some valid json data'
def json = '{"cmHandleProperties" : { "prop1" : "value1", "prop2" : "value2"}}'
- when: 'get resource data POST api is invoked'
+ when: 'the request is posted'
def response = mvc.perform(
post(getResourceDataForCmHandleUrl).contentType(MediaType.APPLICATION_JSON).content(json)
).andReturn().response
then: 'response status is ok'
response.status == OK.value()
- and: 'dmi service called with get resource data for a cm handle'
+ and: 'dmi service method to get resource data is invoked once with correct parameters'
1 * mockDmiService.getResourceData('some-cmHandle',
resourceIdentifier,
'(fields=myfields,depth=5)',
diff --git a/src/test/groovy/org/onap/cps/ncmp/dmi/service/DmiServiceImplSpec.groovy b/src/test/groovy/org/onap/cps/ncmp/dmi/service/DmiServiceImplSpec.groovy
index 1d87b775..8531d35f 100644
--- a/src/test/groovy/org/onap/cps/ncmp/dmi/service/DmiServiceImplSpec.groovy
+++ b/src/test/groovy/org/onap/cps/ncmp/dmi/service/DmiServiceImplSpec.groovy
@@ -1,6 +1,6 @@
/*
* ============LICENSE_START=======================================================
- * Copyright (C) 2021 Nordix Foundation
+ * Copyright (C) 2021-2022 Nordix Foundation
* Modifications Copyright (C) 2021-2022 Bell Canada
* ================================================================================
* Licensed under the Apache License, Version 2.0 (the "License");
@@ -53,58 +53,27 @@ class DmiServiceImplSpec extends Specification {
def mockSdncOperations = Mock(SdncOperations)
def objectUnderTest = new DmiServiceImpl(mockDmiPluginProperties, mockNcmpRestClient, mockSdncOperations, spyObjectMapper)
- def ' Get modules for a cm-handle.'() {
- given: 'a cm handle'
- def cmHandle = 'node1'
- and: 'sdnc operations returns one module schema for the cmhandle'
- def moduleSchema = new ModuleSchema(
- identifier: "example-identifier",
- namespace: "example:namespace",
- version: "example-version")
- mockSdncOperations.getModuleSchemasFromNode(cmHandle) >> List.of(moduleSchema)
- when: 'get modules for cm-handle is called'
- def result = objectUnderTest.getModulesForCmHandle(cmHandle)
- then: 'one module is returned'
- result.schemas.size() == 1
- and: 'module has expected values'
- with(result.schemas[0]) {
- it.getRevision() == moduleSchema.getVersion()
- it.getModuleName() == moduleSchema.getIdentifier()
- it.getNamespace() == moduleSchema.getNamespace();
- }
- }
-
- def 'no modules found for the cmhandle.'() {
- given: 'cm handle id'
- def cmHandle = 'node1'
- and: 'sdnc operations returns no modules'
- mockSdncOperations.getModuleSchemasFromNode(cmHandle) >> Collections.emptyList();
- when: 'get modules for cm-handle is called'
- objectUnderTest.getModulesForCmHandle(cmHandle)
- then: 'module not found exception is thrown'
- thrown(ModulesNotFoundException)
- }
-
def 'Register cm handles with ncmp.'() {
- given: 'cm-handle list and json payload'
+ given: 'some cm-handle ids'
def givenCmHandlesList = ['node1', 'node2']
+ and: 'json payload'
def expectedJson = '{"dmiPlugin":"test-dmi-service","createdCmHandles":[{"cmHandle":"node1"},{"cmHandle":"node2"}]}'
- and: 'mockDmiPluginProperties returns test-dmi-service'
+ and: 'process returns "test-dmi-service" for service name'
mockDmiPluginProperties.getDmiServiceUrl() >> 'test-dmi-service'
- when: 'register cm handles service method with the given cm handles'
+ when: 'the cm handles are registered'
objectUnderTest.registerCmHandles(givenCmHandlesList)
- then: 'register cm handle with ncmp called once and return "created" status'
+ then: 'register cm handle with ncmp is called with the expected json and return "created" status'
1 * mockNcmpRestClient.registerCmHandlesWithNcmp(expectedJson) >> new ResponseEntity<>(HttpStatus.CREATED)
}
def 'Register cm handles with ncmp called with exception #scenario.'() {
- given: 'cm-handle list'
+ given: 'some cm-handle ids'
def cmHandlesList = ['node1', 'node2']
- and: 'dmi plugin service name is "test-dmi-service"'
+ and: 'process returns "test-dmi-service" for service name'
mockDmiPluginProperties.getDmiServiceUrl() >> 'test-dmi-service'
- and: 'ncmp rest client returns #responseEntity'
+ and: 'returns #responseEntity'
mockNcmpRestClient.registerCmHandlesWithNcmp(_ as String) >> responseEntity
- when: 'register cm handles service method called'
+ when: 'the cm handles are registered'
objectUnderTest.registerCmHandles(cmHandlesList)
then: 'a registration exception is thrown'
thrown(CmHandleRegistrationException.class)
@@ -115,29 +84,62 @@ class DmiServiceImplSpec extends Specification {
}
def 'Register cm handles with ncmp with wrong data.'() {
- given: 'objectMapper mock and cm-handle list'
+ given: 'some cm-handle ids'
def cmHandlesList = ['node1', 'node2']
- and: 'objectMapper returns "JsonProcessingException" during parse'
+ and: ' "JsonProcessingException" occurs during parsing'
objectUnderTest.objectMapper = mockObjectMapper
mockObjectMapper.writeValueAsString(_) >> { throw new JsonProcessingException('some error.') }
- when: 'register cm handles service method called'
+ when: 'the cmHandles are registered'
objectUnderTest.registerCmHandles(cmHandlesList)
then: 'a dmi exception is thrown'
thrown(DmiException.class)
}
+ def ' Get modules for a cm-handle.'() {
+ given: 'a cm handle'
+ def cmHandle = 'node1'
+ and: 'process returns one module schema for the cmhandle'
+ def moduleSchema = new ModuleSchema(
+ identifier: "example-identifier",
+ namespace: "example:namespace",
+ version: "example-version")
+ mockSdncOperations.getModuleSchemasFromNode(cmHandle) >> List.of(moduleSchema)
+ when: 'modules for cmHandle is requested'
+ def result = objectUnderTest.getModulesForCmHandle(cmHandle)
+ then: 'one module is returned'
+ result.schemas.size() == 1
+ and: 'module has expected values'
+ with(result.schemas[0]) {
+ it.getRevision() == moduleSchema.getVersion()
+ it.getModuleName() == moduleSchema.getIdentifier()
+ it.getNamespace() == moduleSchema.getNamespace();
+ }
+ }
+
+ def 'no modules found for the cmhandle.'() {
+ given: 'cm handle id'
+ def cmHandle = 'node1'
+ and: 'process returns no modules'
+ mockSdncOperations.getModuleSchemasFromNode(cmHandle) >> Collections.emptyList();
+ when: 'modules for cm-handle is requested'
+ objectUnderTest.getModulesForCmHandle(cmHandle)
+ then: 'module not found exception is thrown'
+ thrown(ModulesNotFoundException)
+ }
+
def 'Get multiple module resources.'() {
- given: 'a cmHandle and module reference list'
+ given: 'a cmHandle'
def cmHandle = 'some-cmHandle'
+ and: 'multiple module references'
def moduleReference1 = new ModuleReference(name: 'name-1', revision: 'revision-1')
def moduleReference2 = new ModuleReference(name: 'name-2', revision: 'revision-2')
def moduleList = [moduleReference1, moduleReference2]
- when: 'get module resources is invoked with the given cm handle and a module list'
+ when: 'module resources is requested'
def result = objectUnderTest.getModuleResources(cmHandle, moduleList)
- then: 'get modules resources is called twice'
+ then: 'SDNC operation service is called same number of module references given'
2 * mockSdncOperations.getModuleResource(cmHandle, _) >>> [new ResponseEntity<String>('{"ietf-netconf-monitoring:output": {"data": "some-data1"}}', HttpStatus.OK),
new ResponseEntity<String>('{"ietf-netconf-monitoring:output": {"data": "some-data2"}}', HttpStatus.OK)]
- and: 'the result is a yang resources object with the expected names, revisions and yang-sources'
+ and: 'the result contains the expected properties'
def yangResources = new YangResources()
def yangResource1 = new YangResource(yangSource: 'some-data1', moduleName: 'name-1', revision: 'revision-1')
def yangResource2 = new YangResource(yangSource: 'some-data2', moduleName: 'name-2', revision: 'revision-2')
@@ -151,11 +153,11 @@ class DmiServiceImplSpec extends Specification {
def cmHandle = 'some-cmHandle'
def moduleReference = new ModuleReference(name: 'NAME', revision: 'REVISION')
def moduleList = [moduleReference]
- when: 'get module resources is invoked with the given cm handle and a module list'
+ when: 'module resources is requested'
objectUnderTest.getModuleResources(cmHandle, moduleList)
- then: 'get modules resources is called once with a response body that contains no data'
+ then: 'SDNC operation service is called once with a response body that contains no data'
1 * mockSdncOperations.getModuleResource(cmHandle, _) >> new ResponseEntity<String>(responseBody, HttpStatus.OK)
- and: 'a module resource not found exception is thrown'
+ and: 'an exception is thrown'
thrown(ModuleResourceNotFoundException)
where: 'the following values are returned'
scenario | responseBody
@@ -164,28 +166,28 @@ class DmiServiceImplSpec extends Specification {
}
def 'Get module resources when sdnc returns #scenario response.'() {
- given: 'get module schema is invoked and returns a response from sdnc'
+ given: 'sdnc returns a #scenario response'
mockSdncOperations.getModuleResource(_ as String, _ as String) >> new ResponseEntity<String>('some-response-body', httpStatus)
- when: 'get module resources is invoked with the given cm handle and a module list'
+ when: 'module resources is requested'
objectUnderTest.getModuleResources('some-cmHandle', [new ModuleReference()] as LinkedList<ModuleReference>)
then: '#expectedException is thrown'
thrown(expectedException)
where: 'the following values are returned'
- scenario | httpStatus || expectedException
- 'not found' | HttpStatus.NOT_FOUND || ModuleResourceNotFoundException
- 'a internal server' | HttpStatus.INTERNAL_SERVER_ERROR || DmiException
+ scenario | httpStatus || expectedException
+ 'not found' | HttpStatus.NOT_FOUND || ModuleResourceNotFoundException
+ 'internal server error' | HttpStatus.INTERNAL_SERVER_ERROR || DmiException
}
def 'Get module resources with JSON processing exception.'() {
- given: 'a json processing exception during conversion'
+ given: 'a json processing exception during process'
def mockObjectWriter = Mock(ObjectWriter)
spyObjectMapper.writer() >> mockObjectWriter
mockObjectWriter.withRootName(_) >> mockObjectWriter
def jsonProcessingException = new JsonProcessingException('')
mockObjectWriter.writeValueAsString(_) >> { throw jsonProcessingException }
- when: 'get module resources is invoked with the given cm handle and a module list'
+ when: 'module resources is requested'
objectUnderTest.getModuleResources('some-cmHandle', [new ModuleReference()] as LinkedList<ModuleReference>)
- then: 'a DMI exception is thrown'
+ then: 'an exception is thrown'
def thrownException = thrown(DmiException.class)
and: 'the exception has the expected message and details'
thrownException.message == 'Unable to process JSON.'
@@ -195,47 +197,41 @@ class DmiServiceImplSpec extends Specification {
}
def 'Get resource data for passthrough operational.'() {
- given: 'cm-handle, passthrough parameter, resourceId, accept header, fields, depth'
- def cmHandle = 'testCmHandle'
- def resourceId = 'testResourceId'
- def optionsParam = '(fields=x/y/z,depth=10,test=abc)'
- def contentQuery = 'content=all'
- and: 'sdnc operation returns OK response'
- mockSdncOperations.getResouceDataForOperationalAndRunning(cmHandle, resourceId, optionsParam, contentQuery) >> new ResponseEntity<>('response json', HttpStatus.OK)
- when: 'get resource data from cm handles service method invoked'
- def response = objectUnderTest.getResourceData(cmHandle,
- resourceId, optionsParam, contentQuery)
- then: 'response have expected json'
+ given: 'sdnc operation returns OK response'
+ mockSdncOperations.getResouceDataForOperationalAndRunning(
+ 'someCmHandle',
+ 'someResourceId',
+ '(fields=x/y/z,depth=10,test=abc)',
+ 'content=all') >> new ResponseEntity<>('response json', HttpStatus.OK)
+ when: 'resource data is requested'
+ def response = objectUnderTest.getResourceData(
+ 'someCmHandle',
+ 'someResourceId',
+ '(fields=x/y/z,depth=10,test=abc)',
+ 'content=all')
+ then: 'response matches the response returned from the SDNC service'
response == 'response json'
}
def 'Get resource data with not found exception.'() {
- given: 'cm-handle, passthrough parameter, resourceId, accept header, fields, depth, query param'
- def cmHandle = 'testCmHandle'
- def resourceId = 'testResourceId'
- def optionsParam = '(fields=x/y/z,depth=10,test=abc)'
- def restConfQueryParam = 'content=config'
- and: 'sdnc operation returns "NOT_FOUND" response'
- mockSdncOperations.getResouceDataForOperationalAndRunning(cmHandle, resourceId, optionsParam, _ as String) >> new ResponseEntity<>(HttpStatus.NOT_FOUND)
- when: 'get resource data from cm handles service method invoked'
- objectUnderTest.getResourceData(cmHandle,
- resourceId, optionsParam, restConfQueryParam)
- then: 'resource data not found'
+ given: 'sdnc operation returns "NOT_FOUND" response'
+ mockSdncOperations.getResouceDataForOperationalAndRunning(*_) >> new ResponseEntity<>(HttpStatus.NOT_FOUND)
+ when: 'resource data is requested'
+ objectUnderTest.getResourceData('someCmHandle', 'someResourceId',
+ '(fields=x/y/z,depth=10,test=abc)', 'content=config')
+ then: 'http client request exception'
thrown(HttpClientRequestException.class)
}
def 'Get resource data for passthrough running.'() {
- given: 'cm-handle, passthrough parameter, resourceId, accept header, fields, depth'
- def cmHandle = 'testCmHandle'
- def resourceId = 'testResourceId'
- def optionsParam = '(fields=x/y/z,depth=10,test=abc)'
- def contentQuery = 'content=config'
- and: 'sdnc operation returns OK response'
- mockSdncOperations.getResouceDataForOperationalAndRunning(cmHandle, resourceId, optionsParam,
- contentQuery) >> new ResponseEntity<>('response json', HttpStatus.OK)
- when: 'get resource data from cm handles service method invoked'
- def response = objectUnderTest.getResourceData(cmHandle,
- resourceId, optionsParam, contentQuery)
+ given: 'sdnc operation returns OK response'
+ mockSdncOperations.getResouceDataForOperationalAndRunning(*_) >> new ResponseEntity<>('response json', HttpStatus.OK)
+ when: 'resource data is requested'
+ def response = objectUnderTest.getResourceData(
+ 'someCmHandle',
+ 'someResourceId',
+ '(fields=x/y/z,depth=10,test=abc)',
+ 'content=config')
then: 'response have expected json'
response == 'response json'
}
@@ -243,10 +239,10 @@ class DmiServiceImplSpec extends Specification {
def 'Write resource data for passthrough running with a #scenario from sdnc.'() {
given: 'sdnc returns a response with #scenario'
mockSdncOperations.writeData(operationEnum, _, _, _, _) >> new ResponseEntity<String>('response json', httpResponse)
- when: 'write resource data for cm handle method invoked'
- def response = objectUnderTest.writeData(operationEnum,'some-cmHandle',
- 'some-resourceIdentifier', 'some-dataType', '{some-data}')
- then: 'the response contains the expected json data from sdnc'
+ when: 'resource data is written to sdnc'
+ def response = objectUnderTest.writeData(operationEnum, 'some-cmHandle',
+ 'some-resourceIdentifier', 'some-dataType', '{some-data}')
+ then: 'the response matches the expected data'
response == 'response json'
where: 'the following values are used'
scenario | httpResponse | operationEnum
@@ -257,21 +253,21 @@ class DmiServiceImplSpec extends Specification {
def 'Write resource data with special characters.'() {
given: 'sdnc returns a created response'
mockSdncOperations.writeData(CREATE, 'some-cmHandle',
- 'some-resourceIdentifier', 'some-dataType', 'data with quote " and \n new line') >> new ResponseEntity<String>('response json', HttpStatus.CREATED)
- when: 'write resource data from cm handles service method invoked'
+ 'some-resourceIdentifier', 'some-dataType', 'data with quote " and \n new line') >> new ResponseEntity<String>('response json', HttpStatus.CREATED)
+ when: 'resource data is written to sdnc'
def response = objectUnderTest.writeData(CREATE, 'some-cmHandle',
- 'some-resourceIdentifier', 'some-dataType', 'data with quote " and \n new line')
- then: 'response have expected json'
+ 'some-resourceIdentifier', 'some-dataType', 'data with quote " and \n new line')
+ then: 'the response matches the expected data'
response == 'response json'
}
def 'Write resource data for passthrough running with a 500 response from sdnc.'() {
- given: 'sdnc returns a 500 response for the write operation'
+ given: 'sdnc returns internal server error response'
mockSdncOperations.writeData(CREATE, _, _, _, _) >> new ResponseEntity<String>('response json', HttpStatus.INTERNAL_SERVER_ERROR)
- when: 'write resource data for passthrough method is invoked'
+ when: 'resource data is written to sdnc'
objectUnderTest.writeData(CREATE, 'some-cmHandle',
- 'some-resourceIdentifier', 'some-dataType', _ as String)
+ 'some-resourceIdentifier', 'some-dataType', _ as String)
then: 'a dmi exception is thrown'
thrown(DmiException.class)
}
-}
+} \ No newline at end of file
diff --git a/src/test/groovy/org/onap/cps/ncmp/dmi/service/client/NcmpRestClientSpec.groovy b/src/test/groovy/org/onap/cps/ncmp/dmi/service/client/NcmpRestClientSpec.groovy
index 32df97b6..4d7e27e2 100644
--- a/src/test/groovy/org/onap/cps/ncmp/dmi/service/client/NcmpRestClientSpec.groovy
+++ b/src/test/groovy/org/onap/cps/ncmp/dmi/service/client/NcmpRestClientSpec.groovy
@@ -1,6 +1,6 @@
/*
* ============LICENSE_START=======================================================
- * Copyright (C) 2021 Nordix Foundation
+ * Copyright (C) 2021-2022 Nordix Foundation
* ================================================================================
* Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License.
@@ -37,8 +37,8 @@ class NcmpRestClientSpec extends Specification {
}
def 'Register a cm handle.'() {
- given: 'json data'
- def jsonData = 'some json'
+ given: 'some request data'
+ def someRequestData = 'some request data'
and: 'configuration data'
mockCpsProperties.baseUrl >> 'http://some-uri'
mockCpsProperties.dmiRegistrationUrl >> 'some-url'
@@ -46,12 +46,12 @@ class NcmpRestClientSpec extends Specification {
mockCpsProperties.authPassword >> 'some-password'
and: 'the rest template returns a valid response entity'
def mockResponseEntity = Mock(ResponseEntity)
- when: 'register cm-handle with ncmp is invoked'
- def result = objectUnderTest.registerCmHandlesWithNcmp(jsonData)
- then: 'the rest template is called with the correct uri and json in the body'
+ when: 'registering a cm handle'
+ def result = objectUnderTest.registerCmHandlesWithNcmp(someRequestData)
+ then: 'the rest template is called with the correct uri and original request data in the body'
1 * mockRestTemplate.exchange({ it.toString() == 'http://some-uri/some-url' },
- HttpMethod.POST, { it.body.contains(jsonData) }, String.class) >> mockResponseEntity
- and: 'the output of the method is equal to the output from the test template'
+ HttpMethod.POST, { it.body.contains(someRequestData) }, String.class) >> mockResponseEntity
+ and: 'the output of the method is equal to the output from the rest template service'
result == mockResponseEntity
}
} \ No newline at end of file
diff --git a/src/test/groovy/org/onap/cps/ncmp/dmi/service/client/SdncRestconfClientSpec.groovy b/src/test/groovy/org/onap/cps/ncmp/dmi/service/client/SdncRestconfClientSpec.groovy
index 8a3170b4..f334f780 100644
--- a/src/test/groovy/org/onap/cps/ncmp/dmi/service/client/SdncRestconfClientSpec.groovy
+++ b/src/test/groovy/org/onap/cps/ncmp/dmi/service/client/SdncRestconfClientSpec.groovy
@@ -1,6 +1,6 @@
/*
* ============LICENSE_START=======================================================
- * Copyright (C) 2021 Nordix Foundation
+ * Copyright (C) 2021-2022 Nordix Foundation
* ================================================================================
* Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License.
@@ -40,35 +40,35 @@ class SdncRestconfClientSpec extends Specification {
def objectUnderTest = new SdncRestconfClient(mockSdncProperties, mockRestTemplate)
def 'SDNC GET operation.'() {
- given: 'a get url'
+ given: 'a get resource url'
def getResourceUrl = '/getResourceUrl'
- and: 'sdnc properties'
+ and: 'test configuration data'
setupTestConfigurationData()
- and: 'the rest template returns a valid response entity'
+ and: 'the process returns a valid response entity'
def mockResponseEntity = Mock(ResponseEntity)
mockRestTemplate.exchange({ it.toString() == 'http://some-uri/getResourceUrl' },
HttpMethod.GET, _ as HttpEntity, String.class) >> mockResponseEntity
- when: 'GET operation is invoked'
+ when: 'the resource is fetched'
def result = objectUnderTest.getOperation(getResourceUrl)
- then: 'the output of the method is equal to the output from the test template'
+ then: 'the output of the method is equal to the output from the rest template service'
result == mockResponseEntity
}
def 'SDNC #scenario operation called.'() {
- given: 'json data'
- def jsonData = 'some-json'
+ given: 'some request data'
+ def someRequestData = 'some request data'
and: 'a url for get module resources'
def getModuleResourceUrl = '/getModuleResourceUrl'
- and: 'configuration data'
+ and: 'test configuration data'
setupTestConfigurationData()
- and: 'the rest template returns a valid response entity'
+ and: 'the process returns a valid response entity'
def mockResponseEntity = Mock(ResponseEntity)
- when: 'get module resources is invoked'
- def result = objectUnderTest.httpOperationWithJsonData(expectedHttpMethod, getModuleResourceUrl, jsonData, new HttpHeaders())
+ when: 'the resource is fetched'
+ def result = objectUnderTest.httpOperationWithJsonData(expectedHttpMethod, getModuleResourceUrl, someRequestData, new HttpHeaders())
then: 'the rest template is called with the correct uri and json in the body'
1 * mockRestTemplate.exchange({ it.toString() == 'http://some-uri/getModuleResourceUrl' },
- expectedHttpMethod, { it.body.contains(jsonData) }, String.class) >> mockResponseEntity
- and: 'the output of the method is the same as the output from the test template'
+ expectedHttpMethod, { it.body.contains(someRequestData) }, String.class) >> mockResponseEntity
+ and: 'the output of the method is the same as the output from the rest template service'
result == mockResponseEntity
where: 'the following values are used'
scenario || expectedHttpMethod
@@ -78,18 +78,18 @@ class SdncRestconfClientSpec extends Specification {
'DELETE' || DELETE
}
- def 'SDNC GET operation with header.'() {
+ def 'SDNC GET operation with headers.'() {
given: 'a get url'
def getResourceUrl = '/getResourceUrl'
- and: 'sdnc properties'
+ and: 'test configuration data'
setupTestConfigurationData()
- and: 'the rest template returns a valid response entity'
+ and: 'the process returns a valid response entity'
def mockResponseEntity = Mock(ResponseEntity)
mockRestTemplate.exchange({ it.toString() == 'http://some-uri/getResourceUrl' },
HttpMethod.GET, _ as HttpEntity, String.class) >> mockResponseEntity
- when: 'GET operation is invoked'
+ when: 'the resource is fetched with headers'
def result = objectUnderTest.getOperation(getResourceUrl, new HttpHeaders())
- then: 'the output of the method is equal to the output from the test template'
+ then: 'the output of the method is equal to the output from the rest template service'
result == mockResponseEntity
}
diff --git a/src/test/groovy/org/onap/cps/ncmp/dmi/service/operation/SdncOperationsSpec.groovy b/src/test/groovy/org/onap/cps/ncmp/dmi/service/operation/SdncOperationsSpec.groovy
index 724d2d4e..9dcb72e6 100644
--- a/src/test/groovy/org/onap/cps/ncmp/dmi/service/operation/SdncOperationsSpec.groovy
+++ b/src/test/groovy/org/onap/cps/ncmp/dmi/service/operation/SdncOperationsSpec.groovy
@@ -52,13 +52,13 @@ class SdncOperationsSpec extends Specification {
SdncOperations objectUnderTest
def 'get modules from node.'() {
- given: 'node id and url'
+ given: 'a node id and url'
def nodeId = 'node1'
def expectedUrl = '/rests/data/network-topology:network-topology/topology=test-topology/node=node1/yang-ext:mount/ietf-netconf-monitoring:netconf-state/schemas'
- and: 'sdnc returns one module in response'
+ and: 'sdnc returns one module during process'
mockSdncRestClient.getOperation(expectedUrl) >>
ResponseEntity.ok(TestUtils.getResourceFileContent('ModuleSchema.json'))
- when: 'get modules from node is called'
+ when: 'module schemas from node are fetched'
def moduleSchemas = objectUnderTest.getModuleSchemasFromNode(nodeId)
then: 'one module is found'
moduleSchemas.size() == 1
@@ -78,9 +78,9 @@ class SdncOperationsSpec extends Specification {
def expectedUrl = '/rests/data/network-topology:network-topology/topology=test-topology/node=node1/yang-ext:mount/ietf-netconf-monitoring:netconf-state/schemas'
and: 'sdnc operation returns #scenario'
mockSdncRestClient.getOperation(expectedUrl) >> ResponseEntity.ok(responseBody)
- when: 'modules from node is called'
+ when: 'the module schemas are requested'
def moduleSchemas = objectUnderTest.getModuleSchemasFromNode(nodeId)
- then: 'no modules are returned'
+ then: 'no module schemas are returned'
moduleSchemas.size() == 0
where:
scenario | responseBody
@@ -93,9 +93,9 @@ class SdncOperationsSpec extends Specification {
given: 'node id and url'
def nodeId = 'node1'
def expectedUrl = '/rests/data/network-topology:network-topology/topology=test-topology/node=node1/yang-ext:mount/ietf-netconf-monitoring:netconf-state/schemas'
- and: 'sdnc operation returns configured response'
+ and: '#scenario is returned during process'
mockSdncRestClient.getOperation(expectedUrl) >> new ResponseEntity<>(sdncResponseBody, sdncHttpStatus)
- when: 'modules for node are fetched'
+ when: 'module schemas from node are fetched'
objectUnderTest.getModuleSchemasFromNode(nodeId)
then: 'SDNCException is thrown'
def thrownException = thrown(SdncException)
@@ -111,28 +111,28 @@ class SdncOperationsSpec extends Specification {
given: 'node id and url'
def nodeId = 'some-node'
def expectedUrl = '/rests/operations/network-topology:network-topology/topology=test-topology/node=some-node/yang-ext:mount/ietf-netconf-monitoring:get-schema'
- when: 'get module resources is called with the expected parameters'
+ when: 'module resource is fetched with the expected parameters'
objectUnderTest.getModuleResource(nodeId, 'some-json-data')
- then: 'the SDNC Rest client is invoked with the correct URL and json data'
+ then: 'the SDNC Rest client is invoked with the correct parameters'
1 * mockSdncRestClient.httpOperationWithJsonData(HttpMethod.POST, expectedUrl, 'some-json-data', _ as HttpHeaders)
}
def 'Get resource data from node to SDNC.'() {
- given: 'expected url, topology-id, sdncOperation object'
+ given: 'expected url'
def expectedUrl = '/rests/data/network-topology:network-topology/topology=test-topology/node=node1/yang-ext:mount/testResourceId?a=1&b=2&content=testContent'
- when: 'called get modules from node'
+ when: 'resource data is fetched for given node ID'
objectUnderTest.getResouceDataForOperationalAndRunning('node1', 'testResourceId',
'(a=1,b=2)', 'content=testContent')
- then: 'the get operation is executed with the correct URL'
+ then: 'the SDNC get operation is executed with the correct URL'
1 * mockSdncRestClient.getOperation(expectedUrl)
}
def 'Write resource data with #scenario operation to SDNC.'() {
- given: 'expected url, topology-id, sdncOperation object'
+ given: 'expected url'
def expectedUrl = '/rests/data/network-topology:network-topology/topology=test-topology/node=node1/yang-ext:mount/testResourceId'
when: 'write resource data for passthrough running is called'
objectUnderTest.writeData(operationEnum, 'node1', 'testResourceId', 'application/json', 'requestData')
- then: 'the #expectedHttpMethod operation is executed with the correct URL and data'
+ then: 'the #expectedHttpMethod operation is executed with the correct parameters'
1 * mockSdncRestClient.httpOperationWithJsonData(expectedHttpMethod, expectedUrl, 'requestData', _ as HttpHeaders)
where: 'the following values are used'
scenario | operationEnum || expectedHttpMethod
@@ -144,10 +144,10 @@ class SdncOperationsSpec extends Specification {
}
def 'build query param list for SDNC where options #scenario'() {
- when: 'build query param list is called with #scenario'
+ when: 'query param list is built'
def result = objectUnderTest.buildQueryParamMap(optionsParamInQuery, 'd=4')
.toSingleValueMap().toString()
- then: 'result equals to expected result'
+ then: 'result matches the expected result'
result == expectedResult
where: 'following parameters are used'
scenario | optionsParamInQuery || expectedResult
@@ -162,10 +162,9 @@ class SdncOperationsSpec extends Specification {
}
def 'options parameters contains a comma #scenario'() {
- // https://jira.onap.org/browse/CPS-719
- when: 'build query param list is called with #scenario'
+ when: 'query param list is built with #scenario'
def result = objectUnderTest.buildQueryParamMap(optionsParamInQuery, 'd=4').toSingleValueMap()
- then: 'expect 2 elements from options where we are ignoring empty query param value +1 from content query param (2+1) = 3 elements'
+ then: 'expect 2 elements from options where we are ignoring empty query param value, +1 from content query param (2+1) = 3 elements'
def expectedNoOfElements = 3
and: 'results contains equal elements as expected'
result.size() == expectedNoOfElements