Selaa lähdekoodia

Test code improvements

Lortseam 4 vuotta sitten
vanhempi
sitoutus
8c6aa98e79

+ 14 - 13
src/test/java/me/lortseam/completeconfig/ConfigTest.java

@@ -5,6 +5,7 @@ import me.lortseam.completeconfig.containers.*;
 import me.lortseam.completeconfig.data.Config;
 import me.lortseam.completeconfig.data.Entry;
 import me.lortseam.completeconfig.data.EntryBase;
+import me.lortseam.completeconfig.groups.EmptyGroup;
 import me.lortseam.completeconfig.io.ConfigSource;
 import nl.altindag.log.LogCaptor;
 import org.apache.logging.log4j.core.util.ReflectionUtil;
@@ -90,43 +91,43 @@ public class ConfigTest {
             public class Entry {
 
                 @Test
-                public void includeAnnotatedFieldIfNonPOJO() {
+                public void includeFieldInNonPOJOIfAnnotated() {
                     Config config = builder.add(new ContainerWithEntry()).build();
                     assertEquals(1, config.getEntries().size());
                 }
 
                 @Test
-                public void excludeNonAnnotatedFieldIfNonPOJO() {
+                public void excludeFieldInNonPOJOIfNotAnnotated() {
                     Config config = builder.add(new ContainerWithField()).build();
                     assertEquals(0, config.getEntries().size());
                 }
 
                 @Test
-                public void includeFieldIfPOJO() {
+                public void includeFieldInPOJO() {
                     Config config = builder.add(new POJOContainerWithEntry()).build();
                     assertEquals(1, config.getEntries().size());
                 }
 
                 @Test
-                public void excludeContainerFieldIfPOJO() {
+                public void excludeFieldInPOJOIfContainer() {
                     Config config = builder.add(new POJOContainerWithEmptyContainer()).build();
                     assertEquals(0, config.getEntries().size());
                 }
 
                 @Test
-                public void ignoreFieldIfAnnotatedWithIgnoreAndPOJO() {
+                public void excludeFieldInPOJOIfIgnoreAnnotated() {
                     Config config = builder.add(new POJOContainerWithIgnoredField()).build();
                     assertEquals(0, config.getEntries().size());
                 }
 
                 @Test
-                public void includeSuperclassField() {
+                public void includeSuperclassFieldIfNonStatic() {
                     Config config = builder.add(new SubclassOfContainerWithEntry()).build();
                     assertEquals(1, config.getEntries().size());
                 }
 
                 @Test
-                public void excludeStaticSuperclassField() {
+                public void excludeSuperclassFieldIfStatic() {
                     Config config = builder.add(new SubclassOfContainerWithStaticEntry()).build();
                     assertEquals(0, config.getEntries().size());
                 }
@@ -137,19 +138,19 @@ public class ConfigTest {
             public class Container {
 
                 @Test
-                public void includeAnnotatedFieldIfNonPOJO() {
+                public void includeFieldInNonPOJOIfAnnotated() {
                     Config config = builder.add(new ContainerWithContainerWithEntry()).build();
                     assertEquals(1, config.getEntries().size());
                 }
 
                 @Test
-                public void excludeNonAnnotatedFieldIfNonPOJO() {
+                public void excludeFieldInNonPOJOIfNotAnnotated() {
                     Config config = builder.add(new ContainerWithNonAnnotatedContainerWithEntry()).build();
                     assertEquals(0, config.getEntries().size());
                 }
 
                 @Test
-                public void includeFieldIfPOJO() {
+                public void includeFieldInPOJO() {
                     Config config = builder.add(new POJOContainerWithContainerWithEntry()).build();
                     assertEquals(1, config.getEntries().size());
                 }
@@ -167,19 +168,19 @@ public class ConfigTest {
                 }
 
                 @Test
-                public void excludeNestedIfPOJO() {
+                public void excludeNestedInPOJOIfNonStatic() {
                     Config config = builder.add(new POJOContainerNestingContainerWithEntry()).build();
                     assertEquals(0, config.getEntries().size());
                 }
 
                 @Test
-                public void includeNestedStaticIfPOJO() {
+                public void includeNestedIfStaticAndPOJO() {
                     Config config = builder.add(new POJOContainerNestingStaticContainerWithEntry()).build();
                     assertEquals(1, config.getEntries().size());
                 }
 
                 @Test
-                public void excludeNestedStaticIfNonPOJO() {
+                public void excludeStaticNestedIfNonPOJO() {
                     Config config = builder.add(new ContainerNestingStaticContainerWithEntry()).build();
                     assertEquals(0, config.getEntries().size());
                 }

+ 1 - 0
src/test/java/me/lortseam/completeconfig/containers/ContainerIncludingGroupWithEntry.java

@@ -1,6 +1,7 @@
 package me.lortseam.completeconfig.containers;
 
 import me.lortseam.completeconfig.api.ConfigEntryContainer;
+import me.lortseam.completeconfig.groups.GroupWithEntry;
 
 public class ContainerIncludingGroupWithEntry implements ConfigEntryContainer {
 

+ 1 - 0
src/test/java/me/lortseam/completeconfig/containers/ContainerWithGroupWithEntry.java

@@ -1,6 +1,7 @@
 package me.lortseam.completeconfig.containers;
 
 import me.lortseam.completeconfig.api.ConfigEntryContainer;
+import me.lortseam.completeconfig.groups.GroupWithEntry;
 
 public class ContainerWithGroupWithEntry implements ConfigEntryContainer {
 

+ 1 - 1
src/test/java/me/lortseam/completeconfig/containers/EmptyGroup.java → src/test/java/me/lortseam/completeconfig/groups/EmptyGroup.java

@@ -1,4 +1,4 @@
-package me.lortseam.completeconfig.containers;
+package me.lortseam.completeconfig.groups;
 
 import me.lortseam.completeconfig.api.ConfigGroup;
 

+ 1 - 1
src/test/java/me/lortseam/completeconfig/containers/GroupWithEntry.java → src/test/java/me/lortseam/completeconfig/groups/GroupWithEntry.java

@@ -1,4 +1,4 @@
-package me.lortseam.completeconfig.containers;
+package me.lortseam.completeconfig.groups;
 
 import me.lortseam.completeconfig.api.ConfigEntry;
 import me.lortseam.completeconfig.api.ConfigGroup;