[geode] branch feature/GEODE-6291 updated: added a unit test for TypeRegistry

[ds...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

dschneider pushed a commit to branch feature/GEODE-6291
in repository https://gitbox.apache.org/repos/asf/geode.git


The following commit(s) were added to refs/heads/feature/GEODE-6291 by this push:
     new 3221de4  added a unit test for TypeRegistry
3221de4 is described below

commit 3221de4c9d4ddc14b182e4d90490000dda3a6ebe
Author: Darrel Schneider <ds...@pivotal.io>
AuthorDate: Mon Jan 28 14:36:34 2019 -0800

    added a unit test for TypeRegistry
---
 .../geode/pdx/internal/TypeRegistryTest.java       | 110 +++++++++++++++++++++
 1 file changed, 110 insertions(+)

diff --git a/geode-core/src/test/java/org/apache/geode/pdx/internal/TypeRegistryTest.java b/geode-core/src/test/java/org/apache/geode/pdx/internal/TypeRegistryTest.java
new file mode 100644
index 0000000..0618034
--- /dev/null
+++ b/geode-core/src/test/java/org/apache/geode/pdx/internal/TypeRegistryTest.java
@@ -0,0 +1,110 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more contributor license
+ * agreements. See the NOTICE file distributed with this work for additional information regarding
+ * copyright ownership. The ASF licenses this file to You under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance with the License. You may obtain a
+ * copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software distributed under the License
+ * is distributed on an "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express
+ * or implied. See the License for the specific language governing permissions and limitations under
+ * the License.
+ */
+package org.apache.geode.pdx.internal;
+
+import static org.assertj.core.api.Assertions.assertThat;
+import static org.assertj.core.api.Assertions.catchThrowable;
+import static org.mockito.Mockito.mock;
+import static org.mockito.Mockito.when;
+
+import java.util.Arrays;
+import java.util.Collections;
+import java.util.HashSet;
+import java.util.Set;
+
+import org.junit.Test;
+import org.junit.experimental.categories.Category;
+
+import org.apache.geode.internal.cache.InternalCache;
+import org.apache.geode.test.junit.categories.SerializationTest;
+
+@Category({SerializationTest.class})
+public class TypeRegistryTest {
+  private final InternalCache cache = mock(InternalCache.class);
+  private TypeRegistration distributedTypeRegistry = mock(TypeRegistration.class);
+  private final TypeRegistry typeRegistry = new TypeRegistry(cache, distributedTypeRegistry);
+  private static final String PDX_CLASS_NAME = "pdxClassName";
+
+  @Test
+  public void findFieldThatMatchesNameReturnsNullGivenNoTypes() {
+    Set<PdxType> pdxTypesForClass = Collections.emptySet();
+    when(distributedTypeRegistry.getPdxTypesForClassName(PDX_CLASS_NAME))
+        .thenReturn(pdxTypesForClass);
+
+    PdxField foundField = this.typeRegistry.findFieldThatMatchesName(PDX_CLASS_NAME, "fieldName");
+
+    assertThat(foundField).isNull();
+  }
+
+  @Test
+  public void findFieldThatMatchesNameReturnsFieldThatExactlyMatches() {
+    PdxType exactMatchType = mock(PdxType.class);
+    PdxField exactMatchField = mock(PdxField.class);
+    when(exactMatchType.getPdxField("fieldName")).thenReturn(exactMatchField);
+    Set<PdxType> pdxTypesForClass = new HashSet<>(Arrays.asList(exactMatchType));
+    when(distributedTypeRegistry.getPdxTypesForClassName(PDX_CLASS_NAME))
+        .thenReturn(pdxTypesForClass);
+
+    PdxField foundField = this.typeRegistry.findFieldThatMatchesName(PDX_CLASS_NAME, "fieldName");
+
+    assertThat(foundField).isSameAs(exactMatchField);
+  }
+
+  @Test
+  public void findFieldThatMatchesNameReturnsFieldThatInexactlyMatches() {
+    PdxType inexactMatchType = mock(PdxType.class);
+    PdxField inexactMatchField = mock(PdxField.class);
+    when(inexactMatchType.getPdxField("fieldName")).thenReturn(null);
+    when(inexactMatchType.getFieldNames()).thenReturn(Arrays.asList("skipThisOne", "FIELDNAME"));
+    when(inexactMatchType.getPdxField("FIELDNAME")).thenReturn(inexactMatchField);
+    Set<PdxType> pdxTypesForClass = new HashSet<>(Arrays.asList(inexactMatchType));
+    when(distributedTypeRegistry.getPdxTypesForClassName(PDX_CLASS_NAME))
+        .thenReturn(pdxTypesForClass);
+
+    PdxField foundField = this.typeRegistry.findFieldThatMatchesName(PDX_CLASS_NAME, "fieldName");
+
+    assertThat(foundField).isSameAs(inexactMatchField);
+  }
+
+  @Test
+  public void findFieldThatMatchesNameReturnsNullIfFieldExistButNoneMatch() {
+    PdxType noMatchType = mock(PdxType.class);
+    when(noMatchType.getPdxField("fieldName")).thenReturn(null);
+    when(noMatchType.getFieldNames()).thenReturn(Arrays.asList("nomatch1", "nomatch2"));
+    Set<PdxType> pdxTypesForClass = new HashSet<>(Arrays.asList(noMatchType));
+    when(distributedTypeRegistry.getPdxTypesForClassName(PDX_CLASS_NAME))
+        .thenReturn(pdxTypesForClass);
+
+    PdxField foundField = this.typeRegistry.findFieldThatMatchesName(PDX_CLASS_NAME, "fieldName");
+
+    assertThat(foundField).isNull();
+  }
+
+  @Test
+  public void findFieldThatMatchesNameThrowsIfMoreThanOneMatch() {
+    PdxType inexactMatchType = mock(PdxType.class);
+    when(inexactMatchType.getPdxField("fieldName")).thenReturn(null);
+    when(inexactMatchType.getFieldNames()).thenReturn(Arrays.asList("fieldname", "FIELDNAME"));
+    Set<PdxType> pdxTypesForClass = new HashSet<>(Arrays.asList(inexactMatchType));
+    when(distributedTypeRegistry.getPdxTypesForClassName(PDX_CLASS_NAME))
+        .thenReturn(pdxTypesForClass);
+
+    Throwable throwable = catchThrowable(
+        () -> this.typeRegistry.findFieldThatMatchesName(PDX_CLASS_NAME, "fieldName"));
+
+    assertThat(throwable).isInstanceOf(IllegalStateException.class)
+        .hasMessage("the pdx fields fieldname, FIELDNAME all match fieldName");
+  }
+}