From 46ba0a5a5a3e5cabf60c4909577b455de4361168 Mon Sep 17 00:00:00 2001 From: James Cole Date: Mon, 3 Sep 2018 18:52:46 +0200 Subject: [PATCH] Mock more repositories in tests. --- app/Validation/FireflyValidator.php | 2 +- .../V1/Controllers/AboutControllerTest.php | 2 +- .../V1/Controllers/AccountControllerTest.php | 2 +- .../Controllers/AttachmentControllerTest.php | 3 +- .../AvailableBudgetControllerTest.php | 2 +- .../Api/V1/Controllers/BillControllerTest.php | 2 +- .../V1/Controllers/BudgetControllerTest.php | 2 +- .../Controllers/BudgetLimitControllerTest.php | 2 +- .../V1/Controllers/CategoryControllerTest.php | 2 +- .../ConfigurationControllerTest.php | 2 +- .../V1/Controllers/CurrencyControllerTest.php | 2 +- .../CurrencyExchangeRateControllerTest.php | 2 +- .../Controllers/JournalLinkControllerTest.php | 2 +- .../V1/Controllers/LinkTypeControllerTest.php | 2 +- .../Controllers/PiggyBankControllerTest.php | 2 +- .../Controllers/PreferencesControllerTest.php | 2 +- .../Controllers/RecurrenceControllerTest.php | 2 +- .../Api/V1/Controllers/RuleControllerTest.php | 2 +- .../Controllers/RuleGroupControllerTest.php | 2 +- .../Controllers/TransactionControllerTest.php | 2 +- .../Api/V1/Controllers/UserControllerTest.php | 2 +- .../Account/CreateControllerTest.php | 15 ++-- .../Account/DeleteControllerTest.php | 15 ++-- .../Account/EditControllerTest.php | 26 +++---- .../Account/IndexControllerTest.php | 4 +- .../Account/ReconcileControllerTest.php | 59 +++++++++++----- .../Account/ShowControllerTest.php | 27 ++++++-- .../Admin/ConfigurationControllerTest.php | 14 +++- .../Controllers/Admin/HomeControllerTest.php | 13 +++- .../Controllers/Admin/LinkControllerTest.php | 69 +++++++++++++++++-- .../Admin/UpdateControllerTest.php | 32 ++++++++- .../Controllers/Admin/UserControllerTest.php | 2 +- .../Controllers/AttachmentControllerTest.php | 38 ++++++++-- .../Auth/ForgotPasswordControllerTest.php | 2 +- .../Auth/TwoFactorControllerTest.php | 3 +- .../Controllers/BillControllerTest.php | 56 ++++++++++++++- .../Budget/AmountControllerTest.php | 2 +- .../Budget/CreateControllerTest.php | 9 ++- .../Budget/DeleteControllerTest.php | 9 ++- .../Controllers/Budget/EditControllerTest.php | 9 ++- .../Budget/IndexControllerTest.php | 17 ++++- .../Controllers/Budget/ShowControllerTest.php | 47 ++++++++----- .../Category/NoCategoryControllerTest.php | 12 +++- .../Category/ShowControllerTest.php | 2 +- .../Controllers/CategoryControllerTest.php | 2 +- .../Chart/AccountControllerTest.php | 2 +- .../Controllers/Chart/BillControllerTest.php | 2 +- .../Chart/BudgetControllerTest.php | 3 +- .../Chart/BudgetReportControllerTest.php | 2 +- .../Chart/CategoryControllerTest.php | 2 +- .../Chart/CategoryReportControllerTest.php | 2 +- .../Chart/ExpenseReportControllerTest.php | 2 +- .../Chart/PiggyBankControllerTest.php | 2 +- .../Chart/ReportControllerTest.php | 2 +- .../Chart/TagReportControllerTest.php | 2 +- .../Controllers/CurrencyControllerTest.php | 2 +- .../Controllers/DebugControllerTest.php | 2 +- .../Controllers/ExportControllerTest.php | 3 +- .../Controllers/HelpControllerTest.php | 2 +- .../Controllers/HomeControllerTest.php | 2 +- .../Import/CallbackControllerTest.php | 2 +- .../Import/IndexControllerTest.php | 2 +- .../Import/JobConfigurationControllerTest.php | 2 +- .../Import/JobStatusControllerTest.php | 2 +- .../Import/PrerequisitesControllerTest.php | 2 +- .../Controllers/JavascriptControllerTest.php | 2 +- .../Json/AutoCompleteControllerTest.php | 2 +- .../Controllers/Json/BoxControllerTest.php | 2 +- .../Json/ExchangeControllerTest.php | 2 +- .../Json/FrontpageControllerTest.php | 2 +- .../Controllers/Json/IntroControllerTest.php | 2 +- .../Json/ReconcileControllerTest.php | 3 +- .../Json/RecurrenceControllerTest.php | 2 +- .../Controllers/JsonControllerTest.php | 2 +- .../Controllers/NewUserControllerTest.php | 2 +- .../Controllers/PiggyBankControllerTest.php | 2 +- .../Popup/ReportControllerTest.php | 4 +- .../Controllers/PreferencesControllerTest.php | 2 +- .../Controllers/ProfileControllerTest.php | 4 +- .../Recurring/CreateControllerTest.php | 2 +- .../Recurring/DeleteControllerTest.php | 2 +- .../Recurring/EditControllerTest.php | 2 +- .../Recurring/IndexControllerTest.php | 2 +- .../Report/AccountControllerTest.php | 2 +- .../Report/BalanceControllerTest.php | 2 +- .../Report/BudgetControllerTest.php | 2 +- .../Report/CategoryControllerTest.php | 2 +- .../Report/ExpenseControllerTest.php | 2 +- .../Report/OperationsControllerTest.php | 2 +- .../Controllers/ReportControllerTest.php | 2 +- .../Controllers/Rule/CreateControllerTest.php | 2 +- .../Controllers/Rule/DeleteControllerTest.php | 2 +- .../Controllers/Rule/EditControllerTest.php | 2 +- .../Controllers/Rule/IndexControllerTest.php | 2 +- .../Controllers/Rule/SelectControllerTest.php | 2 +- .../Controllers/RuleGroupControllerTest.php | 2 +- .../Controllers/SearchControllerTest.php | 2 +- .../Controllers/System/CronControllerTest.php | 2 +- .../Feature/Controllers/TagControllerTest.php | 2 +- .../Transaction/BulkControllerTest.php | 2 +- .../Transaction/ConvertControllerTest.php | 2 +- .../Transaction/LinkControllerTest.php | 2 +- .../Transaction/MassControllerTest.php | 2 +- .../Transaction/SingleControllerTest.php | 2 +- .../Transaction/SplitControllerTest.php | 2 +- .../Controllers/TransactionControllerTest.php | 2 +- tests/Unit/Factory/AccountFactoryTest.php | 2 +- tests/Unit/Factory/AccountMetaFactoryTest.php | 2 +- tests/Unit/Factory/AttachmentFactoryTest.php | 2 +- tests/Unit/Factory/BillFactoryTest.php | 2 +- tests/Unit/Factory/BudgetFactoryTest.php | 2 +- tests/Unit/Factory/CategoryFactoryTest.php | 2 +- .../Factory/PiggyBankEventFactoryTest.php | 2 +- tests/Unit/Factory/PiggyBankFactoryTest.php | 2 +- tests/Unit/Factory/RecurrenceFactoryTest.php | 2 +- tests/Unit/Factory/TagFactoryTest.php | 2 +- .../TransactionCurrencyFactoryTest.php | 2 +- tests/Unit/Factory/TransactionFactoryTest.php | 2 +- .../Factory/TransactionJournalFactoryTest.php | 2 +- .../TransactionJournalMetaFactoryTest.php | 2 +- .../Factory/TransactionTypeFactoryTest.php | 2 +- .../Chart/Basic/ChartJsGeneratorTest.php | 2 +- .../Report/Audit/MonthReportGeneratorTest.php | 2 +- .../Handlers/Events/APIEventHandlerTest.php | 2 +- .../Handlers/Events/AdminEventHandlerTest.php | 2 +- .../Handlers/Events/AutomationHandlerTest.php | 2 +- .../Events/StoredJournalEventHandlerTest.php | 2 +- .../Events/UpdatedJournalEventHandlerTest.php | 2 +- .../Handlers/Events/UserEventHandlerTest.php | 2 +- .../Events/VersionCheckEventHandlerTest.php | 2 +- .../Attachments/AttachmentHelperTest.php | 2 +- tests/Unit/Helpers/Chart/MetaPieChartTest.php | 2 +- .../Unit/Helpers/Filter/AmountFilterTest.php | 2 +- tests/Unit/Helpers/FiscalHelperTest.php | 2 +- tests/Unit/Helpers/Help/HelpTest.php | 2 +- tests/Unit/Helpers/Report/NetWorthTest.php | 2 +- .../Triggers/FromAccountStartsTest.php | 2 +- 137 files changed, 496 insertions(+), 222 deletions(-) diff --git a/app/Validation/FireflyValidator.php b/app/Validation/FireflyValidator.php index c16c7430ba..a8d1bbc87c 100644 --- a/app/Validation/FireflyValidator.php +++ b/app/Validation/FireflyValidator.php @@ -525,7 +525,7 @@ class FireflyValidator extends Validator try { $value = Crypt::decrypt($value); } catch (DecryptException $e) { - Log::debug(sprintf('Could not decrypt. %s', $e->getMessage())); + //Log::debug(sprintf('Could not decrypt. %s', $e->getMessage())); } return $value; diff --git a/tests/Api/V1/Controllers/AboutControllerTest.php b/tests/Api/V1/Controllers/AboutControllerTest.php index 2622b2d56c..a36c5dbaa8 100644 --- a/tests/Api/V1/Controllers/AboutControllerTest.php +++ b/tests/Api/V1/Controllers/AboutControllerTest.php @@ -39,7 +39,7 @@ class AboutControllerTest extends TestCase { parent::setUp(); Passport::actingAs($this->user()); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } /** diff --git a/tests/Api/V1/Controllers/AccountControllerTest.php b/tests/Api/V1/Controllers/AccountControllerTest.php index d6cdbfe354..da9d6157c9 100644 --- a/tests/Api/V1/Controllers/AccountControllerTest.php +++ b/tests/Api/V1/Controllers/AccountControllerTest.php @@ -44,7 +44,7 @@ class AccountControllerTest extends TestCase { parent::setUp(); Passport::actingAs($this->user()); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } /** diff --git a/tests/Api/V1/Controllers/AttachmentControllerTest.php b/tests/Api/V1/Controllers/AttachmentControllerTest.php index 1a06c7db01..09306de5f2 100644 --- a/tests/Api/V1/Controllers/AttachmentControllerTest.php +++ b/tests/Api/V1/Controllers/AttachmentControllerTest.php @@ -45,7 +45,7 @@ class AttachmentControllerTest extends TestCase { parent::setUp(); Passport::actingAs($this->user()); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } /** @@ -99,7 +99,6 @@ class AttachmentControllerTest extends TestCase * Download attachment but file doesn't exist. * * @covers \FireflyIII\Api\V1\Controllers\AttachmentController - * @expectedExceptionMessage Some error message */ public function testDownloadNotExisting(): void { diff --git a/tests/Api/V1/Controllers/AvailableBudgetControllerTest.php b/tests/Api/V1/Controllers/AvailableBudgetControllerTest.php index a37c12f2b7..cbac78120a 100644 --- a/tests/Api/V1/Controllers/AvailableBudgetControllerTest.php +++ b/tests/Api/V1/Controllers/AvailableBudgetControllerTest.php @@ -45,7 +45,7 @@ class AvailableBudgetControllerTest extends TestCase { parent::setUp(); Passport::actingAs($this->user()); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } /** diff --git a/tests/Api/V1/Controllers/BillControllerTest.php b/tests/Api/V1/Controllers/BillControllerTest.php index 01da6566f1..d7cde583c6 100644 --- a/tests/Api/V1/Controllers/BillControllerTest.php +++ b/tests/Api/V1/Controllers/BillControllerTest.php @@ -44,7 +44,7 @@ class BillControllerTest extends TestCase { parent::setUp(); Passport::actingAs($this->user()); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } diff --git a/tests/Api/V1/Controllers/BudgetControllerTest.php b/tests/Api/V1/Controllers/BudgetControllerTest.php index 5c22cda9a1..1c00519a84 100644 --- a/tests/Api/V1/Controllers/BudgetControllerTest.php +++ b/tests/Api/V1/Controllers/BudgetControllerTest.php @@ -43,7 +43,7 @@ class BudgetControllerTest extends TestCase { parent::setUp(); Passport::actingAs($this->user()); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } /** diff --git a/tests/Api/V1/Controllers/BudgetLimitControllerTest.php b/tests/Api/V1/Controllers/BudgetLimitControllerTest.php index 3febef2744..4f20362d67 100644 --- a/tests/Api/V1/Controllers/BudgetLimitControllerTest.php +++ b/tests/Api/V1/Controllers/BudgetLimitControllerTest.php @@ -44,7 +44,7 @@ class BudgetLimitControllerTest extends TestCase { parent::setUp(); Passport::actingAs($this->user()); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } /** diff --git a/tests/Api/V1/Controllers/CategoryControllerTest.php b/tests/Api/V1/Controllers/CategoryControllerTest.php index 5e4161742b..0dfae77a9b 100644 --- a/tests/Api/V1/Controllers/CategoryControllerTest.php +++ b/tests/Api/V1/Controllers/CategoryControllerTest.php @@ -43,7 +43,7 @@ class CategoryControllerTest extends TestCase { parent::setUp(); Passport::actingAs($this->user()); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } /** diff --git a/tests/Api/V1/Controllers/ConfigurationControllerTest.php b/tests/Api/V1/Controllers/ConfigurationControllerTest.php index ba3c4add09..c5297be1ae 100644 --- a/tests/Api/V1/Controllers/ConfigurationControllerTest.php +++ b/tests/Api/V1/Controllers/ConfigurationControllerTest.php @@ -42,7 +42,7 @@ class ConfigurationControllerTest extends TestCase { parent::setUp(); Passport::actingAs($this->user()); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } /** diff --git a/tests/Api/V1/Controllers/CurrencyControllerTest.php b/tests/Api/V1/Controllers/CurrencyControllerTest.php index 347d7c92be..50e247d324 100644 --- a/tests/Api/V1/Controllers/CurrencyControllerTest.php +++ b/tests/Api/V1/Controllers/CurrencyControllerTest.php @@ -46,7 +46,7 @@ class CurrencyControllerTest extends TestCase { parent::setUp(); Passport::actingAs($this->user()); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } diff --git a/tests/Api/V1/Controllers/CurrencyExchangeRateControllerTest.php b/tests/Api/V1/Controllers/CurrencyExchangeRateControllerTest.php index 09da194a12..2020a058ea 100644 --- a/tests/Api/V1/Controllers/CurrencyExchangeRateControllerTest.php +++ b/tests/Api/V1/Controllers/CurrencyExchangeRateControllerTest.php @@ -46,7 +46,7 @@ class CurrencyExchangeRateControllerTest extends TestCase { parent::setUp(); Passport::actingAs($this->user()); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } diff --git a/tests/Api/V1/Controllers/JournalLinkControllerTest.php b/tests/Api/V1/Controllers/JournalLinkControllerTest.php index 770f29d091..79885131a1 100644 --- a/tests/Api/V1/Controllers/JournalLinkControllerTest.php +++ b/tests/Api/V1/Controllers/JournalLinkControllerTest.php @@ -48,7 +48,7 @@ class JournalLinkControllerTest extends TestCase { parent::setUp(); Passport::actingAs($this->user()); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } diff --git a/tests/Api/V1/Controllers/LinkTypeControllerTest.php b/tests/Api/V1/Controllers/LinkTypeControllerTest.php index 003ae40c98..f138c0c9fc 100644 --- a/tests/Api/V1/Controllers/LinkTypeControllerTest.php +++ b/tests/Api/V1/Controllers/LinkTypeControllerTest.php @@ -44,7 +44,7 @@ class LinkTypeControllerTest extends TestCase { parent::setUp(); Passport::actingAs($this->user()); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } diff --git a/tests/Api/V1/Controllers/PiggyBankControllerTest.php b/tests/Api/V1/Controllers/PiggyBankControllerTest.php index 7c24398e80..74931d6000 100644 --- a/tests/Api/V1/Controllers/PiggyBankControllerTest.php +++ b/tests/Api/V1/Controllers/PiggyBankControllerTest.php @@ -46,7 +46,7 @@ class PiggyBankControllerTest extends TestCase { parent::setUp(); Passport::actingAs($this->user()); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } /** diff --git a/tests/Api/V1/Controllers/PreferencesControllerTest.php b/tests/Api/V1/Controllers/PreferencesControllerTest.php index f2fdf4e3d1..4ab718634e 100644 --- a/tests/Api/V1/Controllers/PreferencesControllerTest.php +++ b/tests/Api/V1/Controllers/PreferencesControllerTest.php @@ -44,7 +44,7 @@ class PreferencesControllerTest extends TestCase { parent::setUp(); Passport::actingAs($this->user()); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } /** diff --git a/tests/Api/V1/Controllers/RecurrenceControllerTest.php b/tests/Api/V1/Controllers/RecurrenceControllerTest.php index 59100edca7..9f579c4ace 100644 --- a/tests/Api/V1/Controllers/RecurrenceControllerTest.php +++ b/tests/Api/V1/Controllers/RecurrenceControllerTest.php @@ -44,7 +44,7 @@ class RecurrenceControllerTest extends TestCase { parent::setUp(); Passport::actingAs($this->user()); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } diff --git a/tests/Api/V1/Controllers/RuleControllerTest.php b/tests/Api/V1/Controllers/RuleControllerTest.php index 2f5dc9adee..dfe8356d66 100644 --- a/tests/Api/V1/Controllers/RuleControllerTest.php +++ b/tests/Api/V1/Controllers/RuleControllerTest.php @@ -43,7 +43,7 @@ class RuleControllerTest extends TestCase { parent::setUp(); Passport::actingAs($this->user()); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } /** diff --git a/tests/Api/V1/Controllers/RuleGroupControllerTest.php b/tests/Api/V1/Controllers/RuleGroupControllerTest.php index c21b7785ab..51aefc9165 100644 --- a/tests/Api/V1/Controllers/RuleGroupControllerTest.php +++ b/tests/Api/V1/Controllers/RuleGroupControllerTest.php @@ -43,7 +43,7 @@ class RuleGroupControllerTest extends TestCase { parent::setUp(); Passport::actingAs($this->user()); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } /** diff --git a/tests/Api/V1/Controllers/TransactionControllerTest.php b/tests/Api/V1/Controllers/TransactionControllerTest.php index f4480aa2d7..aa66cc836e 100644 --- a/tests/Api/V1/Controllers/TransactionControllerTest.php +++ b/tests/Api/V1/Controllers/TransactionControllerTest.php @@ -48,7 +48,7 @@ class TransactionControllerTest extends TestCase { parent::setUp(); Passport::actingAs($this->user()); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } /** diff --git a/tests/Api/V1/Controllers/UserControllerTest.php b/tests/Api/V1/Controllers/UserControllerTest.php index 489c205554..1e61472887 100644 --- a/tests/Api/V1/Controllers/UserControllerTest.php +++ b/tests/Api/V1/Controllers/UserControllerTest.php @@ -44,7 +44,7 @@ class UserControllerTest extends TestCase { parent::setUp(); Passport::actingAs($this->user()); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } diff --git a/tests/Feature/Controllers/Account/CreateControllerTest.php b/tests/Feature/Controllers/Account/CreateControllerTest.php index 37576b188b..674b156821 100644 --- a/tests/Feature/Controllers/Account/CreateControllerTest.php +++ b/tests/Feature/Controllers/Account/CreateControllerTest.php @@ -33,9 +33,10 @@ use FireflyIII\Repositories\Journal\JournalRepositoryInterface; use FireflyIII\Repositories\User\UserRepositoryInterface; use Illuminate\Support\Collection; use Log; +use Mockery; use Preferences; use Tests\TestCase; -use Mockery; + /** * * Class CreateControllerTest @@ -48,7 +49,7 @@ class CreateControllerTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } @@ -66,7 +67,7 @@ class CreateControllerTest extends TestCase $journalRepos->shouldReceive('firstNull')->once()->andReturn(new TransactionJournal); // mock hasRole for user repository: - $userRepos->shouldReceive('hasRole')->withArgs([Mockery::any(),'owner'])->andReturn(true)->atLeast()->once(); + $userRepos->shouldReceive('hasRole')->withArgs([Mockery::any(), 'owner'])->andReturn(true)->atLeast()->once(); // get all types: $accountRepos->shouldReceive('getAccountTypeByType')->withArgs(['Debt'])->andReturn(AccountType::find(11))->once(); @@ -90,8 +91,8 @@ class CreateControllerTest extends TestCase public function testStore(): void { // mock stuff - $journalRepos = $this->mock(JournalRepositoryInterface::class); - $repository = $this->mock(AccountRepositoryInterface::class); + $journalRepos = $this->mock(JournalRepositoryInterface::class); + $repository = $this->mock(AccountRepositoryInterface::class); $repository->shouldReceive('store')->once()->andReturn(factory(Account::class)->make()); $journalRepos->shouldReceive('firstNull')->once()->andReturn(new TransactionJournal); @@ -119,8 +120,8 @@ class CreateControllerTest extends TestCase public function testStoreAnother(): void { // mock stuff - $journalRepos = $this->mock(JournalRepositoryInterface::class); - $repository = $this->mock(AccountRepositoryInterface::class); + $journalRepos = $this->mock(JournalRepositoryInterface::class); + $repository = $this->mock(AccountRepositoryInterface::class); $repository->shouldReceive('store')->once()->andReturn(factory(Account::class)->make()); $journalRepos->shouldReceive('firstNull')->once()->andReturn(new TransactionJournal); diff --git a/tests/Feature/Controllers/Account/DeleteControllerTest.php b/tests/Feature/Controllers/Account/DeleteControllerTest.php index 01a8d1c2fb..7ed0e72ec8 100644 --- a/tests/Feature/Controllers/Account/DeleteControllerTest.php +++ b/tests/Feature/Controllers/Account/DeleteControllerTest.php @@ -27,7 +27,6 @@ namespace Tests\Feature\Controllers\Account; use FireflyIII\Models\AccountType; use FireflyIII\Models\TransactionJournal; use FireflyIII\Repositories\Account\AccountRepositoryInterface; -use FireflyIII\Repositories\Currency\CurrencyRepositoryInterface; use FireflyIII\Repositories\Journal\JournalRepositoryInterface; use FireflyIII\Repositories\User\UserRepositoryInterface; use Illuminate\Support\Collection; @@ -47,7 +46,7 @@ class DeleteControllerTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } /** @@ -57,14 +56,14 @@ class DeleteControllerTest extends TestCase public function testDelete(): void { // mock stuff - $journalRepos = $this->mock(JournalRepositoryInterface::class); - $repository = $this->mock(AccountRepositoryInterface::class); - $userRepos = $this->mock(UserRepositoryInterface::class); + $journalRepos = $this->mock(JournalRepositoryInterface::class); + $repository = $this->mock(AccountRepositoryInterface::class); + $userRepos = $this->mock(UserRepositoryInterface::class); $repository->shouldReceive('getAccountsByType')->withArgs([[AccountType::ASSET]])->andReturn(new Collection); $journalRepos->shouldReceive('firstNull')->once()->andReturn(new TransactionJournal); // mock hasRole for user repository: - $userRepos->shouldReceive('hasRole')->withArgs([Mockery::any(),'owner'])->andReturn(true)->atLeast()->once(); + $userRepos->shouldReceive('hasRole')->withArgs([Mockery::any(), 'owner'])->andReturn(true)->atLeast()->once(); $this->be($this->user()); $account = $this->user()->accounts()->where('account_type_id', 3)->whereNull('deleted_at')->first(); @@ -81,8 +80,8 @@ class DeleteControllerTest extends TestCase public function testDestroy(): void { // mock stuff - $journalRepos = $this->mock(JournalRepositoryInterface::class); - $repository = $this->mock(AccountRepositoryInterface::class); + $journalRepos = $this->mock(JournalRepositoryInterface::class); + $repository = $this->mock(AccountRepositoryInterface::class); $repository->shouldReceive('findNull')->withArgs([0])->once()->andReturn(null); $repository->shouldReceive('destroy')->andReturn(true); $journalRepos->shouldReceive('firstNull')->once()->andReturn(new TransactionJournal); diff --git a/tests/Feature/Controllers/Account/EditControllerTest.php b/tests/Feature/Controllers/Account/EditControllerTest.php index 1d2a0b9f90..44cf046a7b 100644 --- a/tests/Feature/Controllers/Account/EditControllerTest.php +++ b/tests/Feature/Controllers/Account/EditControllerTest.php @@ -49,7 +49,7 @@ class EditControllerTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } /** @@ -57,10 +57,10 @@ class EditControllerTest extends TestCase */ public function testEdit(): void { - $journalRepos = $this->mock(JournalRepositoryInterface::class); - $repository = $this->mock(CurrencyRepositoryInterface::class); - $accountRepos = $this->mock(AccountRepositoryInterface::class); - $userRepos = $this->mock(UserRepositoryInterface::class); + $journalRepos = $this->mock(JournalRepositoryInterface::class); + $repository = $this->mock(CurrencyRepositoryInterface::class); + $accountRepos = $this->mock(AccountRepositoryInterface::class); + $userRepos = $this->mock(UserRepositoryInterface::class); $repository->shouldReceive('findNull')->withArgs([1])->andReturn(TransactionCurrency::find(1))->atLeast()->once(); @@ -103,10 +103,10 @@ class EditControllerTest extends TestCase */ public function testEditLiability(): void { - $journalRepos = $this->mock(JournalRepositoryInterface::class); - $repository = $this->mock(CurrencyRepositoryInterface::class); - $accountRepos = $this->mock(AccountRepositoryInterface::class); - $userRepos = $this->mock(UserRepositoryInterface::class); + $journalRepos = $this->mock(JournalRepositoryInterface::class); + $repository = $this->mock(CurrencyRepositoryInterface::class); + $accountRepos = $this->mock(AccountRepositoryInterface::class); + $userRepos = $this->mock(UserRepositoryInterface::class); // mock hasRole for user repository: $userRepos->shouldReceive('hasRole')->withArgs([Mockery::any(), 'owner'])->andReturn(true)->atLeast()->once(); @@ -149,10 +149,10 @@ class EditControllerTest extends TestCase public function testEditNull(): void { // mock stuff - $journalRepos = $this->mock(JournalRepositoryInterface::class); - $repository = $this->mock(CurrencyRepositoryInterface::class); - $accountRepos = $this->mock(AccountRepositoryInterface::class); - $userRepos = $this->mock(UserRepositoryInterface::class); + $journalRepos = $this->mock(JournalRepositoryInterface::class); + $repository = $this->mock(CurrencyRepositoryInterface::class); + $accountRepos = $this->mock(AccountRepositoryInterface::class); + $userRepos = $this->mock(UserRepositoryInterface::class); // mock hasRole for user repository: $userRepos->shouldReceive('hasRole')->withArgs([Mockery::any(), 'owner'])->andReturn(true)->atLeast()->once(); diff --git a/tests/Feature/Controllers/Account/IndexControllerTest.php b/tests/Feature/Controllers/Account/IndexControllerTest.php index c8000218e9..f33c09ccc5 100644 --- a/tests/Feature/Controllers/Account/IndexControllerTest.php +++ b/tests/Feature/Controllers/Account/IndexControllerTest.php @@ -50,7 +50,7 @@ class IndexControllerTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } @@ -71,7 +71,7 @@ class IndexControllerTest extends TestCase $userRepos = $this->mock(UserRepositoryInterface::class); // mock hasRole for user repository: - $userRepos->shouldReceive('hasRole')->withArgs([Mockery::any(),'owner'])->andReturn(true)->atLeast()->once(); + $userRepos->shouldReceive('hasRole')->withArgs([Mockery::any(), 'owner'])->andReturn(true)->atLeast()->once(); $repository->shouldReceive('getAccountsByType')->andReturn(new Collection([$account])); $repository->shouldReceive('getMetaValue')->withArgs([Mockery::any(), 'currency_id'])->andReturn('1'); diff --git a/tests/Feature/Controllers/Account/ReconcileControllerTest.php b/tests/Feature/Controllers/Account/ReconcileControllerTest.php index cb7569c612..97a6c1814b 100644 --- a/tests/Feature/Controllers/Account/ReconcileControllerTest.php +++ b/tests/Feature/Controllers/Account/ReconcileControllerTest.php @@ -47,7 +47,7 @@ class ReconcileControllerTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } /** @@ -106,9 +106,12 @@ class ReconcileControllerTest extends TestCase */ public function testReconcile(): void { - $userRepos = $this->mock(UserRepositoryInterface::class); - $repository = $this->mock(CurrencyRepositoryInterface::class); - $accountRepos = $this->mock(AccountRepositoryInterface::class); + $userRepos = $this->mock(UserRepositoryInterface::class); + $repository = $this->mock(CurrencyRepositoryInterface::class); + $accountRepos = $this->mock(AccountRepositoryInterface::class); + + $accountRepos->shouldReceive('getMetaValue') + ->withArgs([Mockery::any(), 'currency_id'])->andReturn('1')->atLeast()->once(); // mock hasRole for user repository: $userRepos->shouldReceive('hasRole')->withArgs([Mockery::any(), 'owner'])->andReturn(true)->atLeast()->once(); @@ -129,6 +132,10 @@ class ReconcileControllerTest extends TestCase */ public function testReconcileInitialBalance(): void { + $userRepos = $this->mock(UserRepositoryInterface::class); + $repository = $this->mock(CurrencyRepositoryInterface::class); + $accountRepos = $this->mock(AccountRepositoryInterface::class); + $transaction = Transaction::leftJoin('accounts', 'accounts.id', '=', 'transactions.account_id') ->where('accounts.user_id', $this->user()->id)->where('accounts.account_type_id', 6)->first(['account_id']); $this->be($this->user()); @@ -143,9 +150,12 @@ class ReconcileControllerTest extends TestCase */ public function testReconcileNoDates(): void { - $userRepos = $this->mock(UserRepositoryInterface::class); - $repository = $this->mock(CurrencyRepositoryInterface::class); - $accountRepos = $this->mock(AccountRepositoryInterface::class); + $userRepos = $this->mock(UserRepositoryInterface::class); + $repository = $this->mock(CurrencyRepositoryInterface::class); + $accountRepos = $this->mock(AccountRepositoryInterface::class); + + $accountRepos->shouldReceive('getMetaValue') + ->withArgs([Mockery::any(), 'currency_id'])->andReturn('1')->atLeast()->once(); // mock hasRole for user repository: $userRepos->shouldReceive('hasRole')->withArgs([Mockery::any(), 'owner'])->andReturn(true)->atLeast()->once(); @@ -167,9 +177,12 @@ class ReconcileControllerTest extends TestCase */ public function testReconcileNoEndDate(): void { - $userRepos = $this->mock(UserRepositoryInterface::class); - $repository = $this->mock(CurrencyRepositoryInterface::class); - $accountRepos = $this->mock(AccountRepositoryInterface::class); + $userRepos = $this->mock(UserRepositoryInterface::class); + $repository = $this->mock(CurrencyRepositoryInterface::class); + $accountRepos = $this->mock(AccountRepositoryInterface::class); + + $accountRepos->shouldReceive('getMetaValue') + ->withArgs([Mockery::any(), 'currency_id'])->andReturn('1')->atLeast()->once(); // mock hasRole for user repository: $userRepos->shouldReceive('hasRole')->withArgs([Mockery::any(), 'owner'])->andReturn(true)->atLeast()->once(); @@ -207,15 +220,19 @@ class ReconcileControllerTest extends TestCase */ public function testShow(): void { - $userRepos = $this->mock(UserRepositoryInterface::class); - $repository = $this->mock(JournalRepositoryInterface::class); + $userRepos = $this->mock(UserRepositoryInterface::class); + $repository = $this->mock(JournalRepositoryInterface::class); $accountRepos = $this->mock(AccountRepositoryInterface::class); $currencyRepos = $this->mock(CurrencyRepositoryInterface::class); + $accountRepos->shouldReceive('getMetaValue') + ->withArgs([Mockery::any(), 'currency_id'])->andReturn('1')->atLeast()->once(); + $currencyRepos->shouldReceive('findNull')->atLeast()->once()->withArgs([1])->andReturn(TransactionCurrency::find(1)); + // mock hasRole for user repository: $userRepos->shouldReceive('hasRole')->withArgs([Mockery::any(), 'owner'])->andReturn(true)->atLeast()->once(); - $journal = $this->user()->transactionJournals()->where('transaction_type_id', 5)->first(); + $journal = $this->user()->transactionJournals()->where('transaction_type_id', 5)->first(); $repository->shouldReceive('firstNull')->andReturn(new TransactionJournal); $repository->shouldReceive('getAssetTransaction')->once()->andReturn($journal->transactions()->first()); @@ -238,9 +255,9 @@ class ReconcileControllerTest extends TestCase { $accountRepos = $this->mock(AccountRepositoryInterface::class); $currencyRepos = $this->mock(CurrencyRepositoryInterface::class); - $repository = $this->mock(JournalRepositoryInterface::class); + $repository = $this->mock(JournalRepositoryInterface::class); - $journal = $this->user()->transactionJournals()->where('transaction_type_id', 5)->first(); + $journal = $this->user()->transactionJournals()->where('transaction_type_id', 5)->first(); $repository->shouldReceive('firstNull')->andReturn(new TransactionJournal); $repository->shouldReceive('getAssetTransaction')->once()->andReturnNull(); @@ -279,8 +296,8 @@ class ReconcileControllerTest extends TestCase */ public function testSubmit(): void { - $repository = $this->mock(AccountRepositoryInterface::class); - $journalRepos = $this->mock(JournalRepositoryInterface::class); + $repository = $this->mock(AccountRepositoryInterface::class); + $journalRepos = $this->mock(JournalRepositoryInterface::class); $currencyRepos = $this->mock(CurrencyRepositoryInterface::class); $journalRepos->shouldReceive('firstNull')->andReturn(new TransactionJournal); @@ -310,7 +327,7 @@ class ReconcileControllerTest extends TestCase */ public function testUpdate(): void { - $journalRepos = $this->mock(JournalRepositoryInterface::class); + $journalRepos = $this->mock(JournalRepositoryInterface::class); $accountRepos = $this->mock(AccountRepositoryInterface::class); $currencyRepos = $this->mock(CurrencyRepositoryInterface::class); @@ -336,6 +353,9 @@ class ReconcileControllerTest extends TestCase */ public function testUpdateNotReconcile(): void { + $accountRepos = $this->mock(AccountRepositoryInterface::class); + $currencyRepos = $this->mock(CurrencyRepositoryInterface::class); + $journal = $this->user()->transactionJournals()->where('transaction_type_id', '!=', 5)->first(); $data = ['amount' => '5',]; @@ -351,6 +371,9 @@ class ReconcileControllerTest extends TestCase */ public function testUpdateZero(): void { + $accountRepos = $this->mock(AccountRepositoryInterface::class); + $currencyRepos = $this->mock(CurrencyRepositoryInterface::class); + $journal = $this->user()->transactionJournals()->where('transaction_type_id', 5)->first(); $data = ['amount' => '0',]; diff --git a/tests/Feature/Controllers/Account/ShowControllerTest.php b/tests/Feature/Controllers/Account/ShowControllerTest.php index 0e2eb30b7e..caef8a9ed6 100644 --- a/tests/Feature/Controllers/Account/ShowControllerTest.php +++ b/tests/Feature/Controllers/Account/ShowControllerTest.php @@ -51,7 +51,7 @@ class ShowControllerTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } @@ -67,13 +67,15 @@ class ShowControllerTest extends TestCase $this->session(['start' => $date, 'end' => clone $date]); // mock stuff: - $tasker = $this->mock(AccountTaskerInterface::class); + $journalRepos = $this->mock(JournalRepositoryInterface::class); - $currencyRepos = $this->mock(CurrencyRepositoryInterface::class); + $tasker = $this->mock(AccountTaskerInterface::class); $userRepos = $this->mock(UserRepositoryInterface::class); + $currencyRepos = $this->mock(CurrencyRepositoryInterface::class); + $accountRepos = $this->mock(AccountRepositoryInterface::class); // mock hasRole for user repository: - $userRepos->shouldReceive('hasRole')->withArgs([Mockery::any(),'owner'])->andReturn(true)->atLeast()->once(); + $userRepos->shouldReceive('hasRole')->withArgs([Mockery::any(), 'owner'])->andReturn(true)->atLeast()->once(); $currencyRepos->shouldReceive('findNull')->andReturn(TransactionCurrency::find(1)); @@ -121,10 +123,11 @@ class ShowControllerTest extends TestCase $tasker = $this->mock(AccountTaskerInterface::class); $journalRepos = $this->mock(JournalRepositoryInterface::class); $currencyRepos = $this->mock(CurrencyRepositoryInterface::class); + $accountRepos = $this->mock(AccountRepositoryInterface::class); $userRepos = $this->mock(UserRepositoryInterface::class); // mock hasRole for user repository: - $userRepos->shouldReceive('hasRole')->withArgs([Mockery::any(),'owner'])->andReturn(true)->atLeast()->once(); + $userRepos->shouldReceive('hasRole')->withArgs([Mockery::any(), 'owner'])->andReturn(true)->atLeast()->once(); $currencyRepos->shouldReceive('findNull')->andReturn(TransactionCurrency::find(1)); @@ -159,12 +162,16 @@ class ShowControllerTest extends TestCase /** * @covers \FireflyIII\Http\Controllers\Account\ShowController - * @expectedExceptionMessage End is after start! */ public function testShowBrokenBadDates(): void { // mock $journalRepos = $this->mock(JournalRepositoryInterface::class); + $currencyRepos = $this->mock(CurrencyRepositoryInterface::class); + $accountRepos = $this->mock(AccountRepositoryInterface::class); + + $accountRepos->shouldReceive('isLiability')->atLeast()->once()->andReturn(false); + $journalRepos->shouldReceive('firstNull')->once()->andReturn(new TransactionJournal); $this->session(['start' => '2018-01-01', 'end' => '2017-12-01']); @@ -172,6 +179,7 @@ class ShowControllerTest extends TestCase $account = $this->user()->accounts()->where('account_type_id', 3)->orderBy('id', 'ASC')->whereNull('deleted_at')->first(); $response = $this->get(route('accounts.show', [$account->id, '2018-01-01', '2017-12-01'])); $response->assertStatus(500); + $response->assertSee('End is after start!'); } /** @@ -181,6 +189,9 @@ class ShowControllerTest extends TestCase { // mock $journalRepos = $this->mock(JournalRepositoryInterface::class); + $currencyRepos = $this->mock(CurrencyRepositoryInterface::class); + $accountRepos = $this->mock(AccountRepositoryInterface::class); + $journalRepos->shouldReceive('firstNull')->once()->andReturn(new TransactionJournal); $date = new Carbon; $this->session(['start' => $date, 'end' => clone $date]); @@ -209,7 +220,7 @@ class ShowControllerTest extends TestCase $repository = $this->mock(AccountRepositoryInterface::class); // mock hasRole for user repository: - $userRepos->shouldReceive('hasRole')->withArgs([Mockery::any(),'owner'])->andReturn(true)->atLeast()->once(); + $userRepos->shouldReceive('hasRole')->withArgs([Mockery::any(), 'owner'])->andReturn(true)->atLeast()->once(); $journalRepos->shouldReceive('firstNull')->once()->andReturn(new TransactionJournal); $collector->shouldReceive('setAccounts')->andReturnSelf(); @@ -244,6 +255,8 @@ class ShowControllerTest extends TestCase { // mock stuff $journalRepos = $this->mock(JournalRepositoryInterface::class); + $currencyRepos = $this->mock(CurrencyRepositoryInterface::class); + $accountRepos = $this->mock(AccountRepositoryInterface::class); $journalRepos->shouldReceive('firstNull')->once()->andReturn(new TransactionJournal); $date = new Carbon; diff --git a/tests/Feature/Controllers/Admin/ConfigurationControllerTest.php b/tests/Feature/Controllers/Admin/ConfigurationControllerTest.php index ba164baeaf..a7e4350d38 100644 --- a/tests/Feature/Controllers/Admin/ConfigurationControllerTest.php +++ b/tests/Feature/Controllers/Admin/ConfigurationControllerTest.php @@ -24,7 +24,9 @@ namespace Tests\Feature\Controllers\Admin; use FireflyConfig; use FireflyIII\Models\Configuration; +use FireflyIII\Repositories\User\UserRepositoryInterface; use Log; +use Mockery; use Tests\TestCase; /** @@ -38,7 +40,7 @@ class ConfigurationControllerTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } /** @@ -47,8 +49,11 @@ class ConfigurationControllerTest extends TestCase */ public function testIndex(): void { - $this->be($this->user()); + $userRepos = $this->mock(UserRepositoryInterface::class); + $userRepos->shouldReceive('hasRole')->withArgs([Mockery::any(), 'owner'])->andReturn(true)->atLeast()->once(); + + $this->be($this->user()); $falseConfig = new Configuration; $falseConfig->data = false; @@ -70,6 +75,11 @@ class ConfigurationControllerTest extends TestCase */ public function testPostIndex(): void { + $userRepos = $this->mock(UserRepositoryInterface::class); + + $userRepos->shouldReceive('hasRole')->withArgs([Mockery::any(), 'owner'])->andReturn(true)->atLeast()->once(); + $userRepos->shouldReceive('hasRole')->withArgs([Mockery::any(), 'demo'])->andReturn(false)->atLeast()->once(); + $falseConfig = new Configuration; $falseConfig->data = false; diff --git a/tests/Feature/Controllers/Admin/HomeControllerTest.php b/tests/Feature/Controllers/Admin/HomeControllerTest.php index d108af0459..4240952c63 100644 --- a/tests/Feature/Controllers/Admin/HomeControllerTest.php +++ b/tests/Feature/Controllers/Admin/HomeControllerTest.php @@ -24,7 +24,9 @@ namespace Tests\Feature\Controllers\Admin; use Event; use FireflyIII\Events\AdminRequestedTestMessage; +use FireflyIII\Repositories\User\UserRepositoryInterface; use Log; +use Mockery; use Tests\TestCase; /** @@ -38,7 +40,7 @@ class HomeControllerTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } /** @@ -46,6 +48,10 @@ class HomeControllerTest extends TestCase */ public function testIndex(): void { + $userRepos = $this->mock(UserRepositoryInterface::class); + + $userRepos->shouldReceive('hasRole')->withArgs([Mockery::any(), 'owner'])->andReturn(true)->atLeast()->once(); + $this->be($this->user()); $response = $this->get(route('admin.index')); $response->assertStatus(200); @@ -58,6 +64,11 @@ class HomeControllerTest extends TestCase */ public function testTestMessage(): void { + $userRepos = $this->mock(UserRepositoryInterface::class); + + $userRepos->shouldReceive('hasRole')->withArgs([Mockery::any(), 'owner'])->andReturn(true)->atLeast()->once(); + $userRepos->shouldReceive('hasRole')->withArgs([Mockery::any(), 'demo'])->andReturn(false)->atLeast()->once(); + Event::fake(); $this->be($this->user()); diff --git a/tests/Feature/Controllers/Admin/LinkControllerTest.php b/tests/Feature/Controllers/Admin/LinkControllerTest.php index 6ea0c3567d..6d52590d3c 100644 --- a/tests/Feature/Controllers/Admin/LinkControllerTest.php +++ b/tests/Feature/Controllers/Admin/LinkControllerTest.php @@ -24,8 +24,10 @@ namespace Tests\Feature\Controllers\Admin; use FireflyIII\Models\LinkType; use FireflyIII\Repositories\LinkType\LinkTypeRepositoryInterface; +use FireflyIII\Repositories\User\UserRepositoryInterface; use Illuminate\Support\Collection; use Log; +use Mockery; use Tests\TestCase; /** @@ -39,7 +41,7 @@ class LinkControllerTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } /** @@ -47,7 +49,10 @@ class LinkControllerTest extends TestCase */ public function testCreate(): void { + $userRepos = $this->mock(UserRepositoryInterface::class); + $userRepos->shouldReceive('hasRole')->withArgs([Mockery::any(), 'owner'])->andReturn(true)->atLeast()->once(); + $userRepos->shouldReceive('hasRole')->withArgs([Mockery::any(), 'demo'])->andReturn(false)->atLeast()->once(); $this->be($this->user()); $response = $this->get(route('admin.links.create')); $response->assertStatus(200); @@ -58,10 +63,14 @@ class LinkControllerTest extends TestCase */ public function testDeleteEditable(): void { + $userRepos = $this->mock(UserRepositoryInterface::class); $repository = $this->mock(LinkTypeRepositoryInterface::class); // create editable link type just in case: LinkType::create(['editable' => 1, 'inward' => 'hello', 'outward' => 'bye', 'name' => 'Test type']); + $userRepos->shouldReceive('hasRole')->withArgs([Mockery::any(), 'owner'])->andReturn(true)->atLeast()->once(); + $userRepos->shouldReceive('hasRole')->withArgs([Mockery::any(), 'demo'])->andReturn(false)->atLeast()->once(); + $linkType = LinkType::where('editable', 1)->first(); $repository->shouldReceive('get')->once()->andReturn(new Collection([$linkType])); $repository->shouldReceive('countJournals')->andReturn(2); @@ -75,8 +84,12 @@ class LinkControllerTest extends TestCase */ public function testDeleteNonEditable(): void { + $userRepos = $this->mock(UserRepositoryInterface::class); $repository = $this->mock(LinkTypeRepositoryInterface::class); $linkType = LinkType::where('editable', 0)->first(); + + $userRepos->shouldReceive('hasRole')->withArgs([Mockery::any(), 'owner'])->andReturn(true)->atLeast()->once(); + $userRepos->shouldReceive('hasRole')->withArgs([Mockery::any(), 'demo'])->andReturn(false)->atLeast()->once(); $this->be($this->user()); $response = $this->get(route('admin.links.delete', [$linkType->id])); $response->assertStatus(302); @@ -88,8 +101,12 @@ class LinkControllerTest extends TestCase */ public function testDestroy(): void { + $userRepos = $this->mock(UserRepositoryInterface::class); $repository = $this->mock(LinkTypeRepositoryInterface::class); + $userRepos->shouldReceive('hasRole')->withArgs([Mockery::any(), 'owner'])->andReturn(true)->atLeast()->once(); + $userRepos->shouldReceive('hasRole')->withArgs([Mockery::any(), 'demo'])->andReturn(false)->atLeast()->once(); + // create editable link type just in case: LinkType::create(['editable' => 1, 'inward' => 'hellox', 'outward' => 'byex', 'name' => 'Test typeX']); @@ -108,6 +125,11 @@ class LinkControllerTest extends TestCase */ public function testEditEditable(): void { + $userRepos = $this->mock(UserRepositoryInterface::class); + + $userRepos->shouldReceive('hasRole')->withArgs([Mockery::any(), 'owner'])->andReturn(true)->atLeast()->once(); + $userRepos->shouldReceive('hasRole')->withArgs([Mockery::any(), 'demo'])->andReturn(false)->atLeast()->once(); + // create editable link type just in case: LinkType::create(['editable' => 1, 'inward' => 'hello Y', 'outward' => 'bye Y', 'name' => 'Test type Y']); @@ -122,6 +144,11 @@ class LinkControllerTest extends TestCase */ public function testEditNonEditable(): void { + $userRepos = $this->mock(UserRepositoryInterface::class); + + $userRepos->shouldReceive('hasRole')->withArgs([Mockery::any(), 'owner'])->andReturn(true)->atLeast()->once(); + $userRepos->shouldReceive('hasRole')->withArgs([Mockery::any(), 'demo'])->andReturn(false)->atLeast()->once(); + $linkType = LinkType::where('editable', 0)->first(); $this->be($this->user()); $response = $this->get(route('admin.links.edit', [$linkType->id])); @@ -134,6 +161,11 @@ class LinkControllerTest extends TestCase */ public function testIndex(): void { + $userRepos = $this->mock(UserRepositoryInterface::class); + + $userRepos->shouldReceive('hasRole')->withArgs([Mockery::any(), 'owner'])->andReturn(true)->atLeast()->once(); + + $linkTypes = LinkType::inRandomOrder()->take(3)->get(); $repository = $this->mock(LinkTypeRepositoryInterface::class); $repository->shouldReceive('get')->andReturn($linkTypes); @@ -148,6 +180,11 @@ class LinkControllerTest extends TestCase */ public function testShow(): void { + $userRepos = $this->mock(UserRepositoryInterface::class); + + $userRepos->shouldReceive('hasRole')->withArgs([Mockery::any(), 'owner'])->andReturn(true)->atLeast()->once(); + + $linkType = LinkType::first(); $this->be($this->user()); $response = $this->get(route('admin.links.show', [$linkType->id])); @@ -160,8 +197,13 @@ class LinkControllerTest extends TestCase */ public function testStore(): void { + $userRepos = $this->mock(UserRepositoryInterface::class); $repository = $this->mock(LinkTypeRepositoryInterface::class); - $data = [ + + $userRepos->shouldReceive('hasRole')->withArgs([Mockery::any(), 'owner'])->andReturn(true)->atLeast()->once(); + $userRepos->shouldReceive('hasRole')->withArgs([Mockery::any(), 'demo'])->andReturn(false)->atLeast()->once(); + + $data = [ 'name' => 'test ' . random_int(1, 10000), 'inward' => 'test inward' . random_int(1, 10000), 'outward' => 'test outward' . random_int(1, 10000), @@ -182,8 +224,13 @@ class LinkControllerTest extends TestCase */ public function testStoreRedirect(): void { + $userRepos = $this->mock(UserRepositoryInterface::class); $repository = $this->mock(LinkTypeRepositoryInterface::class); - $data = [ + + $userRepos->shouldReceive('hasRole')->withArgs([Mockery::any(), 'owner'])->andReturn(true)->atLeast()->once(); + $userRepos->shouldReceive('hasRole')->withArgs([Mockery::any(), 'demo'])->andReturn(false)->atLeast()->once(); + + $data = [ 'name' => 'test ' . random_int(1, 10000), 'inward' => 'test inward' . random_int(1, 10000), 'outward' => 'test outward' . random_int(1, 10000), @@ -203,8 +250,12 @@ class LinkControllerTest extends TestCase */ public function testUpdate(): void { + $userRepos = $this->mock(UserRepositoryInterface::class); $repository = $this->mock(LinkTypeRepositoryInterface::class); + $userRepos->shouldReceive('hasRole')->withArgs([Mockery::any(), 'owner'])->andReturn(true)->atLeast()->once(); + $userRepos->shouldReceive('hasRole')->withArgs([Mockery::any(), 'demo'])->andReturn(false)->atLeast()->once(); + // create editable link type just in case: $linkType = LinkType::create(['editable' => 1, 'inward' => 'helloxz', 'outward' => 'bzyex', 'name' => 'Test tyzpeX']); $repository->shouldReceive('update')->once()->andReturn(new $linkType); @@ -227,8 +278,13 @@ class LinkControllerTest extends TestCase */ public function testUpdateNonEditable(): void { + $userRepos = $this->mock(UserRepositoryInterface::class); $repository = $this->mock(LinkTypeRepositoryInterface::class); - $linkType = LinkType::where('editable', 0)->first(); + + $userRepos->shouldReceive('hasRole')->withArgs([Mockery::any(), 'owner'])->andReturn(true)->atLeast()->once(); + $userRepos->shouldReceive('hasRole')->withArgs([Mockery::any(), 'demo'])->andReturn(false)->atLeast()->once(); + + $linkType = LinkType::where('editable', 0)->first(); $data = [ 'name' => 'test ' . random_int(1, 10000), @@ -249,7 +305,12 @@ class LinkControllerTest extends TestCase */ public function testUpdateRedirect(): void { + $userRepos = $this->mock(UserRepositoryInterface::class); $repository = $this->mock(LinkTypeRepositoryInterface::class); + + $userRepos->shouldReceive('hasRole')->withArgs([Mockery::any(), 'owner'])->andReturn(true)->atLeast()->once(); + $userRepos->shouldReceive('hasRole')->withArgs([Mockery::any(), 'demo'])->andReturn(false)->atLeast()->once(); + // create editable link type just in case: $linkType = LinkType::create(['editable' => 1, 'inward' => 'healox', 'outward' => 'byaex', 'name' => 'Test tyapeX']); diff --git a/tests/Feature/Controllers/Admin/UpdateControllerTest.php b/tests/Feature/Controllers/Admin/UpdateControllerTest.php index f331bb5e3d..932ad9fbf1 100644 --- a/tests/Feature/Controllers/Admin/UpdateControllerTest.php +++ b/tests/Feature/Controllers/Admin/UpdateControllerTest.php @@ -26,6 +26,7 @@ use Carbon\Carbon; use FireflyConfig; use FireflyIII\Exceptions\FireflyException; use FireflyIII\Models\Configuration; +use FireflyIII\Repositories\User\UserRepositoryInterface; use FireflyIII\Services\Github\Object\Release; use FireflyIII\Services\Github\Request\UpdateRequest; use Log; @@ -43,7 +44,7 @@ class UpdateControllerTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } /** @@ -51,6 +52,10 @@ class UpdateControllerTest extends TestCase */ public function testIndex(): void { + $userRepos = $this->mock(UserRepositoryInterface::class); + + $userRepos->shouldReceive('hasRole')->withArgs([Mockery::any(), 'owner'])->andReturn(true)->atLeast()->once(); + $this->be($this->user()); $config = new Configuration; @@ -74,6 +79,11 @@ class UpdateControllerTest extends TestCase */ public function testPost(): void { + $userRepos = $this->mock(UserRepositoryInterface::class); + + $userRepos->shouldReceive('hasRole')->withArgs([Mockery::any(), 'owner'])->andReturn(true)->atLeast()->once(); + $userRepos->shouldReceive('hasRole')->withArgs([Mockery::any(), 'demo'])->andReturn(false)->atLeast()->once(); + $falseConfig = new Configuration; $falseConfig->data = false; @@ -93,6 +103,11 @@ class UpdateControllerTest extends TestCase */ public function testUpdateCheck(): void { + $userRepos = $this->mock(UserRepositoryInterface::class); + + $userRepos->shouldReceive('hasRole')->withArgs([Mockery::any(), 'owner'])->andReturn(true)->atLeast()->once(); + $userRepos->shouldReceive('hasRole')->withArgs([Mockery::any(), 'demo'])->andReturn(false)->atLeast()->once(); + $falseConfig = new Configuration; $falseConfig->data = false; FireflyConfig::shouldReceive('get')->withArgs(['is_demo_site', false])->once()->andReturn($falseConfig); @@ -123,6 +138,11 @@ class UpdateControllerTest extends TestCase */ public function testUpdateCheckCurrent(): void { + $userRepos = $this->mock(UserRepositoryInterface::class); + + $userRepos->shouldReceive('hasRole')->withArgs([Mockery::any(), 'owner'])->andReturn(true)->atLeast()->once(); + $userRepos->shouldReceive('hasRole')->withArgs([Mockery::any(), 'demo'])->andReturn(false)->atLeast()->once(); + $falseConfig = new Configuration; $falseConfig->data = false; FireflyConfig::shouldReceive('get')->withArgs(['is_demo_site', false])->once()->andReturn($falseConfig); @@ -151,6 +171,11 @@ class UpdateControllerTest extends TestCase */ public function testUpdateCheckError(): void { + $userRepos = $this->mock(UserRepositoryInterface::class); + + $userRepos->shouldReceive('hasRole')->withArgs([Mockery::any(), 'owner'])->andReturn(true)->atLeast()->once(); + $userRepos->shouldReceive('hasRole')->withArgs([Mockery::any(), 'demo'])->andReturn(false)->atLeast()->once(); + $falseConfig = new Configuration; $falseConfig->data = false; FireflyConfig::shouldReceive('get')->withArgs(['is_demo_site', false])->once()->andReturn($falseConfig); @@ -174,6 +199,11 @@ class UpdateControllerTest extends TestCase */ public function testUpdateCheckNewer(): void { + $userRepos = $this->mock(UserRepositoryInterface::class); + + $userRepos->shouldReceive('hasRole')->withArgs([Mockery::any(), 'owner'])->andReturn(true)->atLeast()->once(); + $userRepos->shouldReceive('hasRole')->withArgs([Mockery::any(), 'demo'])->andReturn(false)->atLeast()->once(); + $falseConfig = new Configuration; $falseConfig->data = false; FireflyConfig::shouldReceive('get')->withArgs(['is_demo_site', false])->once()->andReturn($falseConfig); diff --git a/tests/Feature/Controllers/Admin/UserControllerTest.php b/tests/Feature/Controllers/Admin/UserControllerTest.php index 17f5f01c00..7d421689fd 100644 --- a/tests/Feature/Controllers/Admin/UserControllerTest.php +++ b/tests/Feature/Controllers/Admin/UserControllerTest.php @@ -39,7 +39,7 @@ class UserControllerTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } /** diff --git a/tests/Feature/Controllers/AttachmentControllerTest.php b/tests/Feature/Controllers/AttachmentControllerTest.php index d7dc41f2d3..c6550265f7 100644 --- a/tests/Feature/Controllers/AttachmentControllerTest.php +++ b/tests/Feature/Controllers/AttachmentControllerTest.php @@ -26,8 +26,10 @@ use FireflyIII\Models\Attachment; use FireflyIII\Models\TransactionJournal; use FireflyIII\Repositories\Attachment\AttachmentRepositoryInterface; use FireflyIII\Repositories\Journal\JournalRepositoryInterface; +use FireflyIII\Repositories\User\UserRepositoryInterface; use Illuminate\Support\Collection; use Log; +use Mockery; use Tests\TestCase; /** @@ -45,7 +47,7 @@ class AttachmentControllerTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } /** @@ -56,6 +58,10 @@ class AttachmentControllerTest extends TestCase // mock stuff $attachRepository = $this->mock(AttachmentRepositoryInterface::class); $journalRepos = $this->mock(JournalRepositoryInterface::class); + $userRepos = $this->mock(UserRepositoryInterface::class); + + $userRepos->shouldReceive('hasRole')->withArgs([Mockery::any(), 'owner'])->andReturn(true)->atLeast()->once(); + $journalRepos->shouldReceive('firstNull')->once()->andReturn(new TransactionJournal); $this->be($this->user()); $response = $this->get(route('attachments.delete', [1])); @@ -72,6 +78,8 @@ class AttachmentControllerTest extends TestCase // mock stuff $journalRepos = $this->mock(JournalRepositoryInterface::class); $repository = $this->mock(AttachmentRepositoryInterface::class); + $userRepos = $this->mock(UserRepositoryInterface::class); + $repository->shouldReceive('destroy')->andReturn(true); $journalRepos->shouldReceive('firstNull')->once()->andReturn(new TransactionJournal); @@ -90,6 +98,8 @@ class AttachmentControllerTest extends TestCase // mock stuff $journalRepos = $this->mock(JournalRepositoryInterface::class); $repository = $this->mock(AttachmentRepositoryInterface::class); + $userRepos = $this->mock(UserRepositoryInterface::class); + $repository->shouldReceive('exists')->once()->andReturn(true); $repository->shouldReceive('getContent')->once()->andReturn('This is attachment number one.'); $journalRepos->shouldReceive('firstNull')->once()->andReturn(new TransactionJournal); @@ -103,13 +113,14 @@ class AttachmentControllerTest extends TestCase /** * @covers \FireflyIII\Http\Controllers\AttachmentController - * @expectedExceptionMessage Could not find the indicated attachment */ public function testDownloadFail(): void { // mock stuff $journalRepos = $this->mock(JournalRepositoryInterface::class); $repository = $this->mock(AttachmentRepositoryInterface::class); + $userRepos = $this->mock(UserRepositoryInterface::class); + $repository->shouldReceive('exists')->once()->andReturn(false); $journalRepos->shouldReceive('firstNull')->once()->andReturn(new TransactionJournal); @@ -125,6 +136,10 @@ class AttachmentControllerTest extends TestCase { $attachRepository = $this->mock(AttachmentRepositoryInterface::class); $journalRepos = $this->mock(JournalRepositoryInterface::class); + $userRepos = $this->mock(UserRepositoryInterface::class); + + $userRepos->shouldReceive('hasRole')->withArgs([Mockery::any(), 'owner'])->andReturn(true)->atLeast()->once(); + $attachRepository->shouldReceive('getNoteText')->andReturn('OK'); $journalRepos->shouldReceive('firstNull')->once()->andReturn(new TransactionJournal); $this->be($this->user()); @@ -140,6 +155,9 @@ class AttachmentControllerTest extends TestCase public function testIndex() { $repository = $this->mock(AttachmentRepositoryInterface::class); + $userRepos = $this->mock(UserRepositoryInterface::class); + + $userRepos->shouldReceive('hasRole')->withArgs([Mockery::any(), 'owner'])->andReturn(true)->atLeast()->once(); $repository->shouldReceive('get')->andReturn(new Collection([Attachment::first()]))->once(); $repository->shouldReceive('exists')->andReturn(true)->once(); @@ -159,6 +177,8 @@ class AttachmentControllerTest extends TestCase // mock stuff $journalRepos = $this->mock(JournalRepositoryInterface::class); $repository = $this->mock(AttachmentRepositoryInterface::class); + $userRepos = $this->mock(UserRepositoryInterface::class); + $repository->shouldReceive('update')->once(); $journalRepos->shouldReceive('firstNull')->once()->andReturn(new TransactionJournal); @@ -181,11 +201,14 @@ class AttachmentControllerTest extends TestCase */ public function testView(): void { - $repository = $this->mock(AttachmentRepositoryInterface::class); + $repository = $this->mock(AttachmentRepositoryInterface::class); + $userRepos = $this->mock(UserRepositoryInterface::class); + $journalRepos = $this->mock(JournalRepositoryInterface::class); + $repository->shouldReceive('exists')->once()->andReturn(true); $repository->shouldReceive('getContent')->once()->andReturn('This is attachment number one.'); - $journalRepos = $this->mock(JournalRepositoryInterface::class); + $journalRepos->shouldReceive('firstNull')->once()->andReturn(new TransactionJournal); $this->be($this->user()); $response = $this->get(route('attachments.view', [3])); @@ -198,10 +221,13 @@ class AttachmentControllerTest extends TestCase */ public function testViewFail(): void { - $repository = $this->mock(AttachmentRepositoryInterface::class); + $repository = $this->mock(AttachmentRepositoryInterface::class); + $userRepos = $this->mock(UserRepositoryInterface::class); + $journalRepos = $this->mock(JournalRepositoryInterface::class); + $repository->shouldReceive('exists')->once()->andReturn(false); - $journalRepos = $this->mock(JournalRepositoryInterface::class); + $journalRepos->shouldReceive('firstNull')->once()->andReturn(new TransactionJournal); $this->be($this->user()); $response = $this->get(route('attachments.view', [1])); diff --git a/tests/Feature/Controllers/Auth/ForgotPasswordControllerTest.php b/tests/Feature/Controllers/Auth/ForgotPasswordControllerTest.php index 6361f0095e..7e695b0478 100644 --- a/tests/Feature/Controllers/Auth/ForgotPasswordControllerTest.php +++ b/tests/Feature/Controllers/Auth/ForgotPasswordControllerTest.php @@ -37,7 +37,7 @@ class ForgotPasswordControllerTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } /** diff --git a/tests/Feature/Controllers/Auth/TwoFactorControllerTest.php b/tests/Feature/Controllers/Auth/TwoFactorControllerTest.php index f196fd401c..0f7cfa0591 100644 --- a/tests/Feature/Controllers/Auth/TwoFactorControllerTest.php +++ b/tests/Feature/Controllers/Auth/TwoFactorControllerTest.php @@ -39,7 +39,7 @@ class TwoFactorControllerTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } /** @@ -89,7 +89,6 @@ class TwoFactorControllerTest extends TestCase /** * @covers \FireflyIII\Http\Controllers\Auth\TwoFactorController - * @expectedExceptionMessage Your two factor authentication secret is empty */ public function testIndexNoSecret(): void { diff --git a/tests/Feature/Controllers/BillControllerTest.php b/tests/Feature/Controllers/BillControllerTest.php index a4d6e1ea6c..46463ccfc2 100644 --- a/tests/Feature/Controllers/BillControllerTest.php +++ b/tests/Feature/Controllers/BillControllerTest.php @@ -29,8 +29,10 @@ use FireflyIII\Models\Bill; use FireflyIII\Models\Rule; use FireflyIII\Models\TransactionJournal; use FireflyIII\Repositories\Bill\BillRepositoryInterface; +use FireflyIII\Repositories\Currency\CurrencyRepositoryInterface; use FireflyIII\Repositories\Journal\JournalRepositoryInterface; use FireflyIII\Repositories\RuleGroup\RuleGroupRepositoryInterface; +use FireflyIII\Repositories\User\UserRepositoryInterface; use FireflyIII\TransactionRules\TransactionMatcher; use Illuminate\Pagination\LengthAwarePaginator; use Illuminate\Support\Collection; @@ -54,7 +56,7 @@ class BillControllerTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } @@ -67,6 +69,12 @@ class BillControllerTest extends TestCase $attachHelper = $this->mock(AttachmentHelperInterface::class); $journalRepos = $this->mock(JournalRepositoryInterface::class); $ruleGroupRepos = $this->mock(RuleGroupRepositoryInterface::class); + $userRepos = $this->mock(UserRepositoryInterface::class); + $billRepos = $this->mock(BillRepositoryInterface::class); + $currencyRepos = $this->mock(CurrencyRepositoryInterface::class); + + $userRepos->shouldReceive('hasRole')->withArgs([Mockery::any(), 'owner'])->andReturn(true)->atLeast()->once(); + $journalRepos->shouldReceive('firstNull')->once()->andReturn(new TransactionJournal); $this->be($this->user()); @@ -85,6 +93,11 @@ class BillControllerTest extends TestCase $attachHelper = $this->mock(AttachmentHelperInterface::class); $journalRepos = $this->mock(JournalRepositoryInterface::class); $ruleGroupRepos = $this->mock(RuleGroupRepositoryInterface::class); + $userRepos = $this->mock(UserRepositoryInterface::class); + $billRepos = $this->mock(BillRepositoryInterface::class); + $currencyRepos = $this->mock(CurrencyRepositoryInterface::class); + + $userRepos->shouldReceive('hasRole')->withArgs([Mockery::any(), 'owner'])->andReturn(true)->atLeast()->once(); $journalRepos->shouldReceive('firstNull')->once()->andReturn(new TransactionJournal); $this->be($this->user()); @@ -104,6 +117,9 @@ class BillControllerTest extends TestCase $journalRepos = $this->mock(JournalRepositoryInterface::class); $repository = $this->mock(BillRepositoryInterface::class); $ruleGroupRepos = $this->mock(RuleGroupRepositoryInterface::class); + $userRepos = $this->mock(UserRepositoryInterface::class); + $currencyRepos = $this->mock(CurrencyRepositoryInterface::class); + $repository->shouldReceive('destroy')->andReturn(true); $journalRepos->shouldReceive('firstNull')->once()->andReturn(new TransactionJournal); @@ -123,6 +139,12 @@ class BillControllerTest extends TestCase $attachHelper = $this->mock(AttachmentHelperInterface::class); $journalRepos = $this->mock(JournalRepositoryInterface::class); $ruleGroupRepos = $this->mock(RuleGroupRepositoryInterface::class); + $userRepos = $this->mock(UserRepositoryInterface::class); + $billRepos = $this->mock(BillRepositoryInterface::class); + $currencyRepos = $this->mock(CurrencyRepositoryInterface::class); + + $billRepos->shouldReceive('getNoteText')->andReturn('Hello'); + $userRepos->shouldReceive('hasRole')->withArgs([Mockery::any(), 'owner'])->andReturn(true)->atLeast()->once(); $journalRepos->shouldReceive('firstNull')->once()->andReturn(new TransactionJournal); $this->be($this->user()); @@ -144,6 +166,10 @@ class BillControllerTest extends TestCase $journalRepos = $this->mock(JournalRepositoryInterface::class); $repository = $this->mock(BillRepositoryInterface::class); $ruleGroupRepos = $this->mock(RuleGroupRepositoryInterface::class); + $userRepos = $this->mock(UserRepositoryInterface::class); + $currencyRepos = $this->mock(CurrencyRepositoryInterface::class); + + $userRepos->shouldReceive('hasRole')->withArgs([Mockery::any(), 'owner'])->andReturn(true)->atLeast()->once(); $journalRepos->shouldReceive('firstNull')->once()->andReturn(new TransactionJournal); $collection = new Collection([$bill]); $repository->shouldReceive('getPaginator')->andReturn(new LengthAwarePaginator($collection, 1, 50))->once(); @@ -171,10 +197,14 @@ class BillControllerTest extends TestCase $journalRepos = $this->mock(JournalRepositoryInterface::class); $repository = $this->mock(BillRepositoryInterface::class); $ruleGroupRepos = $this->mock(RuleGroupRepositoryInterface::class); + $userRepos = $this->mock(UserRepositoryInterface::class); + $currencyRepos = $this->mock(CurrencyRepositoryInterface::class); + $journalRepos->shouldReceive('firstNull')->once()->andReturn(new TransactionJournal); $repository->shouldReceive('getRulesForBill')->andReturn(new Collection([$rule])); //calls for transaction matcher: + // todo bad to do this: $matcher = $this->mock(TransactionMatcher::class); $matcher->shouldReceive('setLimit')->once()->withArgs([100000]); $matcher->shouldReceive('setRange')->once()->withArgs([100000]); @@ -199,6 +229,9 @@ class BillControllerTest extends TestCase $journalRepos = $this->mock(JournalRepositoryInterface::class); $repository = $this->mock(BillRepositoryInterface::class); $ruleGroupRepos = $this->mock(RuleGroupRepositoryInterface::class); + $userRepos = $this->mock(UserRepositoryInterface::class); + $currencyRepos = $this->mock(CurrencyRepositoryInterface::class); + $journalRepos->shouldReceive('firstNull')->once()->andReturn(new TransactionJournal); $this->be($this->user()); @@ -218,6 +251,10 @@ class BillControllerTest extends TestCase $collector = $this->mock(TransactionCollectorInterface::class); $repository = $this->mock(BillRepositoryInterface::class); $ruleGroupRepos = $this->mock(RuleGroupRepositoryInterface::class); + $userRepos = $this->mock(UserRepositoryInterface::class); + $currencyRepos = $this->mock(CurrencyRepositoryInterface::class); + + $userRepos->shouldReceive('hasRole')->withArgs([Mockery::any(), 'owner'])->andReturn(true)->atLeast()->once(); $repository->shouldReceive('getYearAverage')->andReturn('0'); $repository->shouldReceive('getOverallAverage')->andReturn('0'); $repository->shouldReceive('nextExpectedMatch')->andReturn(new Carbon); @@ -255,6 +292,9 @@ class BillControllerTest extends TestCase $journalRepos = $this->mock(JournalRepositoryInterface::class); $repository = $this->mock(BillRepositoryInterface::class); $ruleGroupRepos = $this->mock(RuleGroupRepositoryInterface::class); + $userRepos = $this->mock(UserRepositoryInterface::class); + $currencyRepos = $this->mock(CurrencyRepositoryInterface::class); + $journalRepos->shouldReceive('firstNull')->once()->andReturn(new TransactionJournal); $repository->shouldReceive('store')->andReturn($bill); $attachHelper->shouldReceive('saveAttachmentsForModel'); @@ -289,7 +329,10 @@ class BillControllerTest extends TestCase $journalRepos = $this->mock(JournalRepositoryInterface::class); $repository = $this->mock(BillRepositoryInterface::class); $ruleGroupRepos = $this->mock(RuleGroupRepositoryInterface::class); - $bill = $this->user()->bills()->first(); + $userRepos = $this->mock(UserRepositoryInterface::class); + $currencyRepos = $this->mock(CurrencyRepositoryInterface::class); + + $bill = $this->user()->bills()->first(); $journalRepos->shouldReceive('firstNull')->once()->andReturn(new TransactionJournal); $repository->shouldReceive('store')->andReturn($bill); $attachHelper->shouldReceive('saveAttachmentsForModel'); @@ -325,6 +368,9 @@ class BillControllerTest extends TestCase $journalRepos = $this->mock(JournalRepositoryInterface::class); $repository = $this->mock(BillRepositoryInterface::class); $ruleGroupRepos = $this->mock(RuleGroupRepositoryInterface::class); + $userRepos = $this->mock(UserRepositoryInterface::class); + $currencyRepos = $this->mock(CurrencyRepositoryInterface::class); + $journalRepos->shouldReceive('firstNull')->once()->andReturn(new TransactionJournal); $repository->shouldReceive('store')->andReturn(null); @@ -357,6 +403,9 @@ class BillControllerTest extends TestCase $journalRepos = $this->mock(JournalRepositoryInterface::class); $repository = $this->mock(BillRepositoryInterface::class); $ruleGroupRepos = $this->mock(RuleGroupRepositoryInterface::class); + $userRepos = $this->mock(UserRepositoryInterface::class); + $currencyRepos = $this->mock(CurrencyRepositoryInterface::class); + $journalRepos->shouldReceive('firstNull')->once()->andReturn(new TransactionJournal); $repository->shouldReceive('store')->andReturn(new Bill); $attachHelper->shouldReceive('saveAttachmentsForModel'); @@ -392,6 +441,9 @@ class BillControllerTest extends TestCase $journalRepos = $this->mock(JournalRepositoryInterface::class); $repository = $this->mock(BillRepositoryInterface::class); $ruleGroupRepos = $this->mock(RuleGroupRepositoryInterface::class); + $userRepos = $this->mock(UserRepositoryInterface::class); + $currencyRepos = $this->mock(CurrencyRepositoryInterface::class); + $journalRepos->shouldReceive('firstNull')->once()->andReturn(new TransactionJournal); $repository->shouldReceive('update')->andReturn(new Bill); $attachHelper->shouldReceive('saveAttachmentsForModel'); diff --git a/tests/Feature/Controllers/Budget/AmountControllerTest.php b/tests/Feature/Controllers/Budget/AmountControllerTest.php index e652504a66..733423d6f9 100644 --- a/tests/Feature/Controllers/Budget/AmountControllerTest.php +++ b/tests/Feature/Controllers/Budget/AmountControllerTest.php @@ -48,7 +48,7 @@ class AmountControllerTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } /** diff --git a/tests/Feature/Controllers/Budget/CreateControllerTest.php b/tests/Feature/Controllers/Budget/CreateControllerTest.php index 0b392a45b1..14bae5caee 100644 --- a/tests/Feature/Controllers/Budget/CreateControllerTest.php +++ b/tests/Feature/Controllers/Budget/CreateControllerTest.php @@ -28,7 +28,9 @@ use FireflyIII\Models\Budget; use FireflyIII\Models\TransactionJournal; use FireflyIII\Repositories\Budget\BudgetRepositoryInterface; use FireflyIII\Repositories\Journal\JournalRepositoryInterface; +use FireflyIII\Repositories\User\UserRepositoryInterface; use Log; +use Mockery; use Tests\TestCase; /** @@ -43,7 +45,7 @@ class CreateControllerTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } @@ -56,6 +58,9 @@ class CreateControllerTest extends TestCase // mock stuff $repository = $this->mock(BudgetRepositoryInterface::class); $journalRepos = $this->mock(JournalRepositoryInterface::class); + $userRepos = $this->mock(UserRepositoryInterface::class); + + $userRepos->shouldReceive('hasRole')->withArgs([Mockery::any(), 'owner'])->andReturn(true)->atLeast()->once(); $journalRepos->shouldReceive('firstNull')->once()->andReturn(new TransactionJournal); $this->be($this->user()); @@ -76,6 +81,8 @@ class CreateControllerTest extends TestCase $budget = factory(Budget::class)->make(); $repository = $this->mock(BudgetRepositoryInterface::class); $journalRepos = $this->mock(JournalRepositoryInterface::class); + $userRepos = $this->mock(UserRepositoryInterface::class); + $journalRepos->shouldReceive('firstNull')->once()->andReturn(new TransactionJournal); $repository->shouldReceive('findNull')->andReturn($budget); $repository->shouldReceive('store')->andReturn($budget); diff --git a/tests/Feature/Controllers/Budget/DeleteControllerTest.php b/tests/Feature/Controllers/Budget/DeleteControllerTest.php index 6cc8436f0f..61d480e978 100644 --- a/tests/Feature/Controllers/Budget/DeleteControllerTest.php +++ b/tests/Feature/Controllers/Budget/DeleteControllerTest.php @@ -26,7 +26,9 @@ namespace Tests\Feature\Controllers\Budget; use FireflyIII\Models\TransactionJournal; use FireflyIII\Repositories\Budget\BudgetRepositoryInterface; use FireflyIII\Repositories\Journal\JournalRepositoryInterface; +use FireflyIII\Repositories\User\UserRepositoryInterface; use Log; +use Mockery; use Tests\TestCase; /** @@ -41,7 +43,7 @@ class DeleteControllerTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } @@ -54,6 +56,9 @@ class DeleteControllerTest extends TestCase // mock stuff $repository = $this->mock(BudgetRepositoryInterface::class); $journalRepos = $this->mock(JournalRepositoryInterface::class); + $userRepos = $this->mock(UserRepositoryInterface::class); + + $userRepos->shouldReceive('hasRole')->withArgs([Mockery::any(), 'owner'])->andReturn(true)->atLeast()->once(); $journalRepos->shouldReceive('firstNull')->once()->andReturn(new TransactionJournal); $this->be($this->user()); @@ -72,6 +77,8 @@ class DeleteControllerTest extends TestCase // mock stuff $journalRepos = $this->mock(JournalRepositoryInterface::class); $repository = $this->mock(BudgetRepositoryInterface::class); + $userRepos = $this->mock(UserRepositoryInterface::class); + $journalRepos->shouldReceive('firstNull')->once()->andReturn(new TransactionJournal); $repository->shouldReceive('destroy')->andReturn(true); diff --git a/tests/Feature/Controllers/Budget/EditControllerTest.php b/tests/Feature/Controllers/Budget/EditControllerTest.php index 55ac0e8cb6..48395d38a1 100644 --- a/tests/Feature/Controllers/Budget/EditControllerTest.php +++ b/tests/Feature/Controllers/Budget/EditControllerTest.php @@ -27,7 +27,9 @@ use FireflyIII\Models\Budget; use FireflyIII\Models\TransactionJournal; use FireflyIII\Repositories\Budget\BudgetRepositoryInterface; use FireflyIII\Repositories\Journal\JournalRepositoryInterface; +use FireflyIII\Repositories\User\UserRepositoryInterface; use Log; +use Mockery; use Tests\TestCase; /** @@ -42,7 +44,7 @@ class EditControllerTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } @@ -55,6 +57,9 @@ class EditControllerTest extends TestCase // mock stuff $repository = $this->mock(BudgetRepositoryInterface::class); $journalRepos = $this->mock(JournalRepositoryInterface::class); + $userRepos = $this->mock(UserRepositoryInterface::class); + + $userRepos->shouldReceive('hasRole')->withArgs([Mockery::any(), 'owner'])->andReturn(true)->atLeast()->once(); $journalRepos->shouldReceive('firstNull')->once()->andReturn(new TransactionJournal); $this->be($this->user()); @@ -75,6 +80,8 @@ class EditControllerTest extends TestCase $budget = factory(Budget::class)->make(); $repository = $this->mock(BudgetRepositoryInterface::class); $journalRepos = $this->mock(JournalRepositoryInterface::class); + $userRepos = $this->mock(UserRepositoryInterface::class); + $journalRepos->shouldReceive('firstNull')->once()->andReturn(new TransactionJournal); $repository->shouldReceive('findNull')->andReturn($budget); $repository->shouldReceive('update'); diff --git a/tests/Feature/Controllers/Budget/IndexControllerTest.php b/tests/Feature/Controllers/Budget/IndexControllerTest.php index e5cb82a19e..8c7fc6db9f 100644 --- a/tests/Feature/Controllers/Budget/IndexControllerTest.php +++ b/tests/Feature/Controllers/Budget/IndexControllerTest.php @@ -30,8 +30,10 @@ use FireflyIII\Models\TransactionJournal; use FireflyIII\Repositories\Account\AccountRepositoryInterface; use FireflyIII\Repositories\Budget\BudgetRepositoryInterface; use FireflyIII\Repositories\Journal\JournalRepositoryInterface; +use FireflyIII\Repositories\User\UserRepositoryInterface; use Illuminate\Support\Collection; use Log; +use Mockery; use Tests\TestCase; /** @@ -46,7 +48,7 @@ class IndexControllerTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } /** @@ -57,6 +59,8 @@ class IndexControllerTest extends TestCase */ public function testIndex(string $range): void { + + Log::debug(sprintf('Now in testIndex(%s)', $range)); // mock stuff $budget = factory(Budget::class)->make(); @@ -76,6 +80,9 @@ class IndexControllerTest extends TestCase $accountRepos = $this->mock(AccountRepositoryInterface::class); $repository = $this->mock(BudgetRepositoryInterface::class); $journalRepos = $this->mock(JournalRepositoryInterface::class); + $userRepos = $this->mock(UserRepositoryInterface::class); + + $userRepos->shouldReceive('hasRole')->withArgs([Mockery::any(), 'owner'])->andReturn(true)->atLeast()->once(); $journalRepos->shouldReceive('firstNull')->once()->andReturn(new TransactionJournal); $accountRepos->shouldReceive('getAccountsByType')->andReturn(new Collection); @@ -122,6 +129,9 @@ class IndexControllerTest extends TestCase $accountRepos = $this->mock(AccountRepositoryInterface::class); $repository = $this->mock(BudgetRepositoryInterface::class); $journalRepos = $this->mock(JournalRepositoryInterface::class); + $userRepos = $this->mock(UserRepositoryInterface::class); + + $userRepos->shouldReceive('hasRole')->withArgs([Mockery::any(), 'owner'])->andReturn(true)->atLeast()->once(); $journalRepos->shouldReceive('firstNull')->once()->andReturn(new TransactionJournal); $accountRepos->shouldReceive('getAccountsByType')->andReturn(new Collection); @@ -170,6 +180,9 @@ class IndexControllerTest extends TestCase $accountRepos = $this->mock(AccountRepositoryInterface::class); $repository = $this->mock(BudgetRepositoryInterface::class); $journalRepos = $this->mock(JournalRepositoryInterface::class); + $userRepos = $this->mock(UserRepositoryInterface::class); + + $userRepos->shouldReceive('hasRole')->withArgs([Mockery::any(), 'owner'])->andReturn(true)->atLeast()->once(); $journalRepos->shouldReceive('firstNull')->once()->andReturn(new TransactionJournal); $accountRepos->shouldReceive('getAccountsByType')->andReturn(new Collection); @@ -216,6 +229,8 @@ class IndexControllerTest extends TestCase $accountRepos = $this->mock(AccountRepositoryInterface::class); $repository = $this->mock(BudgetRepositoryInterface::class); $journalRepos = $this->mock(JournalRepositoryInterface::class); + $userRepos = $this->mock(UserRepositoryInterface::class); + $journalRepos->shouldReceive('firstNull')->once()->andReturn(new TransactionJournal); $accountRepos->shouldReceive('getAccountsByType')->andReturn(new Collection); diff --git a/tests/Feature/Controllers/Budget/ShowControllerTest.php b/tests/Feature/Controllers/Budget/ShowControllerTest.php index 73ef7f5b72..c02c2154b4 100644 --- a/tests/Feature/Controllers/Budget/ShowControllerTest.php +++ b/tests/Feature/Controllers/Budget/ShowControllerTest.php @@ -30,9 +30,11 @@ use FireflyIII\Models\TransactionJournal; use FireflyIII\Repositories\Account\AccountRepositoryInterface; use FireflyIII\Repositories\Budget\BudgetRepositoryInterface; use FireflyIII\Repositories\Journal\JournalRepositoryInterface; +use FireflyIII\Repositories\User\UserRepositoryInterface; use Illuminate\Pagination\LengthAwarePaginator; use Illuminate\Support\Collection; use Log; +use Mockery; use Tests\TestCase; /** @@ -47,7 +49,7 @@ class ShowControllerTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } @@ -66,8 +68,10 @@ class ShowControllerTest extends TestCase $repository = $this->mock(BudgetRepositoryInterface::class); $collector = $this->mock(TransactionCollectorInterface::class); $journalRepos = $this->mock(JournalRepositoryInterface::class); - $journalRepos->shouldReceive('firstNull')->andReturn(null); + $userRepos = $this->mock(UserRepositoryInterface::class); + $userRepos->shouldReceive('hasRole')->withArgs([Mockery::any(), 'owner'])->andReturn(true)->atLeast()->once(); + $journalRepos->shouldReceive('firstNull')->andReturn(null); $collector->shouldReceive('setAllAssetAccounts')->andReturnSelf(); $collector->shouldReceive('setRange')->andReturnSelf(); $collector->shouldReceive('getTransactions')->andReturn(new Collection); @@ -102,8 +106,10 @@ class ShowControllerTest extends TestCase $repository = $this->mock(BudgetRepositoryInterface::class); $collector = $this->mock(TransactionCollectorInterface::class); $journalRepos = $this->mock(JournalRepositoryInterface::class); - $journalRepos->shouldReceive('firstNull')->andReturn(null); + $userRepos = $this->mock(UserRepositoryInterface::class); + $userRepos->shouldReceive('hasRole')->withArgs([Mockery::any(), 'owner'])->andReturn(true)->atLeast()->once(); + $journalRepos->shouldReceive('firstNull')->andReturn(null); $collector->shouldReceive('setAllAssetAccounts')->andReturnSelf(); $collector->shouldReceive('setRange')->andReturnSelf(); $collector->shouldReceive('setLimit')->andReturnSelf(); @@ -139,6 +145,9 @@ class ShowControllerTest extends TestCase $repository = $this->mock(BudgetRepositoryInterface::class); $collector = $this->mock(TransactionCollectorInterface::class); $journalRepos = $this->mock(JournalRepositoryInterface::class); + $userRepos = $this->mock(UserRepositoryInterface::class); + + $userRepos->shouldReceive('hasRole')->withArgs([Mockery::any(), 'owner'])->andReturn(true)->atLeast()->once(); $journalRepos->shouldReceive('firstNull')->andReturn(null); $collector->shouldReceive('setAllAssetAccounts')->andReturnSelf(); @@ -177,9 +186,15 @@ class ShowControllerTest extends TestCase $budgetLimit = factory(BudgetLimit::class)->make(); $journalRepos = $this->mock(JournalRepositoryInterface::class); + $userRepos = $this->mock(UserRepositoryInterface::class); + $collector = $this->mock(TransactionCollectorInterface::class); + $accountRepos = $this->mock(AccountRepositoryInterface::class); + $repository = $this->mock(BudgetRepositoryInterface::class); + + $userRepos->shouldReceive('hasRole')->withArgs([Mockery::any(), 'owner'])->andReturn(true)->atLeast()->once(); $journalRepos->shouldReceive('firstNull')->andReturn(new TransactionJournal); - $collector = $this->mock(TransactionCollectorInterface::class); + $collector->shouldReceive('setAllAssetAccounts')->andReturnSelf(); $collector->shouldReceive('setRange')->andReturnSelf(); $collector->shouldReceive('setLimit')->andReturnSelf(); @@ -188,10 +203,10 @@ class ShowControllerTest extends TestCase $collector->shouldReceive('getPaginatedTransactions')->andReturn(new LengthAwarePaginator([], 0, 10)); $collector->shouldReceive('withBudgetInformation')->andReturnSelf(); - $accountRepos = $this->mock(AccountRepositoryInterface::class); + $accountRepos->shouldReceive('getAccountsByType')->andReturn(new Collection); - $repository = $this->mock(BudgetRepositoryInterface::class); + $repository->shouldReceive('getBudgetLimits')->andReturn(new Collection([$budgetLimit])); $repository->shouldReceive('spentInPeriod')->andReturn('-1'); @@ -208,7 +223,6 @@ class ShowControllerTest extends TestCase /** * @covers \FireflyIII\Http\Controllers\Budget\ShowController - * @expectedExceptionMessage This budget limit is not part of */ public function testShowByBadBudgetLimit(): void { @@ -216,6 +230,8 @@ class ShowControllerTest extends TestCase // mock stuff $repository = $this->mock(BudgetRepositoryInterface::class); $journalRepos = $this->mock(JournalRepositoryInterface::class); + $userRepos = $this->mock(UserRepositoryInterface::class); + $journalRepos->shouldReceive('firstNull')->once()->andReturn(new TransactionJournal); $this->be($this->user()); @@ -233,20 +249,17 @@ class ShowControllerTest extends TestCase { Log::debug(sprintf('Now in testShowByBudgetLimit(%s)', $range)); // mock stuff - $journalRepos = $this->mock(JournalRepositoryInterface::class); - $journalRepos->shouldReceive('firstNull')->once()->andReturn(new TransactionJournal); - - // mock account repository + $journalRepos = $this->mock(JournalRepositoryInterface::class); $accountRepository = $this->mock(AccountRepositoryInterface::class); - $accountRepository->shouldReceive('getAccountsByType')->andReturn(new Collection); + $budgetRepository = $this->mock(BudgetRepositoryInterface::class); + $collector = $this->mock(TransactionCollectorInterface::class); + $userRepos = $this->mock(UserRepositoryInterface::class); - // mock budget repository - $budgetRepository = $this->mock(BudgetRepositoryInterface::class); + $userRepos->shouldReceive('hasRole')->withArgs([Mockery::any(), 'owner'])->andReturn(true)->atLeast()->once(); + $journalRepos->shouldReceive('firstNull')->once()->andReturn(new TransactionJournal); + $accountRepository->shouldReceive('getAccountsByType')->andReturn(new Collection); $budgetRepository->shouldReceive('spentInPeriod')->andReturn('1'); $budgetRepository->shouldReceive('getBudgetLimits')->andReturn(new Collection); - - // mock journal collector: - $collector = $this->mock(TransactionCollectorInterface::class); $collector->shouldReceive('setAllAssetAccounts')->andReturnSelf(); $collector->shouldReceive('setRange')->andReturnSelf(); $collector->shouldReceive('setLimit')->andReturnSelf(); diff --git a/tests/Feature/Controllers/Category/NoCategoryControllerTest.php b/tests/Feature/Controllers/Category/NoCategoryControllerTest.php index 8d138cf3c3..d50c01eea5 100644 --- a/tests/Feature/Controllers/Category/NoCategoryControllerTest.php +++ b/tests/Feature/Controllers/Category/NoCategoryControllerTest.php @@ -31,9 +31,11 @@ use FireflyIII\Models\TransactionJournal; use FireflyIII\Repositories\Account\AccountRepositoryInterface; use FireflyIII\Repositories\Category\CategoryRepositoryInterface; use FireflyIII\Repositories\Journal\JournalRepositoryInterface; +use FireflyIII\Repositories\User\UserRepositoryInterface; use Illuminate\Pagination\LengthAwarePaginator; use Illuminate\Support\Collection; use Log; +use Mockery; use Navigation; use Tests\TestCase; @@ -49,7 +51,7 @@ class NoCategoryControllerTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } @@ -61,17 +63,18 @@ class NoCategoryControllerTest extends TestCase */ public function testNoCategory(string $range): void { - Log::debug(sprintf('Test noCategory(%s)', $range)); // mock stuff $collector = $this->mock(TransactionCollectorInterface::class); $categoryRepos = $this->mock(CategoryRepositoryInterface::class); $accountRepos = $this->mock(AccountRepositoryInterface::class); $journalRepos = $this->mock(JournalRepositoryInterface::class); + $userRepos = $this->mock(UserRepositoryInterface::class); // get the journal with the most recent date for firstNull: $journal = $this->user()->transactionJournals()->orderBy('date', 'DESC')->first(); $journalRepos->shouldReceive('firstNull')->twice()->andReturn($journal); + $userRepos->shouldReceive('hasRole')->withArgs([Mockery::any(), 'owner'])->andReturn(true)->atLeast()->once(); $collector->shouldReceive('setAllAssetAccounts')->andReturnSelf(); $collector->shouldReceive('setTypes')->andReturnSelf(); @@ -108,7 +111,9 @@ class NoCategoryControllerTest extends TestCase $categoryRepos = $this->mock(CategoryRepositoryInterface::class); $accountRepos = $this->mock(AccountRepositoryInterface::class); $journalRepos = $this->mock(JournalRepositoryInterface::class); + $userRepos = $this->mock(UserRepositoryInterface::class); $journalRepos->shouldReceive('firstNull')->twice()->andReturn(TransactionJournal::first()); + $userRepos->shouldReceive('hasRole')->withArgs([Mockery::any(), 'owner'])->andReturn(true)->atLeast()->once(); $collector->shouldReceive('setAllAssetAccounts')->andReturnSelf(); $collector->shouldReceive('setTypes')->andReturnSelf(); @@ -144,7 +149,10 @@ class NoCategoryControllerTest extends TestCase $categoryRepos = $this->mock(CategoryRepositoryInterface::class); $accountRepos = $this->mock(AccountRepositoryInterface::class); $journalRepos = $this->mock(JournalRepositoryInterface::class); + $userRepos = $this->mock(UserRepositoryInterface::class); + $journalRepos->shouldReceive('firstNull')->twice()->andReturn(TransactionJournal::first()); + $userRepos->shouldReceive('hasRole')->withArgs([Mockery::any(), 'owner'])->andReturn(true)->atLeast()->once(); $collector->shouldReceive('setAllAssetAccounts')->andReturnSelf(); $collector->shouldReceive('setTypes')->andReturnSelf(); diff --git a/tests/Feature/Controllers/Category/ShowControllerTest.php b/tests/Feature/Controllers/Category/ShowControllerTest.php index b56e8c745a..c436703309 100644 --- a/tests/Feature/Controllers/Category/ShowControllerTest.php +++ b/tests/Feature/Controllers/Category/ShowControllerTest.php @@ -50,7 +50,7 @@ class ShowControllerTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } /** diff --git a/tests/Feature/Controllers/CategoryControllerTest.php b/tests/Feature/Controllers/CategoryControllerTest.php index d187148674..cd300b3675 100644 --- a/tests/Feature/Controllers/CategoryControllerTest.php +++ b/tests/Feature/Controllers/CategoryControllerTest.php @@ -47,7 +47,7 @@ class CategoryControllerTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } /** diff --git a/tests/Feature/Controllers/Chart/AccountControllerTest.php b/tests/Feature/Controllers/Chart/AccountControllerTest.php index 90d1048b98..db7c38faa8 100644 --- a/tests/Feature/Controllers/Chart/AccountControllerTest.php +++ b/tests/Feature/Controllers/Chart/AccountControllerTest.php @@ -52,7 +52,7 @@ class AccountControllerTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } /** diff --git a/tests/Feature/Controllers/Chart/BillControllerTest.php b/tests/Feature/Controllers/Chart/BillControllerTest.php index 918d949843..b5f5d6073f 100644 --- a/tests/Feature/Controllers/Chart/BillControllerTest.php +++ b/tests/Feature/Controllers/Chart/BillControllerTest.php @@ -43,7 +43,7 @@ class BillControllerTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } /** diff --git a/tests/Feature/Controllers/Chart/BudgetControllerTest.php b/tests/Feature/Controllers/Chart/BudgetControllerTest.php index c049a3f990..3511143252 100644 --- a/tests/Feature/Controllers/Chart/BudgetControllerTest.php +++ b/tests/Feature/Controllers/Chart/BudgetControllerTest.php @@ -49,7 +49,7 @@ class BudgetControllerTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } /** @@ -95,7 +95,6 @@ class BudgetControllerTest extends TestCase /** * @covers \FireflyIII\Http\Controllers\Chart\BudgetController - * @expectedExceptionMessage This budget limit is not part of this budget. */ public function testBudgetLimitWrongLimit(): void { diff --git a/tests/Feature/Controllers/Chart/BudgetReportControllerTest.php b/tests/Feature/Controllers/Chart/BudgetReportControllerTest.php index 62dd0dbe31..450f2fa661 100644 --- a/tests/Feature/Controllers/Chart/BudgetReportControllerTest.php +++ b/tests/Feature/Controllers/Chart/BudgetReportControllerTest.php @@ -48,7 +48,7 @@ class BudgetReportControllerTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } /** diff --git a/tests/Feature/Controllers/Chart/CategoryControllerTest.php b/tests/Feature/Controllers/Chart/CategoryControllerTest.php index f15c33c92f..339ebeea69 100644 --- a/tests/Feature/Controllers/Chart/CategoryControllerTest.php +++ b/tests/Feature/Controllers/Chart/CategoryControllerTest.php @@ -45,7 +45,7 @@ class CategoryControllerTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } /** diff --git a/tests/Feature/Controllers/Chart/CategoryReportControllerTest.php b/tests/Feature/Controllers/Chart/CategoryReportControllerTest.php index 0d1c8665d2..5f69146375 100644 --- a/tests/Feature/Controllers/Chart/CategoryReportControllerTest.php +++ b/tests/Feature/Controllers/Chart/CategoryReportControllerTest.php @@ -45,7 +45,7 @@ class CategoryReportControllerTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } /** diff --git a/tests/Feature/Controllers/Chart/ExpenseReportControllerTest.php b/tests/Feature/Controllers/Chart/ExpenseReportControllerTest.php index 9450875577..dd7506f4c8 100644 --- a/tests/Feature/Controllers/Chart/ExpenseReportControllerTest.php +++ b/tests/Feature/Controllers/Chart/ExpenseReportControllerTest.php @@ -42,7 +42,7 @@ class ExpenseReportControllerTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } diff --git a/tests/Feature/Controllers/Chart/PiggyBankControllerTest.php b/tests/Feature/Controllers/Chart/PiggyBankControllerTest.php index 2eff8eeb62..96deea0b5d 100644 --- a/tests/Feature/Controllers/Chart/PiggyBankControllerTest.php +++ b/tests/Feature/Controllers/Chart/PiggyBankControllerTest.php @@ -40,7 +40,7 @@ class PiggyBankControllerTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } /** diff --git a/tests/Feature/Controllers/Chart/ReportControllerTest.php b/tests/Feature/Controllers/Chart/ReportControllerTest.php index ce432479b4..557d14d9a4 100644 --- a/tests/Feature/Controllers/Chart/ReportControllerTest.php +++ b/tests/Feature/Controllers/Chart/ReportControllerTest.php @@ -41,7 +41,7 @@ class ReportControllerTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } /** diff --git a/tests/Feature/Controllers/Chart/TagReportControllerTest.php b/tests/Feature/Controllers/Chart/TagReportControllerTest.php index e576963ec7..4ec22382b1 100644 --- a/tests/Feature/Controllers/Chart/TagReportControllerTest.php +++ b/tests/Feature/Controllers/Chart/TagReportControllerTest.php @@ -49,7 +49,7 @@ class TagReportControllerTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } /** diff --git a/tests/Feature/Controllers/CurrencyControllerTest.php b/tests/Feature/Controllers/CurrencyControllerTest.php index 0011b0885a..6398095951 100644 --- a/tests/Feature/Controllers/CurrencyControllerTest.php +++ b/tests/Feature/Controllers/CurrencyControllerTest.php @@ -48,7 +48,7 @@ class CurrencyControllerTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } /** diff --git a/tests/Feature/Controllers/DebugControllerTest.php b/tests/Feature/Controllers/DebugControllerTest.php index 97b8c54921..0bf188923b 100644 --- a/tests/Feature/Controllers/DebugControllerTest.php +++ b/tests/Feature/Controllers/DebugControllerTest.php @@ -42,7 +42,7 @@ class DebugControllerTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } /** diff --git a/tests/Feature/Controllers/ExportControllerTest.php b/tests/Feature/Controllers/ExportControllerTest.php index 811edad167..5aff7eaccd 100644 --- a/tests/Feature/Controllers/ExportControllerTest.php +++ b/tests/Feature/Controllers/ExportControllerTest.php @@ -49,7 +49,7 @@ class ExportControllerTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } /** @@ -74,7 +74,6 @@ class ExportControllerTest extends TestCase /** * @covers \FireflyIII\Http\Controllers\ExportController - * @expectedExceptionMessage Against all expectations */ public function testDownloadFailed(): void { diff --git a/tests/Feature/Controllers/HelpControllerTest.php b/tests/Feature/Controllers/HelpControllerTest.php index 0647a4963c..10e7ff049c 100644 --- a/tests/Feature/Controllers/HelpControllerTest.php +++ b/tests/Feature/Controllers/HelpControllerTest.php @@ -42,7 +42,7 @@ class HelpControllerTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } /** diff --git a/tests/Feature/Controllers/HomeControllerTest.php b/tests/Feature/Controllers/HomeControllerTest.php index 3eae28dda3..e449fcbbc2 100644 --- a/tests/Feature/Controllers/HomeControllerTest.php +++ b/tests/Feature/Controllers/HomeControllerTest.php @@ -51,7 +51,7 @@ class HomeControllerTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } diff --git a/tests/Feature/Controllers/Import/CallbackControllerTest.php b/tests/Feature/Controllers/Import/CallbackControllerTest.php index 5ef759b7bb..4149135c7c 100644 --- a/tests/Feature/Controllers/Import/CallbackControllerTest.php +++ b/tests/Feature/Controllers/Import/CallbackControllerTest.php @@ -42,7 +42,7 @@ class CallbackControllerTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } /** diff --git a/tests/Feature/Controllers/Import/IndexControllerTest.php b/tests/Feature/Controllers/Import/IndexControllerTest.php index d73fda83a6..21613fa1aa 100644 --- a/tests/Feature/Controllers/Import/IndexControllerTest.php +++ b/tests/Feature/Controllers/Import/IndexControllerTest.php @@ -49,7 +49,7 @@ class IndexControllerTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } /** diff --git a/tests/Feature/Controllers/Import/JobConfigurationControllerTest.php b/tests/Feature/Controllers/Import/JobConfigurationControllerTest.php index 5d70ab1f15..858b245ccc 100644 --- a/tests/Feature/Controllers/Import/JobConfigurationControllerTest.php +++ b/tests/Feature/Controllers/Import/JobConfigurationControllerTest.php @@ -43,7 +43,7 @@ class JobConfigurationControllerTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } /** diff --git a/tests/Feature/Controllers/Import/JobStatusControllerTest.php b/tests/Feature/Controllers/Import/JobStatusControllerTest.php index ffc6a9597b..12ea9a7245 100644 --- a/tests/Feature/Controllers/Import/JobStatusControllerTest.php +++ b/tests/Feature/Controllers/Import/JobStatusControllerTest.php @@ -48,7 +48,7 @@ class JobStatusControllerTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } /** diff --git a/tests/Feature/Controllers/Import/PrerequisitesControllerTest.php b/tests/Feature/Controllers/Import/PrerequisitesControllerTest.php index 0956aea398..692b32719b 100644 --- a/tests/Feature/Controllers/Import/PrerequisitesControllerTest.php +++ b/tests/Feature/Controllers/Import/PrerequisitesControllerTest.php @@ -45,7 +45,7 @@ class PrerequisitesControllerTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } /** diff --git a/tests/Feature/Controllers/JavascriptControllerTest.php b/tests/Feature/Controllers/JavascriptControllerTest.php index cc7eace7b1..d9477845f9 100644 --- a/tests/Feature/Controllers/JavascriptControllerTest.php +++ b/tests/Feature/Controllers/JavascriptControllerTest.php @@ -48,7 +48,7 @@ class JavascriptControllerTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } diff --git a/tests/Feature/Controllers/Json/AutoCompleteControllerTest.php b/tests/Feature/Controllers/Json/AutoCompleteControllerTest.php index d20a3bb54b..727e662f8c 100644 --- a/tests/Feature/Controllers/Json/AutoCompleteControllerTest.php +++ b/tests/Feature/Controllers/Json/AutoCompleteControllerTest.php @@ -55,7 +55,7 @@ class AutoCompleteControllerTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } /** diff --git a/tests/Feature/Controllers/Json/BoxControllerTest.php b/tests/Feature/Controllers/Json/BoxControllerTest.php index 65961f4f07..d4dbc119c0 100644 --- a/tests/Feature/Controllers/Json/BoxControllerTest.php +++ b/tests/Feature/Controllers/Json/BoxControllerTest.php @@ -47,7 +47,7 @@ class BoxControllerTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } /** diff --git a/tests/Feature/Controllers/Json/ExchangeControllerTest.php b/tests/Feature/Controllers/Json/ExchangeControllerTest.php index a3aa4a4da7..dd41332e8d 100644 --- a/tests/Feature/Controllers/Json/ExchangeControllerTest.php +++ b/tests/Feature/Controllers/Json/ExchangeControllerTest.php @@ -43,7 +43,7 @@ class ExchangeControllerTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } /** diff --git a/tests/Feature/Controllers/Json/FrontpageControllerTest.php b/tests/Feature/Controllers/Json/FrontpageControllerTest.php index 88c95fac14..e052ba7c3f 100644 --- a/tests/Feature/Controllers/Json/FrontpageControllerTest.php +++ b/tests/Feature/Controllers/Json/FrontpageControllerTest.php @@ -42,7 +42,7 @@ class FrontpageControllerTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } /** diff --git a/tests/Feature/Controllers/Json/IntroControllerTest.php b/tests/Feature/Controllers/Json/IntroControllerTest.php index 95898309af..228dbd40b8 100644 --- a/tests/Feature/Controllers/Json/IntroControllerTest.php +++ b/tests/Feature/Controllers/Json/IntroControllerTest.php @@ -40,7 +40,7 @@ class IntroControllerTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } /** diff --git a/tests/Feature/Controllers/Json/ReconcileControllerTest.php b/tests/Feature/Controllers/Json/ReconcileControllerTest.php index 7e24f87377..2104ac814c 100644 --- a/tests/Feature/Controllers/Json/ReconcileControllerTest.php +++ b/tests/Feature/Controllers/Json/ReconcileControllerTest.php @@ -44,7 +44,7 @@ class ReconcileControllerTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } /** @@ -74,7 +74,6 @@ class ReconcileControllerTest extends TestCase * Test overview when it's not an asset. * * @covers \FireflyIII\Http\Controllers\Json\ReconcileController - * @expectedExceptionMessage is not an asset account */ public function testOverviewNotAsset(): void { diff --git a/tests/Feature/Controllers/Json/RecurrenceControllerTest.php b/tests/Feature/Controllers/Json/RecurrenceControllerTest.php index 653e7b5486..07f79e7d4e 100644 --- a/tests/Feature/Controllers/Json/RecurrenceControllerTest.php +++ b/tests/Feature/Controllers/Json/RecurrenceControllerTest.php @@ -42,7 +42,7 @@ class RecurrenceControllerTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } /** diff --git a/tests/Feature/Controllers/JsonControllerTest.php b/tests/Feature/Controllers/JsonControllerTest.php index 7432867f1f..2a062aac56 100644 --- a/tests/Feature/Controllers/JsonControllerTest.php +++ b/tests/Feature/Controllers/JsonControllerTest.php @@ -42,7 +42,7 @@ class JsonControllerTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } /** diff --git a/tests/Feature/Controllers/NewUserControllerTest.php b/tests/Feature/Controllers/NewUserControllerTest.php index 9f210f6c78..c6ab76855d 100644 --- a/tests/Feature/Controllers/NewUserControllerTest.php +++ b/tests/Feature/Controllers/NewUserControllerTest.php @@ -45,7 +45,7 @@ class NewUserControllerTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } diff --git a/tests/Feature/Controllers/PiggyBankControllerTest.php b/tests/Feature/Controllers/PiggyBankControllerTest.php index b93f53c098..de7cf3f59b 100644 --- a/tests/Feature/Controllers/PiggyBankControllerTest.php +++ b/tests/Feature/Controllers/PiggyBankControllerTest.php @@ -53,7 +53,7 @@ class PiggyBankControllerTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } diff --git a/tests/Feature/Controllers/Popup/ReportControllerTest.php b/tests/Feature/Controllers/Popup/ReportControllerTest.php index 1ef29be399..0fbeb07b81 100644 --- a/tests/Feature/Controllers/Popup/ReportControllerTest.php +++ b/tests/Feature/Controllers/Popup/ReportControllerTest.php @@ -49,12 +49,11 @@ class ReportControllerTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } /** * @covers \FireflyIII\Http\Controllers\Popup\ReportController - * @expectedExceptionMessage Could not parse end date */ public function testBadEndDate(): void { @@ -168,7 +167,6 @@ class ReportControllerTest extends TestCase /** * @covers \FireflyIII\Http\Controllers\Popup\ReportController - * @expectedExceptionMessage Firefly cannot handle this type of info-button */ public function testBalanceAmountTagRole(): void { diff --git a/tests/Feature/Controllers/PreferencesControllerTest.php b/tests/Feature/Controllers/PreferencesControllerTest.php index 24e020ceff..269003b476 100644 --- a/tests/Feature/Controllers/PreferencesControllerTest.php +++ b/tests/Feature/Controllers/PreferencesControllerTest.php @@ -46,7 +46,7 @@ class PreferencesControllerTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } diff --git a/tests/Feature/Controllers/ProfileControllerTest.php b/tests/Feature/Controllers/ProfileControllerTest.php index 2f686cc0b4..1fd7826b23 100644 --- a/tests/Feature/Controllers/ProfileControllerTest.php +++ b/tests/Feature/Controllers/ProfileControllerTest.php @@ -49,7 +49,7 @@ class ProfileControllerTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } /** @@ -97,7 +97,6 @@ class ProfileControllerTest extends TestCase /** * @covers \FireflyIII\Http\Controllers\ProfileController - * @expectedExceptionMessage Invalid token */ public function testConfirmEmailChangeNoToken(): void { @@ -541,7 +540,6 @@ class ProfileControllerTest extends TestCase /** * @covers \FireflyIII\Http\Controllers\ProfileController - * @expectedExceptionMessage Invalid token */ public function testUndoEmailChangeBadToken(): void { diff --git a/tests/Feature/Controllers/Recurring/CreateControllerTest.php b/tests/Feature/Controllers/Recurring/CreateControllerTest.php index 3fdd63d91a..0604b5b394 100644 --- a/tests/Feature/Controllers/Recurring/CreateControllerTest.php +++ b/tests/Feature/Controllers/Recurring/CreateControllerTest.php @@ -46,7 +46,7 @@ class CreateControllerTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } /** diff --git a/tests/Feature/Controllers/Recurring/DeleteControllerTest.php b/tests/Feature/Controllers/Recurring/DeleteControllerTest.php index fadee7b0d3..b0229a45ff 100644 --- a/tests/Feature/Controllers/Recurring/DeleteControllerTest.php +++ b/tests/Feature/Controllers/Recurring/DeleteControllerTest.php @@ -40,7 +40,7 @@ class DeleteControllerTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } /** diff --git a/tests/Feature/Controllers/Recurring/EditControllerTest.php b/tests/Feature/Controllers/Recurring/EditControllerTest.php index f305c73003..e62331f4c9 100644 --- a/tests/Feature/Controllers/Recurring/EditControllerTest.php +++ b/tests/Feature/Controllers/Recurring/EditControllerTest.php @@ -43,7 +43,7 @@ class EditControllerTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } /** diff --git a/tests/Feature/Controllers/Recurring/IndexControllerTest.php b/tests/Feature/Controllers/Recurring/IndexControllerTest.php index 40f56f5e41..60cc6b1ff5 100644 --- a/tests/Feature/Controllers/Recurring/IndexControllerTest.php +++ b/tests/Feature/Controllers/Recurring/IndexControllerTest.php @@ -41,7 +41,7 @@ class IndexControllerTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } /** diff --git a/tests/Feature/Controllers/Report/AccountControllerTest.php b/tests/Feature/Controllers/Report/AccountControllerTest.php index 965ac4d6c4..eba73720b5 100644 --- a/tests/Feature/Controllers/Report/AccountControllerTest.php +++ b/tests/Feature/Controllers/Report/AccountControllerTest.php @@ -41,7 +41,7 @@ class AccountControllerTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } /** diff --git a/tests/Feature/Controllers/Report/BalanceControllerTest.php b/tests/Feature/Controllers/Report/BalanceControllerTest.php index b6a124508c..255604946c 100644 --- a/tests/Feature/Controllers/Report/BalanceControllerTest.php +++ b/tests/Feature/Controllers/Report/BalanceControllerTest.php @@ -42,7 +42,7 @@ class BalanceControllerTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } diff --git a/tests/Feature/Controllers/Report/BudgetControllerTest.php b/tests/Feature/Controllers/Report/BudgetControllerTest.php index 308a0c487a..a6daac622d 100644 --- a/tests/Feature/Controllers/Report/BudgetControllerTest.php +++ b/tests/Feature/Controllers/Report/BudgetControllerTest.php @@ -43,7 +43,7 @@ class BudgetControllerTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } diff --git a/tests/Feature/Controllers/Report/CategoryControllerTest.php b/tests/Feature/Controllers/Report/CategoryControllerTest.php index b230e00387..6c85d5313c 100644 --- a/tests/Feature/Controllers/Report/CategoryControllerTest.php +++ b/tests/Feature/Controllers/Report/CategoryControllerTest.php @@ -43,7 +43,7 @@ class CategoryControllerTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } diff --git a/tests/Feature/Controllers/Report/ExpenseControllerTest.php b/tests/Feature/Controllers/Report/ExpenseControllerTest.php index 77de4e78e2..1a12a0837f 100644 --- a/tests/Feature/Controllers/Report/ExpenseControllerTest.php +++ b/tests/Feature/Controllers/Report/ExpenseControllerTest.php @@ -45,7 +45,7 @@ class ExpenseControllerTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } diff --git a/tests/Feature/Controllers/Report/OperationsControllerTest.php b/tests/Feature/Controllers/Report/OperationsControllerTest.php index afdcb13fb2..3dbfbd5375 100644 --- a/tests/Feature/Controllers/Report/OperationsControllerTest.php +++ b/tests/Feature/Controllers/Report/OperationsControllerTest.php @@ -41,7 +41,7 @@ class OperationsControllerTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } diff --git a/tests/Feature/Controllers/ReportControllerTest.php b/tests/Feature/Controllers/ReportControllerTest.php index 01559e3075..a6cebb5bf4 100644 --- a/tests/Feature/Controllers/ReportControllerTest.php +++ b/tests/Feature/Controllers/ReportControllerTest.php @@ -59,7 +59,7 @@ class ReportControllerTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } diff --git a/tests/Feature/Controllers/Rule/CreateControllerTest.php b/tests/Feature/Controllers/Rule/CreateControllerTest.php index 62303792c1..231068962b 100644 --- a/tests/Feature/Controllers/Rule/CreateControllerTest.php +++ b/tests/Feature/Controllers/Rule/CreateControllerTest.php @@ -43,7 +43,7 @@ class CreateControllerTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } diff --git a/tests/Feature/Controllers/Rule/DeleteControllerTest.php b/tests/Feature/Controllers/Rule/DeleteControllerTest.php index 6b411384b2..aa43fe6ede 100644 --- a/tests/Feature/Controllers/Rule/DeleteControllerTest.php +++ b/tests/Feature/Controllers/Rule/DeleteControllerTest.php @@ -38,7 +38,7 @@ class DeleteControllerTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } diff --git a/tests/Feature/Controllers/Rule/EditControllerTest.php b/tests/Feature/Controllers/Rule/EditControllerTest.php index 919e8bc454..aeda652d6d 100644 --- a/tests/Feature/Controllers/Rule/EditControllerTest.php +++ b/tests/Feature/Controllers/Rule/EditControllerTest.php @@ -44,7 +44,7 @@ class EditControllerTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } /** diff --git a/tests/Feature/Controllers/Rule/IndexControllerTest.php b/tests/Feature/Controllers/Rule/IndexControllerTest.php index f4fd74fa71..874845a788 100644 --- a/tests/Feature/Controllers/Rule/IndexControllerTest.php +++ b/tests/Feature/Controllers/Rule/IndexControllerTest.php @@ -46,7 +46,7 @@ class IndexControllerTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } diff --git a/tests/Feature/Controllers/Rule/SelectControllerTest.php b/tests/Feature/Controllers/Rule/SelectControllerTest.php index 922bfeb66f..7ffbd07e21 100644 --- a/tests/Feature/Controllers/Rule/SelectControllerTest.php +++ b/tests/Feature/Controllers/Rule/SelectControllerTest.php @@ -49,7 +49,7 @@ class SelectControllerTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } /** diff --git a/tests/Feature/Controllers/RuleGroupControllerTest.php b/tests/Feature/Controllers/RuleGroupControllerTest.php index 086809e696..f581c63b91 100644 --- a/tests/Feature/Controllers/RuleGroupControllerTest.php +++ b/tests/Feature/Controllers/RuleGroupControllerTest.php @@ -48,7 +48,7 @@ class RuleGroupControllerTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } diff --git a/tests/Feature/Controllers/SearchControllerTest.php b/tests/Feature/Controllers/SearchControllerTest.php index f21132e69b..7a8342029e 100644 --- a/tests/Feature/Controllers/SearchControllerTest.php +++ b/tests/Feature/Controllers/SearchControllerTest.php @@ -42,7 +42,7 @@ class SearchControllerTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } diff --git a/tests/Feature/Controllers/System/CronControllerTest.php b/tests/Feature/Controllers/System/CronControllerTest.php index 7a126fbb5a..7e8ce7f48c 100644 --- a/tests/Feature/Controllers/System/CronControllerTest.php +++ b/tests/Feature/Controllers/System/CronControllerTest.php @@ -45,7 +45,7 @@ class CronControllerTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } /** diff --git a/tests/Feature/Controllers/TagControllerTest.php b/tests/Feature/Controllers/TagControllerTest.php index f8eb171824..30ea3b3f45 100644 --- a/tests/Feature/Controllers/TagControllerTest.php +++ b/tests/Feature/Controllers/TagControllerTest.php @@ -48,7 +48,7 @@ class TagControllerTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } diff --git a/tests/Feature/Controllers/Transaction/BulkControllerTest.php b/tests/Feature/Controllers/Transaction/BulkControllerTest.php index 1be187c0a0..a3daf6d36f 100644 --- a/tests/Feature/Controllers/Transaction/BulkControllerTest.php +++ b/tests/Feature/Controllers/Transaction/BulkControllerTest.php @@ -45,7 +45,7 @@ class BulkControllerTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } diff --git a/tests/Feature/Controllers/Transaction/ConvertControllerTest.php b/tests/Feature/Controllers/Transaction/ConvertControllerTest.php index 30e05b014d..7ff1358dd7 100644 --- a/tests/Feature/Controllers/Transaction/ConvertControllerTest.php +++ b/tests/Feature/Controllers/Transaction/ConvertControllerTest.php @@ -50,7 +50,7 @@ class ConvertControllerTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } diff --git a/tests/Feature/Controllers/Transaction/LinkControllerTest.php b/tests/Feature/Controllers/Transaction/LinkControllerTest.php index b0de8c2dd5..0a8d35e2db 100644 --- a/tests/Feature/Controllers/Transaction/LinkControllerTest.php +++ b/tests/Feature/Controllers/Transaction/LinkControllerTest.php @@ -41,7 +41,7 @@ class LinkControllerTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } diff --git a/tests/Feature/Controllers/Transaction/MassControllerTest.php b/tests/Feature/Controllers/Transaction/MassControllerTest.php index c4e457626b..ca4c5ecc8b 100644 --- a/tests/Feature/Controllers/Transaction/MassControllerTest.php +++ b/tests/Feature/Controllers/Transaction/MassControllerTest.php @@ -46,7 +46,7 @@ class MassControllerTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } diff --git a/tests/Feature/Controllers/Transaction/SingleControllerTest.php b/tests/Feature/Controllers/Transaction/SingleControllerTest.php index 3c73122e9d..8ac8ff5c47 100644 --- a/tests/Feature/Controllers/Transaction/SingleControllerTest.php +++ b/tests/Feature/Controllers/Transaction/SingleControllerTest.php @@ -60,7 +60,7 @@ class SingleControllerTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } diff --git a/tests/Feature/Controllers/Transaction/SplitControllerTest.php b/tests/Feature/Controllers/Transaction/SplitControllerTest.php index f8c15229c4..9aa9df666f 100644 --- a/tests/Feature/Controllers/Transaction/SplitControllerTest.php +++ b/tests/Feature/Controllers/Transaction/SplitControllerTest.php @@ -53,7 +53,7 @@ class SplitControllerTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } diff --git a/tests/Feature/Controllers/TransactionControllerTest.php b/tests/Feature/Controllers/TransactionControllerTest.php index b5a1038fee..e1837aa09c 100644 --- a/tests/Feature/Controllers/TransactionControllerTest.php +++ b/tests/Feature/Controllers/TransactionControllerTest.php @@ -49,7 +49,7 @@ class TransactionControllerTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } diff --git a/tests/Unit/Factory/AccountFactoryTest.php b/tests/Unit/Factory/AccountFactoryTest.php index 7354f3c6ea..0eb2b51082 100644 --- a/tests/Unit/Factory/AccountFactoryTest.php +++ b/tests/Unit/Factory/AccountFactoryTest.php @@ -44,7 +44,7 @@ class AccountFactoryTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } /** diff --git a/tests/Unit/Factory/AccountMetaFactoryTest.php b/tests/Unit/Factory/AccountMetaFactoryTest.php index 0ddc5d788f..76221da3c8 100644 --- a/tests/Unit/Factory/AccountMetaFactoryTest.php +++ b/tests/Unit/Factory/AccountMetaFactoryTest.php @@ -42,7 +42,7 @@ class AccountMetaFactoryTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } /** diff --git a/tests/Unit/Factory/AttachmentFactoryTest.php b/tests/Unit/Factory/AttachmentFactoryTest.php index cb5c3abf48..a9f46c22e7 100644 --- a/tests/Unit/Factory/AttachmentFactoryTest.php +++ b/tests/Unit/Factory/AttachmentFactoryTest.php @@ -42,7 +42,7 @@ class AttachmentFactoryTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } /** diff --git a/tests/Unit/Factory/BillFactoryTest.php b/tests/Unit/Factory/BillFactoryTest.php index cb6289eba0..808b54dc41 100644 --- a/tests/Unit/Factory/BillFactoryTest.php +++ b/tests/Unit/Factory/BillFactoryTest.php @@ -41,7 +41,7 @@ class BillFactoryTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } /** diff --git a/tests/Unit/Factory/BudgetFactoryTest.php b/tests/Unit/Factory/BudgetFactoryTest.php index e4fbef4f06..4c6c6d0e33 100644 --- a/tests/Unit/Factory/BudgetFactoryTest.php +++ b/tests/Unit/Factory/BudgetFactoryTest.php @@ -40,7 +40,7 @@ class BudgetFactoryTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } /** diff --git a/tests/Unit/Factory/CategoryFactoryTest.php b/tests/Unit/Factory/CategoryFactoryTest.php index 3c878df809..1b60dd8e2b 100644 --- a/tests/Unit/Factory/CategoryFactoryTest.php +++ b/tests/Unit/Factory/CategoryFactoryTest.php @@ -39,7 +39,7 @@ class CategoryFactoryTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } /** diff --git a/tests/Unit/Factory/PiggyBankEventFactoryTest.php b/tests/Unit/Factory/PiggyBankEventFactoryTest.php index 1d6f5a8af1..8abcae222f 100644 --- a/tests/Unit/Factory/PiggyBankEventFactoryTest.php +++ b/tests/Unit/Factory/PiggyBankEventFactoryTest.php @@ -44,7 +44,7 @@ class PiggyBankEventFactoryTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } /** diff --git a/tests/Unit/Factory/PiggyBankFactoryTest.php b/tests/Unit/Factory/PiggyBankFactoryTest.php index e9e39382cc..872a236236 100644 --- a/tests/Unit/Factory/PiggyBankFactoryTest.php +++ b/tests/Unit/Factory/PiggyBankFactoryTest.php @@ -40,7 +40,7 @@ class PiggyBankFactoryTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } /** diff --git a/tests/Unit/Factory/RecurrenceFactoryTest.php b/tests/Unit/Factory/RecurrenceFactoryTest.php index 46fe2592f6..297ad1b191 100644 --- a/tests/Unit/Factory/RecurrenceFactoryTest.php +++ b/tests/Unit/Factory/RecurrenceFactoryTest.php @@ -50,7 +50,7 @@ class RecurrenceFactoryTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } /** diff --git a/tests/Unit/Factory/TagFactoryTest.php b/tests/Unit/Factory/TagFactoryTest.php index cc4b984de1..9e97d675a8 100644 --- a/tests/Unit/Factory/TagFactoryTest.php +++ b/tests/Unit/Factory/TagFactoryTest.php @@ -40,7 +40,7 @@ class TagFactoryTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } /** diff --git a/tests/Unit/Factory/TransactionCurrencyFactoryTest.php b/tests/Unit/Factory/TransactionCurrencyFactoryTest.php index a0200409f5..d1347baa08 100644 --- a/tests/Unit/Factory/TransactionCurrencyFactoryTest.php +++ b/tests/Unit/Factory/TransactionCurrencyFactoryTest.php @@ -41,7 +41,7 @@ class TransactionCurrencyFactoryTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } /** diff --git a/tests/Unit/Factory/TransactionFactoryTest.php b/tests/Unit/Factory/TransactionFactoryTest.php index ec09a76624..00e098cdb3 100644 --- a/tests/Unit/Factory/TransactionFactoryTest.php +++ b/tests/Unit/Factory/TransactionFactoryTest.php @@ -50,7 +50,7 @@ class TransactionFactoryTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } /** diff --git a/tests/Unit/Factory/TransactionJournalFactoryTest.php b/tests/Unit/Factory/TransactionJournalFactoryTest.php index 4493a97bb3..f17966bb66 100644 --- a/tests/Unit/Factory/TransactionJournalFactoryTest.php +++ b/tests/Unit/Factory/TransactionJournalFactoryTest.php @@ -52,7 +52,7 @@ class TransactionJournalFactoryTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } /** diff --git a/tests/Unit/Factory/TransactionJournalMetaFactoryTest.php b/tests/Unit/Factory/TransactionJournalMetaFactoryTest.php index 8b6b310d7f..460e31e135 100644 --- a/tests/Unit/Factory/TransactionJournalMetaFactoryTest.php +++ b/tests/Unit/Factory/TransactionJournalMetaFactoryTest.php @@ -42,7 +42,7 @@ class TransactionJournalMetaFactoryTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } /** diff --git a/tests/Unit/Factory/TransactionTypeFactoryTest.php b/tests/Unit/Factory/TransactionTypeFactoryTest.php index 7d65d5ed76..93fc3f9acf 100644 --- a/tests/Unit/Factory/TransactionTypeFactoryTest.php +++ b/tests/Unit/Factory/TransactionTypeFactoryTest.php @@ -41,7 +41,7 @@ class TransactionTypeFactoryTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } /** diff --git a/tests/Unit/Generator/Chart/Basic/ChartJsGeneratorTest.php b/tests/Unit/Generator/Chart/Basic/ChartJsGeneratorTest.php index febac12183..e1e9a1be1f 100644 --- a/tests/Unit/Generator/Chart/Basic/ChartJsGeneratorTest.php +++ b/tests/Unit/Generator/Chart/Basic/ChartJsGeneratorTest.php @@ -39,7 +39,7 @@ class ChartJsGeneratorTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } /** diff --git a/tests/Unit/Generator/Report/Audit/MonthReportGeneratorTest.php b/tests/Unit/Generator/Report/Audit/MonthReportGeneratorTest.php index a30241b664..312ce59a66 100644 --- a/tests/Unit/Generator/Report/Audit/MonthReportGeneratorTest.php +++ b/tests/Unit/Generator/Report/Audit/MonthReportGeneratorTest.php @@ -50,7 +50,7 @@ class MonthReportGeneratorTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } /** diff --git a/tests/Unit/Handlers/Events/APIEventHandlerTest.php b/tests/Unit/Handlers/Events/APIEventHandlerTest.php index 8c971f9698..e585107a91 100644 --- a/tests/Unit/Handlers/Events/APIEventHandlerTest.php +++ b/tests/Unit/Handlers/Events/APIEventHandlerTest.php @@ -44,7 +44,7 @@ class APIEventHandlerTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } /** diff --git a/tests/Unit/Handlers/Events/AdminEventHandlerTest.php b/tests/Unit/Handlers/Events/AdminEventHandlerTest.php index b82bcd4c4a..5ec6162d1a 100644 --- a/tests/Unit/Handlers/Events/AdminEventHandlerTest.php +++ b/tests/Unit/Handlers/Events/AdminEventHandlerTest.php @@ -44,7 +44,7 @@ class AdminEventHandlerTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } diff --git a/tests/Unit/Handlers/Events/AutomationHandlerTest.php b/tests/Unit/Handlers/Events/AutomationHandlerTest.php index 16fbfd723b..59ecc5e85a 100644 --- a/tests/Unit/Handlers/Events/AutomationHandlerTest.php +++ b/tests/Unit/Handlers/Events/AutomationHandlerTest.php @@ -46,7 +46,7 @@ class AutomationHandlerTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } diff --git a/tests/Unit/Handlers/Events/StoredJournalEventHandlerTest.php b/tests/Unit/Handlers/Events/StoredJournalEventHandlerTest.php index a91b4dc99b..f2cceb6d72 100644 --- a/tests/Unit/Handlers/Events/StoredJournalEventHandlerTest.php +++ b/tests/Unit/Handlers/Events/StoredJournalEventHandlerTest.php @@ -44,7 +44,7 @@ class StoredJournalEventHandlerTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } /** diff --git a/tests/Unit/Handlers/Events/UpdatedJournalEventHandlerTest.php b/tests/Unit/Handlers/Events/UpdatedJournalEventHandlerTest.php index b7fffc56e2..0cee2e4bd9 100644 --- a/tests/Unit/Handlers/Events/UpdatedJournalEventHandlerTest.php +++ b/tests/Unit/Handlers/Events/UpdatedJournalEventHandlerTest.php @@ -41,7 +41,7 @@ class UpdatedJournalEventHandlerTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } diff --git a/tests/Unit/Handlers/Events/UserEventHandlerTest.php b/tests/Unit/Handlers/Events/UserEventHandlerTest.php index 7b79511aae..cb93f505db 100644 --- a/tests/Unit/Handlers/Events/UserEventHandlerTest.php +++ b/tests/Unit/Handlers/Events/UserEventHandlerTest.php @@ -54,7 +54,7 @@ class UserEventHandlerTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } /** diff --git a/tests/Unit/Handlers/Events/VersionCheckEventHandlerTest.php b/tests/Unit/Handlers/Events/VersionCheckEventHandlerTest.php index c1c9ed4ff8..24c7be88a6 100644 --- a/tests/Unit/Handlers/Events/VersionCheckEventHandlerTest.php +++ b/tests/Unit/Handlers/Events/VersionCheckEventHandlerTest.php @@ -47,7 +47,7 @@ class VersionCheckEventHandlerTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } diff --git a/tests/Unit/Helpers/Attachments/AttachmentHelperTest.php b/tests/Unit/Helpers/Attachments/AttachmentHelperTest.php index 4daf52ac5c..50483d0065 100644 --- a/tests/Unit/Helpers/Attachments/AttachmentHelperTest.php +++ b/tests/Unit/Helpers/Attachments/AttachmentHelperTest.php @@ -46,7 +46,7 @@ class AttachmentHelperTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } /** diff --git a/tests/Unit/Helpers/Chart/MetaPieChartTest.php b/tests/Unit/Helpers/Chart/MetaPieChartTest.php index 45580a5335..eb9fe5ea2f 100644 --- a/tests/Unit/Helpers/Chart/MetaPieChartTest.php +++ b/tests/Unit/Helpers/Chart/MetaPieChartTest.php @@ -52,7 +52,7 @@ class MetaPieChartTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } /** diff --git a/tests/Unit/Helpers/Filter/AmountFilterTest.php b/tests/Unit/Helpers/Filter/AmountFilterTest.php index ae68185e44..9756561116 100644 --- a/tests/Unit/Helpers/Filter/AmountFilterTest.php +++ b/tests/Unit/Helpers/Filter/AmountFilterTest.php @@ -42,7 +42,7 @@ class AmountFilterTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } /** diff --git a/tests/Unit/Helpers/FiscalHelperTest.php b/tests/Unit/Helpers/FiscalHelperTest.php index e8c93a337c..b2819f344f 100644 --- a/tests/Unit/Helpers/FiscalHelperTest.php +++ b/tests/Unit/Helpers/FiscalHelperTest.php @@ -44,7 +44,7 @@ class FiscalHelperTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } /** diff --git a/tests/Unit/Helpers/Help/HelpTest.php b/tests/Unit/Helpers/Help/HelpTest.php index 292effbad3..aa7338f21c 100644 --- a/tests/Unit/Helpers/Help/HelpTest.php +++ b/tests/Unit/Helpers/Help/HelpTest.php @@ -44,7 +44,7 @@ class HelpTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } /** diff --git a/tests/Unit/Helpers/Report/NetWorthTest.php b/tests/Unit/Helpers/Report/NetWorthTest.php index 75b65ec2fd..d1c58e8db8 100644 --- a/tests/Unit/Helpers/Report/NetWorthTest.php +++ b/tests/Unit/Helpers/Report/NetWorthTest.php @@ -48,7 +48,7 @@ class NetWorthTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } /** diff --git a/tests/Unit/TransactionRules/Triggers/FromAccountStartsTest.php b/tests/Unit/TransactionRules/Triggers/FromAccountStartsTest.php index dc8c423d30..36020b23d7 100644 --- a/tests/Unit/TransactionRules/Triggers/FromAccountStartsTest.php +++ b/tests/Unit/TransactionRules/Triggers/FromAccountStartsTest.php @@ -40,7 +40,7 @@ class FromAccountStartsTest extends TestCase public function setUp(): void { parent::setUp(); - Log::debug(sprintf('Now in %s.', \get_class($this))); + Log::info(sprintf('Now in %s.', \get_class($this))); } /**