fix new type parsing bug

* introduced in e4978ecfbdd755d585470cc963141e65966fe433
* added tests to avoid further similar issues

2 files changed, 22 insertions, 0 deletions

diff --git a/tests/src/cgeo/geocaching/enumerations/CacheSizeTest.java b/tests/src/cgeo/geocaching/enumerations/CacheSizeTest.java
index fb0a16f..c872c9d 100644
--- a/tests/src/cgeo/geocaching/enumerations/CacheSizeTest.java
+++ b/tests/src/cgeo/geocaching/enumerations/CacheSizeTest.java
@@ -17,4 +17,11 @@ public class CacheSizeTest extends AndroidTestCase {
         assertEquals(CacheSize.LARGE, CacheSize.getById("large"));
     }
 
+    public static void testGetByIdComplete() {
+        for (CacheSize size : CacheSize.values()) {
+            assertEquals(size, CacheSize.getById(size.id));
+            assertEquals(size, CacheSize.getById(size.id.toLowerCase()));
+            assertEquals(size, CacheSize.getById(size.id.toUpperCase()));
+        }
+    }
 }
diff --git a/tests/src/cgeo/geocaching/enumerations/CacheTypeTest.java b/tests/src/cgeo/geocaching/enumerations/CacheTypeTest.java
index 3ed949d..dd35669 100644
--- a/tests/src/cgeo/geocaching/enumerations/CacheTypeTest.java
+++ b/tests/src/cgeo/geocaching/enumerations/CacheTypeTest.java
@@ -18,4 +18,19 @@ public class CacheTypeTest extends AndroidTestCase {
         assertEquals(CacheType.CITO, CacheType.getByPattern("cache in trash out event"));
     }
 
+    public static void testGetByIdComplete() {
+        for (CacheType type : CacheType.values()) {
+            assertEquals(type, CacheType.getById(type.id));
+            assertEquals(type, CacheType.getById(type.id.toLowerCase()));
+            assertEquals(type, CacheType.getById(type.id.toUpperCase()));
+        }
+    }
+
+    public static void testGetByPatternComplete() {
+        for (CacheType type : CacheType.values()) {
+            assertEquals(type, CacheType.getByPattern(type.pattern));
+            assertEquals(type, CacheType.getByPattern(type.pattern.toLowerCase()));
+            assertEquals(type, CacheType.getByPattern(type.pattern.toUpperCase()));
+        }
+    }
 }