server: Separate @before and @after methods in Bookmarks unit tests

Until now, before running each individual unit tests, a new experiment would've been created and all its previous (if any) bookmarks were deleted. Now, using @before, a new experiment will be created upon running each individual test, and then, using @after, all the data related to the bookmarks will be deleted. Signed-off-by: Kaveh Shahedi <[email protected]>
eclipse-tracecompass-incubator · Nov 15, 2024 · 35c3434 · 35c3434
1 parent 0044363
commit 35c3434
Showing 1 changed file with 9 additions and 3 deletions.
diff --git a/...ss/incubator/trace/server/jersey/rest/core/tests/services/BookmarkManagerServiceTest.java b/...ss/incubator/trace/server/jersey/rest/core/tests/services/BookmarkManagerServiceTest.java
@@ -26,6 +26,7 @@
 import org.eclipse.tracecompass.incubator.trace.server.jersey.rest.core.tests.stubs.BookmarkModelStub;
 import org.eclipse.tracecompass.incubator.trace.server.jersey.rest.core.tests.stubs.ExperimentModelStub;
 import org.eclipse.tracecompass.incubator.trace.server.jersey.rest.core.tests.utils.RestServerTest;
+import org.junit.After;
 import org.junit.Before;
 import org.junit.Test;
 
@@ -46,8 +47,7 @@ public class BookmarkManagerServiceTest extends RestServerTest {
     private static final String END = "end";
 
     /**
-     * Setup method to run before each test. Creates a clean experiment and removes all
-     * existing bookmarks.
+     * Setup method to run before each test
      */
     @Before
     public void setUp() {
@@ -56,8 +56,14 @@ public void setUp() {
                                        CONTEXT_SWITCHES_UST_NOT_INITIALIZED_STUB);
         assertNotNull("Experiment should not be null", experiment);
         assertNotNull("Experiment UUID should not be null", experiment.getUUID());
+    }
 
-        // Get all existing bookmarks and delete them
+    /**
+     * Tear down method to run after each test
+     */
+    @After
+    public void tearDown() {
+        // Remove all bookmarks
         WebTarget application = getApplicationEndpoint();
         WebTarget bookmarkTarget = application.path(EXPERIMENTS)
                 .path(experiment.getUUID().toString())