Added User Password change tests
This commit is contained in:
parent
7ca1cd543f
commit
adf16e7eb1
|
|
@ -46,6 +46,7 @@ class UserController extends Controller
|
|||
'password.passcheck' => 'This password is incorrect.',
|
||||
'email.unique' => 'This E-mail is already in use.',
|
||||
'first_name.required' => 'Please enter your first name.',
|
||||
'last_name.required' => 'Please enter your last name.',
|
||||
];
|
||||
|
||||
$validation = Validator::make($request->all(), $rules, $messages);
|
||||
|
|
@ -60,7 +61,7 @@ class UserController extends Controller
|
|||
$user = Auth::user();
|
||||
|
||||
if ($request->get('password')) {
|
||||
$user->password = Hash::make(Input::get('new_password'));
|
||||
$user->password = Hash::make($request->get('new_password'));
|
||||
}
|
||||
|
||||
$user->first_name = $request->get('first_name');
|
||||
|
|
|
|||
|
|
@ -7,16 +7,11 @@ use App\Attendize\Utils;
|
|||
|
||||
class UserTest extends TestCase
|
||||
{
|
||||
/**
|
||||
* Test user edit page is successful
|
||||
*
|
||||
* @return void
|
||||
*/
|
||||
public function test_edit_user_is_successful()
|
||||
{
|
||||
$this->actingAs($this->test_user);
|
||||
|
||||
$organiser = factory(App\Models\Organiser::class)->create(['account_id' => 1]);
|
||||
factory(App\Models\Organiser::class)->create(['account_id' => 1]);
|
||||
|
||||
$server = array('HTTP_X-Requested-With' => 'XMLHttpRequest');
|
||||
|
||||
|
|
@ -42,4 +37,111 @@ class UserTest extends TestCase
|
|||
$this->assertEquals($lastName, $user->last_name);
|
||||
$this->assertEquals($email, $user->email);
|
||||
}
|
||||
|
||||
public function test_edit_user_is_successful_when_changing_password()
|
||||
{
|
||||
$this->actingAs($this->test_user);
|
||||
|
||||
$previousPassword = $this->test_user->password;
|
||||
|
||||
factory(App\Models\Organiser::class)->create(['account_id' => 1]);
|
||||
|
||||
$server = array('HTTP_X-Requested-With' => 'XMLHttpRequest');
|
||||
|
||||
$firstName = $this->faker->firstName;
|
||||
$lastName = $this->faker->lastName;
|
||||
$email = 'new@email.com.au';
|
||||
$post = array(
|
||||
'first_name' => $firstName,
|
||||
'last_name' => $lastName,
|
||||
'email' => $email,
|
||||
'password' => $this->test_user_password,
|
||||
'new_password' => 'newpassword',
|
||||
'new_password_confirmation' => 'newpassword',
|
||||
);
|
||||
|
||||
$this->call('post', route('postEditUser'), $post, $server);
|
||||
|
||||
$this->seeJson([
|
||||
'status' => 'success',
|
||||
'message' => 'Successfully Edited User',
|
||||
]);
|
||||
|
||||
$user = App\Models\User::find($this->test_user->id);
|
||||
|
||||
$this->assertEquals($firstName, $user->first_name);
|
||||
$this->assertEquals($lastName, $user->last_name);
|
||||
$this->assertEquals($email, $user->email);
|
||||
$this->assertNotEquals($previousPassword, $user->password);
|
||||
}
|
||||
|
||||
public function test_edit_user_is_unsuccessful_because_of_invalid_email()
|
||||
{
|
||||
$this->actingAs($this->test_user);
|
||||
|
||||
factory(App\Models\Organiser::class)->create(['account_id' => 1]);
|
||||
|
||||
$server = array('HTTP_X-Requested-With' => 'XMLHttpRequest');
|
||||
|
||||
$firstName = $this->faker->firstName;
|
||||
$lastName = $this->faker->lastName;
|
||||
$email = 'new@email';
|
||||
$post = array(
|
||||
'first_name' => $firstName,
|
||||
'last_name' => $lastName,
|
||||
'email' => $email,
|
||||
);
|
||||
|
||||
$this->call('post', route('postEditUser'), $post, $server);
|
||||
|
||||
$this->seeJson([
|
||||
'status' => 'error',
|
||||
]);
|
||||
}
|
||||
|
||||
public function test_edit_user_is_unsuccessful_because_of_no_first_name()
|
||||
{
|
||||
$this->actingAs($this->test_user);
|
||||
|
||||
factory(App\Models\Organiser::class)->create(['account_id' => 1]);
|
||||
|
||||
$server = array('HTTP_X-Requested-With' => 'XMLHttpRequest');
|
||||
|
||||
$lastName = $this->faker->lastName;
|
||||
$email = 'new@email';
|
||||
$post = array(
|
||||
'first_name' => '',
|
||||
'last_name' => $lastName,
|
||||
'email' => $email,
|
||||
);
|
||||
|
||||
$this->call('post', route('postEditUser'), $post, $server);
|
||||
|
||||
$this->seeJson([
|
||||
'status' => 'error',
|
||||
]);
|
||||
}
|
||||
|
||||
public function test_edit_user_is_unsuccessful_because_of_no_last_name()
|
||||
{
|
||||
$this->actingAs($this->test_user);
|
||||
|
||||
factory(App\Models\Organiser::class)->create(['account_id' => 1]);
|
||||
|
||||
$server = array('HTTP_X-Requested-With' => 'XMLHttpRequest');
|
||||
|
||||
$firstName = $this->faker->firstName;
|
||||
$email = 'new@email';
|
||||
$post = array(
|
||||
'first_name' => $firstName,
|
||||
'last_name' => '',
|
||||
'email' => $email,
|
||||
);
|
||||
|
||||
$this->call('post', route('postEditUser'), $post, $server);
|
||||
|
||||
$this->seeJson([
|
||||
'status' => 'error',
|
||||
]);
|
||||
}
|
||||
}
|
||||
|
|
|
|||
Loading…
Reference in New Issue