Deprecated bits related to development of AbstractTestSupport

jamesnetherton · Sep 30, 2024 · 35b3551 · 35b3551
1 parent f2e4fb8
commit 35b3551
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 9 deletions.
diff --git a/docs/modules/ROOT/pages/migration-guide/3.15.0.adoc b/docs/modules/ROOT/pages/migration-guide/3.15.0.adoc
@@ -34,4 +34,6 @@ The following methods were mistakenly not marked as deprecated in 3.15.0:
 * `doPostSetup`
 * `postProcessTest`
 
-This will be corrected in the 3.15.1 release.
+This will be corrected in the 3.15.1 release.
+
+Using JUnit Lifecycle.PER_CLASS in conjunction with CamelQuarkusTestSupport is deprecated and will not be supported in future releases.
diff --git a/...framework/junit5/src/main/java/org/apache/camel/quarkus/test/CamelQuarkusTestSupport.java b/...framework/junit5/src/main/java/org/apache/camel/quarkus/test/CamelQuarkusTestSupport.java
@@ -199,6 +199,7 @@ protected void bindToRegistry(Registry registry) throws Exception {
     /**
      * The same functionality as {@link CamelTestSupport#postProcessTest()} .
      */
+    @Deprecated(since = "3.15.0")
     protected void postProcessTest() throws Exception {
         assertTestClassCamelContextMatchesAppCamelContext();
 
@@ -215,7 +216,7 @@ public CamelContext context() {
         return this.context;
     }
 
-    @Deprecated(since = "4.7.0")
+    @Deprecated(since = "3.15.0")
     public long timeTaken() {
         return watch.taken();
     }
@@ -233,7 +234,7 @@ public final String getCurrentTestName() {
      * @deprecated           Use {@link #setupResources()} instead
      * @throws     Exception if unable to setup the test
      */
-    @Deprecated(since = "4.7.0")
+    @Deprecated(since = "3.15.0")
     public void setUp() throws Exception {
         ExtensionHelper.testStartHeader(getClass(), currentTestName);
 
@@ -266,7 +267,7 @@ public void setUp() throws Exception {
      * @deprecated           Use {@link #cleanupResources()} instead
      * @throws     Exception if unable to setup the test
      */
-    @Deprecated(since = "4.7.0")
+    @Deprecated(since = "3.15.0")
     public void tearDown() throws Exception {
         long time = watch.taken();
 
@@ -297,7 +298,7 @@ public void tearDown() throws Exception {
      *
      * @deprecated use {@link #cleanupResources()} instead.
      */
-    @Deprecated(since = "4.7.0")
+    @Deprecated(since = "3.15.0")
     protected void doPostTearDown() throws Exception {
         // noop
     }
@@ -321,7 +322,7 @@ public void configure() {
      * @see        #createRouteBuilder()
      * @deprecated This method will be made private. Do not use
      */
-    @Deprecated(since = "4.7.0")
+    @Deprecated(since = "3.15.0")
     protected RoutesBuilder[] createRouteBuilders() throws Exception {
         return new RoutesBuilder[] { createRouteBuilder() };
     }
@@ -374,6 +375,7 @@ void internalBeforeEach(ExtensionContext context) throws Exception {
      * If this method is overridden, <i>super.doPreSetup()</i> must be called.
      * </p>
      */
+    @Deprecated(since = "3.15.0")
     protected void doPreSetup() throws Exception {
         if (isUseAdviceWith() || isUseDebugger()) {
             ((FastCamelContext) context).suspend();
@@ -395,6 +397,7 @@ protected void doPreSetup() throws Exception {
      * If this method is overridden, <i>super.doPostSetup()</i> must be called.
      * </p>
      */
+    @Deprecated(since = "3.15.0")
     protected void doPostSetup() throws Exception {
         if (isUseAdviceWith() || isUseDebugger()) {
             ((FastCamelContext) context).resume();
@@ -429,6 +432,7 @@ protected void doPostSetup() throws Exception {
      * @return <code>true</code> to apply advice to existing route(s). <code>false</code> to disable advice.
      */
     @Override
+    @Deprecated(since = "3.15.0")
     public boolean isUseAdviceWith() {
         return false;
     }
@@ -467,7 +471,7 @@ protected final MockEndpoint getMockEndpoint(String uri) {
      *                                 be resolved
      * @throws NoSuchEndpointException is the mock endpoint does not exist
      */
-    @Deprecated(since = "4.7.0")
+    @Deprecated(since = "3.15.0")
     protected final MockEndpoint getMockEndpoint(String uri, boolean create) throws NoSuchEndpointException {
         return TestSupport.getMockEndpoint(context, uri, create);
     }
@@ -477,7 +481,7 @@ protected final MockEndpoint getMockEndpoint(String uri, boolean create) throws
      *
      * @deprecated Use {@link #camelContextConfiguration()} to set an instance of {@link DebugBreakpoint}
      */
-    @Deprecated(since = "4.7.0")
+    @Deprecated(since = "3.15.0")
     protected void debugBefore(
             Exchange exchange, Processor processor, ProcessorDefinition<?> definition, String id, String label) {
     }
@@ -487,7 +491,7 @@ protected void debugBefore(
      *
      * @deprecated Use {@link #camelContextConfiguration()} to set an instance of {@link DebugBreakpoint}
      */
-    @Deprecated(since = "4.7.0")
+    @Deprecated(since = "3.15.0")
     protected void debugAfter(
             Exchange exchange, Processor processor, ProcessorDefinition<?> definition, String id, String label,
             long timeTaken) {