Delay PZ2FlowsToLogging until PSP to avoid dependently computing the core annotation

Summary: We need to avoid hitting a cycle due to computing the core annotation when the fast path passes, but we also want to log the core annotation every time. To break this loop, let's delay the logging to PSP.

Reviewed By: andrewvaslas

Differential Revision: D73955994

fbshipit-source-id: ba508f5833496875eb408be6b55059b056f7a5cb

Author: Kendall Hopkins

third-party/thrift/src/thrift/lib/hack/src/__tests__/ThriftPolicyEnforcerTest.php

@@ -398,6 +398,7 @@ public function testTrustedCallersBitmask(): void {
         )
         ->build(),
     );
+    await PSP()->genRunFromUnitTest(); // flush logs
     expect($mock)->wasCalledWithArgumentsPassingNTimes(
       ($_event, $_sample_rate, $specific_meta) ==> {
         return $specific_meta['thrift_caller_service_id'] === 'foo/bar';
@@ -425,6 +426,7 @@ public function testTrustedCallersBitmask(): void {
       )
     )->genBefore('fake_service', 'fake_method');
 
+    await PSP()->genRunFromUnitTest(); // flush logs
     expect($mock)->wasCalledWithArgumentsPassingNTimes(
       ($_event, $_sample_rate, $specific_meta) ==> {
         return $specific_meta['thrift_caller_service_id'] === 'sr_config_1';
@@ -456,6 +458,7 @@ public function testTrustedCallersBitmask(): void {
       )
     )->genBefore('fake_service', 'fake_method');
 
+    await PSP()->genRunFromUnitTest(); // flush logs
     expect($mock)->wasCalledWithArgumentsPassingNTimes(
       ($log, $_sample_rate) ==> {
         return Shapes::idx($log, 'client_id') === $client_id;