[jira] [Commented] (CAMEL-9313) CamelBlueprintTestSupport - can't initialize ConfigAdmin configurations

["Grzegorz Grzybek (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/CAMEL-9313?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15054579#comment-15054579 ] 

Grzegorz Grzybek commented on CAMEL-9313:
-----------------------------------------

Resolved in:
* master branch (2.17): https://github.com/apache/camel/commit/13760a4a5f1672d54997aa8ede30aefed951505a
* camel-2.16.x branch: https://github.com/apache/camel/commit/2410bfa5b840fd6dd7de86b32fd3e0dee35061c9
* camel-2.15.x branch: https://github.com/apache/camel/commit/bd900b3ce2892ac53830179cce5a1576e1ba2915

> CamelBlueprintTestSupport - can't initialize ConfigAdmin configurations
> -----------------------------------------------------------------------
>
>                 Key: CAMEL-9313
>                 URL: https://issues.apache.org/jira/browse/CAMEL-9313
>             Project: Camel
>          Issue Type: Bug
>          Components: camel-blueprint
>    Affects Versions: 2.15.4
>            Reporter: Andreas Siepert
>            Assignee: Grzegorz Grzybek
>             Fix For: 2.15.6, 2.16.2, 2.17.0
>
>         Attachments: camel-9313.patch
>
>
> The bugfix CAMEL-8948 seems to make older timing problems related to property-placeholders visible.
> To reproduce the problem i changed the test 
> org.apache.camel.test.blueprint.ConfigAdminNoReloadLoadConfigurationFileTest 
> from the component camel-test-blueprint a bit, respectively the context 
> "org/apache/camel/test/blueprint/configadmin-no-reload-loadfile.xml": 
> I added the trace Attribute to the camelContext:
> {code:xml}
> <camelContext xmlns="http://camel.apache.org/schema/blueprint" trace="{{tracing}}"> 
> {code}
> and added also the property to the etc/stuff.cfg 
> {code}
> tracing=true 
> {code}
> Until 2.15.2 this worked fine. From 2.15.3 on the property cannot be 
> replaced any more.
> But, if setting a breakpoint in {{CamelBlueprintTestSupport#createBundleContext}} at loadConfigAdminConfigurationFile() (Line 123 in 2.15.4) - the error occurs even in older versions like 2.14 - so the timing problem seems to be there for a while but did not occur because the loading of the configAdminFile seems to be faster than the event handling during service registration triggered by the code some lines above.
> The issue can also be reproduced when replacing a property's String type 
> with int in the MyCoolBean class and setting its value by using the 
> placeholder like before but with an int value of course. The test run shows 
> that the placeholder "${..\}" will not be replaced and leads to a 
> NumberfFormatException. 
> The production code that is under test works fine in karaf. 



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)