Added tests for user crud

diff --git a/resources/assets/sass/styles.scss b/resources/assets/sass/styles.scss
index d652fbedcc791e026280d0783476f928195771fe..da08ba4571256e7131325574550862a461252e81 100644 (file)
--- a/resources/assets/sass/styles.scss
+++ b/resources/assets/sass/styles.scss
@@ -30,6 +30,11 @@ body.dragging, body.dragging * {
 // User Avatar Images
 .avatar {
   border-radius: 100%;
+  background-color: #EEE;
+  &.med {
+    width: 40px;
+    height: 40px;
+  }
 }
 
 // System wide notifications

diff --git a/resources/views/users/edit.blade.php b/resources/views/users/edit.blade.php
index 2c3c6079f883943515fa65791b31a31c8929a357..5e28059fedf62b4d8a596cbfa1966cbf1ec85e07 100644 (file)
--- a/resources/views/users/edit.blade.php
+++ b/resources/views/users/edit.blade.php
@@ -9,7 +9,7 @@
                 <div class="col-md-6"></div>
                 <div class="col-md-6 faded">
                     <div class="action-buttons">
-                        <a href="/users/{{$user->id}}/delete" class="text-neg text-button"><i class="zmdi zmdi-delete"></i>Delete User</a>
+                        <a href="/users/{{$user->id}}/delete" class="text-neg text-button"><i class="zmdi zmdi-delete"></i>Delete user</a>
                     </div>
                 </div>
             </div>

diff --git a/resources/views/users/index.blade.php b/resources/views/users/index.blade.php
index 49e124e56e53ab7976c566bee6a14a2babded4ca..6df242f10699cf1881fb3f67953553f55fe7fbb8 100644 (file)
--- a/resources/views/users/index.blade.php
+++ b/resources/views/users/index.blade.php
@@ -10,7 +10,7 @@
         <h1>Users</h1>
         @if($currentUser->can('user-create'))
             <p>
-                <a href="/users/create" class="text-pos"><i class="zmdi zmdi-account-add"></i>Add New User</a>
+                <a href="/users/create" class="text-pos"><i class="zmdi zmdi-account-add"></i>Add new user</a>
             </p>
         @endif
         <table class="table">
@@ -22,7 +22,7 @@
             </tr>
             @foreach($users as $user)
                 <tr>
-                    <td style="line-height: 0;"><img class="avatar" src="{{$user->getAvatar(40)}}" alt="{{$user->name}}"></td>
+                    <td style="line-height: 0;"><img class="avatar med" src="{{$user->getAvatar(40)}}" alt="{{$user->name}}"></td>
                     <td>
                         @if($currentUser->can('user-update') || $currentUser->id == $user->id)
                             <a href="/users/{{$user->id}}">

diff --git a/tests/AuthTest.php b/tests/AuthTest.php
index 1c5628d9c83f1c08ba343cee23f3e5ace67bdaba..befa0214d3d781a520f0bb5e50745d408f4d2ab1 100644 (file)
--- a/tests/AuthTest.php
+++ b/tests/AuthTest.php
@@ -102,6 +102,47 @@ class AuthTest extends TestCase
             ->seeInDatabase('users', ['name' => $user->name, 'email' => $user->email, 'email_confirmed' => true]);
     }
 
+    public function testUserControl()
+    {
+        $user = factory(\BookStack\User::class)->make();
+        // Test creation
+        $this->asAdmin()
+            ->visit('/users')
+            ->click('Add new user')
+            ->type($user->name, '#name')
+            ->type($user->email, '#email')
+            ->select(2, '#role')
+            ->type($user->password, '#password')
+            ->type($user->password, '#password-confirm')
+            ->press('Save')
+            ->seeInDatabase('users', $user->toArray())
+            ->seePageIs('/users')
+            ->see($user->name);
+        $user = $user->where('email', '=', $user->email)->first();
+
+        // Test editing
+        $this->asAdmin()
+            ->visit('/users')
+            ->click($user->name)
+            ->seePageIs('/users/' . $user->id)
+            ->see($user->email)
+            ->type('Barry Scott', '#name')
+            ->press('Save')
+            ->seePageIs('/users')
+            ->seeInDatabase('users', ['id' => $user->id, 'name' => 'Barry Scott'])
+            ->notSeeInDatabase('users', ['name' => $user->name]);
+        $user = $user->find($user->id);
+
+        // Test Deletion
+        $this->asAdmin()
+            ->visit('/users/' . $user->id)
+            ->click('Delete user')
+            ->see($user->name)
+            ->press('Confirm')
+            ->seePageIs('/users')
+            ->notSeeInDatabase('users', ['name' => $user->name]);
+    }
+
     public function testLogout()
     {
         $this->asAdmin()