Merge pull request #50 from pilot/renaming

Globar renamed store/create, unstore/delete gateway requests

Author: amacneil

src/Omnipay/Common/AbstractGateway.php

@@ -170,33 +170,33 @@ public function supportsVoid()
     }
 
     /**
-     * Supports Store
+     * Supports CreateCard
      *
-     * @return boolean True if this gateway supports the store() method
+     * @return boolean True if this gateway supports the create() method
      */
-    public function supportsStore()
+    public function supportsCreateCard()
     {
-        return method_exists($this, 'store');
+        return method_exists($this, 'createCard');
     }
 
     /**
-     * Supports Unstore
+     * Supports DeleteCard
      *
-     * @return boolean True if this gateway supports the unstore() method
+     * @return boolean True if this gateway supports the delete() method
      */
-    public function supportsUnstore()
+    public function supportsDeleteCard()
     {
-        return method_exists($this, 'unstore');
+        return method_exists($this, 'deleteCard');
     }
 
     /**
-     * Supports Update
+     * Supports UpdateCard
      *
      * @return boolean True if this gateway supports the update() method
      */
-    public function supportsUpdate()
+    public function supportsUpdateCard()
     {
-        return method_exists($this, 'update');
+        return method_exists($this, 'updateCard');
     }
 
     /**

tests/Omnipay/GatewayTestCase.php

@@ -149,27 +149,39 @@ public function testSupportsVoid()
         }
     }
 
-    public function testSupportsStore()
+    public function testSupportsCreateCard()
     {
-        $supportsStore = $this->gateway->supportsStore();
-        $this->assertInternalType('boolean', $supportsStore);
+        $supportsCreate = $this->gateway->supportsCreateCard();
+        $this->assertInternalType('boolean', $supportsCreate);
 
-        if ($supportsStore) {
-            $this->assertInstanceOf('Omnipay\Common\Message\RequestInterface', $this->gateway->store());
+        if ($supportsCreate) {
+            $this->assertInstanceOf('Omnipay\Common\Message\RequestInterface', $this->gateway->createCard());
         } else {
-            $this->assertFalse(method_exists($this->gateway, 'store'));
+            $this->assertFalse(method_exists($this->gateway, 'createCard'));
         }
     }
 
-    public function testSupportsUnstore()
+    public function testSupportsDeleteCard()
     {
-        $supportsUnstore = $this->gateway->supportsUnstore();
-        $this->assertInternalType('boolean', $supportsUnstore);
+        $supportsDelete = $this->gateway->supportsDeleteCard();
+        $this->assertInternalType('boolean', $supportsDelete);
 
-        if ($supportsUnstore) {
-            $this->assertInstanceOf('Omnipay\Common\Message\RequestInterface', $this->gateway->unstore());
+        if ($supportsDelete) {
+            $this->assertInstanceOf('Omnipay\Common\Message\RequestInterface', $this->gateway->deleteCard());
         } else {
-            $this->assertFalse(method_exists($this->gateway, 'unstore'));
+            $this->assertFalse(method_exists($this->gateway, 'deleteCard'));
+        }
+    }
+
+    public function testSupportsUpdateCard()
+    {
+        $supportsUpdate = $this->gateway->supportsUpdateCard();
+        $this->assertInternalType('boolean', $supportsUpdate);
+
+        if ($supportsUpdate) {
+            $this->assertInstanceOf('Omnipay\Common\Message\RequestInterface', $this->gateway->updateCard());
+        } else {
+            $this->assertFalse(method_exists($this->gateway, 'updateCard'));
         }
     }
 
@@ -290,9 +302,26 @@ public function testVoidParameters()
         }
     }
 
-    public function testStoreParameters()
+    public function testCreateCardParameters()
+    {
+        if ($this->gateway->supportsCreateCard()) {
+            foreach ($this->gateway->getDefaultParameters() as $key => $default) {
+                // set property on gateway
+                $getter = 'get'.ucfirst($key);
+                $setter = 'set'.ucfirst($key);
+                $value = uniqid();
+                $this->gateway->$setter($value);
+
+                // request should have matching property, with correct value
+                $request = $this->gateway->createCard();
+                $this->assertSame($value, $request->$getter());
+            }
+        }
+    }
+
+    public function testDeleteCardParameters()
     {
-        if ($this->gateway->supportsStore()) {
+        if ($this->gateway->supportsDeleteCard()) {
             foreach ($this->gateway->getDefaultParameters() as $key => $default) {
                 // set property on gateway
                 $getter = 'get'.ucfirst($key);
@@ -301,15 +330,15 @@ public function testStoreParameters()
                 $this->gateway->$setter($value);
 
                 // request should have matching property, with correct value
-                $request = $this->gateway->store();
+                $request = $this->gateway->deleteCard();
                 $this->assertSame($value, $request->$getter());
             }
         }
     }
 
-    public function testUnstoreParameters()
+    public function testUpdateCardParameters()
     {
-        if ($this->gateway->supportsUnstore()) {
+        if ($this->gateway->supportsUpdateCard()) {
             foreach ($this->gateway->getDefaultParameters() as $key => $default) {
                 // set property on gateway
                 $getter = 'get'.ucfirst($key);
@@ -318,7 +347,7 @@ public function testUnstoreParameters()
                 $this->gateway->$setter($value);
 
                 // request should have matching property, with correct value
-                $request = $this->gateway->unstore();
+                $request = $this->gateway->updateCard();
                 $this->assertSame($value, $request->$getter());
             }
         }