[ 
https://issues.apache.org/jira/browse/OPENJPA-1557?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12850164#action_12850164
 ] 

Michael Dick commented on OPENJPA-1557:
---------------------------------------

1.3.x has been building for me with Donald's last round of changes, fwiw. 

I'll take a look again this weekend and maybe merge to 1.2.x / 1.0.x. 

> Logging configuration is difficult for running tests
> ----------------------------------------------------
>
>                 Key: OPENJPA-1557
>                 URL: https://issues.apache.org/jira/browse/OPENJPA-1557
>             Project: OpenJPA
>          Issue Type: Improvement
>    Affects Versions: 1.2.2, 2.0.0-beta2
>            Reporter: Craig L Russell
>            Assignee: Craig L Russell
>             Fix For: 1.3.0, 2.0.0-beta3
>
>         Attachments: OPENJPA-1557.diff.txt
>
>
> Logging is difficult to configure and is incorrectly documented for running 
> tests.
> There is a system property that can be set on the command line when running 
> tests using surefire: openjpa.loglevel. This property, set via 
> -Dopenjpa.loglevel, is used in the pom.xml to send logging properties to the 
> runtime. The simple case works, e.g. -Dopenjpa.loglevel=error. This affects 
> enhancement and the test runtime to report only errors. 
> However, setting this property also results in warnings:
> enhance.all.entities:
>      [echo]  running enhancer
> 12  WARN   [main] openjpa.Runtime - The configuration property named 
> "openjpa.loglevel" was not recognized and will be ignored, although the name 
> closely matches a valid property called "openjpa.Log".
> 2  WARN   [main] openjpa.Runtime - The configuration property named 
> "openjpa.loglevel" was not recognized and will be ignored, although the name 
> closely matches a valid property called "openjpa.Log".
> This is because the property is both passed to the runtime and also used to 
> configure the log level via the surefire plugin (the relevant part of the 
> pom.xml follows):
>                         <property>
>                             <name>openjpa.Log</name>
>                             <value>DefaultLevel=${openjpa.loglevel}</value>
>                         </property>
> Via "property injection" it's also possible to set any number of other log 
> properties using this mechanism, although it's a bit odd. Just concatenate 
> the extra properties after the level. For example,
> mvn test -Dopenjpa.loglevel=trace,file=openjpa.log,Runtime=info
> This will set the default log level to trace, send the log output to the file 
> openjpa.log, and set the Runtime log level to info.
> The surefire plugin effectively disables the openjpa.Log so the user cannot 
> use this property.
> The warning can be removed by changing the PersistenceTestCase.createNamedEMF:
>         Map map = new HashMap(System.getProperties());
>         map.remove("openjpa.level");
> But the enhancer doesn't go through this path.
> Before I change the online documentation I'd like to see if anyone else has 
> any ideas how to make this easier.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to