Add site tests

Author: asloobq

src/main/java/com/uid2/client/KeyParser.java

@@ -6,7 +6,9 @@
 import java.time.Instant;
 import java.util.ArrayList;
 import java.util.Base64;
+import java.util.HashSet;
 import java.util.List;
+import java.util.Set;
 
 
 class KeyParser {
@@ -82,12 +84,12 @@ private static Site getSiteFromJson(JsonObject siteJson) {
             return null;
         }
         JsonArray domainOrAppNamesJArray = siteJson.getAsJsonArray("domain_names");
-        List<String> domainOrAppNames = new ArrayList<>();
+        Set<String> domainOrAppNamesSet = new HashSet<>();
         for (int i = 0; i < domainOrAppNamesJArray.size(); ++i) {
-            domainOrAppNames.add(domainOrAppNamesJArray.get(i).getAsString());
+            domainOrAppNamesSet.add(domainOrAppNamesJArray.get(i).getAsString());
         }
 
-        return new Site(siteId, domainOrAppNames);
+        return new Site(siteId, domainOrAppNamesSet);
     }
 
     static private int getAsInt(JsonObject body, String memberName) {

src/main/java/com/uid2/client/Site.java

@@ -1,21 +1,17 @@
 package com.uid2.client;
 
-import java.util.HashSet;
-import java.util.List;
-import java.util.Set;
+import lombok.AllArgsConstructor;
 import lombok.Getter;
 
+import java.util.Set;
+
 @Getter
+@AllArgsConstructor
 public class Site {
     private final int id;
 
     private final Set<String> domainOrAppNames;
 
-    public Site(int id, List<String> domainOrAppNames) {
-        this.id = id;
-        this.domainOrAppNames = new HashSet<>(domainOrAppNames);
-    }
-
     public boolean allowDomainOrAppName(String domainOrAppName) {
         // Using streams because HashSet's contains() is case sensitive
         return domainOrAppNames.stream().anyMatch(domainOrAppName::equalsIgnoreCase);

src/test/java/com/uid2/client/SiteTests.java

@@ -0,0 +1,50 @@
+package com.uid2.client;
+
+import org.junit.jupiter.api.Assertions;
+import org.junit.jupiter.api.BeforeAll;
+import org.junit.jupiter.params.ParameterizedTest;
+import org.junit.jupiter.params.provider.CsvSource;
+import org.junit.jupiter.params.provider.ValueSource;
+
+import java.util.Arrays;
+import java.util.HashSet;
+
+public class SiteTests {
+
+    private static Site site;
+
+    @BeforeAll
+    public static void setup() {
+        site = new Site(101, new HashSet<>(Arrays.asList(
+         "example.com",
+         "example.org",
+         "com.123.Game.App.android",
+         "123456789"
+        )));
+    }
+
+
+    @ParameterizedTest
+    @ValueSource(strings = {
+            "example.com",
+            "example.org",
+            "com.123.Game.App.android",
+            "123456789",
+            "EXAMPLE.COM",
+            "com.123.game.app.android",
+        })
+    public void testAllowDomainOrAppNameSuccess(String domainOrAppName) {
+        Assertions.assertTrue(site.allowDomainOrAppName(domainOrAppName));
+    }
+
+    @ParameterizedTest
+    @CsvSource({
+            "*",
+            "example",
+            "example*",
+            "example.net"
+    })
+    public void testAllowDomainOrAppNameFailure(String domainOrAppName) {
+        Assertions.assertFalse(site.allowDomainOrAppName(domainOrAppName));
+    }
+}