summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
-rw-r--r--src/com/isode/stroke/jid/JID.java13
-rw-r--r--test/com/isode/stroke/jid/JIDTest.java10
2 files changed, 15 insertions, 8 deletions
diff --git a/src/com/isode/stroke/jid/JID.java b/src/com/isode/stroke/jid/JID.java
index 59eef52..8b62ca1 100644
--- a/src/com/isode/stroke/jid/JID.java
+++ b/src/com/isode/stroke/jid/JID.java
@@ -1,3 +1,3 @@
/*
- * Copyright (c) 2010-2012, Isode Limited, London, England.
+ * Copyright (c) 2010-2016, Isode Limited, London, England.
* All rights reserved.
@@ -16,6 +16,2 @@ package com.isode.stroke.jid;
-import com.isode.stroke.idn.ICUConverter;
-import com.isode.stroke.idn.IDNConverter;
-import com.ibm.icu.text.StringPrepParseException;
-import com.isode.stroke.base.NotNull;
import java.util.Arrays;
@@ -23,2 +19,6 @@ import java.util.List;
+import com.isode.stroke.base.NotNull;
+import com.isode.stroke.idn.ICUConverter;
+import com.isode.stroke.idn.IDNConverter;
+
/**
@@ -149,2 +149,5 @@ public class JID implements Comparable<JID> {
hasResource_ = (resource != null);
+ if (hasResource_ && resource.isEmpty()) {
+ valid_ = false;
+ }
nameprepAndSetComponents(node, domain, resource);
diff --git a/test/com/isode/stroke/jid/JIDTest.java b/test/com/isode/stroke/jid/JIDTest.java
index 96de429..69bee05 100644
--- a/test/com/isode/stroke/jid/JIDTest.java
+++ b/test/com/isode/stroke/jid/JIDTest.java
@@ -14,8 +14,6 @@ package com.isode.stroke.jid;
import static org.junit.Assert.assertEquals;
-import static org.junit.Assert.assertNotNull;
-import static org.junit.Assert.assertNull;
import static org.junit.Assert.assertFalse;
import static org.junit.Assert.assertTrue;
+
import org.junit.Test;
-import com.isode.stroke.jid.JID;
@@ -157,2 +155,8 @@ public class JIDTest {
}
+
+ @Test
+ public void testConstructorWithStrings_EmptyResource() {
+ JID testling = new JID("foo","bar","");
+ assertFalse(testling.isValid());
+ }