Fixing bug where a 'naked' properties fragment cannot be written

The check on content being null was overly restrictive, as v1/documents allows for null content. This may cause some other DMSDK test to fail, in which case that test will be modified as null content is allowable here.

Author: rjrudin

marklogic-client-api/src/main/java/com/marklogic/client/datamovement/impl/WriteBatcherImpl.java

@@ -207,8 +207,9 @@ public WriteBatcher addAs(String uri, Object content) {
 
   @Override
   public WriteBatcher add(DocumentWriteOperation writeOperation) {
-    if ( writeOperation.getUri() == null ) throw new IllegalArgumentException("uri must not be null");
-    if ( writeOperation.getContent() == null ) throw new IllegalArgumentException("contentHandle must not be null");
+	  if (writeOperation.getUri() == null) throw new IllegalArgumentException("uri must not be null");
+	  // Prior to 6.6.1 and higher, threw an exception here if the content was null. But that was not necessary - the
+	  // v1/documents endpoint supports writing a 'naked' properties fragment with no content.
     initialize();
     requireNotStopped();
     queue.add(writeOperation);

marklogic-client-api/src/test/java/com/marklogic/client/test/datamovement/WriteNakedPropertiesTest.java

@@ -0,0 +1,45 @@
+package com.marklogic.client.test.datamovement;
+
+import com.marklogic.client.DatabaseClient;
+import com.marklogic.client.datamovement.DataMovementManager;
+import com.marklogic.client.datamovement.WriteBatcher;
+import com.marklogic.client.io.DocumentMetadataHandle;
+import com.marklogic.client.test.Common;
+import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.Test;
+
+import javax.xml.namespace.QName;
+
+import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertTrue;
+
+public class WriteNakedPropertiesTest {
+
+	@BeforeEach
+	void setup() {
+		Common.newRestAdminClient().newXMLDocumentManager().delete("/naked.xml");
+	}
+
+	@Test
+	void test() {
+		DatabaseClient client = Common.newClient();
+		DataMovementManager dmm = client.newDataMovementManager();
+		WriteBatcher writeBatcher = dmm.newWriteBatcher();
+		dmm.startJob(writeBatcher);
+
+		DocumentMetadataHandle metadata = new DocumentMetadataHandle();
+		metadata.getProperties().put(new QName("org:example", "hello"), "world");
+		writeBatcher.add("/naked.xml", metadata, null);
+		writeBatcher.flushAndWait();
+		dmm.stopJob(writeBatcher);
+
+		DatabaseClient evalClient = Common.newEvalClient();
+		String properties = evalClient.newServerEval()
+			.xquery("xdmp:document-properties('/naked.xml')").evalAs(String.class);
+		assertTrue(properties.contains("world"), "Should be able to read the 'naked' properties fragment, " +
+			"which verifies that it was written correctly, even with the content handle being null.");
+
+		String output = evalClient.newServerEval().xquery("fn:doc-available('/naked.xml')").evalAs(String.class);
+		assertEquals("false", output, "No document exists, only a properties fragment.");
+	}
+}