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

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

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

   ## 🚨 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/42282878700/job/128116116875?pr=3126)
   
   🏷️ Commit: 782b6de8e73bd75c05b63955b4a1e7433fb2dd5d
   ▶️ Tests:  63216 executed
   🟡 Checks: 24/25 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/)._
   




> STC accepts a Closure<R> value passed to or returned as a functional 
> interface whose method does not return R
> -------------------------------------------------------------------------------------------------------------
>
>                 Key: GROOVY-12599
>                 URL: https://issues.apache.org/jira/browse/GROOVY-12599
>             Project: Groovy
>          Issue Type: Bug
>    Affects Versions: 4.0.33, 5.0.8, 6.0.0
>            Reporter: Scott Murphy Heiberg
>            Priority: Major
>
> {code:groovy}
> import java.util.function.Supplier
> @groovy.transform.CompileStatic // or @TypeChecked
> class C {
>     static String take(Supplier<String> s) { s.get().toLowerCase() }
>     static Supplier<String> make() { Closure<Integer> c = { 1 }; c }
>     static void main(args) {
>         Closure<Integer> c = { 1 }
>         take(c)
>         make().get().toLowerCase()
>     }
> }
> {code}
> Actual: the code compiles. Under static compilation both calls throw
> {noformat}
> java.lang.ClassCastException: class java.lang.Integer cannot be cast to class 
> java.lang.String
> {noformat}
> (MissingMethodException under @TypeChecked). The same happens for an instance 
> of a non-generic subclass such as {{class D extends Closure<Integer>}} passed 
> as the argument.
> Expected: compile errors, as for a closure literal ("Cannot return value of 
> type int for closure expecting java.lang.String"):
> {noformat}
> Cannot call C#take(java.util.function.Supplier<java.lang.String>) with 
> arguments [groovy.lang.Closure<java.lang.Integer>]
> Cannot return value of type groovy.lang.Closure<java.lang.Integer> for method 
> returning java.util.function.Supplier<java.lang.String>
> {noformat}
> For a method argument, 
> StaticTypeCheckingSupport.typeCheckMethodArgumentWithGenerics accepts any 
> Closure for a SAM-type parameter and assumes the return type was checked 
> earlier. That is only true for a closure literal. For a return statement, 
> generics are compared only when the value's type implements the return type, 
> which Closure does not for most SAM types.
> Seen in 4.0.33, 5.0.8 and 6.0.0.
> PR: https://github.com/apache/groovy/pull/3126



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

Reply via email to