upgrade tests

Author: Babichev Maxim

.gitignore

@@ -2,3 +2,4 @@ composer.phar
 /vendor/
 composer.lock
 .idea/
+build/

phpunit.xml

@@ -19,4 +19,8 @@
             <directory suffix=".php">src/</directory>
         </whitelist>
     </filter>
+    <logging>
+        <log type="coverage-html" target="./build/html/"/>
+        <log type="coverage-clover" target="./build/logs/clover.xml"/>
+    </logging>
 </phpunit>

src/WalletServiceProvider.php

@@ -11,6 +11,7 @@ class WalletServiceProvider extends ServiceProvider
      * Bootstrap services.
      *
      * @return void
+     * @codeCoverageIgnore
      */
     public function boot(): void
     {

tests/ProductTest.php

@@ -2,34 +2,55 @@
 
 namespace Bavix\Wallet\Test;
 
-use Bavix\Wallet\Models\Transfer;
 use Bavix\Wallet\Test\Models\Buyer;
 use Bavix\Wallet\Test\Models\Item;
 
 class ProductTest extends TestCase
 {
 
     /**
-     * @return Buyer
+     * @return void
      */
-    public function getBuyer(): Buyer
+    public function testPay(): void
     {
-        return factory(Buyer::class)->create();
+        $buyer = factory(Buyer::class)->create();
+        $product = factory(Item::class)->create([
+            'quantity' => 1,
+        ]);
+
+        $this->assertEquals($buyer->balance, 0);
+        $buyer->deposit($product->price);
+
+        $this->assertEquals($buyer->balance, $product->price);
+        $this->assertNotNull($buyer->pay($product));
+
+        $this->assertEquals($buyer->balance, 0);
+        $this->assertNull($buyer->safePay($product));
     }
 
     /**
      * @return void
      */
-    public function testPay(): void
+    public function testRefund(): void
     {
-        $buyer = $this->getBuyer();
+        $buyer = factory(Buyer::class)->create();
         $product = factory(Item::class)->create([
             'quantity' => 1,
         ]);
+
         $this->assertEquals($buyer->balance, 0);
         $buyer->deposit($product->price);
+
         $this->assertEquals($buyer->balance, $product->price);
-        $this->assertTrue($buyer->pay($product)->exists);
+        $this->assertNotNull($buyer->pay($product));
+
+        $this->assertTrue($buyer->refund($product));
+        $this->assertEquals($buyer->balance, $product->price);
+
+        $this->assertFalse($buyer->safeRefund($product));
+        $this->assertEquals($buyer->balance, $product->price);
+
+        $this->assertNotNull($buyer->pay($product));
         $this->assertEquals($buyer->balance, 0);
     }
 

tests/TestCase.php

@@ -16,8 +16,14 @@ public function setUp(): void
     {
         parent::setUp();
         $this->withFactories(__DIR__ . '/factories');
-        $this->loadMigrationsFrom(__DIR__ . '/migrations');
-        $this->loadMigrationsFrom(\dirname(__DIR__) . '/database/migrations');
+        $this->loadMigrationsFrom([
+            '--database' => 'testbench',
+            '--path' => __DIR__ . '/migrations'
+        ]);
+        $this->loadMigrationsFrom([
+            '--database' => 'testbench',
+            '--path' => \dirname(__DIR__) . '/database/migrations'
+        ]);
     }
 
     /**

tests/WalletTest.php

@@ -7,20 +7,12 @@
 class WalletTest extends TestCase
 {
 
-    /**
-     * @return User
-     */
-    public function getUser(): User
-    {
-        return factory(User::class)->create();
-    }
-
     /**
      * @return void
      */
     public function testDeposit(): void
     {
-        $user = $this->getUser();
+        $user = factory(User::class)->create();
         $this->assertEquals($user->balance, 0);
 
         $user->deposit(10);
@@ -44,7 +36,7 @@ public function testDeposit(): void
      */
     public function testInvalidDeposit(): void
     {
-        $user = $this->getUser();
+        $user = factory(User::class)->create();
         $user->deposit(-1);
     }
 
@@ -54,7 +46,7 @@ public function testInvalidDeposit(): void
      */
     public function testWithdraw(): void
     {
-        $user = $this->getUser();
+        $user = factory(User::class)->create();
         $this->assertEquals($user->balance, 0);
 
         $user->deposit(100);
@@ -78,7 +70,7 @@ public function testWithdraw(): void
      */
     public function testInvalidWithdraw(): void
     {
-        $user = $this->getUser();
+        $user = factory(User::class)->create();
         $user->withdraw(-1);
     }
 
@@ -116,6 +108,18 @@ public function testTransfer(): void
 
         $first->withdraw($first->balance);
         $this->assertEquals($first->balance, 0);
+
+        $this->assertNull($first->safeTransfer($second, 100));
+        $this->assertEquals($first->balance, 0);
+        $this->assertEquals($second->balance, 0);
+
+        $this->assertNotNull($first->forceTransfer($second, 100));
+        $this->assertEquals($first->balance, -100);
+        $this->assertEquals($second->balance, 100);
+
+        $this->assertNotNull($second->forceTransfer($first, 100));
+        $this->assertEquals($first->balance, 0);
+        $this->assertEquals($second->balance, 0);
     }
 
     /**