This is an automated email from the ASF dual-hosted git repository.

davsclaus pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/camel.git


The following commit(s) were added to refs/heads/main by this push:
     new 8098f39f2ec Remove tests as its allowed now due to CAMEL-16374
8098f39f2ec is described below

commit 8098f39f2ec7d0ab55141ec6c5fd83297cf144f7
Author: Claus Ibsen <claus.ib...@gmail.com>
AuthorDate: Sun Nov 19 14:01:42 2023 +0100

    Remove tests as its allowed now due to CAMEL-16374
---
 .../SpringOnCompletionCompleteAndFailureTest.java  | 50 ---------------------
 .../SpringOnCompletionCompleteAndFailureTest.xml   | 40 -----------------
 .../OnCompletionCompleteAndFailureTest.java        | 51 ----------------------
 3 files changed, 141 deletions(-)

diff --git 
a/components/camel-spring-xml/src/test/java/org/apache/camel/spring/processor/SpringOnCompletionCompleteAndFailureTest.java
 
b/components/camel-spring-xml/src/test/java/org/apache/camel/spring/processor/SpringOnCompletionCompleteAndFailureTest.java
deleted file mode 100644
index b334928e0ab..00000000000
--- 
a/components/camel-spring-xml/src/test/java/org/apache/camel/spring/processor/SpringOnCompletionCompleteAndFailureTest.java
+++ /dev/null
@@ -1,50 +0,0 @@
-/*
- * Licensed to the Apache Software Foundation (ASF) under one or more
- * contributor license agreements.  See the NOTICE file distributed with
- * this work for additional information regarding copyright ownership.
- * The ASF licenses this file to You under the Apache License, Version 2.0
- * (the "License"); you may not use this file except in compliance with
- * the License.  You may obtain a copy of the License at
- *
- *      http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- */
-package org.apache.camel.spring.processor;
-
-import org.apache.camel.spring.SpringTestSupport;
-import org.junit.jupiter.api.Assertions;
-import org.junit.jupiter.api.Test;
-import org.springframework.context.support.AbstractXmlApplicationContext;
-import org.springframework.context.support.ClassPathXmlApplicationContext;
-
-import static org.junit.jupiter.api.Assertions.assertEquals;
-
-public class SpringOnCompletionCompleteAndFailureTest extends 
SpringTestSupport {
-
-    @Override
-    public void setUp() throws Exception {
-        try {
-            super.setUp();
-            Assertions.fail("Should throw exception");
-        } catch (Exception e) {
-            assertEquals("Only 1 onCompletion is allowed per route.", 
e.getMessage());
-        }
-    }
-
-    @Override
-    protected AbstractXmlApplicationContext createApplicationContext() {
-        return new ClassPathXmlApplicationContext(
-                
"org/apache/camel/spring/processor/SpringOnCompletionCompleteAndFailureTest.xml");
-    }
-
-    @Test
-    public void testOnCompletion() throws Exception {
-        // noop
-    }
-
-}
diff --git 
a/components/camel-spring-xml/src/test/resources/org/apache/camel/spring/processor/SpringOnCompletionCompleteAndFailureTest.xml
 
b/components/camel-spring-xml/src/test/resources/org/apache/camel/spring/processor/SpringOnCompletionCompleteAndFailureTest.xml
deleted file mode 100644
index f513e947a70..00000000000
--- 
a/components/camel-spring-xml/src/test/resources/org/apache/camel/spring/processor/SpringOnCompletionCompleteAndFailureTest.xml
+++ /dev/null
@@ -1,40 +0,0 @@
-<?xml version="1.0" encoding="UTF-8"?>
-<!--
-
-    Licensed to the Apache Software Foundation (ASF) under one or more
-    contributor license agreements.  See the NOTICE file distributed with
-    this work for additional information regarding copyright ownership.
-    The ASF licenses this file to You under the Apache License, Version 2.0
-    (the "License"); you may not use this file except in compliance with
-    the License.  You may obtain a copy of the License at
-
-         http://www.apache.org/licenses/LICENSE-2.0
-
-    Unless required by applicable law or agreed to in writing, software
-    distributed under the License is distributed on an "AS IS" BASIS,
-    WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-    See the License for the specific language governing permissions and
-    limitations under the License.
-
--->
-<beans xmlns="http://www.springframework.org/schema/beans";
-       xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance";
-       xsi:schemaLocation="
-       http://www.springframework.org/schema/beans 
http://www.springframework.org/schema/beans/spring-beans.xsd
-       http://camel.apache.org/schema/spring 
http://camel.apache.org/schema/spring/camel-spring.xsd
-    ">
-
-    <camelContext xmlns="http://camel.apache.org/schema/spring";>
-    <jmxAgent id="jmx" disabled="true"/>
-        <route>
-            <from uri="direct:start"/>
-            <onCompletion onCompleteOnly="true">
-                <to uri="mock:ok"/>
-            </onCompletion>
-            <onCompletion onFailureOnly="true">
-                <to uri="mock:error"/>
-            </onCompletion>
-            <to uri="mock:result"/>
-        </route>
-    </camelContext>
-</beans>
diff --git 
a/core/camel-core/src/test/java/org/apache/camel/processor/OnCompletionCompleteAndFailureTest.java
 
b/core/camel-core/src/test/java/org/apache/camel/processor/OnCompletionCompleteAndFailureTest.java
deleted file mode 100644
index 3d6a4fdf5f9..00000000000
--- 
a/core/camel-core/src/test/java/org/apache/camel/processor/OnCompletionCompleteAndFailureTest.java
+++ /dev/null
@@ -1,51 +0,0 @@
-/*
- * Licensed to the Apache Software Foundation (ASF) under one or more
- * contributor license agreements.  See the NOTICE file distributed with
- * this work for additional information regarding copyright ownership.
- * The ASF licenses this file to You under the Apache License, Version 2.0
- * (the "License"); you may not use this file except in compliance with
- * the License.  You may obtain a copy of the License at
- *
- *      http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- */
-package org.apache.camel.processor;
-
-import org.apache.camel.ContextTestSupport;
-import org.apache.camel.builder.RouteBuilder;
-import org.junit.jupiter.api.Test;
-
-import static org.junit.jupiter.api.Assertions.assertEquals;
-
-public class OnCompletionCompleteAndFailureTest extends ContextTestSupport {
-
-    @Override
-    public boolean isUseRouteBuilder() {
-        return false;
-    }
-
-    @Test
-    public void testMultipleRouteOnComplete() throws Exception {
-        try {
-            context.addRoutes(new RouteBuilder() {
-                @Override
-                public void configure() throws Exception {
-                    from("direct:start")
-                            
.onCompletion().onCompleteOnly().to("mock:ok").end()
-                            
.onCompletion().onFailureOnly().to("mock:error").end()
-                            .onCompletion().to("mock:all").end()
-                            .to("mock:result");
-                }
-            });
-            //fail("Should throw exception"); Now allowed: CAMEL-16374
-        } catch (IllegalArgumentException e) {
-            assertEquals("Only 1 onCompletion is allowed per route.", 
e.getMessage());
-        }
-    }
-
-}

Reply via email to