diff options
author | Paul Pazderski | 2019-11-06 13:45:06 +0000 |
---|---|---|
committer | Paul Pazderski | 2019-11-06 14:10:42 +0000 |
commit | 1f4e4bc45d44e6e7f0a416750dbb166fa1784b87 (patch) | |
tree | 79fe4d55f72f3d36dff4c38bd6bba57d162ef994 | |
parent | cb886eecb5f665d6955c50857c164d50abc2cf88 (diff) | |
download | eclipse.platform.debug-1f4e4bc45d44e6e7f0a416750dbb166fa1784b87.tar.gz eclipse.platform.debug-1f4e4bc45d44e6e7f0a416750dbb166fa1784b87.tar.xz eclipse.platform.debug-1f4e4bc45d44e6e7f0a416750dbb166fa1784b87.zip |
Improve tear down in IOConsoleTests and ProcessConsoleTestsY20191107-0440Y20191107-0420I20191109-0650I20191109-0330I20191108-1800I20191107-1800I20191106-1805
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.java | 3 | ||||
-rw-r--r-- | org.eclipse.debug.tests/src/org/eclipse/debug/tests/console/ProcessConsoleTests.java | 3 |
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()); } /** |