diff options
| author | Dave Arter <davea@mysociety.org> | 2016-07-14 11:27:12 +0100 |
|---|---|---|
| committer | Dave Arter <davea@mysociety.org> | 2016-07-19 18:10:04 +0100 |
| commit | def23b4c2ed67e3920489d07cf04689117274fc8 (patch) | |
| tree | 3169b8634ea21351eb65e8f2e9c3040e282d618f /t/app/controller/report_new.t | |
| parent | a001715332d9423dd4c68542f7eceac4c2e3900c (diff) | |
Use improved TestMech::create_user_ok instead of User::find_or_create
This improves create_user_ok to work more like create_body_ok where extra params
can be passed in and set on the model.
Diffstat (limited to 't/app/controller/report_new.t')
| -rw-r--r-- | t/app/controller/report_new.t | 9 |
1 files changed, 3 insertions, 6 deletions
diff --git a/t/app/controller/report_new.t b/t/app/controller/report_new.t index 6ea4c9523..2aebfa00b 100644 --- a/t/app/controller/report_new.t +++ b/t/app/controller/report_new.t @@ -654,8 +654,7 @@ subtest "test password errors for a user who is signing in as they report" => su # check that the user does not exist my $test_email = 'test-2@example.com'; - my $user = FixMyStreet::App->model('DB::User')->find_or_create( { email => $test_email } ); - ok $user, "test user does exist"; + my $user = $mech->create_user_ok($test_email); # setup the user. ok $user->update( { @@ -707,8 +706,7 @@ subtest "test report creation for a user who is signing in as they report" => su # check that the user does not exist my $test_email = 'test-2@example.com'; - my $user = FixMyStreet::App->model('DB::User')->find_or_create( { email => $test_email } ); - ok $user, "test user does exist"; + my $user = $mech->create_user_ok($test_email); # setup the user. ok $user->update( { @@ -909,8 +907,7 @@ subtest "test report creation for a category that is non public" => sub { # check that the user does not exist my $test_email = 'test-2@example.com'; - my $user = FixMyStreet::App->model('DB::User')->find_or_create( { email => $test_email } ); - ok $user, "test user does exist"; + my $user = $mech->create_user_ok($test_email); $contact1->update( { non_public => 1 } ); |