Re escritura de la librería

* Añadidas mas excepciones para granular un poco los errores. * Agregadas validaciones un poco mas estrictas a las entradas de datos. * Actualizadas las pruebas. * Cambiado el esquema general del API, parece un poco más fluente. * Actualizados los Test Suites * Mejorado el Autoloading, ahora es solo PSR-4 * Agregado script en composer.json para phpunit. * Cambios ligeros en el phpunit.xml
abr4xas · Dec 30, 2016 · 15b93c3 · 15b93c3
2 parents 9e6a958 + f937331
commit 15b93c3
Show file tree

Hide file tree

Showing 10 changed files with 43 additions and 22 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -4,6 +4,17 @@
 
 <small>Lamento haber olvidado hacer los tags correspondientes :( </small>
 
+### v3.0.0
+### Nuevo
+* Se re escribe la librería.
+* Añadidas mas excepciones para granular un poco los errores.
+* Agregadas validaciones un poco mas estrictas a las entradas de datos.
+* Actualizadas las pruebas.
+* Cambiado el esquema general del API, parece un poco más fluente.
+* Actualizados los Test Suites
+* Mejorado el Autoloading, ahora es solo PSR-4
+* Agregado script en composer.json para phpunit.
+* Cambios ligeros en el phpunit.xml
 
 ### v2.0.1
 ### Nuevo

diff --git a/README.md b/README.md
@@ -44,21 +44,18 @@ creamos un archivo `index.php`
 
 require 'vendor/autoload.php';
 
-use \Instapago\InstapagoGateway\InstapagoPayment;
-
-$api = new InstapagoPayment('<keyId>','<publicKeyId>');
-```
-
-#### sin composer
-
-```php
-<?php
-
-require_once 'Instapago/autoload.php';
-
-use \Instapago\InstapagoGateway\InstapagoPayment;
-
-$api = new InstapagoPayment('<keyId>','<publicKeyId>');
+use \Instapago\Api;
+
+try{
+  $api = new Api('<keyId>','<publicKeyId>');
+
+  $respuesta = $api->directPayment($paymentData);
+  // hacer algo con $respuesta
+}catch(\Instapago\Exceptions\InstapagoException $e){
+
+  echo "Ocurrió un problema procesando el pago.";
+  // manejar el error 
+}
 ```
 
 Podemos revisar rápidamente si todo funciona correctamente escribiendo:

diff --git a/composer.json b/composer.json
@@ -20,7 +20,7 @@
         "guzzlehttp/guzzle": "^6.2"
     },
     "require-dev": {
-        "phpunit/phpunit": "5.6.1"
+        "phpunit/phpunit": "5.7.5"
     },
     "autoload": {
         "psr-4": {

diff --git a/docs/.gitignore → help/.gitignore b/docs/.gitignore → help/.gitignore
diff --git a/docs/AUTORES.md → help/AUTORES.md b/docs/AUTORES.md → help/AUTORES.md
diff --git a/docs/CONTRIBUCION.md → help/CONTRIBUCION.md b/docs/CONTRIBUCION.md → help/CONTRIBUCION.md
diff --git a/docs/DOCUMENTACION.md → help/DOCUMENTACION.md b/docs/DOCUMENTACION.md → help/DOCUMENTACION.md
@@ -123,10 +123,12 @@ Efectúa un pago directo con tarjeta de crédito, los pagos directos son inmedia
 
 ```php
 try{
-  $api = new Instapago('<keyId>','<publicKeyId>');
+  $api = new Api('<keyId>','<publicKeyId>');
+
   $respuesta = $api->directPayment($paymentData);
   // hacer algo con $respuesta
 }catch(\Instapago\Exceptions\InstapagoException $e){
+
   echo "Ocurrió un problema procesando el pago.";
   // manejar el error 
 }
@@ -138,10 +140,12 @@ Efectúa una reserva o retención de pago en la tarjeta de crédito del cliente,
 
 ```php
 try{
-  $api = new Instapago('<keyId>','<publicKeyId>');
+  $api = new Api('<keyId>','<publicKeyId>');
+
   $respuesta = $api->reservePayment($paymentData);
   // hacer algo con $respuesta
 }catch(\Instapago\Exceptions\InstapagoException $e){
+
   echo "Ocurrió un problema procesando el pago.";
   // manejar el error 
 }
@@ -156,11 +160,13 @@ try{
 
 ```php
 try{
-  $api = new Instapago('<keyId>','<publicKeyId>');
+  $api = new Api('<keyId>','<publicKeyId>');
+
   $respuesta = $api-continuePayment([
     'id' => 'af614bca-0e2b-4232-bc8c-dbedbdf73b48',
     'amount' => '200.00'
   ]);
+
 }catch(\Instapago\Exceptions\InstapagoException $e){
   // manejar errores
 }
@@ -172,10 +178,12 @@ Consulta información sobre un pago previamente generado.
 
 ```php
 try{
-  $api = new Instapago('<keyId>','<publicKeyId>');
+  $api = new Api('<keyId>','<publicKeyId>');
+
   $idPago = 'af614bca-0e2b-4232-bc8c-dbedbdf73b48';
 
   $respuesta = $api->query($idPago);
+
 }catch(\Instapago\Exceptions\InstapagoException $e){
   // manejar errores
 }
@@ -188,10 +196,12 @@ Este método permite cancelar un pago, haya sido directo o retenido.
 
 ```php
 try{
-  $api = new Instapago('<keyId>','<publicKeyId>');
+  $api = new Api('<keyId>','<publicKeyId>');
+
   $idPago = 'af614bca-0e2b-4232-bc8c-dbedbdf73b48';
 
   $info = $api->cancel($idPago);
+
 }catch(\Instapago\Exceptions\InstapagoException $e){
   // manejar errores
 }

diff --git a/docs/hYNsH6B.png → help/hYNsH6B.png b/docs/hYNsH6B.png → help/hYNsH6B.png
diff --git a/phpunit.xml b/phpunit.xml
@@ -1,4 +1,4 @@
-<phpunit bootstrap="vendor/autoload.php">
+<phpunit bootstrap="vendor/autoload.php" colors="true">
   <testsuites>
     <testsuite name="instapago">
       <directory>tests/</directory>

diff --git a/tests/.gitignore b/tests/.gitignore
@@ -0,0 +1,3 @@
+*
+!ApiTest.php
+!.gitignore