diff options
author | Matthew Somerville <matthew@mysociety.org> | 2017-10-04 15:31:44 +0100 |
---|---|---|
committer | Matthew Somerville <matthew@mysociety.org> | 2017-10-05 13:07:41 +0100 |
commit | 861b3ace16097b5558d749210509afe4f0bdf70f (patch) | |
tree | 650eb3c0a8c0bbaeea63b4181bab62bda045dc43 /t/app/controller | |
parent | aed38e8dfca568ef4aabd8e222c03c6ac7bb69ad (diff) |
Return page with "No results" if no ref found.
This lookup is often performed by inspectors with appcache switched on,
which means they were getting the intercepted Glitch page rather than a
404; give them a no results page instead.
Diffstat (limited to 't/app/controller')
-rw-r--r-- | t/app/controller/around.t | 23 |
1 files changed, 15 insertions, 8 deletions
diff --git a/t/app/controller/around.t b/t/app/controller/around.t index fbb4e76cd..cdaeaf363 100644 --- a/t/app/controller/around.t +++ b/t/app/controller/around.t @@ -92,15 +92,22 @@ foreach my $test ( }; } -subtest 'check non public reports are not displayed on around page' => sub { - my $params = { - postcode => 'EH1 1BB', - latitude => 55.9519637512, - longitude => -3.17492254484, - }; - my @edinburgh_problems = - $mech->create_problems_for_body( 5, 2651, 'Around page', $params ); +my @edinburgh_problems = $mech->create_problems_for_body( 5, 2651, 'Around page', { + postcode => 'EH1 1BB', + latitude => 55.9519637512, + longitude => -3.17492254484, +}); +subtest 'check lookup by reference' => sub { + $mech->get_ok('/'); + $mech->submit_form_ok( { with_fields => { pc => 'ref:12345' } }, 'bad ref'); + $mech->content_contains('Searching found no reports'); + my $id = $edinburgh_problems[0]->id; + $mech->submit_form_ok( { with_fields => { pc => "ref:$id" } }, 'good ref'); + is $mech->uri->path, "/report/$id", "redirected to report page"; +}; + +subtest 'check non public reports are not displayed on around page' => sub { $mech->get_ok('/'); FixMyStreet::override_config { ALLOWED_COBRANDS => [ { 'fixmystreet' => '.' } ], |