aboutsummaryrefslogtreecommitdiffstats
path: root/t/app/controller
diff options
context:
space:
mode:
authorDave Arter <davea@mysociety.org>2018-01-29 08:30:24 +0000
committerDave Arter <davea@mysociety.org>2018-02-07 17:22:10 +0000
commitdf49eefc0c45485da113e91dddd4e4432ac678ac (patch)
tree7516645c232cbb603354916d05bcd6d365bca11b /t/app/controller
parent01311af63412f39b75ff3b12bb6dc4051eab13b3 (diff)
Don’t require SMS_AUTHENTICATION to view/edit user email/phone_verified
Previously this would force phone_verified to false for all user edits, as well as forcing email_verified to true - which would be the wrong thing to do if the user had been created/verified via the ‘report as another user’ flow.
Diffstat (limited to 't/app/controller')
-rw-r--r--t/app/controller/admin/users.t14
1 files changed, 14 insertions, 0 deletions
diff --git a/t/app/controller/admin/users.t b/t/app/controller/admin/users.t
index 63295e26d..8759e260d 100644
--- a/t/app/controller/admin/users.t
+++ b/t/app/controller/admin/users.t
@@ -200,8 +200,10 @@ FixMyStreet::override_config {
fields => {
name => '',
email => 'test@example.com',
+ email_verified => 1,
body => $haringey->id,
phone => '',
+ phone_verified => undef,
flagged => undef,
is_superuser => undef,
area_id => '',
@@ -218,8 +220,10 @@ FixMyStreet::override_config {
fields => {
name => 'Changed User',
email => 'test@example.com',
+ email_verified => 1,
body => $haringey->id,
phone => '',
+ phone_verified => undef,
flagged => undef,
is_superuser => undef,
area_id => '',
@@ -236,8 +240,10 @@ FixMyStreet::override_config {
fields => {
name => 'Changed User',
email => 'changed@example.com',
+ email_verified => 1,
body => $haringey->id,
phone => '',
+ phone_verified => undef,
flagged => undef,
is_superuser => undef,
area_id => '',
@@ -254,8 +260,10 @@ FixMyStreet::override_config {
fields => {
name => 'Changed User',
email => 'changed@example.com',
+ email_verified => 1,
body => $southend->id,
phone => '',
+ phone_verified => undef,
flagged => undef,
is_superuser => undef,
area_id => '',
@@ -272,8 +280,10 @@ FixMyStreet::override_config {
fields => {
name => 'Changed User',
email => 'changed@example.com',
+ email_verified => 1,
body => $southend->id,
phone => '',
+ phone_verified => undef,
flagged => 'on',
is_superuser => undef,
area_id => '',
@@ -290,8 +300,10 @@ FixMyStreet::override_config {
fields => {
name => 'Changed User',
email => 'changed@example.com',
+ email_verified => 1,
body => $southend->id,
phone => '',
+ phone_verified => undef,
flagged => undef,
is_superuser => undef,
area_id => '',
@@ -311,8 +323,10 @@ FixMyStreet::override_config {
fields => {
name => 'Changed User',
email => 'changed@example.com',
+ email_verified => 1,
body => $southend->id,
phone => '',
+ phone_verified => undef,
flagged => undef,
is_superuser => 'on',
area_id => '',