Merge pull request #38 from nreynis/xlsx-support

feat(xlsx): add supports for xlsx

Author: yannickroger

src/RREST.php

@@ -24,6 +24,7 @@ class RREST
         'json' => ['application/json', 'application/x-json'],
         'xml' => ['text/xml', 'application/xml', 'application/x-xml'],
         'csv' => ['text/csv', 'application/csv'],
+        'xlsx' => ['application/vnd.openxmlformats-officedocument.spreadsheetml.sheet'],
     ];
 
     /**

src/Response.php

@@ -41,7 +41,7 @@ class Response
     /**
      * @var string[]
      */
-    protected $supportedFormat = ['json', 'xml', 'csv'];
+    protected $supportedFormat = ['json', 'xml', 'csv', 'xlsx'];
 
     /**
      * @var RouterInterface
@@ -264,10 +264,10 @@ public function serialize($data, $format)
             $data = json_decode(json_encode($data, JSON_UNESCAPED_SLASHES | JSON_UNESCAPED_UNICODE), true);
 
             return $serializer->serialize($data, $format);
-        } elseif ($format === 'csv') {
+        } elseif ($format === 'csv' || $format === 'xlsx') {
             if (!is_string($data)) {
                 throw new \RuntimeException(
-                    'auto serialization for CSV format is not supported'
+                    'auto serialization for '.strtoupper($format).' format is not supported'
                 );
             }
 

tests/units/Response.php

@@ -121,6 +121,34 @@ public function testSerializeForCsv()
         ;
     }
 
+    public function testNoXlsxSerialize()
+    {
+        $this->newTestedInstance($this->router, 'xlsx', 200);
+
+        $this
+            ->given($this->testedInstance)
+            ->exception(
+                function () {
+                    $this->testedInstance->serialize([], 'xlsx');
+                }
+            )
+            ->isInstanceOf('\RuntimeException')
+            ->message->contains('auto serialization for XLSX format is not supported');
+    }
+
+    public function testSerializeForXlsx()
+    {
+        $this->newTestedInstance($this->router, 'xlsx', 200);
+
+        $xlsx = "Placeholder for binary content";
+
+        $this
+            ->given($this->testedInstance)
+            ->string($this->testedInstance->serialize($xlsx, 'xlsx'))
+            ->isEqualTo($xlsx);
+        ;
+    }
+
     public function testAssertReponseSchema()
     {
         $this->newTestedInstance($this->router, 'json', 200);