Don't show postcode column in RSS/alert emails, may have no bearing on report.

3 files changed, 18 insertions, 60 deletions

diff --git a/t/app/controller/alert_new.t b/t/app/controller/alert_new.t
index 7ba887824..d66590c57 100644
--- a/t/app/controller/alert_new.t
+++ b/t/app/controller/alert_new.t
@@ -448,7 +448,7 @@ subtest "Test normal alert signups and that alerts are sent" => sub {
         $count++ if $_->body =~ /The following updates have been left on this problem:/;
         $count++ if $_->body =~ /The following new problems have been reported to City of\s*Edinburgh Council:/;
         $count++ if $_->body =~ /The following nearby problems have been added:/;
-        $count++ if $_->body =~ /\s+-\s+Testing,\s+EH1\s+1BB/;
+        $count++ if $_->body =~ /\s+-\s+Testing/;
     }
     is $count, 5, 'Five emails with the right things in them';
 
diff --git a/t/app/controller/rss.t b/t/app/controller/rss.t
index c6ab20574..77e2c7ee1 100644
--- a/t/app/controller/rss.t
+++ b/t/app/controller/rss.t
@@ -41,7 +41,7 @@ my $report = FixMyStreet::App->model('DB::Problem')->find_or_create( {
 
 
 $mech->get_ok("/rss/pc/EH11BB/2");
-$mech->content_contains( "Testing, 10th October, EH1 1BB" );
+$mech->content_contains( "Testing, 10th October" );
 $mech->content_lacks( 'Nearest road to the pin' );
 
 $report->geocode( 
@@ -106,19 +106,12 @@ $report->geocode(
           'authenticationResultCode' => 'ValidCredentials'
         }
 );
-$report->postcode('eh11bb');
 $report->update();
 
 $mech->get_ok("/rss/pc/EH11BB/2");
-$mech->content_contains( "Testing, 10th October, EH1 1BB" );
+$mech->content_contains( "Testing, 10th October" );
 $mech->content_contains( '18 North Bridge, Edinburgh' );
 
-$report->postcode('Princes St, Edinburgh');
-$report->update();
-
-$mech->get_ok("/rss/pc/EH11BB/2");
-$mech->content_contains( "Testing, 10th October, Princes St, Edinburgh" );
-
 $report->delete();
 $user1->delete();
 
diff --git a/t/app/model/alert_type.t b/t/app/model/alert_type.t
index 272ad4900..7df4c44c0 100644
--- a/t/app/model/alert_type.t
+++ b/t/app/model/alert_type.t
@@ -155,59 +155,24 @@ my $council_alert = FixMyStreet::App->model('DB::Alert')->find_or_create(
     }
 );
 
-for my $test (
-    {
-        postcode           => 'SW1A 1AA',
-        expected_postcode  => 'SW1A 1AA',
-    },
-    {
-        postcode           => 'sw1a 1AA',
-        expected_postcode  => 'SW1A 1AA',
-    },
-    {
-        postcode           => 'SW1A 1aa',
-        expected_postcode  => 'SW1A 1AA',
-    },
-    {
-        postcode           => 'SW1A1AA',
-        expected_postcode  => 'SW1A 1AA',
-    },  
-    {
-        postcode           => 'Buckingham Gate',
-        expected_postcode  => 'Buckingham\s+Gate',
-    },  
-    {
-        postcode           => 'Buckingham gate',
-        expected_postcode  => 'Buckingham\s+gate',
-    },  
-) {
-    subtest "correct text for postcode $test->{postcode}" => sub {
-        $mech->clear_emails_ok;
-
-        my $sent = FixMyStreet::App->model('DB::AlertSent')->search(
-            {
-                alert_id => $council_alert->id,
-                parameter => $report->id,
-            }
-        )->delete;
-
-        $report->postcode( $test->{postcode} );
-        $report->update;
-
-        FixMyStreet::App->model('DB::AlertType')->email_alerts();
+subtest "correct text for title after URL" => sub {
+    $mech->clear_emails_ok;
 
-        $mech->email_count_is( 1 );
-        my $email = $mech->get_email;
-        my $pc = $test->{expected_postcode};
-        (my $title = $report->title) =~ s/ /\\s+/;
-        my $body = $email->body;
+    my $sent = FixMyStreet::App->model('DB::AlertSent')->search(
+        {
+            alert_id => $council_alert->id,
+            parameter => $report->id,
+        }
+    )->delete;
+    FixMyStreet::App->model('DB::AlertType')->email_alerts();
 
-        like $body, qr#report/$report_id\s+-\s+$title,\s+$pc#, 'email contains expected postcode';
-    };
-}
+    $mech->email_count_is( 1 );
+    my $email = $mech->get_email;
+    (my $title = $report->title) =~ s/ /\\s+/;
+    my $body = $email->body;
 
-$report->postcode( 'SW1A 1AA' );
-$report->update;
+    like $body, qr#report/$report_id\s+-\s+$title#, 'email contains expected title';
+};
 
 $report->geocode(
 {