[jira] [Created] (CAMEL-11763) Some camel-milo unit tests pass even when exceptions are thrown

["James Netherton (JIRA)" <ji...@apache.org>]

James Netherton created CAMEL-11763:
---------------------------------------

             Summary: Some camel-milo unit tests pass even when exceptions are thrown 
                 Key: CAMEL-11763
                 URL: https://issues.apache.org/jira/browse/CAMEL-11763
             Project: Camel
          Issue Type: Test
          Components: camel-milo
    Affects Versions: 2.20.0
            Reporter: James Netherton
            Priority: Minor


I've been running the Milo [ServerLocalTest|https://github.com/apache/camel/blob/master/components/camel-milo/src/test/java/org/apache/camel/component/milo/server/ServerLocalTest.java]. There's a few tests that use the 'Variant' type as the message body. Although the tests seem to pass, they shouldn't be because an exception is thrown in the background:

{code}
java.lang.IllegalArgumentException: Variant cannot contain Variant
	at com.google.common.base.Preconditions.checkArgument(Preconditions.java:122)
	at org.eclipse.milo.opcua.stack.core.types.builtin.Variant.<init>(Variant.java:49)
	at org.apache.camel.component.milo.server.internal.CamelServerItem.update(CamelServerItem.java:140)
	at org.apache.camel.component.milo.server.MiloServerProducer.process(MiloServerProducer.java:36)
	at org.apache.camel.util.AsyncProcessorConverterHelper$ProcessorToAsyncProcessorBridge.process(AsyncProcessorConverterHelper.java:61)
	at org.apache.camel.processor.SharedCamelInternalProcessor.process(SharedCamelInternalProcessor.java:186)
	at org.apache.camel.processor.SharedCamelInternalProcessor.process(SharedCamelInternalProcessor.java:86)
	at org.apache.camel.impl.ProducerCache$1.doInProducer(ProducerCache.java:541)
	at org.apache.camel.impl.ProducerCache$1.doInProducer(ProducerCache.java:506)
	at org.apache.camel.impl.ProducerCache.doInProducer(ProducerCache.java:369)
	at org.apache.camel.impl.ProducerCache.sendExchange(ProducerCache.java:506)
	at org.apache.camel.impl.ProducerCache.send(ProducerCache.java:229)
	at org.apache.camel.impl.DefaultProducerTemplate.send(DefaultProducerTemplate.java:144)
	at org.apache.camel.impl.DefaultProducerTemplate.send(DefaultProducerTemplate.java:130)
	at org.apache.camel.test.junit4.CamelTestSupport.sendBody(CamelTestSupport.java:792)
	at org.apache.camel.component.milo.server.ServerLocalTest.testAcceptVariantString(ServerLocalTest.java:67)
	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.lang.reflect.Method.invoke(Method.java:498)
	at org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:50)
	at org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12)
	at org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java:47)
	at org.junit.internal.runners.statements.InvokeMethod.evaluate(InvokeMethod.java:17)
	at org.junit.internal.runners.statements.RunBefores.evaluate(RunBefores.java:26)
	at org.junit.internal.runners.statements.RunAfters.evaluate(RunAfters.java:27)
{code}

If you tweak the tests to verify that a message was received by the mock endpoint, then it'll fail because of this.

I'm no Milo expert, so my knowledge here is limited.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)