From 2cb9ac26cd4b5b79fc9c813a76a8a0043ef2fa6c Mon Sep 17 00:00:00 2001
From: snipe <snipe@snipe.net>
Date: Wed, 9 Oct 2024 19:57:04 +0100
Subject: [PATCH] Renamed test

Signed-off-by: snipe <snipe@snipe.net>
---
 tests/Feature/Settings/LdapSettingsTest.php  | 48 ++++++++++++++++++++
 tests/Feature/Settings/LdapSettingsTests.php | 29 ------------
 2 files changed, 48 insertions(+), 29 deletions(-)
 create mode 100644 tests/Feature/Settings/LdapSettingsTest.php
 delete mode 100644 tests/Feature/Settings/LdapSettingsTests.php

diff --git a/tests/Feature/Settings/LdapSettingsTest.php b/tests/Feature/Settings/LdapSettingsTest.php
new file mode 100644
index 000000000..49189c33c
--- /dev/null
+++ b/tests/Feature/Settings/LdapSettingsTest.php
@@ -0,0 +1,48 @@
+<?php
+
+namespace Tests\Feature\Settings;
+
+use Tests\TestCase;
+use App\Models\User;
+
+
+class LdapSettingsTest extends TestCase
+{
+    public function testPermissionRequiredToViewLdapSettings()
+    {
+        $this->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 2cda521da..000000000
--- a/tests/Feature/Settings/LdapSettingsTests.php
+++ /dev/null
@@ -1,29 +0,0 @@
-<?php
-
-namespace Tests\Feature\Settings;
-
-use Tests\TestCase;
-use App\Models\User;
-
-
-class LdapSettingsTests extends TestCase
-{
-    public function testPermissionRequiredToViewLdapSettings()
-    {
-        $this->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');
-    }
-
-}