use ScreenshotOnFailure TestWatcher

Author: EcljpseB0T

tests/org.eclipse.swt.tests/JUnit Tests/org/eclipse/swt/tests/junit/Test_org_eclipse_swt_custom_StyledText.java

@@ -103,13 +103,8 @@ public class Test_org_eclipse_swt_custom_StyledText extends Test_org_eclipse_swt
 private boolean listenerCalled;
 private boolean listener2Called;
 
-@Rule public TestWatcher screenshotRule = new TestWatcher() {
-	@Override
-	protected void failed(Throwable e, org.junit.runner.Description description) {
-		super.failed(e, description);
-		Screenshots.takeScreenshot(description.getTestClass(), description.getMethodName());
-	}
-};
+@Rule
+public TestWatcher screenshotRule = Screenshots.onFailure();
 
 @Override
 @Before

tests/org.eclipse.swt.tests/JUnit Tests/org/eclipse/swt/tests/junit/Test_org_eclipse_swt_custom_StyledText_multiCaretsSelections.java

@@ -40,13 +40,7 @@
 public class Test_org_eclipse_swt_custom_StyledText_multiCaretsSelections {
 
 	@Rule
-	public TestWatcher screenshotRule = new TestWatcher() {
-		@Override
-		protected void failed(Throwable e, org.junit.runner.Description description) {
-			super.failed(e, description);
-			Screenshots.takeScreenshot(description.getTestClass(), description.getMethodName());
-		}
-	};
+	public TestWatcher screenshotRule = Screenshots.onFailure();
 
 	Shell shell;
 	StyledText text;

tests/org.eclipse.swt.tests/JUnit Tests/org/eclipse/swt/tests/junit/Test_org_eclipse_swt_widgets_Tree.java

@@ -38,6 +38,7 @@
 import org.junit.Rule;
 import org.junit.Test;
 import org.junit.rules.TestName;
+import org.junit.rules.TestWatcher;
 
 /**
  * Automated Test Suite for class org.eclipse.swt.widgets.Tree
@@ -915,6 +916,9 @@ public void test_disposeItemNotTriggerSelection() {
 	assertFalse(selectionCalled[0]);
 }
 
+@Rule
+public TestWatcher screenshotRule = Screenshots.onFailure();
+
 @Test
 public void test_Virtual() {
 	tree.dispose();
@@ -964,7 +968,6 @@ public void test_Virtual() {
 	}
 	// temp code to capture screenshot
 	if (SwtTestUtil.isCocoa) {
-		Screenshots.takeScreenshot(getClass(), testName.getMethodName());
 		// check if setData is called for root item
 		assertTrue("SetData not called for top item", top[0] != null);
 	}