Possible flaky test fix?

And removed unnecessary logging

Author: “Akshay

iterableapi/src/androidTest/java/com/iterable/iterableapi/IterableApiResponseTest.java

@@ -37,8 +37,14 @@ public class IterableApiResponseTest {
     private MockWebServer server;
 
     @Before
-    public void setUp() {
+    public void setUp() throws IOException {
         server = new MockWebServer();
+        // Explicitly start the server to ensure it's ready
+        try {
+            server.start();
+        } catch (IllegalStateException e) {
+            // Server may already be started by url() call below, which is fine
+        }
         IterableApi.overrideURLEndpointPath(server.url("").toString());
         createIterableApi();
     }
@@ -138,7 +144,7 @@ public void onFailure(@NonNull String reason, @Nullable JSONObject data) {
         new IterableRequestTask().execute(request);
 
         server.takeRequest(5, TimeUnit.SECONDS);
-        assertTrue("onFailure is called", signal.await(1, TimeUnit.SECONDS));
+        assertTrue("onFailure is called", signal.await(5, TimeUnit.SECONDS));
     }
 
 

iterableapi/src/main/java/com/iterable/iterableapi/IterableInAppFragmentHTMLNotification.java

@@ -198,9 +198,6 @@ public View onCreateView(@NonNull LayoutInflater inflater, @Nullable ViewGroup c
         webView = new IterableWebView(getContext());
         webView.setId(R.id.webView);
 
-        // Debug the HTML content
-        IterableLogger.d(TAG, "HTML content preview: " + (htmlString.length() > 200 ? htmlString.substring(0, 200) + "..." : htmlString));
-
         webView.createWithHtml(this, htmlString);
 
         if (orientationListener == null) {
@@ -241,7 +238,6 @@ public void run() {
         RelativeLayout webViewContainer = new RelativeLayout(this.getContext());
 
         int gravity = getVerticalLocation(insetPadding);
-        IterableLogger.d(TAG, "Initial setup - gravity: " + gravity + " for inset padding: " + insetPadding);
 
         // Set FrameLayout gravity based on positioning
         FrameLayout.LayoutParams containerParams = new FrameLayout.LayoutParams(
@@ -251,13 +247,10 @@ public void run() {
 
         if (gravity == Gravity.CENTER_VERTICAL) {
             containerParams.gravity = Gravity.CENTER;
-            IterableLogger.d(TAG, "Applied CENTER gravity to container");
         } else if (gravity == Gravity.TOP) {
             containerParams.gravity = Gravity.TOP | Gravity.CENTER_HORIZONTAL;
-            IterableLogger.d(TAG, "Applied TOP gravity to container");
         } else if (gravity == Gravity.BOTTOM) {
             containerParams.gravity = Gravity.BOTTOM | Gravity.CENTER_HORIZONTAL;
-            IterableLogger.d(TAG, "Applied BOTTOM gravity to container");
         }
 
         // Add WebView to the RelativeLayout container with WRAP_CONTENT for proper sizing
@@ -268,13 +261,9 @@ public void run() {
         webViewParams.addRule(RelativeLayout.CENTER_IN_PARENT);
         webViewContainer.addView(webView, webViewParams);
 
-        IterableLogger.d(TAG, "Added WebView with WRAP_CONTENT and CENTER_IN_PARENT rule");
-
         // Add the container to the FrameLayout
         frameLayout.addView(webViewContainer, containerParams);
 
-        IterableLogger.d(TAG, "Created FrameLayout with positioned RelativeLayout container");
-
         if (savedInstanceState == null || !savedInstanceState.getBoolean(IN_APP_OPEN_TRACKED, false)) {
             IterableApi.sharedInstance.trackInAppOpen(messageId, location);
         }