Updated translations.

This commit is contained in:
James Cole
2019-08-12 17:12:11 +02:00
parent 6e78d4efa7
commit 76dad84ba6
10 changed files with 76 additions and 278 deletions

View File

@@ -309,206 +309,4 @@ class MetaPieChartTest extends TestCase
$this->assertTrue(true);
}
// /**
// * @covers \FireflyIII\Helpers\Chart\MetaPieChart
// */
// public function testGenerateExpenseAccountWithOthers(): void
// {
// $this->markTestIncomplete('Needs to be rewritten for v4.8.0');
//
// return;
// $som = (new Carbon())->startOfMonth();
// $eom = (new Carbon())->endOfMonth();
// $collection = $this->fakeTransactions();
// $others = $this->fakeOthers();
// $accounts = [
// 1 => factory(Account::class)->make(),
// 2 => factory(Account::class)->make(),
// ];
//
// // mock collector so the correct set of journals is returned:
// // then verify the results.
// $collector = $this->mock(TransactionCollectorInterface::class);
// $collector->shouldReceive('addFilter')->withArgs([NegativeAmountFilter::class])->andReturnSelf()->once();
// $collector->shouldReceive('addFilter')->withArgs([PositiveAmountFilter::class])->andReturnSelf()->once();
// $collector->shouldReceive('addFilter')->withArgs([OpposingAccountFilter::class])->andReturnSelf()->once();
// $collector->shouldReceive('removeFilter')->withArgs([NegativeAmountFilter::class])->andReturnSelf()->once();
// $collector->shouldReceive('setUser')->andReturnSelf()->twice();
// $collector->shouldReceive('setAccounts')->andReturnSelf()->twice();
// $collector->shouldReceive('setRange')->andReturnSelf()->twice();
// $collector->shouldReceive('setBudgets')->andReturnSelf()->once();
// $collector->shouldReceive('setCategories')->andReturnSelf()->once();
// $collector->shouldReceive('setTypes')->withArgs([[TransactionType::WITHDRAWAL, TransactionType::TRANSFER]])->andReturnSelf()->once();
// $collector->shouldReceive('withOpposingAccount')->andReturnSelf()->once();
// $collector->shouldReceive('getTransactions')->andReturn($collection)->once();
//
// $collector->shouldReceive('setTypes')->withArgs([[TransactionType::WITHDRAWAL]])->andReturnSelf()->once();
// $collector->shouldReceive('getTransactions')->andReturn($others)->once();
//
// // mock all repositories:
// $accountRepos = $this->mock(AccountRepositoryInterface::class);
//
// $accountRepos->shouldReceive('setUser');
// $accountRepos->shouldReceive('findNull')->withArgs([1])->andReturn($accounts[1]);
// $accountRepos->shouldReceive('findNull')->withArgs([2])->andReturn($accounts[2]);
//
// $helper = new MetaPieChart();
// $helper->setCollectOtherObjects(true);
// $helper->setUser($this->user());
// $helper->setStart($som);
// $helper->setEnd($eom);
// $chart = $helper->generate('expense', 'account');
//
// // since the set is pretty basic the result is easy to validate:
// $keys = array_keys($chart);
// $this->assertEquals($keys[0], $accounts[1]->name);
// $this->assertEquals($keys[1], $accounts[2]->name);
// $this->assertSame(0, bccomp('1000', $chart[$accounts[1]->name]));
// $this->assertSame(0, bccomp('1000', $chart[$accounts[2]->name]));
// $this->assertSame(0, bccomp('-5000', $chart['Everything else']));
//
// $this->assertTrue(true);
// }
//
//
// /**
// * @covers \FireflyIII\Helpers\Chart\MetaPieChart
// */
// public function testGenerateIncomeAccount(): void
// {
// $this->markTestIncomplete('Needs to be rewritten for v4.8.0');
//
// return;
// $som = (new Carbon())->startOfMonth();
// $eom = (new Carbon())->endOfMonth();
// $collection = $this->fakeTransactions();
// $accounts = [
// 1 => factory(Account::class)->make(),
// 2 => factory(Account::class)->make(),
// ];
//
// // mock collector so the correct set of journals is returned:
// // then verify the results.
// $collector = $this->mock(TransactionCollectorInterface::class);
//
// $collector->shouldReceive('addFilter')->withArgs([NegativeAmountFilter::class])->andReturnSelf()->once();
// $collector->shouldReceive('addFilter')->withArgs([OpposingAccountFilter::class])->andReturnSelf()->once();
// $collector->shouldReceive('removeFilter')->withArgs([TransferFilter::class])->andReturnSelf()->once();
// $collector->shouldReceive('setUser')->andReturnSelf()->once();
// $collector->shouldReceive('setAccounts')->andReturnSelf()->once();
// $collector->shouldReceive('setRange')->andReturnSelf()->once();
// $collector->shouldReceive('setBudgets')->andReturnSelf()->once();
// $collector->shouldReceive('setCategories')->andReturnSelf()->once();
// $collector->shouldReceive('setTypes')->withArgs([[TransactionType::DEPOSIT, TransactionType::TRANSFER]])->andReturnSelf()->once();
// $collector->shouldReceive('withOpposingAccount')->andReturnSelf()->once();
// $collector->shouldReceive('getTransactions')->andReturn($collection);
//
// // mock all repositories:
// $accountRepos = $this->mock(AccountRepositoryInterface::class);
//
// $accountRepos->shouldReceive('setUser');
// $accountRepos->shouldReceive('findNull')->withArgs([1])->andReturn($accounts[1]);
// $accountRepos->shouldReceive('findNull')->withArgs([2])->andReturn($accounts[2]);
//
// $helper = new MetaPieChart();
// $helper->setUser($this->user());
// $helper->setStart($som);
// $helper->setEnd($eom);
// $chart = $helper->generate('income', 'account');
//
// // since the set is pretty basic the result is easy to validate:
// $keys = array_keys($chart);
// $this->assertEquals($keys[0], $accounts[1]->name);
// $this->assertEquals($keys[1], $accounts[2]->name);
// $this->assertSame(0, bccomp('1000', $chart[$accounts[1]->name]));
// $this->assertSame(0, bccomp('1000', $chart[$accounts[2]->name]));
//
// $this->assertTrue(true);
// }
//
// /**
// * @covers \FireflyIII\Helpers\Chart\MetaPieChart
// */
// public function testGenerateIncomeAccountWithOthers(): void
// {
// $this->markTestIncomplete('Needs to be rewritten for v4.8.0');
//
// return;
// $som = (new Carbon())->startOfMonth();
// $eom = (new Carbon())->endOfMonth();
// $collection = $this->fakeTransactions();
// $others = $this->fakeOthers();
// $accounts = [
// 1 => factory(Account::class)->make(),
// 2 => factory(Account::class)->make(),
// ];
//
// // mock collector so the correct set of journals is returned:
// // then verify the results.
// $collector = $this->mock(TransactionCollectorInterface::class);
// $collector->shouldReceive('addFilter')->withArgs([NegativeAmountFilter::class])->andReturnSelf()->once();
// $collector->shouldReceive('addFilter')->withArgs([OpposingAccountFilter::class])->andReturnSelf()->once();
// $collector->shouldReceive('removeFilter')->withArgs([TransferFilter::class])->andReturnSelf()->once();
// $collector->shouldReceive('setUser')->andReturnSelf()->twice();
// $collector->shouldReceive('setAccounts')->andReturnSelf()->twice();
// $collector->shouldReceive('setRange')->andReturnSelf()->twice();
// $collector->shouldReceive('setBudgets')->andReturnSelf()->once();
// $collector->shouldReceive('setCategories')->andReturnSelf()->once();
// $collector->shouldReceive('setTypes')->withArgs([[TransactionType::DEPOSIT, TransactionType::TRANSFER]])->andReturnSelf()->once();
// $collector->shouldReceive('withOpposingAccount')->andReturnSelf()->once();
// $collector->shouldReceive('getTransactions')->andReturn($collection)->once();
//
// $collector->shouldReceive('setTypes')->withArgs([[TransactionType::DEPOSIT]])->andReturnSelf()->once();
// $collector->shouldReceive('getTransactions')->andReturn($others)->once();
//
// // mock all repositories:
// $accountRepos = $this->mock(AccountRepositoryInterface::class);
//
// $accountRepos->shouldReceive('setUser');
// $accountRepos->shouldReceive('findNull')->withArgs([1])->andReturn($accounts[1]);
// $accountRepos->shouldReceive('findNull')->withArgs([2])->andReturn($accounts[2]);
//
// $helper = new MetaPieChart();
// $helper->setCollectOtherObjects(true);
// $helper->setUser($this->user());
// $helper->setStart($som);
// $helper->setEnd($eom);
// $chart = $helper->generate('income', 'account');
//
// // since the set is pretty basic the result is easy to validate:
// $keys = array_keys($chart);
// $this->assertEquals($keys[0], $accounts[1]->name);
// $this->assertEquals($keys[1], $accounts[2]->name);
// $this->assertSame(0, bccomp('1000', $chart[$accounts[1]->name]));
// $this->assertSame(0, bccomp('1000', $chart[$accounts[2]->name]));
// $this->assertSame(0, bccomp('1000', $chart['Everything else']));
//
// $this->assertTrue(true);
// }
//
// /**
// * @return Collection
// */
// private function fakeOthers(): Collection
// {
// $set = new Collection;
//
// for ($i = 0; $i < 30; ++$i) {
// $transaction = new Transaction;
//
// // basic fields.
// $transaction->opposing_account_id = 3;
// $transaction->transaction_journal_budget_id = 3;
// $transaction->transaction_budget_id = 3;
// $transaction->transaction_journal_category_id = 3;
// $transaction->transaction_category_id = 3;
// $transaction->transaction_amount = '100';
// $set->push($transaction);
// }
//
// return $set;
// }
}