diff options
author | M Somerville <matthew-github@dracos.co.uk> | 2020-09-11 18:11:03 +0100 |
---|---|---|
committer | M Somerville <matthew-github@dracos.co.uk> | 2020-09-25 16:20:40 +0100 |
commit | 2abe443efb8bbbd7cf3048eec271fd28a777932f (patch) | |
tree | ed7c25c63e9774e851152224078621b2016229be /t/app/controller/report_update_text.t | |
parent | 82a3e9bfcf094538db75755f63984a428670dedd (diff) |
Show error if text confirmation code sending fails
Diffstat (limited to 't/app/controller/report_update_text.t')
-rw-r--r-- | t/app/controller/report_update_text.t | 47 |
1 files changed, 25 insertions, 22 deletions
diff --git a/t/app/controller/report_update_text.t b/t/app/controller/report_update_text.t index 3f9a73773..fbf5ca0c6 100644 --- a/t/app/controller/report_update_text.t +++ b/t/app/controller/report_update_text.t @@ -58,22 +58,25 @@ my $comment = FixMyStreet::DB->resultset('Comment')->find_or_create( { my $comment_id = $comment->id; ok $comment, "created test update - $comment_id"; +my %defaults = ( + username => '', + update => 'Update', + name => 'Name', + photo1 => '', + photo2 => '', + photo3 => '', + fixed => undef, + add_alert => 1, + may_show_name => undef, + password_sign_in => '', + password_register => '', +); for my $test ( { desc => 'Invalid phone', fields => { username_register => '01214960000000', - username => '', - update => 'Update', - name => 'Name', - photo1 => '', - photo2 => '', - photo3 => '', - fixed => undef, - add_alert => 1, - may_show_name => undef, - password_sign_in => '', - password_register => '', + %defaults, }, changes => {}, field_errors => [ 'Please check your phone number is correct' ] @@ -82,21 +85,20 @@ for my $test ( desc => 'landline number', fields => { username_register => '01214960000', - username => '', - update => 'Update', - name => 'Name', - photo1 => '', - photo2 => '', - photo3 => '', - fixed => undef, - add_alert => 1, - may_show_name => undef, - password_register => '', - password_sign_in => '', + %defaults, }, changes => {}, field_errors => [ 'Please enter a mobile number' ] }, + { + desc => 'fails to send', + fields => { + username_register => '+18165550101', + %defaults, + }, + changes => {}, + field_errors => [ 'Sending a confirmation text failed: "Unable to send (21408)"' ] + }, ) { subtest "submit an update - $test->{desc}" => sub { @@ -104,6 +106,7 @@ for my $test ( FixMyStreet::override_config { SMS_AUTHENTICATION => 1, + TWILIO_ACCOUNT_SID => 'AC123', PHONE_COUNTRY => 'GB', }, sub { $mech->submit_form_ok( { with_fields => $test->{fields} }, 'submit update' ); |