Refactor: Improve code quality and compliance with SonarQube standards

- Consolidated multiple `if` statements to enhance readability and maintainability.
- Replaced traditional `instanceof` checks followed by casting with the new pattern matching syntax for improved type safety and clarity.
- Removed unnecessary casts and streamlined code in `addYangLeafList` and `addDataNodeFromNormalizedNode` methods.

Issue-ID: CPS-1957
Change-Id: I46696dc184b6b3c3d0fe3a6bd12267a4e0d76193
Signed-off-by: sourabh_sourabh <sourabh.sourabh@est.tech>

Resize cps expectation

- set expectation to 70 (cmhanldes/second) for discovery case

Issue-ID: CPS-2350

Change-Id: I3d0f9f5e20de87f79ebbe569c7905f4eefed35e8
Signed-off-by: halil.cakal <halil.cakal@est.tech>

Merge "Enable 2 minute initial module processing delay"

Enable 2 minute initial module processing delay

Issue-ID: CPS-2403
Signed-off-by: danielhanrahan <daniel.hanrahan@est.tech>
Change-Id: I614313461bd36ca351a8d42d9683b88460090b50

Update SpotBugs dependency

- update spotbugs to 4.8.6
- remove old findbugs dependency (spotbugs replaces findbugs)
- remove an unused SupressFBWarnings

Issue-ID: CPS-2458
Signed-off-by: danielhanrahan <daniel.hanrahan@est.tech>
Change-Id: If9d5571ae4293a2653a117a988414af2452647c0

Release Stage 7 for 3.5.4

Issue-ID: CPS-2457
Change-Id: Id9a267c587f16f3fb5edb558b33cd0003864ed93
Signed-off-by: seanbeirne <sean.beirne@est.tech>

Release Stage 6 for 3.5.4

Issue-ID: CPS-2457
Change-Id: I9aa7265d8179c55a71a7722e3ef42c44b0f8dbfe
Signed-off-by: seanbeirne <sean.beirne@est.tech>

Release Stage 5 for 3.5.4

Issue-ID: CPS-2457
Change-Id: I8bc9b2588539b64a6c8de936b6df97111665f1ed
Signed-off-by: seanbeirne <sean.beirne@est.tech>

Release notes and documentation updates for 3.5.4

- Release notes update
- Update openapi info.version to 3.5.4
- Update scheduled-tasks docs to reflect changes in retry mechanism
- Update deployment docs to update configurable timer parameters
- Remove unused locked-module-sync.sleep-time-ms

Issue-ID: CPS-2457
Signed-off-by: danielhanrahan <daniel.hanrahan@est.tech>
Change-Id: Ieb559bbfe348848c4b8669410861edfeeb6c9bf3

Refactor and Optimize ModuleSyncWatchdog for Improved Lock Handling and Queue Management

- Enhanced logging for populateWorkQueueIfNeeded and resetPreviouslyLockedCmHandles
methods, improving clarity and error handling.
- Improved readability and maintainability of the locking mechanism with
  Hazelcast's FencedLock.
- Optimized error handling in catch blocks, logging detailed exception messages and stack
traces for better troubleshooting.
- Refined lock acquisition and release flow, with clear log messages for both successful
and failed lock operations, ensuring safe handling of Hazelcast distributed locks.

Issue-ID: CPS-2403
Change-Id: Ie089f36a817d4965782235b51ee987ef054516b1
Signed-off-by: sourabh_sourabh <sourabh.sourabh@est.tech>

Merge "Update documentation related to hazelcast"

Update documentation related to hazelcast

- removed old entries of data structures which are removed now.
- updated the documentation to correctly reflect our infrastructure
  startegy related to hazelcast.
- fixed few NB comments on the merged patch.

Issue-ID: CPS-2408
Change-Id: Id7ce4ac695ad94742490ac5d2be89cd23ffda3e5
Signed-off-by: mpriyank <priyank.maheshwari@est.tech>

Merge "Add missing duplication test in integration test suite"

Merge "Added additional logging"

Add missing duplication test in integration test suite

- Added test for  ModuleSerive.createSchemaSetFromModules()
- Unintendedly Updated one swagger file in docs because
1) DT team forgot to update the relevant file
2) Gers committed pom update to automatically update swagger files
3) I ran 'mvn install'

Issue-ID: CPS-2399
Change-Id: I33261c32bbca58ff21aff3cde9588617199f104c
Signed-off-by: ToineSiebelink <toine.siebelink@est.tech>

Added additional logging

- Added logging for non existing xpath while updating data notes and its
  decendants.

Issue-ID: CPS-2403
Change-Id: I1ecaf2aed77aec8d266f6a758fe80ee96717d9c7
Signed-off-by: sourabh_sourabh <sourabh.sourabh@est.tech>

Merge "add documentaiton for cps-and-ncmp deployment"

add documentaiton for cps-and-ncmp deployment

- the optimum percentage of the JVM allocation documented
- remove anchorDataCache from the documentation
  since no more used

Issue-ID: CPS-2180

Change-Id: Ie1b8c43136a38f8da5a865493bed1df030a497d6
Signed-off-by: halil.cakal <halil.cakal@est.tech>

Merge "Copy Swagger file to docs folder"

Merge "use one hazelcast instance per jvm"

use one hazelcast instance per jvm

- updated all the cache(distributed datastructures) to use the same instance config now.
- there will be just one instance per JVM now
- Better cleanup of hz instance in the testware
- Refactored the testware to verify the configs
- Expected Impact on lowering the memory usage, less number of TCP
  communications between members
- NOTE: we need to do a full regression as the changes impacts all the
  use cases which involves cache

Issue-ID: CPS-2408
Change-Id: I7564992a9990f44ef3defb4f50cb7d094cad7b92
Signed-off-by: mpriyank <priyank.maheshwari@est.tech>

Copy Swagger file to docs folder

To avoid manual copy/paste errors updates to docs folder for RTD are generated and copied when compiling CPS

Issue-ID: CPS-2332

Change-Id: I1a64f7cc74a82341403f8125adc5216ef046429c
Signed-off-by: egernug <gerard.nugent@est.tech>

Merge "Autodiscover cps-and-ncmp containers when load balancing"

Merge "Added architecture tests to validate ONLY NCMP REST/Service dependencies"

Added architecture tests to validate ONLY NCMP REST/Service dependencies

  - removed old tests and added new test cases for the NCMP controller and service layers
  - other dependencies will be added in later commits

Issue-ID: CPS-2422
Change-Id: Iabc4b53e9414477cf3d0b2e1e5a03ba8b4995169
Signed-off-by: leventecsanyi <levente.csanyi@est.tech>

Merge "Added K6 script test to check a throughput of CPS kafka consumer"

Merge "Implementation of Data validation feature in Create a Node API"

Autodiscover cps-and-ncmp containers when load balancing

- remove hard-coded container names from load balancer(nginx) and
  prometheus

Issue-ID: CPS-2437

Change-Id: Ia64b949c2eb3e4d67e123b046e66d5bfdda2c801
Signed-off-by: halil.cakal <halil.cakal@est.tech>

Fine tune JVM parameters for cps-and-ncmp

- limit the total memory with 2G
- set heap memory 75% of the total RAM

Issue-ID: CPS-2180

Change-Id: Ia8a6c2bd40866052caba4f332ac563e1ed829212
Signed-off-by: halil.cakal <halil.cakal@est.tech>

Implementation of Data validation feature in Create a Node API

Added support to validate JSON/XML data without the need of persisting
it in the databse.
 - added "dryRunInQuery" flag as a new query parameter
 - added new method as part of CpsDataService layer to perform data
   validation
 - added new method in yang parser "validateData" to validate
   data without persisting it

Issue-ID: CPS-2361
Change-Id: I43dd33cc6120576d0fac606d5c4b0168d107311d
Signed-off-by: Arpit Singh <as00745003@techmahindra.com>

Merge "Policy Executor: handle errors, part 2 (fighting between IntelliJ and Checkstyle best practices)"

Merge "Re-adjust cps expectations"

Merge "RTD update for delta API 2"

Bump CPS to 3.5.4-SNAPSHOT

- bump cps to 3.5.4-SNAPSHOT
- updated release notes

Issue-ID: CPS-2434
Change-Id: I3d5e6f37711bb29b61b598cc5af57ecfdc35cbba
Signed-off-by: mpriyank <priyank.maheshwari@est.tech>

[CPS] Step-3: OSLO release changes

- cps maven stage docker yaml is created for release 3.5.3

Issue-ID:CPS-2434

Change-Id: I4048b9f54b9c0b0eb433fef9d9c403d54c39bc35
Signed-off-by: sourabh_sourabh <sourabh.sourabh@est.tech>

[CPS] Step-2: OSLO release changes

- cps maven stage master yaml is created for release 3.5.3

Issue-ID:CPS-2434
Change-Id: I5353eb64252b3e071b5203d931f0f646ef290c13
Signed-off-by: sourabh_sourabh <sourabh.sourabh@est.tech>

[CPS] OSLO release changes

- Updated open api for cps, ncmp and policy apis.
- Updated release notes with bus and features.

Issue-ID: CPS-2434
Change-Id: Ib4d80f3939b6e267f228177c817e43832a9384db
Signed-off-by: sourabh_sourabh <sourabh.sourabh@est.tech>

Merge "[BUG] Fix memory leak related to using arrays in Hibernate"

Revert "[1/2] Move Swagger API to docs folder"

This reverts commit b76392e2d1629d4eb67b10c450cdd954ef678966.

Reason for revert: Release blocker

Change-Id: Ic477dbaaad34992c8f440981d92f12b977cdf96e
Signed-off-by: emaclee <lee.anjella.macabuhay@est.tech>

RTD update for delta API 2

 - Documentation update for Delta Feature and Delta API 1
 - Added documentation for API 2: Delta between anchor
   and JSON payload

Issue-ID: CPS-2212
Signed-off-by: Arpit Singh <as00745003@techmahindra.com>
Change-Id: Id74cd930ce48e5cb414aa62c5381b79675346a37

[BUG] Fix memory leak related to using arrays in Hibernate

The use of arrays like String[] instead of Collection<String> in
JpaRepository methods is causing a memory leak, most likely due to a
bug in the hypersistence-utils dependency which provides the feature.
Note code using arrays in JDBC (via jdbcTemplate) is not affected.

This patch removes most uses of arrays in Java, except one needed for
FragmentPrefetchRepository using JDBC setArray(), which is safe.

Issue-ID: CPS-2430
Signed-off-by: danielhanrahan <daniel.hanrahan@est.tech>
Change-Id: I94f8c3d4c8c32ebe0978c08d3226a196a95bf3b9

Revert "Fix inputSpec for openapi code gen"

This reverts commit 4baa75b06dcfb9365a8ea5cde03d77d7f21fa01c.

Reason for revert: Release blocker

Change-Id: I6156461de64dcec41e3f84cbd0cf90f63938bea6
Signed-off-by: emaclee <lee.anjella.macabuhay@est.tech>

Revert "CI: Add test (silent) checkov scan as part of the verify process"

This reverts commit c8e2b3383830395bb4bc37371a9c6119a316cb53.

Issue-ID: CIMAN-33
Change-Id: I5120097ad05394e3667a868c4b7edd44ef1aa070
Signed-off-by: Jessica Wagantall <jwagantall@linuxfoundation.org>

Policy Executor: handle errors, part 2
(fighting between IntelliJ and Checkstyle best practices)

- non-2xx responses are processed using web client exceptions
- handle unknown host exception
- upgraded spotbugs (checkstyle and related mvn plugin)
- fixed some small spotbugs due to upgrade
- added commented instructions in docker compose to enable debugging
- added some environment variables for policy executor configuration
- extract out Sleeper in stub service to achieve 100% coverage
- added cause to Policy Executor exceptions where applicable
- ignored (new) spotbug rule about catch NPE because of issue in 3pp
- ignored (new) spotbug rule about \n in string due to multiline string block

Issue-ID: CPS-2412
Change-Id: I6835a73320c436cbeea12cc7a06f15899eec7bf1
Signed-off-by: ToineSiebelink <toine.siebelink@est.tech>

Added K6 script test to check a throughput of CPS kafka consumer

- Added new K6 script to produce cm avc enents. (It is not a part
  regular automated tests)
- Added a json resource (avc event)

Issue-ID: CPS-2329
Change-Id: I6446bc120382257c5039df17fd34b84b7c6f6550
Signed-off-by: sourabh_sourabh <sourabh.sourabh@est.tech>

Re-adjust cps expectations

- aligning cps expectation for new plots

Issue-ID: CPS-2350

Change-Id: I5b300c4f5432bf2bdfe8a8b0d52b2ce60bd338d6
Signed-off-by: halil.cakal <halil.cakal@est.tech>

Update JIRA and Wiki links host name from wiki.onap.org to lf-onap.atlassian.net

Issue-ID: CPS-2432
Change-Id: I53f380a4fe0968d1ec9f1393651c7c43f03d0d95
Signed-off-by: sourabh_sourabh <sourabh.sourabh@est.tech>

Merge "BugFix: 500 Server error when upgrade and service restart with data in DB"

Fix inputSpec for openapi code gen

Existing inputSpec only accessible from cps root. Changing path to be accessible elsewhere

Issue-ID: CPS-2332

Change-Id: I94ada93ea3c88d2fdca84d49c21e1ffc2923f4b2
Signed-off-by: egernug <gerard.nugent@est.tech>

Merge "[1/2] Move Swagger API to docs folder"

CI: Add test (silent) checkov scan as part of the verify process

Issue-ID: CIMAN-33
Change-Id: I70b171824acb913f67fd28c119bfbcbc1cc1e470
Signed-off-by: Jessica Wagantall <jwagantall@linuxfoundation.org>

BugFix: 500 Server error when upgrade and service restart with data in DB

- return NONE for get effective trust level api if the trustlevel caches
  empty (restart case)

Issue-ID: CPS-2409

Change-Id: I72a755ca6ba5d4a78f5458a235c0d1f43aaa8d53
Signed-off-by: halil.cakal <halil.cakal@est.tech>

Merge "[k6] CM handles searches using different filters"

Policy Executor: handle errors

- configurable default answer
- apply default answer upon non 2xx response
- delayed default webclient read timeout
- add custom timeout method with original read timeout in seconds
- apply default answer upon timeout
- add integration test with short timeout error scenario

Issue-ID: CPS-2412
Change-Id: I62527a27e426c2f01fda2182ebd2513242c29ac1
Signed-off-by: ToineSiebelink <toine.siebelink@est.tech>

Remove Hazelcast cache for prefix resolver (CPS-2417 #2)

This patch removes the AnchorDataCache from CPS, which is used for
prefix resolution in get/query operations.

As such, Hazelcast is no longer a dependency of CPS, only NCMP.

- Changed PrefixResolver to be more efficient.
- Removed AnchorDataCache and associated classes.
- Moved HazelcastCacheConfig to NCMP.
- Removed Hazelcast dependency from cps-service/pom.xml

This shows good performance improvements in some APIs such as v2 GET
which is nearly 2x faster (also 5x faster including base patch).

Issue-ID: CPS-2417
Signed-off-by: danielhanrahan <daniel.hanrahan@est.tech>
Change-Id: I24768469f24e90b70f7a6187faa4f5b3d75777d2

Fix intermittent error in integration test

- make collection verification order independent

Issue-ID: CPS-475
Change-Id: Ib070fd5360463c93ba6193a84a1dfcd11cd802fa
Signed-off-by: ToineSiebelink <toine.siebelink@est.tech>

Merge "Reduce anchor lookups related to PrefixResolver (CPS-2417 #1)"

Merge "Revert "Correctly report trust level if DMI is down""

[k6] CM handles searches using different filters

Each 5 VUs for searches uses a different filter:
no filter, module, property, cps path, and trust level.
This is needed since each search may have different
performance. This also helps reduce load versus doing
a combined module and property search all the time.

Issue-ID: CPS-2349
Signed-off-by: danielhanrahan <daniel.hanrahan@est.tech>
Change-Id: I33a75260f97697d2061d10b80ba7eeb2f6c346f5

Reduce anchor lookups related to PrefixResolver (CPS-2417 #1)

Instead of looking up same Anchor many times inside a for-loop,
do it once outside the loop.

This greatly improves performance in some cases, such as v2 GET API:
- /cps/api/v2/dataspaces/{dataspace}/anchors/{anchor}/node
Testing shows 3x faster response time.

Issue-ID: CPS-2417
Signed-off-by: danielhanrahan <daniel.hanrahan@est.tech>
Change-Id: I80d97d8cc24372eed70626ed840cad985cbe0a4b

Revert "Correctly report trust level if DMI is down"

This reverts commit https://gerrit.onap.org/r/c/cps/+/138853
Reason for revert: Commit caused performance issues in CM handle search

Manual rebase was required to integrate changes for alternate-id
support made since original commit was merged.

Issue-ID: CPS-2375
Signed-off-by: danielhanrahan <daniel.hanrahan@est.tech>
Change-Id: I5ef743ea053589fa17c10587317a3534c2714c4b

Merge "Make nginx load balance using least-connections"

Retry mechanism (with back off algorithm) is removed with more frequent watchdog poll

- Increased watchdog frequency for locked cm handle.
- Removed retry backoff algorithm for locked cm handle.

Issue-ID: CPS-2395
Change-Id: I54d0ec8f9de53a7d181639c14aaaa93736f03e19
Signed-off-by: sourabh_sourabh <sourabh.sourabh@est.tech>

Make nginx load balance using least-connections

By default, nginx uses round robin for load balancing. Round robin
works well when all requests take same amount of time, but is not
optimal for dynamic loads.

With least-connected load balancing, nginx will try not to overload
a busy server with excessive requests, sending requests to a less
busy server instead.

Issue-ID: CPS-2415
Signed-off-by: danielhanrahan <daniel.hanrahan@est.tech>
Change-Id: I0e7d6b36a79aa6541f2f5f667746f2792d55e54b

Merge "[Cps Path Parser] Fixes for parent path & normalization"

Merge "Change execution order of the legacy async batch read operation in k6 suite"

Change execution order of the legacy async batch read operation in k6 suite

- run the test after the parallel cases
- add 30 seconds safe-period of time waiting for completion of
  active threads (remained from previous test cases)

Issue-ID: CPS-2414

Change-Id: I1260b37cd4f1974eebcccf867b36172b407a9646
Signed-off-by: halil.cakal <halil.cakal@est.tech>

[1/2] Move Swagger API to docs folder

Code generation moved to docs folder for cps-core

Issue-ID: CPS-2332

Change-Id: I9a8c5007a3039e0f46114c589de0756d6de91127
Signed-off-by: egernug <gerard.nugent@est.tech>

[Cps Path Parser] Fixes for parent path & normalization

This commit fixes issues with Cps Path Parser related to
path normalization and parent path generation when using
descendant paths and ancestor axis.

Issue-ID: CPS-2365
Signed-off-by: danielhanrahan <daniel.hanrahan@est.tech>
Change-Id: I728fc379b134bd62c39a7085650930450c8a8597

Merge "Refactored cps-ri package structure"

Refactored cps-ri package structure

  - fixed import order and moved package structure

Issue-ID: CPS-2293
Change-Id: Ie2f9f057f261577054530feee7480850ba4b41e1
Signed-off-by: leventecsanyi <levente.csanyi@est.tech>

Merge "Policy Executor API: Add Prefix in Yaml"

Policy Executor API: Add Prefix in Yaml

- Remove servers from yaml.
- Add prefix to path in yaml.
- Remove basepath prefix from Controller Impl.
- Added policy server port to 8093.
- Exposed docker env. vars for policy services.

Issue-ID: CPS-2291
Change-Id: Idcc9e23f9d63bad2480537dc8d39523fe82b4b83
Signed-off-by: ToineSiebelink <toine.siebelink@est.tech>
Signed-off-by: sourabh_sourabh <sourabh.sourabh@est.tech>

Merge "Fix violation reported by sonarqube"

Fix violation reported by sonarqube

Issue-ID: CPS-89

Change-Id: Ia6e3bb26ffb433f3610f53f408fad12489f2e3b3
Signed-off-by: halil.cakal <halil.cakal@est.tech>

Add CPS limit to the plots

- as of now the cps limits given manually according to
  the latest result
- there will be another commit will change the plot
  job in cicd repo to reflect the values on the plots

Issue-ID: CPS-2350

Change-Id: Ic371f46894f1825dca1d26ec65ae0f62aa06c327
Signed-off-by: halil.cakal <halil.cakal@est.tech>

Calculate throughput for legacy batch read data operations

- stress the cps-and-ncmp module for batch read data operation
- for this aim use shared-ireations during async http request
- as parallel to this, consume expected number of messages
  as fast as possible

Issue-ID: CPS-2268

Change-Id: I1b6724479dac7391cbb6407fda52d15176aa8985
Signed-off-by: halil.cakal <halil.cakal@est.tech>

Merge "Renaming Delta Operations as per RFC 9144"

[k6] Fix incorrect results when errors occur in k6 tests (case 2)

De-registration is reported as 200,000 handles per second if all operations fail
- Added a running total for successful de-registrations of CM Handles

Issue-ID: CPS-2346

Change-Id: Ib4302d3b605ff1eaf4ecadac11b4e851a69210df
Signed-off-by: egernug <gerard.nugent@est.tech>

Merge "Fix high-cardinality metrics in k6, causing high memory use"

Merge "Uplift liquibase-core to 4.29.0"

Merge "NCMP should not use CPS SPI classes"

Renaming Delta Operations as per RFC 9144

RFC 9144 recommends naming convention for operations that can be performed on any datastore.
The recommended names are:
- create
- remove
- replace
Whereas in CPS Delta feature these operaitions were named:
- add
- remove
- update
So, "add" and "update" operations have been changed to "create" and "replace" respectively.

Issue-ID: CPS-2393
Change-Id: Ie39f5f8ccc91e96cf484ca7fbc833a6be8758054
Signed-off-by: Arpit Singh <as00745003@techmahindra.com>

Fix high-cardinality metrics in k6, causing high memory use

- Added unique IDs as metric tags for all the endpoints.
- Re-arranged order of public and prive js methods.

Issue-ID: CPS-2331
Change-Id: Ib876a647fb35110c50670c7222986e8a8a6f5ca0
Signed-off-by: sourabh_sourabh <sourabh.sourabh@est.tech>

Uplift liquibase-core to 4.29.0

- uplift liquibase-core to 4.29.0 to resolve the internal licensing
  issue

Issue-ID: CPS-2392
Change-Id: Ia06bbb095a480c910d0df3d23ac719a746508856
Signed-off-by: mpriyank <priyank.maheshwari@est.tech>

Refactor FragmentQueryBuilder

This is a refactoring of FragmentQueryBuilder, there are
no functional changes. This refactoring is needed before
main patch of CPS-2365.

Issue-ID: CPS-2365
Signed-off-by: danielhanrahan <daniel.hanrahan@est.tech>
Change-Id: I648616159719e084ba0e7cb4b09649cd17560cab

NCMP should not use CPS SPI classes

CpsDataPersistenceService is an internal interface of CPS, part of
the SPI, intended only to be used by CPS. Instead, NCMP should use
the public API of CPS, i.e. CpsDataService and CpsQueryService.

This is an architectural issue, but it will affect upcoming changes,
where changes to the SPI implementation should not affect public API.

Issue-ID: CPS-2398
Signed-off-by: danielhanrahan <daniel.hanrahan@est.tech>
Change-Id: I89339e1a3566a25e33168c24c8b8401ca52224e9

[k6] Minor correction to plot titles

Plot title say Module filter instead of Module and Property.

Issue-ID: CPS-2349
Signed-off-by: danielhanrahan <daniel.hanrahan@est.tech>
Change-Id: Ie23080a6085960b89132ad38d07d0c2b6f12471b

Merge "[Cps Path Parser] Move NCMP-specific logic to NCMP"

Merge "[k6] Align tests with updated requirements"

[Cps Path Parser] Move NCMP-specific logic to NCMP

Some special case code to disable ancestor-axis was added for
CM-handle search (see CPS-2308). It is now relocated to NCMP.
This makes other needed improvements of Cps Path Parser easier.

- Move special case code into NCMP
- Add integration test to ensure CM-handle search works

Issue-ID: CPS-2365
Signed-off-by: danielhanrahan <daniel.hanrahan@est.tech>
Change-Id: I168d6156be559166f115aa42e21cd987d98b7d41

Merge "Support alternate id for CPS-E05 GetCmHandleDetailsById"

Merge "[Cps Path Parser] Refactoring leaf conditions"

Merge "K6 Performance Graph fails frequently"

K6 Performance Graph fails frequently

- Overrides initial processing delay of every new module set tag for DMI plugin stub from 2 min (default) to 0 as a temp. solution.

Issue-ID: CPS-2395
Change-Id: Ic875262314406ec55457e31a69c00d55293c1b74
Signed-off-by: sourabh_sourabh <sourabh.sourabh@est.tech>

[Cps Path Parser] Remove unneeded validation in grammar

The invalid prefix/postfix is not needed; existing tests show
that PathParsingException is thrown in for invalid paths.

Issue-ID: CPS-2365
Signed-off-by: danielhanrahan <daniel.hanrahan@est.tech>
Change-Id: I202f0ca0bc5eb768fc3af711180405882dafc22d

[k6] Align tests with updated requirements

- Test load increased to 5 CM handle searches and 5 Id searches to
  reflect current requirements.
- CM handles searches now use a combined search filter including
  1 module and 1 public property, so we are testing with a heavier
  load than required, for early warning of trouble.
- Test load reduced for passthrough operations to 4 VUs per operation
  to reflect addition of batch operation.
- Added test of passthrough write operation with alternate ID.
- Additional refactoring.

Issue-ID: CPS-2349
Signed-off-by: danielhanrahan <daniel.hanrahan@est.tech>
Change-Id: I88d2fe431f74821e8e0e976441efcf82aa320849

Merge "Support Alternate-id for CPS-E05 module search"

Merge "Remove event notifications when removing trust level entries"

Support Alternate-id for CPS-E05 module search

Issue-ID: CPS-2370
Change-Id: I0ca3a1df232ad9fd9f76695cd6ea21d36919130d
Signed-off-by: seanbeirne <sean.beirne@est.tech>