diff --git a/tests/Feature/Settings/LdapSettingsTest.php b/tests/Feature/Settings/LdapSettingsTest.php new file mode 100644 index 0000000000..49189c33c9 --- /dev/null +++ b/tests/Feature/Settings/LdapSettingsTest.php @@ -0,0 +1,48 @@ +actingAs(User::factory()->create()) + ->get(route('settings.ldap.index')) + ->assertForbidden(); + } + + public function testLdapSettingsCanBeSaved() + { + $response = $this->actingAs(User::factory()->superuser()->create()) + ->post(route('settings.ldap.save', [ + 'ldap_enabled' => 1, + 'ldap_username_field' => 'samaccountName', + 'ldap_filter' => 'uid=', + ])) + ->assertStatus(302) + ->assertValid('ldap_enabled') + ->assertRedirect(route('settings.index')) + ->assertSessionHasNoErrors(); + $this->followRedirects($response)->assertSee('alert-success'); + } + + public function testLdapSettingsAreValidate() + { + $response = $this->actingAs(User::factory()->superuser()->create()) + ->from(route('settings.ldap.index')) + ->post(route('settings.ldap.save', [ + 'ldap_enabled' => 1, + 'ldap_username_field' => 'sAMAccountName', + 'ldap_filter' => '', + ])) + ->assertStatus(302) + ->assertRedirect(route('settings.ldap.index')) + ->assertSessionHasErrors(); + $this->followRedirects($response)->assertSee('alert-danger'); + } + +} diff --git a/tests/Feature/Settings/LdapSettingsTests.php b/tests/Feature/Settings/LdapSettingsTests.php deleted file mode 100644 index 2cda521da1..0000000000 --- a/tests/Feature/Settings/LdapSettingsTests.php +++ /dev/null @@ -1,29 +0,0 @@ -actingAs(User::factory()->create()) - ->get(route('settings.ldap.index')) - ->assertForbidden(); - } - - public function testLdapSettingsCanBeSaved() - { - $response = $this->actingAs(User::factory()->superuser()->create()) - ->post(route('settings.ldap.save', ['ldap_enabled' => 1])) - ->assertStatus(302) - ->assertValid('alert_email') - ->assertRedirect(route('settings.index')) - ->assertSessionHasNoErrors(); - $this->followRedirects($response)->assertSee('alert-success'); - } - -}