Clarified comment

src/app/reporting/SynchronizedReportSchedulerImpl.cpp

@@ -191,7 +191,8 @@ void SynchronizedReportSchedulerImpl::TimerFired()
 
         if (node->IsReportableNow(now))
         {
-            // We assume we fired the timer early if no handler is reportable at the moment
+            // We set firedEarly false here because we assume we fired the timer early if no handler is reportable at the moment,
+            // which becomes false if we find a handler that is reportable
             firedEarly = false;
             node->SetEngineRunScheduled(true);
             ChipLogProgress(DataManagement, "Handler: %p with min: 0x" ChipLogFormatX64 " and max: 0x" ChipLogFormatX64 "", (node),

src/app/tests/TestReadInteraction.cpp

@@ -2590,6 +2590,7 @@ void TestReadInteraction::TestSubscribeUrgentWildcardEvent(nlTestSuite * apSuite
             // Should get those urgent events reported and the non urgent reported for synchronisation
             NL_TEST_ASSERT(apSuite, delegate.mGotEventResponse);
             NL_TEST_ASSERT(apSuite, nonUrgentDelegate.mGotEventResponse);
+            ctx.GetIOContext().DriveIO();
         }
     }