Reviewed all tests for Travis

tests/Feature/Banking/AccountsTest.php

@@ -73,18 +73,17 @@ class AccountsTest extends FeatureTestCase
 
     private function getAccountRequest()
     {
-        return[
-        'company_id' => $this->company->id,
-        'name' => $this->faker->text(5),
-        'number' => '1',
-        'currency_code' => setting('general.default_currency'),
-        'opening_balance' => 0,
-        'bank_name' => $this->faker->text(5),
-        'bank_phone' => null,
-        'bank_address' => null,
-        'default_account' => $this->faker->randomElement(['yes', 'no']),
-        'enabled' => $this->faker->boolean ? 1 : 0,
+        return [
+	        'company_id' => $this->company->id,
+	        'name' => $this->faker->text(5),
+	        'number' => '1',
+	        'currency_code' => setting('general.default_currency'),
+	        'opening_balance' => 0,
+	        'bank_name' => $this->faker->text(5),
+	        'bank_phone' => null,
+	        'bank_address' => null,
+	        'default_account' => $this->faker->randomElement(['yes', 'no']),
+	        'enabled' => $this->faker->boolean ? 1 : 0,
         ];
     }
 }
-

tests/Feature/Banking/ReconciliationsTest.php

@@ -1,6 +1,7 @@
 <?php
 
 namespace Tests\Feature\Banking;
+
 use App\Models\Banking\Reconciliation;
 use Tests\Feature\FeatureTestCase;
 
@@ -85,4 +86,4 @@ class ReconciliationsTest extends FeatureTestCase
             'reconciled' => '1',
         ];
     }
-}
+}

tests/Feature/Banking/TransfersTest.php

@@ -120,7 +120,7 @@ class TransfersTest extends FeatureTestCase
             'payment_method' => 'offlinepayment.cash.1',
             'reference' => null,
             'currency_code' => setting('general.default_currency'),
-            'currency_rate' => '1',
+            'currency_rate' => '1'
         ];
     }
 
@@ -140,7 +140,7 @@ class TransfersTest extends FeatureTestCase
             'category_id' => $this->company->categories()->type('income')->first()->id,
             'reference' => $this->faker->text(5),
             'payment_method' => setting('general.default_payment_method'),
-            'attachment' => $attachment,
+            'attachment' => $attachment
         ];
     }
 
@@ -160,7 +160,7 @@ class TransfersTest extends FeatureTestCase
             'category_id' => $this->company->categories()->type('expense')->first()->id,
             'payment_method' => setting('general.default_payment_method'),
             'reference' => $this->faker->text(5),
-            'attachment' => $attachment,
+            'attachment' => $attachment
         ];
     }
-}
+}

tests/Feature/Commands/BillReminderTest.php

@@ -50,7 +50,6 @@ class BillReminderTest extends FeatureTestCase
 
         $items = [['name' => $this->faker->text(5), 'item_id' => null, 'quantity' => '1', 'price' => $amount, 'currency' => 'USD']];
 
-
         $data = [
             'vendor_id' => '0',
             'billed_at' => $this->faker->date(),
@@ -74,7 +73,6 @@ class BillReminderTest extends FeatureTestCase
             'company_id' => $this->company->id,
         ];
 
-
         return $data;
     }
 }

tests/Feature/Commands/InvoiceReminderTest.php

@@ -50,7 +50,6 @@ class InvoiceReminderTest extends FeatureTestCase
 
         $items = [['name' => $this->faker->text(5), 'item_id' => null, 'quantity' => '1', 'price' => $amount, 'currency' => 'USD']];
 
-
         $data = [
             'customer_id' => '0',
             'invoiced_at' => $this->faker->date(),

tests/Feature/Common/DashboardTest.php

@@ -13,4 +13,4 @@ class DashboardTest extends FeatureTestCase
             ->assertStatus(200)
             ->assertSeeText(trans('general.dashboard'));
     }
-}
+}

tests/Feature/Common/ItemsTest.php

@@ -90,4 +90,4 @@ class ItemsTest extends FeatureTestCase
             'enabled' => $this->faker->boolean ? 1 : 0
         ];
     }
-}
+}

tests/Feature/Expenses/BillsTest.php

@@ -121,4 +121,4 @@ class BillsTest extends FeatureTestCase
 
         return $data;
     }
-}
+}

tests/Feature/Expenses/PaymentsTest.php

@@ -8,7 +8,6 @@ use Tests\Feature\FeatureTestCase;
 
 class PaymentsTest extends FeatureTestCase
 {
-
     public function testItShouldSeePaymentListPage()
     {
         $this->loginAs()
@@ -82,5 +81,4 @@ class PaymentsTest extends FeatureTestCase
             'attachment' => $attachment,
         ];
     }
-
-}
+}

tests/Feature/Expenses/VendorsTest.php

@@ -80,7 +80,6 @@ class VendorsTest extends FeatureTestCase
             ->assertRedirect(route('vendors.index'));
 
         $this->assertFlashLevel('success');
-
     }
 
     private function getVendorRequest()
@@ -97,5 +96,4 @@ class VendorsTest extends FeatureTestCase
             'enabled' => $this->faker->boolean ? 1 : 0
         ];
     }
-
-}
+}

tests/Feature/FeatureTestCase.php

@@ -27,6 +27,11 @@ abstract class FeatureTestCase extends TestCase
 		$this->faker = Factory::create();
 		$this->user = User::first();
 		$this->company = $this->user->first()->companies()->first();
+
+		// Set Company settings
+        setting()->forgetAll();
+        setting()->setExtraColumns(['company_id' => $this->company->id]);
+        setting()->load(true);
 	}
 
 	/**
@@ -44,7 +49,7 @@ abstract class FeatureTestCase extends TestCase
 
 		if (!$company) {
 		    $company = $user->companies()->first();
-        	}
+		}
 
 		$this->startSession();
 

tests/Feature/Incomes/CustomersTest.php

@@ -144,4 +144,4 @@ class CustomersTest extends FeatureTestCase
 				'password_confirmation' => $password
 			];
 	}
-}
+}

tests/Feature/Incomes/InvoicesTest.php

@@ -58,7 +58,6 @@ class InvoicesTest extends FeatureTestCase
             ->assertStatus(302);
 
         $this->assertFlashLevel('success');
-
     }
 
     public function testItShouldDeleteInvoice()
@@ -71,7 +70,6 @@ class InvoicesTest extends FeatureTestCase
             ->assertRedirect(url('incomes/invoices'));
 
         $this->assertFlashLevel('success');
-
     }
 
     private function getInvoiceRequest($recurring = 0)
@@ -112,4 +110,4 @@ class InvoicesTest extends FeatureTestCase
 
         return $data;
     }
-}
+}

tests/Feature/Incomes/RevenuesTest.php

@@ -81,4 +81,4 @@ class RevenuesTest extends FeatureTestCase
             'attachment' => $attachment,
         ];
     }
-}
+}

tests/Feature/Reports/ExpenseSummaryTest.php

@@ -13,4 +13,4 @@ class ExpenseSummaryTest extends FeatureTestCase
             ->assertStatus(200)
             ->assertSeeText(trans('reports.summary.expense'));
     }
-}
+}

tests/Feature/Reports/IncomeExpenseSumaryTest.php

@@ -13,4 +13,4 @@ class IncomeExpenseSummaryTest extends FeatureTestCase
             ->assertStatus(200)
             ->assertSeeText(trans('reports.summary.income_expense'));
     }
-}
+}

tests/Feature/Reports/IncomeSummaryTest.php

@@ -13,4 +13,4 @@ class IncomeSummaryTest extends FeatureTestCase
             ->assertStatus(200)
             ->assertSeeText(trans('reports.summary.income'));
     }
-}
+}

tests/Feature/Reports/ProfitLossTest.php

@@ -13,4 +13,4 @@ class ProfitLossTest extends FeatureTestCase
             ->assertStatus(200)
             ->assertSeeText(trans('reports.profit_loss'));
     }
-}
+}

tests/Feature/Reports/TaxSummaryTest.php

@@ -13,4 +13,4 @@ class TaxSummaryTest extends FeatureTestCase
             ->assertStatus(200)
             ->assertSeeText(trans('reports.summary.tax'));
     }
-}
+}

tests/Feature/Settings/CategoriesTest.php

@@ -81,4 +81,4 @@ class CategoriesTest extends FeatureTestCase
             'enabled' => $this->faker->boolean ? 1 : 0
         ];
     }
-}
+}

tests/Feature/Settings/CurrenciesTest.php

@@ -7,7 +7,6 @@ use Tests\Feature\FeatureTestCase;
 
 class CurrenciesTest extends FeatureTestCase
 {
-
     public function testItShouldSeeCurrencyListPage()
     {
         $this->loginAs()
@@ -62,7 +61,6 @@ class CurrenciesTest extends FeatureTestCase
         $this->assertFlashLevel('success');
     }
 
-
     private function getCurrencyRequest()
     {
         return [
@@ -80,4 +78,4 @@ class CurrenciesTest extends FeatureTestCase
             'default_currency' => $this->faker->boolean ? 1 : 0
         ];
     }
-}
+}

tests/Feature/Settings/TaxesTest.php

@@ -71,4 +71,4 @@ class TaxesTest extends FeatureTestCase
             'enabled' => $this->faker->boolean ? 1 : 0
         ];
     }
-}
+}