aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMatthew Somerville <matthew@mysociety.org>2016-01-27 11:38:11 +0000
committerMatthew Somerville <matthew@mysociety.org>2016-01-27 11:38:11 +0000
commit2679b30e530861ba7c2bef6f36a60242872bd6a7 (patch)
tree7cec1bc797b30f7efb8b4eae0549bcebd1c96e07
parentb8083a462aebf542f6f39c4201e1f09248b7e0e7 (diff)
Move test mocks to their own directory.
-rw-r--r--t/Mock/Facebook.pm (renamed from t/Facebook.pm)2
-rw-r--r--t/Mock/MapIt.pm (renamed from t/MapIt.pm)2
-rw-r--r--t/Mock/Nominatim.pm (renamed from t/Nominatim.pm)2
-rw-r--r--t/app/controller/alert.t4
-rw-r--r--t/app/controller/around.t4
-rw-r--r--t/app/controller/auth_social.t8
-rw-r--r--t/app/controller/report_import.t4
-rw-r--r--t/cobrand/fixamingata.t4
8 files changed, 15 insertions, 15 deletions
diff --git a/t/Facebook.pm b/t/Mock/Facebook.pm
index 8c258699b..68c63f2d5 100644
--- a/t/Facebook.pm
+++ b/t/Mock/Facebook.pm
@@ -1,4 +1,4 @@
-package t::Facebook;
+package t::Mock::Facebook;
use JSON;
use Web::Simple;
diff --git a/t/MapIt.pm b/t/Mock/MapIt.pm
index a10118013..a3bfb10d4 100644
--- a/t/MapIt.pm
+++ b/t/Mock/MapIt.pm
@@ -1,4 +1,4 @@
-package t::MapIt;
+package t::Mock::MapIt;
use JSON;
use Web::Simple;
diff --git a/t/Nominatim.pm b/t/Mock/Nominatim.pm
index 2041b4677..3f3cef69a 100644
--- a/t/Nominatim.pm
+++ b/t/Mock/Nominatim.pm
@@ -1,4 +1,4 @@
-package t::Nominatim;
+package t::Mock::Nominatim;
use JSON;
use Web::Simple;
diff --git a/t/app/controller/alert.t b/t/app/controller/alert.t
index 5bf2af428..c7dc9ed09 100644
--- a/t/app/controller/alert.t
+++ b/t/app/controller/alert.t
@@ -6,7 +6,7 @@ use LWP::Protocol::PSGI;
use FixMyStreet::TestMech;
my $mech = FixMyStreet::TestMech->new;
-use t::Nominatim;
+use t::Mock::Nominatim;
# check that we can get the page
$mech->get_ok('/alert');
@@ -42,7 +42,7 @@ FixMyStreet::override_config {
$mech->content_contains('ward:2651:20728:City_of_Edinburgh:City_Centre');
subtest "Test Nominatim lookup" => sub {
- LWP::Protocol::PSGI->register(t::Nominatim->run_if_script, host => 'nominatim.openstreetmap.org');
+ LWP::Protocol::PSGI->register(t::Mock::Nominatim->run_if_script, host => 'nominatim.openstreetmap.org');
$mech->get_ok('/alert/list?pc=High Street');
$mech->content_contains('We found more than one match for that location');
};
diff --git a/t/app/controller/around.t b/t/app/controller/around.t
index a70116525..89ca5246e 100644
--- a/t/app/controller/around.t
+++ b/t/app/controller/around.t
@@ -3,7 +3,7 @@ use warnings;
use Test::More;
use LWP::Protocol::PSGI;
-use t::MapIt;
+use t::Mock::MapIt;
use FixMyStreet::TestMech;
my $mech = FixMyStreet::TestMech->new;
@@ -84,7 +84,7 @@ foreach my $test (
)
{
subtest "check lat/lng for '$test->{pc}'" => sub {
- LWP::Protocol::PSGI->register(t::MapIt->run_if_script, host => 'mapit.uk');
+ LWP::Protocol::PSGI->register(t::Mock::MapIt->run_if_script, host => 'mapit.uk');
$mech->get_ok('/');
FixMyStreet::override_config {
diff --git a/t/app/controller/auth_social.t b/t/app/controller/auth_social.t
index 173572b02..84fdd4dfe 100644
--- a/t/app/controller/auth_social.t
+++ b/t/app/controller/auth_social.t
@@ -5,8 +5,8 @@ use LWP::Protocol::PSGI;
use LWP::Simple;
use JSON::MaybeXS;
-use t::Facebook;
-use t::MapIt;
+use t::Mock::Facebook;
+use t::Mock::MapIt;
use FixMyStreet::TestMech;
my $mech = FixMyStreet::TestMech->new;
@@ -17,7 +17,7 @@ END { FixMyStreet::App->log->enable('info'); }
my ($report) = $mech->create_problems_for_body(1, '2345', 'Test');
-LWP::Protocol::PSGI->register(t::MapIt->to_psgi_app, host => 'mapit.uk');
+LWP::Protocol::PSGI->register(t::Mock::MapIt->to_psgi_app, host => 'mapit.uk');
FixMyStreet::override_config {
FACEBOOK_APP_ID => 'facebook-app-id',
@@ -40,7 +40,7 @@ for my $fb_state ( 'refused', 'no email', 'existing UID', 'okay' ) {
}
# Set up a mock to catch (most, see below) requests to Facebook
- my $fb = t::Facebook->new;
+ my $fb = t::Mock::Facebook->new;
$fb->returns_email(0) if $fb_state eq 'no email' || $fb_state eq 'existing UID';
LWP::Protocol::PSGI->register($fb->to_psgi_app, host => 'www.facebook.com');
LWP::Protocol::PSGI->register($fb->to_psgi_app, host => 'graph.facebook.com');
diff --git a/t/app/controller/report_import.t b/t/app/controller/report_import.t
index 1d6695996..220ebaac5 100644
--- a/t/app/controller/report_import.t
+++ b/t/app/controller/report_import.t
@@ -3,7 +3,7 @@ use warnings;
use Test::More;
use LWP::Protocol::PSGI;
-use t::MapIt;
+use t::Mock::MapIt;
use FixMyStreet::TestMech;
use FixMyStreet::App;
use Web::Scraper;
@@ -92,7 +92,7 @@ subtest "Test creating bad partial entries" => sub {
};
subtest "Submit a correct entry" => sub {
- LWP::Protocol::PSGI->register(t::MapIt->run_if_script, host => 'mapit.uk');
+ LWP::Protocol::PSGI->register(t::Mock::MapIt->run_if_script, host => 'mapit.uk');
$mech->get_ok('/import');
diff --git a/t/cobrand/fixamingata.t b/t/cobrand/fixamingata.t
index ea3c2da92..50b7713d1 100644
--- a/t/cobrand/fixamingata.t
+++ b/t/cobrand/fixamingata.t
@@ -8,7 +8,7 @@ BEGIN {
FixMyStreet->test_mode(1);
}
-use t::MapIt;
+use t::Mock::MapIt;
use mySociety::Locale;
use FixMyStreet::TestMech;
@@ -102,7 +102,7 @@ subtest "Test ajax decimal points" => sub {
# A note to the future - the run_if_script line must be within a subtest
# otherwise it fails to work
- LWP::Protocol::PSGI->register(t::MapIt->run_if_script, host => 'mapit.sweden');
+ LWP::Protocol::PSGI->register(t::Mock::MapIt->run_if_script, host => 'mapit.sweden');
FixMyStreet::override_config {
ALLOWED_COBRANDS => [ 'fixamingata' ],