Move skip to start of subtest so it doesn't give a 'tests were run but no plan' warning.

1 files changed, 4 insertions, 4 deletions

diff --git a/t/app/model/problem.t b/t/app/model/problem.t
index 8aabce4f7..9aa52c3cf 100644
--- a/t/app/model/problem.t
+++ b/t/app/model/problem.t
@@ -483,6 +483,10 @@ foreach my $test ( {
     },
 ) {
     subtest $test->{ desc } => sub {
+        if ( $test->{cobrand} && $test->{cobrand} =~ /lichfielddc/ && !FixMyStreet::Cobrand->exists('lichfielddc') ) {
+            plan skip_all => 'Skipping Lichfield tests without Lichfield cobrand';
+        }
+
         $mech->clear_emails_ok;
 
         FixMyStreet::App->model('DB::Problem')->search(
@@ -491,10 +495,6 @@ foreach my $test ( {
             }
         )->update( { whensent => \'ms_current_timestamp()' } );
 
-        if ( $test->{cobrand} && $test->{cobrand} =~ /lichfielddc/ && !FixMyStreet::Cobrand->exists('lichfielddc') ) {
-            plan skip_all => 'Skipping Lichfield tests without Lichfield cobrand';
-        }
-
         $problem->discard_changes;
         $problem->update( {
             council => $test->{ council },