Merge pull request #127 from kopecny-shockworks/master

Fixed CRM.Company.update

Author: mesilov

src/classes/crm/company/company.php

@@ -63,8 +63,10 @@ public function update($bitrix24CompanyId, $fields = array())
 	{
 		$fullResult = $this->client->call(
 			'crm.company.update',
-			array('id' => $bitrix24CompanyId),
-			array('fields' => $fields)
+			array(
+                            'id' => $bitrix24CompanyId,
+                            'fields' => $fields,
+                        )
 		);
 		return $fullResult;
 	}

tests/src/classes/crm/company/companyTest.php

@@ -0,0 +1,51 @@
+<?php
+/*
+ * This file is part of the bitrix24-php-sdk package.
+ *
+ * © Mesilov Maxim <mesilov.maxim@gmail.com>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+namespace Bitrix24\CRM;
+
+use Bitrix24\CRM\Company;
+use Bitrix24\Contracts\iBitrix24;
+
+
+/**
+ * @package Bitrix24
+ */
+class CompanyTest extends \PHPUnit_Framework_TestCase
+{
+    public function testUpdateCallsClientWithIdAndFields()
+    {
+        $testId = \md5(\time());
+        $testField = \md5($testId);
+        $testFields = [
+            $testField => \md5($testField),
+        ];
+
+        //silly mocking in PHPUnit 4.8
+        $methods = array();
+        $ref = new \ReflectionClass('Bitrix24\\Contracts\\iBitrix24');
+        foreach ($ref->getMethods() as $method) {
+            $methods[] = $method->getName();
+        }
+        $client = $this->getMockBuilder('Bitrix24\\Contracts\\iBitrix24')
+            ->setMethods($methods)
+            ->getMock();
+        $client->expects($this->once())
+            ->method('call')
+            ->with(
+                'crm.company.update',
+                array(
+                    'id' => $testId,
+                    'fields' => $testFields,
+                )
+            );
+
+        $company = new Company($client);
+        $company->update($testId, $testFields);
+    }
+}