Simplify createFeaturesSet() 96/100396/1
authorRobert Varga <robert.varga@pantheon.tech>
Mon, 4 Apr 2022 22:55:39 +0000 (00:55 +0200)
committerRobert Varga <robert.varga@pantheon.tech>
Mon, 4 Apr 2022 22:55:39 +0000 (00:55 +0200)
We can express this method in terms of a Stream transformation, saving
us an intermediate HashSet.

Change-Id: I26649e4d9d43bf288053dc76c1324db1d7cf9e1e
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
parser/yang-parser-rfc7950/src/test/java/org/opendaylight/yangtools/yang/parser/stmt/rfc7950/Bug6869Test.java

index 668f7db76a6c5fe11889bb6b01154e877d1bf42c..0ab52503e7184b8abba7588c2ae1bd028e9b41a1 100644 (file)
@@ -16,9 +16,10 @@ import static org.junit.Assert.assertNotNull;
 import static org.junit.Assert.assertTrue;
 
 import com.google.common.collect.ImmutableSet;
+import java.util.Arrays;
 import java.util.Collection;
-import java.util.HashSet;
 import java.util.Set;
+import java.util.stream.Collectors;
 import org.junit.Test;
 import org.opendaylight.yangtools.yang.common.QName;
 import org.opendaylight.yangtools.yang.model.api.DataSchemaNode;
@@ -71,12 +72,9 @@ public class Bug6869Test extends AbstractYangTest {
     }
 
     private static Set<QName> createFeaturesSet(final String... featureNames) {
-        final Set<QName> supportedFeatures = new HashSet<>();
-        for (final String featureName : featureNames) {
-            supportedFeatures.add(QName.create("foo", featureName));
-        }
-
-        return ImmutableSet.copyOf(supportedFeatures);
+        return Arrays.stream(featureNames)
+            .map(featureName -> QName.create("foo", featureName))
+            .collect(Collectors.toUnmodifiableSet());
     }
 
     @Test