diff --git a/core/camel-api/src/main/java/org/apache/camel/catalog/EndpointValidationResult.java b/core/camel-api/src/main/java/org/apache/camel/catalog/EndpointValidationResult.java index c674f91556154..3bfbec6b2ce80 100644 --- a/core/camel-api/src/main/java/org/apache/camel/catalog/EndpointValidationResult.java +++ b/core/camel-api/src/main/java/org/apache/camel/catalog/EndpointValidationResult.java @@ -59,9 +59,7 @@ public void addLenient(String name) { if (lenient == null) { lenient = new LinkedHashSet<>(); } - if (!lenient.contains(name)) { - lenient.add(name); - } + lenient.add(name); } public void addNotConsumerOnly(String name) { diff --git a/core/camel-api/src/main/java/org/apache/camel/catalog/PropertiesValidationResult.java b/core/camel-api/src/main/java/org/apache/camel/catalog/PropertiesValidationResult.java index d6303663a01a2..115a420113a40 100644 --- a/core/camel-api/src/main/java/org/apache/camel/catalog/PropertiesValidationResult.java +++ b/core/camel-api/src/main/java/org/apache/camel/catalog/PropertiesValidationResult.java @@ -128,9 +128,7 @@ public void addDeprecated(String name) { if (deprecated == null) { deprecated = new LinkedHashSet<>(); } - if (!deprecated.contains(name)) { - deprecated.add(name); - } + deprecated.add(name); } private String computeErrors(String value) { diff --git a/core/camel-core/src/test/java/org/apache/camel/util/CaseInsensitiveMapTest.java b/core/camel-core/src/test/java/org/apache/camel/util/CaseInsensitiveMapTest.java index 68af18d0531b1..27cb86dcd9b26 100644 --- a/core/camel-core/src/test/java/org/apache/camel/util/CaseInsensitiveMapTest.java +++ b/core/camel-core/src/test/java/org/apache/camel/util/CaseInsensitiveMapTest.java @@ -311,11 +311,11 @@ public void testRomeks() { assertTrue(map.containsKey("foo")); assertTrue(map.containsKey("FOO")); - assertTrue(map.keySet().contains("FOO")); - assertTrue(map.keySet().contains("FoO")); - assertTrue(map.keySet().contains("Foo")); - assertTrue(map.keySet().contains("foo")); - assertTrue(map.keySet().contains("fOO")); + assertTrue(map.containsKey("FOO")); + assertTrue(map.containsKey("FoO")); + assertTrue(map.containsKey("Foo")); + assertTrue(map.containsKey("foo")); + assertTrue(map.containsKey("fOO")); map.put("FOO", "cake"); assertEquals(1, map.size()); @@ -335,7 +335,7 @@ public void testRomeksUsingRegularHashMap() { assertTrue(map.containsKey("foo")); assertFalse(map.containsKey("FOO")); - assertFalse(map.keySet().contains("FOO")); + assertFalse(map.containsKey("FOO")); map.put("FOO", "cake"); assertEquals(2, map.size());