fixed bug with unconfirmed charges/deposits

Author: rez1dent3

src/Traits/HasWallet.php

@@ -168,8 +168,11 @@ protected function assemble(Wallet $wallet, Transaction $withdraw, Transaction $
      */
     protected function change(int $amount, ?array $meta, bool $confirmed): Transaction
     {
-        $this->getBalanceAttribute();
-        static::$cachedBalances[$this->getKey()] += $amount;
+        if ($confirmed) {
+            $this->getBalanceAttribute();
+            static::$cachedBalances[$this->getKey()] += $amount;
+        }
+
         return $this->transactions()->create([
             'type' => $amount > 0 ? 'deposit' : 'withdraw',
             'payable_type' => $this->getMorphClass(),

tests/ProductTest.php

@@ -95,4 +95,24 @@ public function testForceRefund(): void
         $this->assertEquals($buyer->balance, 0);
     }
 
+    /**
+     * @return void
+     * @expectedException \Bavix\Wallet\Exceptions\ProductEnded
+     */
+    public function testOutOfStock(): void
+    {
+        /**
+         * @var Buyer $buyer
+         * @var Item $product
+         */
+        $buyer = factory(Buyer::class)->create();
+        $product = factory(Item::class)->create([
+            'quantity' => 1,
+        ]);
+
+        $buyer->deposit($product->price);
+        $buyer->pay($product);
+        $buyer->pay($product);
+    }
+
 }

tests/WalletTest.php

@@ -141,4 +141,39 @@ public function testTransferYourself(): void
         $this->assertEquals($user->balance, 0);
     }
 
+    /**
+     * @return void
+     * @expectedException \Bavix\Wallet\Exceptions\BalanceIsEmpty
+     */
+    public function testBalanceIsEmpty(): void
+    {
+        /**
+         * @var User $user
+         */
+        $user = factory(User::class)->create();
+        $this->assertEquals($user->balance, 0);
+        $user->withdraw(1);
+    }
+
+    /**
+     * @return void
+     */
+    public function testConfirmed(): void
+    {
+        /**
+         * @var User $user
+         */
+        $user = factory(User::class)->create();
+        $this->assertEquals($user->balance, 0);
+
+        $user->deposit(1);
+        $this->assertEquals($user->balance, 1);
+
+        $user->withdraw(1, null, false);
+        $this->assertEquals($user->balance, 1);
+
+        $user->withdraw(1);
+        $this->assertEquals($user->balance, 0);
+    }
+
 }