testcases for #188

tests/Solarium/Tests/QueryType/Update/Query/Document/DocumentTest.php

@@ -203,6 +203,21 @@ class DocumentTest extends \PHPUnit_Framework_TestCase
         );
     }
 
+    public function testRemoveFieldBySettingNullValueWithModifier()
+    {
+        $this->doc->setKey('key', 123);
+        $this->doc->setField('name', null, null, Document::MODIFIER_SET);
+
+        $expectedFields = $this->fields;
+        $expectedFields['key'] = 123;
+        $expectedFields['name'] = null;
+
+        $this->assertEquals(
+            $expectedFields,
+            $this->doc->getFields()
+        );
+    }
+
     public function testRemoveFieldBySettingToNull()
     {
         $this->doc->setField('name', NULL);

tests/Solarium/Tests/QueryType/Update/RequestBuilderTest.php

@@ -219,6 +219,21 @@ class RequestBuilderTest extends \PHPUnit_Framework_TestCase
         );
     }
 
+    public function testBuildAddXmlWithFieldModifierAndNullValue()
+    {
+        $doc = new Document();
+        $doc->setKey('employeeId','05991');
+        $doc->addField('skills', null, null,  Document::MODIFIER_SET);
+
+        $command = new AddCommand();
+        $command->addDocument($doc);
+
+        $this->assertEquals(
+            '<add><doc><field name="employeeId">05991</field><field name="skills" update="set" null="true"></field></doc></add>',
+            $this->builder->buildAddXml($command)
+        );
+    }
+
     public function testBuildDeleteXml()
     {
         $command = new DeleteCommand;