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

ASF GitHub Bot commented on GROOVY-12573:
-----------------------------------------

codeconsole commented on PR #3101:
URL: https://github.com/apache/groovy/pull/3101#issuecomment-6526527072

   @daniellansun Merged master, which starts a new CI run (a fork author cannot 
re-run jobs).




> LogTest loses its spy handler when the logger is garbage collected before the 
> class under test logs
> ---------------------------------------------------------------------------------------------------
>
>                 Key: GROOVY-12573
>                 URL: https://issues.apache.org/jira/browse/GROOVY-12573
>             Project: Groovy
>          Issue Type: Test
>            Reporter: Scott Murphy Heiberg
>            Priority: Major
>
> {{LogTest.registerLogFormatterSpy}} adds a handler to 
> {{Logger.getLogger(name)}} without keeping the logger. The {{LogManager}} 
> holds loggers weakly, so a garbage collection before the class under test 
> initializes drops the logger, and the class gets a new one without the spy; 
> {{testDefaultCategory}} then fails (seen on CI).
> PR: https://github.com/apache/groovy/pull/3101



--
This message was sent by Atlassian Jira
(v10.3.26#10030026)

Reply via email to