Alissa Bonas has uploaded a new change for review.

Change subject: core: test duplicate keys in AppErrors.properties
......................................................................

core: test duplicate keys in AppErrors.properties

Change-Id: I5a0a83704c48a0c8250fa3a2a71ac2328c24f6a7
Signed-off-by: Alissa Bonas <abo...@redhat.com>
---
A 
backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/DuplicateKeysTest.java
A 
backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/ExtendedProperties.java
2 files changed, 65 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/42/11342/1

diff --git 
a/backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/DuplicateKeysTest.java
 
b/backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/DuplicateKeysTest.java
new file mode 100644
index 0000000..b195c01
--- /dev/null
+++ 
b/backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/DuplicateKeysTest.java
@@ -0,0 +1,44 @@
+package org.ovirt.engine.core;
+
+import org.junit.Test;
+
+import java.io.File;
+import java.io.IOException;
+import java.io.InputStream;
+
+import static org.junit.Assert.fail;
+
+/**
+ * Created by IntelliJ IDEA.
+ * User: abonas
+ * Date: 1/6/13
+ * Time: 6:46 PM
+ * To change this template use File | Settings | File Templates.
+ */
+public class DuplicateKeysTest {
+       @Test
+       public void testkDuplicateKeys() {
+           String fileName = "AppErrors.properties";
+           File file = new File("../main/resources/bundles/" + fileName);
+           file.exists();
+           InputStream is = 
this.getClass().getClassLoader().getResourceAsStream(fileName);
+//           InputStream is = 
this.getClass().getClassLoader().getResourceAsStream(fileName);
+           ExtendedProperties props = new ExtendedProperties();
+           try{
+               props.load(is);
+           }
+           catch(NullPointerException npe) {
+               System.out.println(npe);
+               fail();
+           }
+           catch(IOException ioe) {
+              fail("Test failed with " + ioe.getMessage());
+           }
+           catch(RuntimeException exception) {
+               fail(exception.getMessage());
+           }
+
+
+       }
+
+}
diff --git 
a/backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/ExtendedProperties.java
 
b/backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/ExtendedProperties.java
new file mode 100644
index 0000000..40e82cd
--- /dev/null
+++ 
b/backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/ExtendedProperties.java
@@ -0,0 +1,21 @@
+package org.ovirt.engine.core;
+
+import java.util.Properties;
+
+/**
+ * Created by IntelliJ IDEA.
+ * User: abonas
+ * Date: 1/6/13
+ * Time: 6:48 PM
+ * To change this template use File | Settings | File Templates.
+ */
+public class ExtendedProperties extends Properties {
+
+    @Override
+    public Object put(Object key, Object value) {
+        if(this.containsKey(key))   {
+            throw new RuntimeException("The key " + key + " already exists");
+        }
+        return super.put(key, value);
+    }
+}


--
To view, visit http://gerrit.ovirt.org/11342
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I5a0a83704c48a0c8250fa3a2a71ac2328c24f6a7
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Alissa Bonas <abo...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to