First tests for transaction controller.

app/controllers/TransactionController.php

@@ -39,6 +39,8 @@ class TransactionController extends BaseController
      *
      * @param TransactionJournal $journal
      *
+     * @codeCoverageIgnore
+     *
      * @return array|\Illuminate\Http\JsonResponse
      */
     public function alreadyRelated(TransactionJournal $journal)
@@ -130,8 +132,10 @@ class TransactionController extends BaseController
     {
         $type   = $transactionJournal->transactionType->type;
         $return = 'withdrawal';
-        $this->_repository->destroy($transactionJournal);
 
+        Session::flash('success', 'Transaction "' . e($transactionJournal->description) . '" destroyed.');
+
+        $this->_repository->destroy($transactionJournal);
 
         switch ($type) {
             case 'Deposit':
@@ -149,6 +153,8 @@ class TransactionController extends BaseController
      * TODO this needs cleaning up and thinking over.
      *
      * @return \Illuminate\Http\JsonResponse
+     * @codeCoverageIgnore
+     *
      */
     public function doRelate()
     {
@@ -259,6 +265,8 @@ class TransactionController extends BaseController
      * @param TransactionJournal $journal
      *
      * @return \Illuminate\View\View
+     * @codeCoverageIgnore
+     *
      */
     public function relate(TransactionJournal $journal)
     {
@@ -282,6 +290,8 @@ class TransactionController extends BaseController
      *
      * @param TransactionJournal $journal
      *
+     * @codeCoverageIgnore
+     *
      * @return \Illuminate\Http\JsonResponse
      */
     public function relatedSearch(TransactionJournal $journal)
@@ -385,6 +395,7 @@ class TransactionController extends BaseController
      *
      * @param TransactionJournal $journal
      *
+     * @codeCoverageIgnore
      * @return \Illuminate\Http\JsonResponse
      * @throws Exception
      */
@@ -420,39 +431,24 @@ class TransactionController extends BaseController
         $data             = Input::except('_token');
         $data['currency'] = 'EUR';
         $data['what']     = strtolower($journal->transactionType->type);
+        $messages         = $this->_repository->validate($data);
 
-        // always validate:
-        $messages = $this->_repository->validate($data);
-
-        // flash messages:
         Session::flash('warnings', $messages['warnings']);
         Session::flash('successes', $messages['successes']);
         Session::flash('errors', $messages['errors']);
         if ($messages['errors']->count() > 0) {
             Session::flash('error', 'Could not update transaction: ' . $messages['errors']->first());
         }
-
-
-        // return to update screen:
         if ($data['post_submit_action'] == 'validate_only' || $messages['errors']->count() > 0) {
             return Redirect::route('transactions.edit', $journal->id)->withInput();
         }
-
-        // update
         $this->_repository->update($journal, $data);
         Session::flash('success', 'Transaction "' . e($data['description']) . '" updated.');
-
         Event::fire('transactionJournal.update', [$journal]); // new and used.
-
-        /*
-         * Also trigger on both transactions.
-         */
         /** @var Transaction $transaction */
         foreach ($journal->transactions()->get() as $transaction) {
             Event::fire('transaction.update', [$transaction]);
         }
-
-        // go back to list
         if ($data['post_submit_action'] == 'update') {
             return Redirect::route('transactions.index', $data['what']);
         }

app/database/seeds/TestContentSeeder.php

@@ -102,13 +102,15 @@ class TestContentSeeder extends Seeder
                 );
 
 
+
+
                 PiggyBankEvent::create(['piggy_bank_id' => 1, 'date' => $startDate->format('Y-m-d'), 'amount' => 100]);
                 PiggyBankRepetition::create(
                     [
                         'piggy_bank_id' => $piggy->id,
                         'startdate'     => Carbon::now()->format('Y-m-d'),
                         'targetdate'    => null,
-                        'currentamount' => 0
+                        'currentamount' => 100
                     ]
                 );
 
@@ -337,10 +339,35 @@ class TestContentSeeder extends Seeder
                 }
 
                 // create some big expenses, move some money around.
-                $this->createTransaction($savings, $checking, 1259, $transfer, 'Money for new PC', $end->format('Y-m') . '-11', $dollar);
-                $this->createTransaction($checking, $store, 1259, $withdrawal, 'New PC', $end->format('Y-m') . '-12', $euro);
+                $one = $this->createTransaction($savings, $checking, 1259, $transfer, 'Money for new PC', $end->format('Y-m') . '-11', $dollar);
+                $two = $this->createTransaction($checking, $store, 1259, $withdrawal, 'New PC', $end->format('Y-m') . '-12', $euro);
 
-                // create two budgets
+                // create a group for these two:
+                $group = TransactionGroup::create(
+                    [
+                        'user_id'  => $user->id,
+                        'relation' => 'balance'
+                    ]
+                );
+                $group->transactionjournals()->save($one);
+                $group->transactionjournals()->save($two);
+
+
+                // piggy bank event
+                // add money to this piggy bank
+                // create a piggy bank event to match:
+                $intoPiggy = $this->createTransaction(
+                    $checking, $savings, 100, $transfer, 'Money for piggy',
+                    Carbon::now()->addDay()->format('Y-m-d'), $euro, $groceriesBudget, $house
+                );
+                $event     = PiggyBankEvent::create(
+                    [
+                        'piggy_bank_id'          => $piggy->id,
+                        'transaction_journal_id' => $intoPiggy->id,
+                        'date'                   => Carbon::now()->addDay()->format('Y-m-d'),
+                        'amount'                 => 100
+                    ]
+                );
 
                 // create two categories
 

app/views/transactions/create.blade.php

@@ -1,7 +1,7 @@
 @extends('layouts.default')
 @section('content')
 {{ Breadcrumbs::renderIfExists(Route::getCurrentRoute()->getName(), $what) }}
-{{Form::open(['class' => 'form-horizontal','url' => route('transactions.store',$what)])}}
+{{Form::open(['class' => 'form-horizontal','id' => 'store','url' => route('transactions.store',$what)])}}
 {{Form::hidden('reminder',Input::get('reminder_id'))}}
 
 <div class="row">

app/views/transactions/delete.blade.php

@@ -1,7 +1,7 @@
 @extends('layouts.default')
 @section('content')
 {{ Breadcrumbs::renderIfExists(Route::getCurrentRoute()->getName(), $journal) }}
-{{Form::open(['class' => 'form-horizontal','url' => route('transactions.destroy',$journal->id)])}}
+{{Form::open(['class' => 'form-horizontal','id' => 'destroy','url' => route('transactions.destroy',$journal->id)])}}
 
 <div class="row">
     <div class="col-lg-6 col-md-6 col-sm-12">

app/views/transactions/edit.blade.php

@@ -1,7 +1,7 @@
 @extends('layouts.default')
 @section('content')
 {{ Breadcrumbs::renderIfExists(Route::getCurrentRoute()->getName(), $journal) }}
-{{Form::open(['class' => 'form-horizontal','url' => route('transactions.update',$journal->id)])}}
+{{Form::open(['class' => 'form-horizontal','id' => 'update','url' => route('transactions.update',$journal->id)])}}
 
 
 <div class="row">