Skip to main content
aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorPaul Pazderski2019-11-06 13:45:06 +0000
committerPaul Pazderski2019-11-06 14:10:42 +0000
commit1f4e4bc45d44e6e7f0a416750dbb166fa1784b87 (patch)
tree79fe4d55f72f3d36dff4c38bd6bba57d162ef994
parentcb886eecb5f665d6955c50857c164d50abc2cf88 (diff)
downloadeclipse.platform.debug-Y20191107-0440.tar.gz
eclipse.platform.debug-Y20191107-0440.tar.xz
eclipse.platform.debug-Y20191107-0440.zip
Before 'asserting' in tearDown do the cleanup stuff which would not be executed if the assertion fails. Change-Id: I9f72d0a073957aea4d216be4cff58729494bbe82 Signed-off-by: Paul Pazderski <paul-eclipse@ppazderski.de>
-rw-r--r--org.eclipse.debug.tests/src/org/eclipse/debug/tests/console/IOConsoleTests.java3
-rw-r--r--org.eclipse.debug.tests/src/org/eclipse/debug/tests/console/ProcessConsoleTests.java3
2 files changed, 4 insertions, 2 deletions
diff --git a/org.eclipse.debug.tests/src/org/eclipse/debug/tests/console/IOConsoleTests.java b/org.eclipse.debug.tests/src/org/eclipse/debug/tests/console/IOConsoleTests.java
index 0962231bd..ce203b915 100644
--- a/org.eclipse.debug.tests/src/org/eclipse/debug/tests/console/IOConsoleTests.java
+++ b/org.eclipse.debug.tests/src/org/eclipse/debug/tests/console/IOConsoleTests.java
@@ -109,9 +109,10 @@ public class IOConsoleTests extends AbstractDebugTest {
@Override
protected void tearDown() throws Exception {
- assertEquals("Test triggered errors in IOConsole.", 0, loggedErrors.get());
Platform.removeLogListener(errorLogListener);
super.tearDown();
+
+ assertEquals("Test triggered errors in IOConsole.", 0, loggedErrors.get());
}
/**
diff --git a/org.eclipse.debug.tests/src/org/eclipse/debug/tests/console/ProcessConsoleTests.java b/org.eclipse.debug.tests/src/org/eclipse/debug/tests/console/ProcessConsoleTests.java
index 2227d43e7..10ff24470 100644
--- a/org.eclipse.debug.tests/src/org/eclipse/debug/tests/console/ProcessConsoleTests.java
+++ b/org.eclipse.debug.tests/src/org/eclipse/debug/tests/console/ProcessConsoleTests.java
@@ -82,9 +82,10 @@ public class ProcessConsoleTests extends AbstractDebugTest {
@Override
protected void tearDown() throws Exception {
- assertEquals("Test triggered errors.", 0, loggedErrors.get());
Platform.removeLogListener(errorLogListener);
super.tearDown();
+
+ assertEquals("Test triggered errors.", 0, loggedErrors.get());
}
/**

Back to the top