aboutsummaryrefslogtreecommitdiffstats
path: root/perllib/FixMyStreet/TestMech.pm
diff options
context:
space:
mode:
authorDave Arter <davea@mysociety.org>2016-07-14 11:27:12 +0100
committerDave Arter <davea@mysociety.org>2016-07-19 18:10:04 +0100
commitdef23b4c2ed67e3920489d07cf04689117274fc8 (patch)
tree3169b8634ea21351eb65e8f2e9c3040e282d618f /perllib/FixMyStreet/TestMech.pm
parenta001715332d9423dd4c68542f7eceac4c2e3900c (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 'perllib/FixMyStreet/TestMech.pm')
-rw-r--r--perllib/FixMyStreet/TestMech.pm7
1 files changed, 3 insertions, 4 deletions
diff --git a/perllib/FixMyStreet/TestMech.pm b/perllib/FixMyStreet/TestMech.pm
index 2ad820d1f..937780a31 100644
--- a/perllib/FixMyStreet/TestMech.pm
+++ b/perllib/FixMyStreet/TestMech.pm
@@ -63,11 +63,10 @@ Create a test user (or find it and return if it already exists).
sub create_user_ok {
my $self = shift;
- my ($email) = @_;
+ my ( $email, %extra ) = @_;
- my $user =
- FixMyStreet::DB->resultset('User')
- ->find_or_create( { email => $email } );
+ my $params = { email => $email, %extra };
+ my $user = FixMyStreet::DB->resultset('User')->find_or_create($params);
ok $user, "found/created user for $email";
return $user;