Fix #854

Author: exodist

Changes

@@ -1,5 +1,7 @@
 {{$NEXT}}
 
+    - Fix done_testing(0) producing 2 plans and an incorrect message
+
 1.302194  2023-03-13 20:06:57-07:00 America/Los_Angeles
 
     - Fix failing test on 5.10

lib/Test/Builder.pm

@@ -1670,7 +1670,7 @@ sub _ending {
     return unless $plan || $count || $failed;
 
     # Ran tests but never declared a plan or hit done_testing
-    if( !$hub->plan and $hub->count ) {
+    if( !defined($hub->plan) and $hub->count ) {
         $self->diag("Tests were run but no plan was declared and done_testing() was not seen.");
 
         if($real_exit_code) {

lib/Test2/Hub.pm

@@ -432,7 +432,7 @@ sub finalize {
         $count  = $self->{+COUNT};
         $failed = $self->{+FAILED};
 
-        if (($plan && $plan eq 'NO PLAN') || ($do_plan && !$plan)) {
+        if ((defined($plan) && $plan eq 'NO PLAN') || ($do_plan && !defined($plan))) {
             $self->send(
                 Test2::Event::Plan->new(
                     trace => $trace,