Sporadically failing Browser Tests logging #1466

amartya4256 · amartya4256 · commit d35c5c532975 · 2025-05-23T09:25:29.000+02:00
This commit adds relevant loggings for debugging purposes to the Browser tests which sporadically fail for Edge Browser. contributes to #2098, #2113 and #2063
diff --git a/tests/org.eclipse.swt.tests/JUnit Tests/org/eclipse/swt/tests/junit/Test_org_eclipse_swt_browser_Browser.java b/tests/org.eclipse.swt.tests/JUnit Tests/org/eclipse/swt/tests/junit/Test_org_eclipse_swt_browser_Browser.java
@@ -587,10 +587,14 @@ public void test_LocationListener_changing() {
 @Test
 public void test_LocationListener_changed() {
 	AtomicBoolean changedFired = new AtomicBoolean(false);
-	browser.addLocationListener(changedAdapter(e ->	changedFired.set(true)));
+	browser.addLocationListener(changedAdapter(e ->	{
+			changedFired.set(true);
+			System.out.println(e.location);
+		}));
 	shell.open();
 	browser.setText("Hello world");
 	boolean passed = waitForPassCondition(changedFired::get);
+	System.out.println(browser.getUrl());
 	assertTrue("LocationListener.changed() event was never fired", passed);
 }
 @Test
@@ -658,6 +662,7 @@ public void test_LocationListener_then_ProgressListener() {
 	browser.addLocationListener(changedAdapter(event ->	locationChanged.set(true)));
 
 	browser.addProgressListener(completedAdapter(event -> {
+		System.out.println(browser.getText());
 		if (locationChanged.get()) {
 			progressChangedAfterLocationChanged.set(true);
 		}
@@ -672,7 +677,7 @@ public void test_LocationListener_then_ProgressListener() {
 			+ "Location changed: " + locationChanged.get() + "\n"
 			+ "ProgressChangedAfterLocationChanged: " + progressChangedAfterLocationChanged.get() + "\n"
 			+ "progressChanged: " + progressChanged.get();
-
+	System.out.println(browser.getText());
 	assertTrue(errorMsg, progressChangedAfterLocationChanged.get());
 }
 
@@ -995,13 +1000,14 @@ public void completed(ProgressEvent event) {
 
 		@Override
 		public void changed(ProgressEvent event) {
-
+			System.out.println(browser.getText());
 		}
 	};
 	browser.addProgressListener(l);
 	browser.setText("<html><body>This test ensures that the completed listener is called.</body></html>");
 	shell.open();
 	boolean passed = waitForPassCondition(childCompleted::get);
+	System.out.println(browser.getUrl());
 	assertTrue(passed);
 }
 
