CPS Validator Changes
[cps.git] / cps-service / src / test / groovy / org / onap / cps / api / impl / CpsAdminServiceImplSpec.groovy
index cbe1ebb..2979c09 100755 (executable)
@@ -25,20 +25,23 @@ package org.onap.cps.api.impl
 import org.onap.cps.api.CpsDataService
 import org.onap.cps.spi.CpsAdminPersistenceService
 import org.onap.cps.spi.model.Anchor
-import org.onap.cps.spi.model.CmHandleQueryParameters
+import org.onap.cps.spi.utils.CpsValidator
 import spock.lang.Specification
 import java.time.OffsetDateTime
 
 class CpsAdminServiceImplSpec extends Specification {
     def mockCpsAdminPersistenceService = Mock(CpsAdminPersistenceService)
     def mockCpsDataService = Mock(CpsDataService)
-    def objectUnderTest = new CpsAdminServiceImpl(mockCpsAdminPersistenceService, mockCpsDataService)
+    def mockCpsValidator = Mock(CpsValidator)
+    def objectUnderTest = new CpsAdminServiceImpl(mockCpsAdminPersistenceService, mockCpsDataService,mockCpsValidator)
 
     def 'Create dataspace method invokes persistence service.'() {
         when: 'create dataspace method is invoked'
             objectUnderTest.createDataspace('someDataspace')
         then: 'the persistence service method is invoked with same parameters'
             1 * mockCpsAdminPersistenceService.createDataspace('someDataspace')
+        and: 'the CpsValidator is called on the dataspaceName'
+            1 * mockCpsValidator.validateNameCharacters('someDataspace')
     }
 
     def 'Create anchor method invokes persistence service.'() {
@@ -46,30 +49,44 @@ class CpsAdminServiceImplSpec extends Specification {
             objectUnderTest.createAnchor('someDataspace', 'someSchemaSet', 'someAnchorName')
         then: 'the persistence service method is invoked with same parameters'
             1 * mockCpsAdminPersistenceService.createAnchor('someDataspace', 'someSchemaSet', 'someAnchorName')
+        and: 'the CpsValidator is called on the dataspaceName, schemaSetName and anchorName'
+            1 * mockCpsValidator.validateNameCharacters('someDataspace', 'someSchemaSet', 'someAnchorName')
     }
 
     def 'Retrieve all anchors for dataspace.'() {
-        given: 'that anchor is associated with the dataspace'
+        given: 'that an anchor is associated with the dataspace'
             def anchors = [new Anchor()]
             mockCpsAdminPersistenceService.getAnchors('someDataspace') >> anchors
-        expect: 'the collection provided by persistence service is returned as result'
-            objectUnderTest.getAnchors('someDataspace') == anchors
+        when: 'get Anchors is called for a dataspace name'
+            def result = objectUnderTest.getAnchors('someDataspace')
+        then: 'the collection provided by persistence service is returned as result'
+            result == anchors
+        and: 'the CpsValidator is called on the dataspaceName'
+            1 * mockCpsValidator.validateNameCharacters('someDataspace')
     }
 
     def 'Retrieve all anchors for schema-set.'() {
         given: 'that anchor is associated with the dataspace and schemaset'
             def anchors = [new Anchor()]
             mockCpsAdminPersistenceService.getAnchors('someDataspace', 'someSchemaSet') >> anchors
-        expect: 'the collection provided by persistence service is returned as result'
-            objectUnderTest.getAnchors('someDataspace', 'someSchemaSet') == anchors
+        when: 'get anchors is called for a dataspace name and schema set name'
+            def result = objectUnderTest.getAnchors('someDataspace', 'someSchemaSet')
+        then: 'the collection provided by persistence service is returned as result'
+            result == anchors
+        and: 'the CpsValidator is called on the dataspaceName, schemaSetName'
+            1 * mockCpsValidator.validateNameCharacters('someDataspace', 'someSchemaSet')
     }
 
     def 'Retrieve anchor for dataspace and provided anchor name.'() {
         given: 'that anchor name is associated with the dataspace'
             Anchor anchor = new Anchor()
             mockCpsAdminPersistenceService.getAnchor('someDataspace','someAnchor') >>  anchor
-        expect: 'the anchor provided by persistence service is returned as result'
-            assert objectUnderTest.getAnchor('someDataspace','someAnchor') == anchor
+        when: 'get anchor is called for a dataspace name and anchor name'
+            def result = objectUnderTest.getAnchor('someDataspace','someAnchor')
+        then: 'the anchor provided by persistence service is returned as result'
+            result == anchor
+        and: 'the CpsValidator is called on the dataspaceName, anchorName'
+            1 * mockCpsValidator.validateNameCharacters('someDataspace', 'someAnchor')
     }
 
     def 'Delete anchor.'() {
@@ -79,14 +96,19 @@ class CpsAdminServiceImplSpec extends Specification {
             1 * mockCpsDataService.deleteDataNodes('someDataspace','someAnchor', _ as OffsetDateTime )
         and: 'the persistence service method is invoked with same parameters to delete anchor'
              1 * mockCpsAdminPersistenceService.deleteAnchor('someDataspace','someAnchor')
+        and: 'the CpsValidator is called on the dataspaceName, anchorName'
+            1 * mockCpsValidator.validateNameCharacters('someDataspace', 'someAnchor')
     }
 
     def 'Query all anchor identifiers for a dataspace and module names.'() {
         given: 'the persistence service is invoked with the expected parameters and returns a list of anchors'
             mockCpsAdminPersistenceService.queryAnchors('some-dataspace-name', ['some-module-name']) >> [new Anchor(name:'some-anchor-identifier')]
-        expect: 'get anchor identifiers returns the same anchor identifier returned by the persistence layer'
-            objectUnderTest.queryAnchorNames('some-dataspace-name', ['some-module-name']) == ['some-anchor-identifier']
-
+        when: 'query anchor names is called using a dataspace name and module name'
+            def result = objectUnderTest.queryAnchorNames('some-dataspace-name', ['some-module-name'])
+        then: 'get anchor identifiers returns the same anchor identifier returned by the persistence layer'
+            result == ['some-anchor-identifier']
+        and: 'the CpsValidator is called on the dataspaceName'
+            1 * mockCpsValidator.validateNameCharacters('some-dataspace-name')
     }
 
     def 'Delete dataspace.'() {
@@ -94,15 +116,7 @@ class CpsAdminServiceImplSpec extends Specification {
             objectUnderTest.deleteDataspace('someDataspace')
         then: 'associated persistence service method is invoked with correct parameter'
             1 * mockCpsAdminPersistenceService.deleteDataspace('someDataspace')
+        and: 'the CpsValidator is called on the dataspaceName'
+            1 * mockCpsValidator.validateNameCharacters('someDataspace')
     }
-
-    def 'Query CM Handles.'() {
-        given: 'a cm handle query'
-            def cmHandleQueryParameters = new CmHandleQueryParameters()
-        when: 'query cm handles is invoked'
-            objectUnderTest.queryCmHandles(cmHandleQueryParameters)
-        then: 'associated persistence service method is invoked with correct parameter'
-            1 * mockCpsAdminPersistenceService.queryCmHandles(cmHandleQueryParameters)
-    }
-
 }