diff options
Diffstat (limited to 'cps-path-parser')
-rw-r--r-- | cps-path-parser/src/main/java/org/onap/cps/cpspath/parser/CpsPathBuilder.java | 4 | ||||
-rw-r--r-- | cps-path-parser/src/test/groovy/org/onap/cps/cpspath/parser/CpsPathQuerySpec.groovy | 19 |
2 files changed, 22 insertions, 1 deletions
diff --git a/cps-path-parser/src/main/java/org/onap/cps/cpspath/parser/CpsPathBuilder.java b/cps-path-parser/src/main/java/org/onap/cps/cpspath/parser/CpsPathBuilder.java index de261e64b3..0bb09235ff 100644 --- a/cps-path-parser/src/main/java/org/onap/cps/cpspath/parser/CpsPathBuilder.java +++ b/cps-path-parser/src/main/java/org/onap/cps/cpspath/parser/CpsPathBuilder.java @@ -165,6 +165,10 @@ public class CpsPathBuilder extends CpsPathBaseListener { cpsPathQuery.setContainerNames(containerNames); cpsPathQuery.setBooleanOperators(booleanOperators); cpsPathQuery.setComparativeOperators(comparativeOperators); + if (cpsPathQuery.hasAncestorAxis() && cpsPathQuery.getXpathPrefix() + .endsWith("/" + cpsPathQuery.getAncestorSchemaNodeIdentifier())) { + cpsPathQuery.setAncestorSchemaNodeIdentifier(""); + } return cpsPathQuery; } diff --git a/cps-path-parser/src/test/groovy/org/onap/cps/cpspath/parser/CpsPathQuerySpec.groovy b/cps-path-parser/src/test/groovy/org/onap/cps/cpspath/parser/CpsPathQuerySpec.groovy index ae7ee598ab..15f6b11823 100644 --- a/cps-path-parser/src/test/groovy/org/onap/cps/cpspath/parser/CpsPathQuerySpec.groovy +++ b/cps-path-parser/src/test/groovy/org/onap/cps/cpspath/parser/CpsPathQuerySpec.groovy @@ -1,6 +1,6 @@ /* * ============LICENSE_START======================================================= - * Copyright (C) 2021-2023 Nordix Foundation + * Copyright (C) 2021-2024 Nordix Foundation * Modifications Copyright (C) 2023 TechMahindra Ltd * ================================================================================ * Licensed under the Apache License, Version 2.0 (the "License"); @@ -253,4 +253,21 @@ class CpsPathQuerySpec extends Specification { '/test[@name2="value2" and @name1="value1"]' || 'name2' | 'name1' } + def 'Ancestor axis matching prefix'() { + when: 'building a cps path query' + def result = parseXPathAndBuild(xpath) + then: 'ancestor axis is removed when same as prefix' + assert result.hasAncestorAxis() == expectAncestorAxis + where: 'the following xpaths are used' + xpath || expectAncestorAxis + '//abc/def/ancestor::abc' || true + '//abc/def/ancestor::def' || false + '//abc/def/ancestor::ef' || true + } + + def parseXPathAndBuild(xpath) { + def cpsPathBuilder = CpsPathUtil.getCpsPathBuilder(xpath) + cpsPathBuilder.build() + } + } |