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/contact.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/contact.t')
| -rw-r--r-- | t/app/controller/contact.t | 15 |
1 files changed, 3 insertions, 12 deletions
diff --git a/t/app/controller/contact.t b/t/app/controller/contact.t index 4ac69a9f8..1b0f09a85 100644 --- a/t/app/controller/contact.t +++ b/t/app/controller/contact.t @@ -52,12 +52,7 @@ for my $test ( ) { subtest 'check reporting a problem displays correctly' => sub { - my $user = FixMyStreet::App->model('DB::User')->find_or_create( - { - name => $test->{name}, - email => $test->{email} - } - ); + my $user = $mech->create_user_ok($test->{email}, name => $test->{name}); my $problem = FixMyStreet::App->model('DB::Problem')->create( { @@ -80,12 +75,8 @@ for my $test ( if ( $test->{update} ) { my $update_info = $test->{update}; - my $update_user = FixMyStreet::App->model('DB::User')->find_or_create( - { - name => $update_info->{name}, - email => $update_info->{email} - } - ); + my $update_user = $mech->create_user_ok($update_info->{email}, + name => $update_info->{name}); $update = FixMyStreet::App->model('DB::Comment')->create( { |