Merge pull request #66 from nafiesl/restructure_user_edit_page

Restructure user edit page

app/Http/Requests/Users/UpdateRequest.php

@@ -26,9 +26,9 @@ class UpdateRequest extends FormRequest
     public function rules()
     {
         return [
-            'nickname'    => 'required|string|max:255',
-            'name'        => 'required|string|max:255',
-            'gender_id'   => 'required|numeric',
+            'nickname'    => 'sometimes|required|string|max:255',
+            'name'        => 'sometimes|required|string|max:255',
+            'gender_id'   => 'sometimes|required|numeric',
             'dob'         => 'nullable|date|date_format:Y-m-d',
             'yob'         => 'nullable|date_format:Y',
             'dod'         => 'nullable|date|date_format:Y-m-d',
@@ -54,19 +54,10 @@ class UpdateRequest extends FormRequest
     {
         $formData = parent::validated();
 
-        if ($formData['dod']) {
-            $formData['yod'] = substr($formData['dod'], 0, 4);
-        } else {
-            $formData['yod'] = $formData['yod'];
-        }
-
-        if ($formData['dob']) {
-            $formData['yob'] = substr($formData['dob'], 0, 4);
-        } else {
-            $formData['yob'] = $formData['yob'];
-        }
+        $formData['yod'] = $this->getYod($formData);
+        $formData['yob'] = $this->getYob($formData);
 
-        if ($formData['password']) {
+        if (isset($formData['password']) && $formData['password']) {
             $formData['password'] = bcrypt($formData['password']);
         } else {
             unset($formData['password']);
@@ -74,4 +65,30 @@ class UpdateRequest extends FormRequest
 
         return $formData;
     }
+
+    private function getYob($formData)
+    {
+        if (isset($formData['yob'])) {
+            return $formData['yob'];
+        }
+
+        if (isset($formData['dob']) && $formData['dob']) {
+            return substr($formData['dob'], 0, 4);
+        }
+
+        return;
+    }
+
+    private function getYod($formData)
+    {
+        if (isset($formData['yod'])) {
+            return $formData['yod'];
+        }
+
+        if (isset($formData['dod']) && $formData['dod']) {
+            return substr($formData['dod'], 0, 4);
+        }
+
+        return;
+    }
 }

resources/lang/en/user.php

@@ -11,6 +11,7 @@ return [
     'grand_childs' => 'Grand Childs',
     'siblings'     => 'Siblings',
     'dead'         => 'Dead',
+    'death'        => 'Death',
     'child_name'   => 'Child Name',
     'child_gender' => 'Child Gender',
     'grand_mother' => 'Grand Mother',

resources/lang/id/user.php

@@ -11,6 +11,7 @@ return [
     'grand_childs' => 'Cucu-cucu',
     'siblings'     => 'Saudara',
     'dead'         => 'Meninggal',
+    'death'        => 'Kematian',
     'child_name'   => 'Nama Anak',
     'child_gender' => 'Jenis Kelamin Anak',
     'grand_mother' => 'Nenek',

resources/views/users/edit.blade.php

@@ -5,48 +5,7 @@
     @can('delete', $user)
         <div class="row">
             <div class="col-md-6 col-md-offset-3">
-                <div class="panel panel-default">
-                    <div class="panel-heading"><h3 class="panel-title">{{ __('user.delete') }} : {{ $user->name }}</h3></div>
-                    <div class="panel-body">
-                        <table class="table table-condensed">
-                            <tr><td>{{ __('user.name') }}</td><td>{{ $user->name }}</td></tr>
-                            <tr><td>{{ __('user.nickname') }}</td><td>{{ $user->nickname }}</td></tr>
-                            <tr><td>{{ __('user.gender') }}</td><td>{{ $user->gender }}</td></tr>
-                            <tr><td>{{ __('user.father') }}</td><td>{{ $user->father_id ? $user->father->name : '' }}</td></tr>
-                            <tr><td>{{ __('user.mother') }}</td><td>{{ $user->mother_id ? $user->mother->name : '' }}</td></tr>
-                            <tr><td>{{ __('user.childs_count') }}</td><td>{{ $childsCount = $user->childs()->count() }}</td></tr>
-                            <tr><td>{{ __('user.spouses_count') }}</td><td>{{ $spousesCount = $user->marriages()->count() }}</td></tr>
-                            <tr><td>{{ __('user.managed_user') }}</td><td>{{ $managedUserCount = $user->managedUsers()->count() }}</td></tr>
-                            <tr><td>{{ __('user.managed_couple') }}</td><td>{{ $managedCoupleCount = $user->managedCouples()->count() }}</td></tr>
-                        </table>
-                        @if ($childsCount + $spousesCount + $managedUserCount + $managedCoupleCount)
-                            {{ __('user.replace_delete_text') }}
-                            {{ Form::open([
-                                'route' => ['users.destroy', $user],
-                                'method' => 'delete',
-                                'onsubmit' => 'return confirm("'.__('user.replace_confirm').'")',
-                            ]) }}
-                            {!! FormField::select('replacement_user_id', $replacementUsers, [
-                                'label' => false,
-                                'placeholder' => __('user.replacement'),
-                            ]) !!}
-                            {{ Form::submit(__('user.replace_delete_button'), [
-                                'name' => 'replace_delete_button',
-                                'class' => 'btn btn-danger',
-                            ]) }}
-                            {{ link_to_route('users.edit', __('app.cancel'), [$user], ['class' => 'btn btn-default pull-right']) }}
-                            {{ Form::close() }}
-                        @else
-                            {!! FormField::delete(
-                                ['route' => ['users.destroy', $user]],
-                                __('user.delete_confirm_button'),
-                                ['class' => 'btn btn-danger'],
-                                ['user_id' => $user->id]
-                            ) !!}
-                            {{ link_to_route('users.edit', __('app.cancel'), [$user], ['class' => 'btn btn-default']) }}
-                        @endif
-                    </div>
-                </div>
+                @include('users.partials.delete_confirmation')
             </div>
         </div>
     @endcan
@@ -58,71 +17,25 @@
         {{ __('user.edit') }} {{ $user->profileLink() }}
     </h2>
     <div class="row">
-        {{ Form::model($user, ['route' => ['users.update', $user->id], 'method' =>'patch', 'autocomplete' => 'off']) }}
-        <div class="col-md-4">
-            <div class="panel panel-default">
-                <div class="panel-heading"><h3 class="panel-title">{{ __('user.edit') }}</h3></div>
-                <div class="panel-body">
-                    {!! FormField::text('name', ['label' => __('user.name')]) !!}
-                    {!! FormField::text('nickname', ['label' => __('user.nickname')]) !!}
-                    <div class="row">
-                        <div class="col-md-6">{!! FormField::radios('gender_id', [1 => __('app.male_code'), __('app.female_code')], ['label' => __('user.gender')]) !!}</div>
-                        <div class="col-md-4">
-                            {!! FormField::text('birth_order', ['label' => __('user.birth_order'), 'type' => 'number', 'min' => 1]) !!}
-                        </div>
-                    </div>
-                    <div class="row">
-                        <div class="col-md-6">{!! FormField::text('yob', ['label' => __('user.yob'), 'placeholder' => __('app.example').' 1959']) !!}</div>
-                        <div class="col-md-6">{!! FormField::text('dob', ['label' => __('user.dob'), 'placeholder' => __('app.example').' 1959-07-20']) !!}</div>
-                    </div>
-                    <div class="row">
-                        <div class="col-md-6">{!! FormField::text('yod', ['label' => __('user.yod'), 'placeholder' => __('app.example').' 2003']) !!}</div>
-                        <div class="col-md-6">{!! FormField::text('dod', ['label' => __('user.dod'), 'placeholder' => __('app.example').' 2003-10-17']) !!}</div>
+        <div class="col-md-2">@include('users.partials.edit_nav_tabs')</div>
+        <div class="col-md-10">
+            <div class="row">
+                {{ Form::model($user, ['route' => ['users.update', $user->id], 'method' =>'patch', 'autocomplete' => 'off']) }}
+                <div class="col-md-6">
+                    @includeWhen(request('tab') == null, 'users.partials.edit_profile')
+                    @includeWhen(request('tab') == 'death', 'users.partials.edit_death')
+                    @includeWhen(request('tab') == 'contact_address', 'users.partials.edit_contact_address')
+                    @includeWhen(request('tab') == 'login_account', 'users.partials.edit_login_account')
+                    <div class="text-right">
+                        {{ Form::submit(__('app.update'), ['class' => 'btn btn-primary']) }}
+                        {{ link_to_route('users.show', __('app.cancel'), [$user->id], ['class' => 'btn btn-default']) }}
                     </div>
                 </div>
-            </div>
-            <div class="text-right">
-                {{ Form::submit(__('app.update'), ['class' => 'btn btn-primary']) }}
-                {{ link_to_route('users.show', __('app.cancel'), [$user->id], ['class' => 'btn btn-default']) }}
-            </div>
-        </div>
-        <div class="col-md-4">
-            <div class="panel panel-default">
-                <div class="panel-heading"><h3 class="panel-title">{{ __('app.address') }} &amp; {{ __('app.contact') }}</h3></div>
-                <div class="panel-body">
-                    {!! FormField::textarea('address', ['label' => __('app.address')]) !!}
-                    {!! FormField::text('city', ['label' => __('app.city'), 'placeholder' => __('app.example').' Jakarta']) !!}
-                    {!! FormField::text('phone', ['label' => __('app.phone'), 'placeholder' => __('app.example').' 081234567890']) !!}
-                </div>
-            </div>
-            <div class="panel panel-default">
-                <div class="panel-heading"><h3 class="panel-title">{{ __('app.login_account') }}</h3></div>
-                <div class="panel-body">
-                    {!! FormField::email('email', ['label' => __('auth.email'), 'placeholder' => __('app.example').' nama@mail.com']) !!}
-                    {!! FormField::password('password', ['label' => __('auth.password'), 'placeholder' => '******', 'value' => '']) !!}
-                </div>
-            </div>
-        </div>
-        {{ Form::close() }}
-        <div class="col-md-4">
-            <div class="panel panel-default">
-                <div class="panel-heading"><h3 class="panel-title">{{ __('user.update_photo') }}</h3></div>
-                {{ Form::open(['route' => ['users.photo-upload', $user], 'method' => 'patch', 'files' => true]) }}
-                <div class="panel-body text-center">
-                    {{ userPhoto($user, ['style' => 'width:100%;max-width:300px']) }}
-                </div>
-                <div class="panel-body">
-                    {!! FormField::file('photo', ['required' => true, 'label' => __('user.reupload_photo'), 'info' => ['text' => 'Format jpg, maks: 200 Kb.', 'class' => 'warning']]) !!}
-                </div>
-                <div class="panel-footer">
-                    {!! Form::submit(__('user.update_photo'), ['class' => 'btn btn-success']) !!}
-                    {{ link_to_route('users.show', __('app.cancel'), [$user], ['class' => 'btn btn-default']) }}
-                </div>
                 {{ Form::close() }}
+                <div class="col-md-6">
+                    @includeWhen(request('tab') == null, 'users.partials.update_photo')
+                </div>
             </div>
-            @can('delete', $user)
-                {{ link_to_route('users.edit', __('user.delete'), [$user, 'action' => 'delete'], ['class' => 'btn btn-danger pull-right', 'id' => 'del-user-'.$user->id]) }}
-            @endcan
         </div>
     </div>
 @endif

resources/views/users/partials/delete_confirmation.blade.php

@@ -0,0 +1,42 @@
+<div class="panel panel-default">
+    <div class="panel-heading"><h3 class="panel-title">{{ __('user.delete') }} : {{ $user->name }}</h3></div>
+    <div class="panel-body">
+        <table class="table table-condensed">
+            <tr><td>{{ __('user.name') }}</td><td>{{ $user->name }}</td></tr>
+            <tr><td>{{ __('user.nickname') }}</td><td>{{ $user->nickname }}</td></tr>
+            <tr><td>{{ __('user.gender') }}</td><td>{{ $user->gender }}</td></tr>
+            <tr><td>{{ __('user.father') }}</td><td>{{ $user->father_id ? $user->father->name : '' }}</td></tr>
+            <tr><td>{{ __('user.mother') }}</td><td>{{ $user->mother_id ? $user->mother->name : '' }}</td></tr>
+            <tr><td>{{ __('user.childs_count') }}</td><td>{{ $childsCount = $user->childs()->count() }}</td></tr>
+            <tr><td>{{ __('user.spouses_count') }}</td><td>{{ $spousesCount = $user->marriages()->count() }}</td></tr>
+            <tr><td>{{ __('user.managed_user') }}</td><td>{{ $managedUserCount = $user->managedUsers()->count() }}</td></tr>
+            <tr><td>{{ __('user.managed_couple') }}</td><td>{{ $managedCoupleCount = $user->managedCouples()->count() }}</td></tr>
+        </table>
+        @if ($childsCount + $spousesCount + $managedUserCount + $managedCoupleCount)
+            {{ __('user.replace_delete_text') }}
+            {{ Form::open([
+                'route' => ['users.destroy', $user],
+                'method' => 'delete',
+                'onsubmit' => 'return confirm("'.__('user.replace_confirm').'")',
+            ]) }}
+            {!! FormField::select('replacement_user_id', $replacementUsers, [
+                'label' => false,
+                'placeholder' => __('user.replacement'),
+            ]) !!}
+            {{ Form::submit(__('user.replace_delete_button'), [
+                'name' => 'replace_delete_button',
+                'class' => 'btn btn-danger',
+            ]) }}
+            {{ link_to_route('users.edit', __('app.cancel'), [$user], ['class' => 'btn btn-default pull-right']) }}
+            {{ Form::close() }}
+        @else
+            {!! FormField::delete(
+                ['route' => ['users.destroy', $user]],
+                __('user.delete_confirm_button'),
+                ['class' => 'btn btn-danger'],
+                ['user_id' => $user->id]
+            ) !!}
+            {{ link_to_route('users.edit', __('app.cancel'), [$user], ['class' => 'btn btn-default']) }}
+        @endif
+    </div>
+</div>

resources/views/users/partials/edit_contact_address.blade.php

@@ -0,0 +1,8 @@
+<div class="panel panel-default">
+    <div class="panel-heading"><h3 class="panel-title">{{ __('app.address') }} &amp; {{ __('app.contact') }}</h3></div>
+    <div class="panel-body">
+        {!! FormField::textarea('address', ['label' => __('app.address')]) !!}
+        {!! FormField::text('city', ['label' => __('app.city'), 'placeholder' => __('app.example').' Jakarta']) !!}
+        {!! FormField::text('phone', ['label' => __('app.phone'), 'placeholder' => __('app.example').' 081234567890']) !!}
+    </div>
+</div>

resources/views/users/partials/edit_death.blade.php

@@ -0,0 +1,4 @@
+<div class="row">
+    <div class="col-md-6">{!! FormField::text('yod', ['label' => __('user.yod'), 'placeholder' => __('app.example').' 2003']) !!}</div>
+    <div class="col-md-6">{!! FormField::text('dod', ['label' => __('user.dod'), 'placeholder' => __('app.example').' 2003-10-17']) !!}</div>
+</div>

resources/views/users/partials/edit_login_account.blade.php

@@ -0,0 +1,7 @@
+<div class="panel panel-default">
+    <div class="panel-heading"><h3 class="panel-title">{{ __('app.login_account') }}</h3></div>
+    <div class="panel-body">
+        {!! FormField::email('email', ['label' => __('auth.email'), 'placeholder' => __('app.example').' nama@mail.com']) !!}
+        {!! FormField::password('password', ['label' => __('auth.password'), 'placeholder' => '******', 'value' => '']) !!}
+    </div>
+</div>

resources/views/users/partials/edit_nav_tabs.blade.php

@@ -0,0 +1,19 @@
+<!-- Nav tabs -->
+<ul class="nav nav-pills nav-stacked">
+    <li class="{{ request('tab') == null ? 'active' : '' }}">
+        {!! link_to_route('users.edit', __('user.edit'), [$user->id]) !!}
+    </li>
+    <li class="{{ request('tab') == 'contact_address' ? 'active' : '' }}">
+        {!! link_to_route('users.edit', __('app.address').' &amp; '.__('app.contact'), [$user->id, 'tab' => 'contact_address']) !!}
+    </li>
+    <li class="{{ request('tab') == 'login_account' ? 'active' : '' }}">
+        {!! link_to_route('users.edit', __('app.login_account'), [$user->id, 'tab' => 'login_account']) !!}
+    </li>
+    <li class="{{ request('tab') == 'death' ? 'active' : '' }}">
+        {!! link_to_route('users.edit', __('user.death'), [$user->id, 'tab' => 'death']) !!}
+    </li>
+</ul>
+<br>
+@can('delete', $user)
+{{ link_to_route('users.edit', __('user.delete'), [$user, 'action' => 'delete'], ['class' => 'btn btn-danger', 'id' => 'del-user-'.$user->id]) }}
+@endcan

resources/views/users/partials/edit_profile.blade.php

@@ -0,0 +1,17 @@
+<div class="panel panel-default">
+    <div class="panel-heading"><h3 class="panel-title">{{ __('user.edit') }}</h3></div>
+    <div class="panel-body">
+        {!! FormField::text('name', ['label' => __('user.name')]) !!}
+        {!! FormField::text('nickname', ['label' => __('user.nickname')]) !!}
+        <div class="row">
+            <div class="col-md-6">{!! FormField::radios('gender_id', [1 => __('app.male_code'), __('app.female_code')], ['label' => __('user.gender')]) !!}</div>
+            <div class="col-md-4">
+                {!! FormField::text('birth_order', ['label' => __('user.birth_order'), 'type' => 'number', 'min' => 1]) !!}
+            </div>
+        </div>
+        <div class="row">
+            <div class="col-md-6">{!! FormField::text('yob', ['label' => __('user.yob'), 'placeholder' => __('app.example').' 1959']) !!}</div>
+            <div class="col-md-6">{!! FormField::text('dob', ['label' => __('user.dob'), 'placeholder' => __('app.example').' 1959-07-20']) !!}</div>
+        </div>
+    </div>
+</div>

resources/views/users/partials/update_photo.blade.php

@@ -0,0 +1,15 @@
+<div class="panel panel-default">
+    <div class="panel-heading"><h3 class="panel-title">{{ __('user.update_photo') }}</h3></div>
+    {{ Form::open(['route' => ['users.photo-upload', $user], 'method' => 'patch', 'files' => true]) }}
+    <div class="panel-body text-center">
+        {{ userPhoto($user, ['style' => 'width:100%;max-width:300px']) }}
+    </div>
+    <div class="panel-body">
+        {!! FormField::file('photo', ['required' => true, 'label' => __('user.reupload_photo'), 'info' => ['text' => 'Format jpg, maks: 200 Kb.', 'class' => 'warning']]) !!}
+    </div>
+    <div class="panel-footer">
+        {!! Form::submit(__('user.update_photo'), ['class' => 'btn btn-success']) !!}
+        {{ link_to_route('users.show', __('app.cancel'), [$user], ['class' => 'btn btn-default']) }}
+    </div>
+    {{ Form::close() }}
+</div>

tests/Feature/UsersProfileTest.php

@@ -12,6 +12,21 @@ class UsersProfileTest extends TestCase
     use RefreshDatabase;
 
     /** @test */
+    public function guest_can_search_users_profile()
+    {
+        $jono = factory(User::class)->create(['name' => 'Jono']);
+        $jeni = factory(User::class)->create(['name' => 'Jeni']);
+        $johan = factory(user::class)->create(['name' => 'Johan']);
+
+        $this->visitRoute('users.search', ['q' => 'jo']);
+        $this->seeRouteIs('users.search', ['q' => 'jo']);
+
+        $this->seeText('Jono');
+        $this->seeText('Johan');
+        $this->dontSeeText('Jeni');
+    }
+
+    /** @test */
     public function user_can_view_other_users_profile()
     {
         $user = factory(User::class)->create();
@@ -32,13 +47,6 @@ class UsersProfileTest extends TestCase
             'gender_id'   => 1,
             'dob'         => '1959-06-09',
             'yob'         => '',
-            'dod'         => '2003-10-17',
-            'yod'         => '',
-            'address'     => 'Jln. Angkasa, No. 70',
-            'city'        => 'Nama Kota',
-            'phone'       => '081234567890',
-            'email'       => '',
-            'password'    => '',
             'birth_order' => 3,
         ]);
 
@@ -48,24 +56,114 @@ class UsersProfileTest extends TestCase
             'gender_id'   => 1,
             'dob'         => '1959-06-09',
             'yob'         => '1959',
-            'dod'         => '2003-10-17',
-            'yod'         => '2003',
-            'address'     => 'Jln. Angkasa, No. 70',
-            'city'        => 'Nama Kota',
-            'phone'       => '081234567890',
-            'email'       => null,
-            'password'    => null,
             'birth_order' => 3,
         ]);
     }
 
     /** @test */
+    public function user_can_update_yob_only()
+    {
+        $user = $this->loginAsUser();
+        $this->visit(route('users.edit', $user->id));
+        $this->seePageIs(route('users.edit', $user->id));
+
+        $this->submitForm(trans('app.update'), [
+            'dob' => '',
+            'yob' => '2003',
+        ]);
+
+        $this->seeInDatabase('users', [
+            'id'  => $user->id,
+            'dob' => null,
+            'yob' => '2003',
+        ]);
+    }
+
+    /** @test */
+    public function user_can_edit_contact_address()
+    {
+        $user = $this->loginAsUser();
+        $this->visit(route('users.edit', [$user->id, 'tab' => 'contact_address']));
+        $this->seePageIs(route('users.edit', [$user->id, 'tab' => 'contact_address']));
+
+        $this->submitForm(trans('app.update'), [
+            'address' => 'Jln. Angkasa, No. 70',
+            'city'    => 'Nama Kota',
+            'phone'   => '081234567890',
+        ]);
+
+        $this->seeInDatabase('users', [
+            'id'      => $user->id,
+            'address' => 'Jln. Angkasa, No. 70',
+            'city'    => 'Nama Kota',
+            'phone'   => '081234567890',
+        ]);
+    }
+
+    /** @test */
+    public function user_can_edit_login_account()
+    {
+        $user = $this->loginAsUser();
+        $this->visit(route('users.edit', [$user->id, 'tab' => 'login_account']));
+        $this->seePageIs(route('users.edit', [$user->id, 'tab' => 'login_account']));
+
+        $this->submitForm(trans('app.update'), [
+            'email'    => '',
+            'password' => '',
+        ]);
+
+        $this->seeInDatabase('users', [
+            'id'       => $user->id,
+            'email'    => null,
+            'password' => null,
+        ]);
+    }
+
+    /** @test */
+    public function user_can_edit_death()
+    {
+        $user = $this->loginAsUser();
+        $this->visit(route('users.edit', [$user->id, 'tab' => 'death']));
+        $this->seePageIs(route('users.edit', [$user->id, 'tab' => 'death']));
+
+        $this->submitForm(trans('app.update'), [
+            'dod' => '2003-10-17',
+            'yod' => '',
+        ]);
+
+        $this->seeInDatabase('users', [
+            'id'  => $user->id,
+            'dod' => '2003-10-17',
+            'yod' => '2003',
+        ]);
+    }
+
+    /** @test */
+    public function user_can_update_yod_only()
+    {
+        $user = $this->loginAsUser();
+        $this->visit(route('users.edit', [$user->id, 'tab' => 'death']));
+        $this->seePageIs(route('users.edit', [$user->id, 'tab' => 'death']));
+
+        $this->submitForm(trans('app.update'), [
+            'dod' => '',
+            'yod' => '2003',
+        ]);
+
+        $this->seeInDatabase('users', [
+            'id'  => $user->id,
+            'dod' => null,
+            'yod' => '2003',
+        ]);
+    }
+
+    /** @test */
     public function manager_can_add_login_account_on_a_user()
     {
         $manager = $this->loginAsUser();
         $user = factory(User::class)->create(['manager_id' => $manager->id]);
-        $this->visit(route('users.edit', $user->id));
-        $this->seePageIs(route('users.edit', $user->id));
+        $this->visit(route('users.edit', [$user->id, 'tab' => 'login_account']));
+        $this->seePageIs(route('users.edit', [$user->id, 'tab' => 'login_account']));
 
         $this->submitForm(trans('app.update'), [
             'email'    => 'user@mail.com',
@@ -82,8 +180,8 @@ class UsersProfileTest extends TestCase
     {
         $manager = $this->loginAsUser();
         $user = factory(User::class)->create(['manager_id' => $manager->id]);
-        $this->visit(route('users.edit', $user->id));
-        $this->seePageIs(route('users.edit', $user->id));
+        $this->visit(route('users.edit', [$user->id, 'tab' => 'login_account']));
+        $this->seePageIs(route('users.edit', [$user->id, 'tab' => 'login_account']));
 
         $this->submitForm(trans('app.update'), [
             'email'    => 'user@mail.com',
@@ -103,8 +201,8 @@ class UsersProfileTest extends TestCase
             'manager_id' => $manager->id,
             'password'   => 'some random string password',
         ]);
-        $this->visit(route('users.edit', $user->id));
-        $this->seePageIs(route('users.edit', $user->id));
+        $this->visit(route('users.edit', [$user->id, 'tab' => 'login_account']));
+        $this->seePageIs(route('users.edit', [$user->id, 'tab' => 'login_account']));
 
         $this->submitForm(trans('app.update'), [
             'email'    => 'user@mail.com',
@@ -137,11 +235,4 @@ class UsersProfileTest extends TestCase
         $this->assertNotNull($user->photo_path);
         Storage::assertExists($user->photo_path);
     }
-
-    /** @test */
-    public function guest_can_search_users_profile()
-    {
-        $this->visit(route('users.search'));
-        $this->seePageIs(route('users.search'));
-    }
 }