silva
/
adrema
2
1
Fork 0
Code Issues 23 Pull Requests 1 Projects 1 Releases 106 Wiki Activity

Add test for login check

This commit is contained in:
Philipp Lang 2023-05-07 21:17:28 +02:00
parent ab53a80c4d
commit 48b1808497
3 changed files with 7 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
app/Http/Middleware/RedirectIfNotInitializedMiddleware.php
View File

@ -11,7 +11,7 @@ class RedirectIfNotInitializedMiddleware
/** /**
* @var array<int, string> * @var array<int, string>
*/ */
public array $dontRedirect = ['initialize.form', 'initialize.store', 'nami.search']; public array $dontRedirect = ['initialize.form', 'initialize.store', 'nami.login-check', 'nami.search'];
/** /**
* Handle an incoming request. * Handle an incoming request.

2
routes/web.php
View File

@ -17,6 +17,7 @@ use App\Dashboard\Actions\IndexAction as DashboardIndexAction;
use App\Efz\ShowEfzDocumentAction; use App\Efz\ShowEfzDocumentAction;
use App\Initialize\Actions\InitializeAction; use App\Initialize\Actions\InitializeAction;
use App\Initialize\Actions\InitializeFormAction; use App\Initialize\Actions\InitializeFormAction;
use App\Initialize\Actions\NamiLoginCheckAction;
use App\Initialize\Actions\NamiSearchAction; use App\Initialize\Actions\NamiSearchAction;
use App\Member\Actions\ExportAction; use App\Member\Actions\ExportAction;
use App\Member\Actions\MemberResyncAction; use App\Member\Actions\MemberResyncAction;
@ -39,6 +40,7 @@ Route::group(['namespace' => 'App\\Http\\Controllers'], function (): void {
Route::group(['middleware' => 'auth:web'], function (): void { Route::group(['middleware' => 'auth:web'], function (): void {
Route::get('/', DashboardIndexAction::class)->name('home'); Route::get('/', DashboardIndexAction::class)->name('home');
Route::post('/nami/login-check', NamiLoginCheckAction::class)->name('nami.login-check');
Route::post('/nami/search', NamiSearchAction::class)->name('nami.search'); Route::post('/nami/search', NamiSearchAction::class)->name('nami.search');
Route::post('/api/member/search', SearchAction::class)->name('member.search'); Route::post('/api/member/search', SearchAction::class)->name('member.search');
Route::get('/initialize', InitializeFormAction::class)->name('initialize.form'); Route::get('/initialize', InitializeFormAction::class)->name('initialize.form');

8
tests/Feature/Initializer/ValidateLoginTest.php
View File

@ -21,7 +21,7 @@ class ValidateLoginTest extends TestCase
{ {
Auth::success(333, 'secret'); Auth::success(333, 'secret');
$this->postJson('/nami-login-check', [ $this->postJson('/nami/login-check', [
'mglnr' => 333, 'mglnr' => 333,
'password' => 'secret', 'password' => 'secret',
])->assertStatus(204); ])->assertStatus(204);
@ -29,7 +29,7 @@ class ValidateLoginTest extends TestCase
public function testItNeedsPasswordAndMglnr(): void public function testItNeedsPasswordAndMglnr(): void
{ {
$this->postJson('/nami-login-check', [ $this->postJson('/nami/login-check', [
'mglnr' => '', 'mglnr' => '',
'password' => '', 'password' => '',
])->assertJsonValidationErrors(['mglnr', 'password']); ])->assertJsonValidationErrors(['mglnr', 'password']);
@ -37,7 +37,7 @@ class ValidateLoginTest extends TestCase
public function testMglnrShouldBeNumeric(): void public function testMglnrShouldBeNumeric(): void
{ {
$this->postJson('/nami-login-check', [ $this->postJson('/nami/login-check', [
'mglnr' => 'aaa', 'mglnr' => 'aaa',
'password' => 'secret', 'password' => 'secret',
])->assertJsonValidationErrors(['mglnr']); ])->assertJsonValidationErrors(['mglnr']);
@ -45,7 +45,7 @@ class ValidateLoginTest extends TestCase
public function testLoginCanFail(): void public function testLoginCanFail(): void
{ {
$this->postJson('/nami-login-check', [ $this->postJson('/nami/login-check', [
'mglnr' => '111', 'mglnr' => '111',
'password' => 'secret', 'password' => 'secret',
])->assertJsonValidationErrors(['nami' => 'NaMi Login fehlgeschlagen.']); ])->assertJsonValidationErrors(['nami' => 'NaMi Login fehlgeschlagen.']);