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/dashboard.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/dashboard.t')
| -rw-r--r-- | t/app/controller/dashboard.t | 11 |
1 files changed, 3 insertions, 8 deletions
diff --git a/t/app/controller/dashboard.t b/t/app/controller/dashboard.t index 3829873e5..5ea5cb9f5 100644 --- a/t/app/controller/dashboard.t +++ b/t/app/controller/dashboard.t @@ -16,14 +16,9 @@ my $test_ward = 20723; my $body = $mech->create_body_ok($test_council, 'City of Edinburgh Council'); $mech->delete_user( $test_user ); -my $user = FixMyStreet::App->model('DB::User')->create( { - email => $test_user, - password => $test_pass, -} ); - -my $p_user = FixMyStreet::App->model('DB::User')->find_or_create( { - email => 'p_user@example.com' -} ); +my $user = $mech->create_user_ok($test_user, password => $test_pass); + +my $p_user = $mech->create_user_ok('p_user@example.com'); # Dashboard tests assume we are not too early in year, to allow reporting # within same year, as a convenience. |