Differential D2674 Diff 7738 src/tests/Feature/Controller/AuthAttemptsTest.php

Changeset View

Standalone View

src/tests/Feature/Controller/AuthAttemptsTest.php

This file was added.

				<?php

				namespace Tests\Feature\Controller;

				use App\User;
				use App\AuthAttempt;
				use Tests\TestCase;

				class AuthAttemptsTest extends TestCase
				{
				/**
				* {@inheritDoc}
				*/
				public function setUp(): void
				{
				parent::setUp();

				$this->deleteTestUser('UsersControllerTest1@userscontroller.com');
				$this->deleteTestDomain('userscontroller.com');
				}

				/**
				* {@inheritDoc}
				*/
				public function tearDown(): void
				{
				$this->deleteTestUser('UsersControllerTest1@userscontroller.com');
				$this->deleteTestDomain('userscontroller.com');

				parent::tearDown();
				}

				public function testRecord(): void
				{
				machniakUnsubmitted Done Inline Actions This is not a controller test. Should be in src/tests/Feature/AuthAttemptTest.php, imo. machniak: This is not a controller test. Should be in src/tests/Feature/AuthAttemptTest.php, imo.
				$user = $this->getTestUser('UsersControllerTest1@userscontroller.com');
				$authAttempt = \App\AuthAttempt::recordAuthAttempt($user, "10.0.0.1");
				$this->assertEquals($authAttempt->user_id, $user->id);
				$this->assertEquals($authAttempt->ip, "10.0.0.1");
				$authAttempt->refresh();
				$this->assertEquals($authAttempt->status, "NEW");

				$authAttempt2 = \App\AuthAttempt::recordAuthAttempt($user, "10.0.0.1");
				$this->assertEquals($authAttempt->id, $authAttempt2->id);

				$authAttempt3 = \App\AuthAttempt::recordAuthAttempt($user, "10.0.0.2");
				$this->assertNotEquals($authAttempt->id, $authAttempt3->id);
				}


				public function testAcceptDeny(): void
				{
				machniakUnsubmitted Done Inline Actions I used to comment controller tests with something like this: /** * Test domain config update (POST /api/v4/domains/<domain>/config) / machniak:* I used to comment controller tests with something like this: ``` /** * Test domain config…
				$user = $this->getTestUser('UsersControllerTest1@userscontroller.com');
				$authAttempt = \App\AuthAttempt::recordAuthAttempt($user, "10.0.0.1");

				$response = $this->actingAs($user)->post("api/v4/auth-attempts/{$authAttempt->id}/confirm");
				$response->assertStatus(200);
				$authAttempt->refresh();
				$this->assertTrue($authAttempt->isAccepted());

				$response = $this->actingAs($user)->post("api/v4/auth-attempts/{$authAttempt->id}/deny");
				$response->assertStatus(200);
				$authAttempt->refresh();
				$this->assertTrue($authAttempt->isDenied());
				}


				public function testDetails(): void
				{
				$user = $this->getTestUser('UsersControllerTest1@userscontroller.com');
				$authAttempt = \App\AuthAttempt::recordAuthAttempt($user, "10.0.0.1");

				$response = $this->actingAs($user)->get("api/v4/auth-attempts/{$authAttempt->id}/details");
				$response->assertStatus(200);

				machniakUnsubmitted Done Inline Actions All controller tests should include tests for all 403/404 conditions. machniak: All controller tests should include tests for all 403/404 conditions.
				$json = $response->json();

				$authAttempt->refresh();

				$this->assertEquals($user->email, $json['username']);
				$this->assertEquals($authAttempt->ip, $json['ip']);
				$this->assertEquals(json_encode($authAttempt->updated_at), "\"" . $json['timestamp'] . "\"");
				$this->assertEquals("CH", $json['country']);
				}


				public function testList(): void
				{
				$user = $this->getTestUser('UsersControllerTest1@userscontroller.com');
				$authAttempt = \App\AuthAttempt::recordAuthAttempt($user, "10.0.0.1");
				$authAttempt2 = \App\AuthAttempt::recordAuthAttempt($user, "10.0.0.2");

				$response = $this->actingAs($user)->get("api/v4/auth-attempts");
				$response->assertStatus(200);

				$json = $response->json();

				/* var_export($json); */

				machniakUnsubmitted Done Inline Actions This can be removed. machniak: This can be removed.
				$this->assertEquals(count($json), 2);
				$this->assertEquals($json[0]['id'], $authAttempt->id);
				machniakUnsubmitted Done Inline Actions This can be `$this->assertCount(2, $json);` machniak: This can be `$this->assertCount(2, $json);`
				$this->assertEquals($json[1]['id'], $authAttempt2->id);
				machniakUnsubmitted Done Inline Actions This assertion sometimes fail. It's because there's `orderBy('updated_at', 'desc')`, so I think the order of assertions needs to be reverted here. machniak: This assertion sometimes fail. It's because there's `orderBy('updated_at', 'desc')`, so I think…
				mollekopfAuthorUnsubmitted Done Inline Actions The timestamp is sometimes the same, so I changed it so the order doesn't matter). mollekopf: The timestamp is sometimes the same, so I changed it so the order doesn't matter).
				}
				}
				machniakUnsubmitted Done Inline Actions This however will also pass if $json[0][id] == $json[1][id] ;) so, it could be more precise. machniak: This however will also pass if $json[0][id] == $json[1][id] ;) so, it could be more precise.