[jira] [Commented] (GROOVY-12573) LogTest loses its spy handler when the logger is garbage collected before the class under test logs

[ASF GitHub Bot (Jira)]

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

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

testlens-app[bot] commented on PR #3101:
URL: https://github.com/apache/groovy/pull/3101#issuecomment-6512412099

   ## 🚨 All tests passed but jobs failed 🚨
   
   ### Failed Jobs without Test Failures
   
   ❌ [Build and test / lts \(17, windows-latest, 
1\)](https://github.com/apache/groovy/actions/runs/40977363959/job/123808617652?pr=3101)
   
   🏷️ Commit: 76046e8dc8d29ce03a06f7ec7e19f59cd1b9ea6c
   ▶️ Tests:  63386 executed
   🟡 Checks: 16/24 completed
   
   ### Rerun Controls
   
   Click the checkbox to trigger a rerun:
   
   🔲 **Rerun jobs**
   
   ---
   _Learn more about TestLens at 
[testlens.app/docs](https://testlens.app/docs/features/pr-comment/)._
   




> 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)