]> BookStack Code Mirror - bookstack/commitdiff
Preserve original display_name_attribute configuration values. 1317/head
authorDaniel Fanara <redacted>
Sat, 9 Mar 2019 06:13:30 +0000 (01:13 -0500)
committerDaniel Fanara <redacted>
Sat, 9 Mar 2019 06:13:30 +0000 (01:13 -0500)
tests/Auth/LdapTest.php

index d23c9ec84538bff9e00caf813a5323e332149bd4..54cf84266e2109838982ccccf4d4967649290252 100644 (file)
@@ -375,6 +375,7 @@ class LdapTest extends BrowserKitTest
 
     public function test_login_uses_specified_display_name_attribute()
     {
+        $originalAttribute = config('services.ldap.display_name_attribute');
         app('config')->set([
             'services.ldap.display_name_attribute' => 'displayName'
         ]);
@@ -407,12 +408,13 @@ class LdapTest extends BrowserKitTest
             ->seeInDatabase('users', ['email' => $this->mockUser->email, 'email_confirmed' => false, 'external_auth_id' => $this->mockUser->name, 'name' => 'displayNameAttribute']);
         
         app('config')->set([
-            'services.ldap.display_name_attribute' => 'cn'
+            'services.ldap.display_name_attribute' => $originalAttribute
         ]);
     }
 
     public function test_login_uses_default_display_name_attribute_if_specified_not_present()
     {
+        $originalAttribute = config('services.ldap.display_name_attribute');
         app('config')->set([
             'services.ldap.display_name_attribute' => 'displayName'
         ]);
@@ -444,7 +446,7 @@ class LdapTest extends BrowserKitTest
             ->seeInDatabase('users', ['email' => $this->mockUser->email, 'email_confirmed' => false, 'external_auth_id' => $this->mockUser->name, 'name' => $this->mockUser->name]);
     
         app('config')->set([
-            'services.ldap.display_name_attribute' => 'cn'
+            'services.ldap.display_name_attribute' => $originalAttribute
         ]);
     }