Merge pull request #17 from packagist/t/convert-package-json

Package: automatically convert custom package data

Author: naderman

src/Api/Packages.php

@@ -43,6 +43,10 @@ public function createVcsPackage($url, $credentials = null)
 
     public function createCustomPackage($customJson, $credentials = null)
     {
+        if (is_array($customJson) || is_object($customJson)) {
+            $customJson = json_encode($customJson);
+        }
+
         return $this->post('/packages/', ['repoType' => 'package', 'repoConfig' => $customJson, 'credentials' => $credentials]);
     }
 

tests/Api/PackagesTest.php

@@ -97,7 +97,10 @@ public function testCreateVcsPackage()
         $this->assertSame($expected, $api->createVcsPackage('localhost'));
     }
 
-    public function testCreateCustomPackage()
+    /**
+     * @dataProvider customProvider
+     */
+    public function testCreateCustomPackage($customJson)
     {
         $expected = [
             'id' => 'job-id',
@@ -109,9 +112,17 @@ public function testCreateCustomPackage()
         $api->expects($this->once())
             ->method('post')
             ->with($this->equalTo('/packages/'), $this->equalTo(['repoType' => 'package', 'repoConfig' => '{}', 'credentials' => null]))
-            ->will($this->returnValue($expected));
+            ->willReturn($expected);
 
-        $this->assertSame($expected, $api->createCustomPackage('{}'));
+        $this->assertSame($expected, $api->createCustomPackage($customJson));
+    }
+
+    public function customProvider()
+    {
+        return [
+            ['{}'],
+            [new \stdClass()],
+        ];
     }
 
     public function testEditVcsPackage()