refactor: deprecate RestStepListener (#3540) (#3543)

since it no longer does anything Co-authored-by: James Shaw <[email protected]>
serenity-bdd · Sep 25, 2024 · f0f17ed · f0f17ed
1 parent 6fc140c
commit f0f17ed
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 5 deletions.
diff --git a/serenity-rest-assured/src/main/java/net/serenitybdd/rest/RestStepListener.java b/serenity-rest-assured/src/main/java/net/serenitybdd/rest/RestStepListener.java
@@ -2,6 +2,10 @@
 
 import net.thucydides.model.steps.StepListenerAdapter;
 
+/**
+ * @deprecated With no replacement, since {@code RestStepListener} no longer performs any operations.
+ */
+@Deprecated(since="4.2.2")
 public class RestStepListener extends StepListenerAdapter  {
 
 }
diff --git a/serenity-rest-assured/src/main/java/net/serenitybdd/rest/utils/RestReportingHelper.java b/serenity-rest-assured/src/main/java/net/serenitybdd/rest/utils/RestReportingHelper.java
@@ -9,7 +9,6 @@
 import net.serenitybdd.core.Serenity;
 import net.serenitybdd.model.rest.RestMethod;
 import net.serenitybdd.model.rest.RestQuery;
-import net.serenitybdd.rest.RestStepListener;
 import net.serenitybdd.rest.decorators.request.RequestSpecificationDecorated;
 import net.serenitybdd.rest.event.RecordRestQueryEvent;
 import net.serenitybdd.rest.filters.FieldsRecordingFilter;
@@ -39,10 +38,6 @@
  */
 public class RestReportingHelper {
 
-    public RestReportingHelper() {
-        getParallelEventBus().registerListener(new RestStepListener());
-    }
-
     private static boolean shouldRecordResponseBodyFor(Response result) {
         final ContentType type = ContentType.fromContentType(result.contentType());
         return type != null && (ContentType.JSON == type || ContentType.XML == type