Merge pull request #10 from AndrewNovikof/fix_tests

fix capture and refund, update readme, update tests

README.md

@@ -44,7 +44,7 @@ $gateway = Omnipay::create('Sberbank');
 $gateway->authorize(
     [
        'orderNumber' => $localOrderNumber, // local order number
-       'amount' => $order_amount * 100, // The amount of payment in kopecks (or cents)
+       'amount' => $order_amount, // The amount of payment (you can use integer, decimal with 2 precisions or string equal to decimal
        'returnUrl' => $callback_url // succesfull callback url
     ]
 );
@@ -113,7 +113,7 @@ $gateway = Omnipay::create('Sberbank');
 $response = $gateway->authorize(
     [
        'orderNumber' => $localOrderNumber, // local order number
-       'amount' => $order_amount * 100, // The amount of payment in kopecks (or cents)
+       'amount' => $order_amount, // The amount of payment (you can use integer, decimal with 2 precisions or string equal to decimal
        'returnUrl' => $callback_url, // succesfull callback url
        'description' => 'Order Description'
     ]
@@ -161,7 +161,7 @@ $gateway = Omnipay::create('Sberbank');
 $response = $gateway->authorize(
     [
        'orderNumber' => $localOrderNumber, // local order number
-       'amount' => $order_amount * 100, // The amount of payment in kopecks (or cents)
+       'amount' => $order_amount, // The amount of payment (you can use integer, decimal with 2 precisions or string equal to decimal
        'returnUrl' => $callback_url, // succesfull callback url
        'description' => 'Order Description'
     ]
@@ -210,7 +210,7 @@ $gateway = Omnipay::create('Sberbank');
 $response = $gateway->capture(
     [
        'orderId' => $localOrderNumber, // gateway order number
-       'amount' => $order_amount * 100, // The amount of payment in kopecks (or cents)
+       'amount' => $order_amount, // The amount of payment (you can use integer, decimal with 2 precisions or string equal to decimal
     ]
 )->setUserName('merchant_login')
  ->setPassword('merchant_password')

composer.json

@@ -32,7 +32,7 @@
   },
   "autoload-dev": {
     "psr-4": {
-      "Tests\\Examples\\": "tests/Examples"
+      "Omnipay\\Sberbank\\Tests\\": "tests/"
     }
   },
   "minimum-stability": "dev",

src/Message/AbstractRequest.php

@@ -2,7 +2,6 @@
 
 namespace Omnipay\Sberbank\Message;
 
-use Omnipay\Common\Exception\InvalidRequestException;
 use Omnipay\Common\Exception\RuntimeException;
 use Omnipay\Common\Message\AbstractRequest as BaseAbstractRequest;
 

src/Message/CaptureRequest.php

@@ -18,7 +18,7 @@ public function getData()
 
         $data = [
             'orderId' => $this->getOrderId(),
-            'amount' => $this->getAmount(),
+            'amount' => $this->getAmountInteger(),
         ];
 
         return $data;

src/Message/RefundRequest.php

@@ -18,7 +18,7 @@ public function getData()
 
         $data = [
             'orderId' => $this->getOrderId(),
-            'amount' => $this->getAmount(),
+            'amount' => $this->getAmountInteger(),
         ];
 
         return $data;

tests/Message/CaptureRequestTest.php

@@ -69,7 +69,9 @@ protected function getRequestParameters()
      */
     public function testData()
     {
-        $this->assertEquals($this->request->getData(), $this->getRequestParameters());
+        $params = $this->getRequestParameters();
+        $params['amount'] = $params['amount'] * 100;
+        $this->assertEquals($this->request->getData(), $params);
     }
 
     /**

tests/Message/RefundRequestTest.php

@@ -74,7 +74,9 @@ public function getMethod()
      */
     public function testData()
     {
-        $this->assertEquals($this->request->getData(), $this->getRequestParameters());
+        $params = $this->getRequestParameters();
+        $params['amount'] = $params['amount'] * 100;
+        $this->assertEquals($this->request->getData(), $params);
     }
 
     /**