Merge "Fix: Make bookstore data consistent"
[cps.git] / cps-service / src / test / groovy / org / onap / cps / api / impl / CpsAdminServiceImplSpec.groovy
index def99e2..eb41e20 100755 (executable)
@@ -1,8 +1,9 @@
 /*
  *  ============LICENSE_START=======================================================
- *  Copyright (C) 2020-2022 Nordix Foundation
+ *  Copyright (C) 2020-2023 Nordix Foundation
  *  Modifications Copyright (C) 2020-2022 Bell Canada.
  *  Modifications Copyright (C) 2021 Pantheon.tech
+ *  Modifications Copyright (C) 2022 TechMahindra Ltd.
  *  ================================================================================
  *  Licensed under the Apache License, Version 2.0 (the "License");
  *  you may not use this file except in compliance with the License.
@@ -24,31 +25,26 @@ package org.onap.cps.api.impl
 
 import org.onap.cps.api.CpsDataService
 import org.onap.cps.spi.CpsAdminPersistenceService
-import org.onap.cps.spi.exceptions.DataValidationException
+import org.onap.cps.spi.exceptions.ModuleNamesNotFoundException
 import org.onap.cps.spi.model.Anchor
-import org.onap.cps.spi.model.CmHandleQueryParameters
+import org.onap.cps.spi.model.Dataspace
+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')
-    }
-
-    def 'Create a dataspace with an invalid dataspace name.'() {
-        when: 'create dataspace method is invoked with incorrectly named dataspace'
-            objectUnderTest.createDataspace('Dataspace Name with spaces')
-        then: 'a data validation exception is thrown'
-            thrown(DataValidationException)
-        and: 'the persistence service method is not invoked'
-            0 * mockCpsAdminPersistenceService.createDataspace(_)
+        and: 'the CpsValidator is called on the dataspaceName'
+            1 * mockCpsValidator.validateNameCharacters('someDataspace')
     }
 
     def 'Create anchor method invokes persistence service.'() {
@@ -56,76 +52,73 @@ 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')
-    }
-
-    def 'Create an anchor with an invalid anchor name.'() {
-        when: 'create anchor method is invoked with incorrectly named dataspace'
-            objectUnderTest.createAnchor('someDataspace', 'someSchemaSet', 'Anchor Name With Spaces')
-        then: 'a data validation exception is thrown'
-            thrown(DataValidationException)
-        and: 'the persistence service method is not invoked'
-            0 * mockCpsAdminPersistenceService.createAnchor(_, _, _)
+        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
-    }
-
-    def 'Retrieve all anchors with an invalid dataspace name.'() {
-        when: 'get anchors is invoked with an invalid dataspace name'
-            objectUnderTest.getAnchors('Dataspace name with spaces')
-        then: 'a data validation exception is thrown'
-            thrown(DataValidationException)
-        and: 'cps admin persistence get anchors is not invoked'
-            0 * mockCpsAdminPersistenceService.getAnchors(_)
+        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
-    }
-    def 'Retrieve all anchors for schema-set with invalid #scenario.'() {
-        when: 'the collection provided by persistence service is returned as result'
-            objectUnderTest.getAnchors(dataspaceName, schemaSetName)
-        then: 'a data validation exception is thrown'
-            thrown(DataValidationException)
-        and: 'cps admin persistence get anchors is not invoked'
-            0 * mockCpsAdminPersistenceService.getAnchors(_, _)
-        where: 'the following parameters are used'
-            scenario                         | dataspaceName                 | schemaSetName
-            'dataspace name'                 | 'dataspace names with spaces' | 'schemaSetName'
-            'schema set name'                | 'dataspaceName'               | 'schema set name with spaces'
-            'dataspace and schema set name'  | 'dataspace name with spaces'  | 'schema set name with spaces'
+        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 all anchors for multiple schema-sets.'() {
+        given: 'that anchor is associated with the dataspace and schemasets'
+            def anchors = [new Anchor(), new Anchor()]
+            mockCpsAdminPersistenceService.getAnchors('someDataspace', _ as Collection<String>) >> anchors
+        when: 'get anchors is called for a dataspace name and schema set names'
+            def result = objectUnderTest.getAnchors('someDataspace', ['schemaSet1', 'schemaSet2'])
+        then: 'the collection provided by persistence service is returned as result'
+            result == anchors
+        and: 'the CpsValidator is called on the dataspace name and schema-set names'
+            1 * mockCpsValidator.validateNameCharacters('someDataspace')
+            1 * mockCpsValidator.validateNameCharacters(_)
     }
 
-
     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 'Retrieve dataspace.'() {
+        given: 'a dataspace is already created'
+            def dataspace = new Dataspace(name: "someDataspace")
+            mockCpsAdminPersistenceService.getDataspace('someDataspace') >> dataspace
+        expect: 'the dataspace provided by persistence service is returned as result'
+          assert objectUnderTest.getDataspace('someDataspace') == dataspace
     }
 
-    def 'Retrieve anchor with invalid #scenario.'() {
-        when: 'get anchors is invoked with an invalid dataspace name'
-            objectUnderTest.getAnchor(dataspaceName, anchorName)
-        then: 'a data validation exception is thrown'
-            thrown(DataValidationException)
-        and: 'cps admin persistence get anchor is not invoked'
-            0 * mockCpsAdminPersistenceService.getAnchor(_, _)
-        where: 'the following parameters are used'
-            scenario                     | dataspaceName                 | anchorName
-            'dataspace name'             | 'dataspace names with spaces' | 'anchorName'
-            'anchor name'                | 'dataspaceName'               | 'anchor name with spaces'
-            'dataspace and anchor name'  | 'dataspace name with spaces'  | 'anchor name with spaces'
+    def 'Retrieve all dataspaces.'() {
+        given: 'that all given dataspaces are already created'
+        def dataspaces = [new Dataspace(name: "test-dataspace1"), new Dataspace(name: "test-dataspace2")]
+            mockCpsAdminPersistenceService.getAllDataspaces() >> dataspaces
+        expect: 'the dataspace provided by persistence service is returned as result'
+           assert objectUnderTest.getAllDataspaces() == dataspaces
     }
 
     def 'Delete anchor.'() {
@@ -135,39 +128,46 @@ 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 'Delete anchor with invalid #scenario.'() {
-        when: 'delete anchor is invoked'
-            objectUnderTest.deleteAnchor(dataspaceName, anchorName)
-        then: 'a data validation exception is thrown'
-            thrown(DataValidationException)
-        and: 'delete data nodes is invoked on the data service with expected parameters'
-            0 * mockCpsDataService.deleteDataNodes(_,_, _ as OffsetDateTime )
+    def 'Delete multiple anchors.'() {
+        when: 'delete anchors is invoked'
+            objectUnderTest.deleteAnchors('someDataspace', ['anchor1', 'anchor2'])
+        then: 'delete data nodes is invoked on the data service with expected parameters'
+            1 * mockCpsDataService.deleteDataNodes('someDataspace', _ as Collection<String>, _ as OffsetDateTime)
         and: 'the persistence service method is invoked with same parameters to delete anchor'
-            0 * mockCpsAdminPersistenceService.deleteAnchor(_,_)
-        where: 'the following parameters are used'
-            scenario                     | dataspaceName                 | anchorName
-            'dataspace name'             | 'dataspace names with spaces' | 'anchorName'
-            'anchor name'                | 'dataspaceName'               | 'anchor name with spaces'
-            'dataspace and anchor name'  | 'dataspace name with spaces'  | 'anchor name with spaces'
+            1 * mockCpsAdminPersistenceService.deleteAnchors('someDataspace',_ as Collection<String>)
+        and: 'the CpsValidator is called on the dataspace name and anchor names'
+            1 * mockCpsValidator.validateNameCharacters('someDataspace')
+            1 * mockCpsValidator.validateNameCharacters(_)
     }
 
     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']
-
-    }
-
-    def 'Query all anchor identifiers for a dataspace and module names with an invalid dataspace name.'() {
-        when: 'delete anchor is invoked'
-            objectUnderTest.queryAnchorNames('some dataspace name', _ as Collection<String>)
-        then: 'a data validation exception is thrown'
-            thrown(DataValidationException)
-        and: 'delete data nodes is not invoked'
-            0 * mockCpsAdminPersistenceService.queryAnchors(_, _)
+        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 'Query all anchors with Module Names Not Found Exception in persistence layer.'() {
+        given: 'the persistence layer throws a Module Names Not Found Exception'
+            def originalException = new ModuleNamesNotFoundException('exception-ds', [ 'm1', 'm2'])
+            mockCpsAdminPersistenceService.queryAnchors(*_) >> { throw originalException}
+        when: 'attempt query anchors'
+            objectUnderTest.queryAnchorNames('some-dataspace-name', [])
+        then: 'the same exception is thrown (up)'
+            def thrownUp = thrown(ModuleNamesNotFoundException)
+            assert thrownUp == originalException
+        and: 'the exception details contains the relevant data'
+            assert thrownUp.details.contains('exception-ds')
+            assert thrownUp.details.contains('m1')
+            assert thrownUp.details.contains('m2')
     }
 
     def 'Delete dataspace.'() {
@@ -175,14 +175,7 @@ class CpsAdminServiceImplSpec extends Specification {
             objectUnderTest.deleteDataspace('someDataspace')
         then: 'associated persistence service method is invoked with correct parameter'
             1 * mockCpsAdminPersistenceService.deleteDataspace('someDataspace')
-    }
-
-    def 'Delete dataspace with invalid dataspace id.'() {
-        when: 'delete dataspace is invoked'
-            objectUnderTest.deleteDataspace('some dataspace name')
-        then: 'a data validation exception is thrown'
-            thrown(DataValidationException)
-        and: 'associated persistence service method is not invoked'
-            0 * mockCpsAdminPersistenceService.deleteDataspace(_)
+        and: 'the CpsValidator is called on the dataspaceName'
+            1 * mockCpsValidator.validateNameCharacters('someDataspace')
     }
 }