Clean up helper methods

Author: sarahchen6

dd-java-agent/instrumentation/play-2.3/src/test/groovy/datadog/trace/instrumentation/play23/test/client/PlayWSClientTest.groovy

@@ -94,7 +94,7 @@ class PlayWSClientTest extends HttpClientTest implements TestingNettyHttpNamingC
       tagsAssert.tag("error.stack", String)
       tagsAssert.tag("error.message", String)
     } else {
-      assertErrorTags(tagsAssert, exception)
+      super.assertErrorTags(tagsAssert, exception)
     }
   }
 }

dd-java-agent/instrumentation/play-2.4/src/test/groovy/datadog/trace/instrumentation/play25/client/PlayWSClientTest.groovy

@@ -72,7 +72,7 @@ abstract class PlayWSClientTest extends HttpClientTest {
       tagsAssert.tag("error.stack", String)
       tagsAssert.tag("error.message", String)
     } else {
-      assertErrorTags(tagsAssert, exception)
+      super.assertErrorTags(tagsAssert, exception)
     }
   }
 }

dd-java-agent/instrumentation/play-ws-1/src/test/groovy/PlayWSClientTest.groovy

@@ -113,12 +113,7 @@ class PlayScalaStreamedWSClientTest extends PlayWSClientTestBase {
       .withHttpHeaders(JavaConverters.mapAsScalaMap(headers).toSeq())
       .stream()
       .transform({ theTry ->
-        if (callback != null) {
-          // Execute callback in a separate thread to clear trace context
-          def thread = new Thread({ callback.call() })
-          thread.start()
-          thread.join()
-        }
+        runInAdHocThread(callback)
         theTry
       }, ExecutionContext.global())
 

dd-java-agent/instrumentation/play-ws/src/test/groovy/PlayWSClientTestBase.groovy

@@ -96,7 +96,7 @@ abstract class PlayWSClientTestBase extends HttpClientTest {
       tagsAssert.tag("error.stack", String)
       tagsAssert.tag("error.message", String)
     } else {
-      assertErrorTags(tagsAssert, exception)
+      super.assertErrorTags(tagsAssert, exception)
     }
   }