Merge branch 'ethanclevenger91-master'

Author: asimlqt

src/Google/Spreadsheet/ListEntry.php

@@ -75,7 +75,7 @@ public function update($values)
 
         foreach($values as $colName => $value) {
             $entry .= sprintf(
-                '<gsx:%s>%s</gsx:%s>',
+                '<gsx:%s><![CDATA[%s]]></gsx:%s>',
                 $colName,
                 $value,
                 $colName

src/Google/Spreadsheet/ListFeed.php

@@ -68,7 +68,7 @@ public function insert($row)
         $entry = '<entry xmlns="http://www.w3.org/2005/Atom" xmlns:gsx="http://schemas.google.com/spreadsheets/2006/extended">';
         foreach($row as $colName => $value) {
             $entry .= sprintf(
-                '<gsx:%s>%s</gsx:%s>',
+                '<gsx:%s><![CDATA[%s]]></gsx:%s>',
                 $colName,
                 $value,
                 $colName

tests/Google/Spreadsheet/ListEntryTest.php

@@ -0,0 +1,44 @@
+<?php
+namespace Google\Spreadsheet;
+
+use PHPUnit_Framework_TestCase;
+
+class ListEntryTest extends PHPUnit_Framework_TestCase
+{
+    
+    public function testGetEditUrl()
+    {
+        $xml = file_get_contents(__DIR__.'/xml/list-feed.xml');
+        $listFeed = new ListFeed($xml);
+        $listEntry = current($listFeed->getEntries());
+        
+        $this->assertEquals(
+            'https://spreadsheets.google.com/feeds/list/G3345eEsfsk60/od6/private/full/cokwr/bnkj8i7jo6c',
+            $listEntry->getEditUrl()
+        );
+    }
+    
+    public function testUpdate()
+    {
+        $mockServiceRequest = $this->getMockBuilder('Google\Spreadsheet\DefaultServiceRequest')
+                ->setMethods(array("put"))
+                ->disableOriginalConstructor()
+                ->getMock();
+        
+        $mockServiceRequest->expects($this->once())
+            ->method('put')
+            ->with(
+                $this->equalTo('https://spreadsheets.google.com/feeds/list/G3345eEsfsk60/od6/private/full/cokwr/bnkj8i7jo6c'),
+                $this->stringContains('<gsx:nname><![CDATA[Asim]]></gsx:nname>')
+            );
+        
+        ServiceRequestFactory::setInstance($mockServiceRequest);
+        
+        $listFeed = new ListFeed(file_get_contents(__DIR__.'/xml/list-feed.xml'));
+        $entry = current($listFeed->getEntries());
+        $data = $entry->getValues();
+        $data["nname"] = "Asim";
+        $entry->update($data);
+    }
+
+}

tests/Google/Spreadsheet/ListFeedTest.php

@@ -28,7 +28,7 @@ public function testInsert()
             ->method('post')
             ->with(
                 $this->equalTo('https://spreadsheets.google.com/feeds/list/G3345eEsfsk60/od6/private/full'),
-                $this->stringContains('<gsx:occupation>software engineer</gsx:occupation>')
+                $this->stringContains('<gsx:occupation><![CDATA[software engineer]]></gsx:occupation>')
             );
 
         ServiceRequestFactory::setInstance($mockServiceRequest);