banner view wait for listeners in itest

Author: DellaBitta

admob/integration_test/src/integration_test.cc

@@ -636,6 +636,9 @@ TEST_F(FirebaseAdMobTest, TestBannerView) {
   app_framework::ProcessEvents(2000);
 
   WaitForCompletion(banner->Destroy(), "Destroy BannerView");
+
+  LogDebug("And again to ensure destruction callbacks are recorded.");
+  app_framework::ProcessEvents(2000);
   banner->SetBoundingBoxListener(nullptr);
   delete banner;
 
@@ -734,6 +737,9 @@ TEST_F(FirebaseAdMobTest, TestBannerViewAdOpenedAdClosed) {
     app_framework::ProcessEvents(1000);
   }
 
+  LogDebug("Waiting for a moment to ensure all callbacks are recorded.");
+  app_framework::ProcessEvents(2000);
+
   // Ensure all of the expected events were triggered on Android.
   EXPECT_EQ(ad_listener.num_on_ad_clicked_, 1);
   EXPECT_EQ(ad_listener.num_on_ad_impression_, 1);

admob/src/android/interstitial_ad_internal_android.cc

@@ -101,7 +101,8 @@ Future<LoadAdResult> InterstitialAdInternalAndroid::LoadAd(
 
   if (!initialized_) {
     SafeFutureHandle<LoadAdResult> handle =
-      future_data_.future_impl.SafeAlloc<LoadAdResult>(kInterstitialAdFnLoadAd, LoadAdResult());
+        future_data_.future_impl.SafeAlloc<LoadAdResult>(
+            kInterstitialAdFnLoadAd, LoadAdResult());
     CompleteFuture(kAdMobErrorUninitialized, kAdUninitializedErrorMessage,
                    handle, &future_data_, LoadAdResult());
     return MakeFuture(&future_data_.future_impl, handle);