---

schmidt-sebastian · schmidt-sebastian · commit 467e6a0d549f · 2017-10-27T16:16:25.000-07:00
yaml --- r: 8813 b: refs/heads/lesv-patch-1 c: 8dad469 h: refs/heads/master i: 8811: fc97a4d
diff --git a/[refs] b/[refs]
@@ -66,7 +66,7 @@ refs/tags/v0.22.0: 18b298fe4bfe8ec2f20b0e0bf7ffdcce5cc3c5fe
 refs/heads/vam-google-patch-1: d0c8fee3a4074d0bf7360ce8c4f7f7223d0ee7b9
 refs/heads/vam-google-patch-CODEOWNERS: 2ac1616e25229e51d08a984708ef1918f91a35ee
 refs/heads/danoscarmike-patch-1: 7342a9916bce4ed00002c7202e2a16c5d46afaea
-refs/heads/lesv-patch-1: e30b794e2d5549c3cff944494dec95320455e2f2
+refs/heads/lesv-patch-1: 8dad46922b01428b84f0f7a7cc036573fc4b8942
 refs/heads/ml-update-branch: 079dd6610017f5c51b9d1938c12d6d55b61513cf
 refs/heads/vkedia-patch-2: 7d8241388a9769a5c069334761b06c7012c878e7
 refs/heads/vkedia-patch-3: 4d128043acaa7db9160faf439d2ca6104e8a88cb
diff --git a/branches/lesv-patch-1/google-cloud-firestore/src/main/java/com/google/cloud/firestore/BasePath.java b/branches/lesv-patch-1/google-cloud-firestore/src/main/java/com/google/cloud/firestore/BasePath.java
@@ -16,6 +16,7 @@
 
 package com.google.cloud.firestore;
 
+import com.google.common.base.Preconditions;
 import com.google.common.collect.ImmutableList;
 import javax.annotation.Nullable;
 
@@ -52,6 +53,7 @@ B getParent() {
    * @param path A relative path
    */
   B append(String path) {
+    Preconditions.checkArgument(path != null && !path.isEmpty(), "'path' must be a non-empty String" );
     ImmutableList.Builder<String> components = ImmutableList.builder();
     components.addAll(this.getSegments());
     components.add(splitChildPath(path));
diff --git a/branches/lesv-patch-1/google-cloud-firestore/src/test/java/com/google/cloud/firestore/CollectionReferenceTest.java b/branches/lesv-patch-1/google-cloud-firestore/src/test/java/com/google/cloud/firestore/CollectionReferenceTest.java
@@ -19,10 +19,12 @@
 import static org.junit.Assert.assertEquals;
 import static org.junit.Assert.assertNull;
 import static org.junit.Assert.assertTrue;
+import static org.junit.Assert.fail;
 import static org.mockito.Mockito.doReturn;
 
 import com.google.api.core.ApiFuture;
 import com.google.api.gax.rpc.UnaryCallable;
+import com.google.cloud.firestore.Transaction.Function;
 import com.google.cloud.firestore.spi.v1beta1.FirestoreRpc;
 import com.google.firestore.v1beta1.CommitRequest;
 import com.google.firestore.v1beta1.CommitResponse;
@@ -68,6 +70,13 @@ public void getDocument() {
     assertEquals("doc", documentReference.getId());
     documentReference = collectionReference.document();
     assertEquals(20, documentReference.getId().length());
+
+    try {
+      collectionReference.document("");
+      fail();
+    } catch (IllegalArgumentException e) {
+      assertTrue(e.getMessage().endsWith("'path' must be a non-empty String"));
+    }
   }
 
   @Test
diff --git a/branches/lesv-patch-1/google-cloud-firestore/src/test/java/com/google/cloud/firestore/it/ITSystemTest.java b/branches/lesv-patch-1/google-cloud-firestore/src/test/java/com/google/cloud/firestore/it/ITSystemTest.java
@@ -423,6 +423,7 @@ public String updateCallback(Transaction transaction) {
                 }
               })
           .get();
+      fail();
     } catch (Exception e) {
       assertTrue(e.getMessage().endsWith("User exception"));
     }

Original file line number	Diff line number	Diff line change
`@@ -423,6 +423,7 @@ public String updateCallback(Transaction transaction) {`
`423`	`423`	`}`
`424`	`424`	`})`
`425`	`425`	`.get();`
	`426`	`+ fail();`
`426`	`427`	`} catch (Exception e) {`
`427`	`428`	`assertTrue(e.getMessage().endsWith("User exception"));`
`428`	`429`	`}`