summaryrefslogtreecommitdiffstats
path: root/models-base/src/test/java
diff options
context:
space:
mode:
authorliamfallon <liam.fallon@est.tech>2019-04-08 13:58:53 +0000
committerliamfallon <liam.fallon@est.tech>2019-04-08 13:58:53 +0000
commit9ce39af891ccf063d46e18ecf5a2a47eb1408930 (patch)
treeeb55cc63c83ea34f73aeaaae5039c96283b66ef8 /models-base/src/test/java
parenteb7127ac85b3df30a09277721a5f9271033843e7 (diff)
Add bug fixes and tests for filters
Fixed bugs on filtering where lack of null checks was blocking all results. Added unit test for PDP related JPA objects. Fixed cascading and orphan control on JPA objects. Added partial testing of PdpProvider. Added partial testing of filters. Changed tag for content of operational policies from "Content" to "content". Issue-ID: POLICY-1095 Change-Id: Ieb22e06955a8434b490bae7d0f6b77d4479515e8 Signed-off-by: liamfallon <liam.fallon@est.tech>
Diffstat (limited to 'models-base/src/test/java')
-rw-r--r--models-base/src/test/java/org/onap/policy/models/base/PfObjectFilterTest.java21
1 files changed, 13 insertions, 8 deletions
diff --git a/models-base/src/test/java/org/onap/policy/models/base/PfObjectFilterTest.java b/models-base/src/test/java/org/onap/policy/models/base/PfObjectFilterTest.java
index c3ccb4aa5..3d16f8e3f 100644
--- a/models-base/src/test/java/org/onap/policy/models/base/PfObjectFilterTest.java
+++ b/models-base/src/test/java/org/onap/policy/models/base/PfObjectFilterTest.java
@@ -81,21 +81,21 @@ public class PfObjectFilterTest {
doList.add(do5);
DummyPfObjectFilter dof = new DummyPfObjectFilter();
- assertFalse(dof.filterOnRegexp("Hello", "Goodbye"));
- assertTrue(dof.filterOnRegexp("Hello", "Hello"));
+ assertFalse(dof.filterString("Hello", "Goodbye"));
+ assertTrue(dof.filterString("Hello", "Hello"));
assertThatThrownBy(() -> {
- dof.filterOnRegexp(null, null);
+ dof.filterString(null, null);
}).hasMessage("value is marked @NonNull but is null");
assertThatThrownBy(() -> {
- dof.filterOnRegexp("hello", null);
- }).hasMessage("regexp is marked @NonNull but is null");
-
- assertThatThrownBy(() -> {
- dof.filterOnRegexp(null, "hello");
+ dof.filterString(null, "hello");
}).hasMessage("value is marked @NonNull but is null");
+ assertEquals(false, dof.filterString("Hello", "Goodbye"));
+ assertEquals(true, dof.filterString("Hello", "Hello"));
+ assertEquals(true, dof.filterString("Hello", null));
+
List<DummyPfObject> latestVersionList = dof.latestVersionFilter(doList);
assertEquals(3, latestVersionList.size());
assertEquals("aaaaa", latestVersionList.get(0).getName());
@@ -104,5 +104,10 @@ public class PfObjectFilterTest {
assertEquals("1.0.0", latestVersionList.get(1).getVersion());
assertEquals("name1", latestVersionList.get(2).getName());
assertEquals("0.1.2", latestVersionList.get(2).getVersion());
+
+ latestVersionList.remove(2);
+ latestVersionList.remove(1);
+ List<DummyPfObject> newestVersionList = dof.latestVersionFilter(latestVersionList);
+ assertEquals(latestVersionList, newestVersionList);
}
}