From 81287602d7f353e5a4543b2d0ed8d2198ab61252 Mon Sep 17 00:00:00 2001 From: James Cole Date: Wed, 20 Jan 2016 16:15:23 +0100 Subject: [PATCH] Updated tests. --- .../Controllers/BudgetControllerTest.php | 128 ++++++++++-------- .../Controllers/CategoryControllerTest.php | 103 +++++++------- .../acceptance/Controllers/ControllerTest.php | 63 --------- 3 files changed, 121 insertions(+), 173 deletions(-) delete mode 100644 tests/acceptance/Controllers/ControllerTest.php diff --git a/tests/acceptance/Controllers/BudgetControllerTest.php b/tests/acceptance/Controllers/BudgetControllerTest.php index bfceaab89d..68ea18b6f0 100644 --- a/tests/acceptance/Controllers/BudgetControllerTest.php +++ b/tests/acceptance/Controllers/BudgetControllerTest.php @@ -15,145 +15,153 @@ class BudgetControllerTest extends TestCase { /** * @covers FireflyIII\Http\Controllers\BudgetController::amount - * @todo Implement testAmount(). */ public function testAmount() { - // Remove the following lines when you implement this test. - $this->markTestIncomplete( - 'This test has not been implemented yet.' - ); + $args = [ + 'amount' => 1200, + '_token' => Session::token(), + ]; + $this->be($this->user()); + + $response = $this->call('POST', '/budgets/amount/1', $args); + $this->assertEquals(200, $response->status()); } /** * @covers FireflyIII\Http\Controllers\BudgetController::create - * @todo Implement testCreate(). */ public function testCreate() { - // Remove the following lines when you implement this test. - $this->markTestIncomplete( - 'This test has not been implemented yet.' - ); + $this->be($this->user()); + $response = $this->call('GET', '/budgets/create'); + $this->assertEquals(200, $response->status()); } /** * @covers FireflyIII\Http\Controllers\BudgetController::delete - * @todo Implement testDelete(). */ public function testDelete() { - // Remove the following lines when you implement this test. - $this->markTestIncomplete( - 'This test has not been implemented yet.' - ); + $this->be($this->user()); + $response = $this->call('GET', '/budgets/delete/1'); + $this->assertEquals(200, $response->status()); } /** * @covers FireflyIII\Http\Controllers\BudgetController::destroy - * @todo Implement testDestroy(). */ public function testDestroy() { - // Remove the following lines when you implement this test. - $this->markTestIncomplete( - 'This test has not been implemented yet.' - ); + $this->be($this->user()); + + $args = [ + '_token' => Session::token(), + ]; + + $this->session(['budgets.delete.url' => 'http://localhost']); + + $response = $this->call('POST', '/budgets/destroy/2', $args); + $this->assertSessionHas('success'); + $this->assertEquals(302, $response->status()); } /** * @covers FireflyIII\Http\Controllers\BudgetController::edit - * @todo Implement testEdit(). */ public function testEdit() { - // Remove the following lines when you implement this test. - $this->markTestIncomplete( - 'This test has not been implemented yet.' - ); + $this->be($this->user()); + $response = $this->call('GET', '/budgets/edit/1'); + $this->assertEquals(200, $response->status()); } /** * @covers FireflyIII\Http\Controllers\BudgetController::index - * @todo Implement testIndex(). */ public function testIndex() { - // Remove the following lines when you implement this test. - $this->markTestIncomplete( - 'This test has not been implemented yet.' - ); + $this->be($this->user()); + $response = $this->call('GET', '/budgets'); + $this->assertEquals(200, $response->status()); } /** * @covers FireflyIII\Http\Controllers\BudgetController::noBudget - * @todo Implement testNoBudget(). */ public function testNoBudget() { - // Remove the following lines when you implement this test. - $this->markTestIncomplete( - 'This test has not been implemented yet.' - ); + $this->be($this->user()); + $response = $this->call('GET', '/budgets/list/noBudget'); + $this->assertEquals(200, $response->status()); } /** * @covers FireflyIII\Http\Controllers\BudgetController::postUpdateIncome - * @todo Implement testPostUpdateIncome(). */ public function testPostUpdateIncome() { - // Remove the following lines when you implement this test. - $this->markTestIncomplete( - 'This test has not been implemented yet.' - ); + $args = [ + 'amount' => 1200, + '_token' => Session::token(), + ]; + $this->be($this->user()); + + $response = $this->call('POST', '/budgets/income', $args); + $this->assertEquals(302, $response->status()); } /** * @covers FireflyIII\Http\Controllers\BudgetController::show - * @todo Implement testShow(). */ public function testShow() { - // Remove the following lines when you implement this test. - $this->markTestIncomplete( - 'This test has not been implemented yet.' - ); + $this->be($this->user()); + $response = $this->call('GET', '/budgets/show/1'); + $this->assertEquals(200, $response->status()); } /** * @covers FireflyIII\Http\Controllers\BudgetController::store - * @todo Implement testStore(). */ public function testStore() { - // Remove the following lines when you implement this test. - $this->markTestIncomplete( - 'This test has not been implemented yet.' - ); + $this->be($this->user()); + $this->session(['budgets.create.url' => 'http://localhost']); + $args = [ + '_token' => Session::token(), + 'name' => 'Some kind of test budget.', + ]; + + $response = $this->call('POST', '/budgets/store', $args); + $this->assertEquals(302, $response->status()); + $this->assertSessionHas('success'); } /** * @covers FireflyIII\Http\Controllers\BudgetController::update - * @todo Implement testUpdate(). */ public function testUpdate() { - // Remove the following lines when you implement this test. - $this->markTestIncomplete( - 'This test has not been implemented yet.' - ); + $this->be($this->user()); + $this->session(['budgets.edit.url' => 'http://localhost']); + $args = [ + '_token' => Session::token(), + 'name' => 'Some kind of test budget.', + ]; + + $response = $this->call('POST', '/budgets/update/1', $args); + $this->assertEquals(302, $response->status()); + $this->assertSessionHas('success'); } /** * @covers FireflyIII\Http\Controllers\BudgetController::updateIncome - * @todo Implement testUpdateIncome(). */ public function testUpdateIncome() { - // Remove the following lines when you implement this test. - $this->markTestIncomplete( - 'This test has not been implemented yet.' - ); + $this->be($this->user()); + $response = $this->call('GET', '/budgets/income'); + $this->assertEquals(200, $response->status()); } } diff --git a/tests/acceptance/Controllers/CategoryControllerTest.php b/tests/acceptance/Controllers/CategoryControllerTest.php index 9ad44bf07c..a9c571321a 100644 --- a/tests/acceptance/Controllers/CategoryControllerTest.php +++ b/tests/acceptance/Controllers/CategoryControllerTest.php @@ -16,121 +16,124 @@ class CategoryControllerTest extends TestCase /** * @covers FireflyIII\Http\Controllers\CategoryController::create - * @todo Implement testCreate(). */ public function testCreate() { - // Remove the following lines when you implement this test. - $this->markTestIncomplete( - 'This test has not been implemented yet.' - ); + $this->be($this->user()); + $response = $this->call('GET', '/categories/create'); + $this->assertEquals(200, $response->status()); } /** * @covers FireflyIII\Http\Controllers\CategoryController::delete - * @todo Implement testDelete(). */ public function testDelete() { - // Remove the following lines when you implement this test. - $this->markTestIncomplete( - 'This test has not been implemented yet.' - ); + $this->be($this->user()); + $response = $this->call('GET', '/categories/delete/1'); + $this->assertEquals(200, $response->status()); } /** * @covers FireflyIII\Http\Controllers\CategoryController::destroy - * @todo Implement testDestroy(). */ public function testDestroy() { - // Remove the following lines when you implement this test. - $this->markTestIncomplete( - 'This test has not been implemented yet.' - ); + $this->be($this->user()); + + $args = [ + '_token' => Session::token(), + ]; + + $this->session(['categories.delete.url' => 'http://localhost']); + + $response = $this->call('POST', '/categories/destroy/2', $args); + $this->assertSessionHas('success'); + $this->assertEquals(302, $response->status()); } /** * @covers FireflyIII\Http\Controllers\CategoryController::edit - * @todo Implement testEdit(). */ public function testEdit() { - // Remove the following lines when you implement this test. - $this->markTestIncomplete( - 'This test has not been implemented yet.' - ); + $this->be($this->user()); + $response = $this->call('GET', '/categories/edit/1'); + $this->assertEquals(200, $response->status()); } /** * @covers FireflyIII\Http\Controllers\CategoryController::index - * @todo Implement testIndex(). */ public function testIndex() { - // Remove the following lines when you implement this test. - $this->markTestIncomplete( - 'This test has not been implemented yet.' - ); + $this->be($this->user()); + $response = $this->call('GET', '/categories'); + $this->assertEquals(200, $response->status()); } /** * @covers FireflyIII\Http\Controllers\CategoryController::noCategory - * @todo Implement testNoCategory(). */ public function testNoCategory() { - // Remove the following lines when you implement this test. - $this->markTestIncomplete( - 'This test has not been implemented yet.' - ); + $this->be($this->user()); + $response = $this->call('GET', '/categories/list/noCategory'); + $this->assertEquals(200, $response->status()); } /** * @covers FireflyIII\Http\Controllers\CategoryController::show - * @todo Implement testShow(). */ public function testShow() { - // Remove the following lines when you implement this test. - $this->markTestIncomplete( - 'This test has not been implemented yet.' - ); + $this->be($this->user()); + $response = $this->call('GET', '/categories/show/1'); + $this->assertEquals(200, $response->status()); + } /** * @covers FireflyIII\Http\Controllers\CategoryController::showWithDate - * @todo Implement testShowWithDate(). */ public function testShowWithDate() { - // Remove the following lines when you implement this test. - $this->markTestIncomplete( - 'This test has not been implemented yet.' - ); + $this->be($this->user()); + $response = $this->call('GET', '/categories/show/1/20150101'); + $this->assertEquals(200, $response->status()); } /** * @covers FireflyIII\Http\Controllers\CategoryController::store - * @todo Implement testStore(). */ public function testStore() { - // Remove the following lines when you implement this test. - $this->markTestIncomplete( - 'This test has not been implemented yet.' - ); + $this->be($this->user()); + $this->session(['categories.create.url' => 'http://localhost']); + $args = [ + '_token' => Session::token(), + 'name' => 'Some kind of test cat.', + ]; + + $response = $this->call('POST', '/categories/store', $args); + $this->assertEquals(302, $response->status()); + $this->assertSessionHas('success'); } /** * @covers FireflyIII\Http\Controllers\CategoryController::update - * @todo Implement testUpdate(). */ public function testUpdate() { - // Remove the following lines when you implement this test. - $this->markTestIncomplete( - 'This test has not been implemented yet.' - ); + $this->be($this->user()); + $this->session(['categories.edit.url' => 'http://localhost']); + $args = [ + '_token' => Session::token(), + 'name' => 'Some kind of test category.', + ]; + + $response = $this->call('POST', '/categories/update/1', $args); + $this->assertEquals(302, $response->status()); + $this->assertSessionHas('success'); } } diff --git a/tests/acceptance/Controllers/ControllerTest.php b/tests/acceptance/Controllers/ControllerTest.php deleted file mode 100644 index 374c3050df..0000000000 --- a/tests/acceptance/Controllers/ControllerTest.php +++ /dev/null @@ -1,63 +0,0 @@ -markTestIncomplete( - 'This test has not been implemented yet.' - ); - } - - /** - * @covers FireflyIII\Http\Controllers\Controller::authorizeForUser - * @todo Implement testAuthorizeForUser(). - */ - public function testAuthorizeForUser() - { - // Remove the following lines when you implement this test. - $this->markTestIncomplete( - 'This test has not been implemented yet.' - ); - } - - /** - * @covers FireflyIII\Http\Controllers\Controller::validate - * @todo Implement testValidate(). - */ - public function testValidate() - { - // Remove the following lines when you implement this test. - $this->markTestIncomplete( - 'This test has not been implemented yet.' - ); - } - - /** - * @covers FireflyIII\Http\Controllers\Controller::validateWithBag - * @todo Implement testValidateWithBag(). - */ - public function testValidateWithBag() - { - // Remove the following lines when you implement this test. - $this->markTestIncomplete( - 'This test has not been implemented yet.' - ); - } -}