You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@groovy.apache.org by "ASF GitHub Bot (Jira)" <ji...@apache.org> on 2025/06/07 12:30:00 UTC

[jira] [Commented] (GROOVY-11695) [GINQ] `groupby` alias can not be found in `switch` expression of `orderby`

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

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

codecov-commenter commented on PR #2252:
URL: https://github.com/apache/groovy/pull/2252#issuecomment-2952430056

   ## [Codecov](https://app.codecov.io/gh/apache/groovy/pull/2252?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) Report
   Attention: Patch coverage is `80.00000%` with `1 line` in your changes missing coverage. Please review.
   > Project coverage is 69.0431%. Comparing base [(`3707260`)](https://app.codecov.io/gh/apache/groovy/commit/3707260f51b65568111e70929fc82792a6b37395?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) to head [(`f6ec781`)](https://app.codecov.io/gh/apache/groovy/commit/f6ec7814d5dde7df85e3a7e351a9e931c1b0e22d?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache).
   
   | [Files with missing lines](https://app.codecov.io/gh/apache/groovy/pull/2252?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Patch % | Lines |
   |---|---|---|
   | [...oovy/ginq/provider/collection/GinqAstWalker.groovy](https://app.codecov.io/gh/apache/groovy/pull/2252?src=pr&el=tree&filepath=subprojects%2Fgroovy-ginq%2Fsrc%2Fmain%2Fgroovy%2Forg%2Fapache%2Fgroovy%2Fginq%2Fprovider%2Fcollection%2FGinqAstWalker.groovy&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-c3VicHJvamVjdHMvZ3Jvb3Z5LWdpbnEvc3JjL21haW4vZ3Jvb3Z5L29yZy9hcGFjaGUvZ3Jvb3Z5L2dpbnEvcHJvdmlkZXIvY29sbGVjdGlvbi9HaW5xQXN0V2Fsa2VyLmdyb292eQ==) | 80.0000% | [1 Missing :warning: ](https://app.codecov.io/gh/apache/groovy/pull/2252?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
   
   <details><summary>Additional details and impacted files</summary>
   
   
   [![Impacted file tree graph](https://app.codecov.io/gh/apache/groovy/pull/2252/graphs/tree.svg?width=650&height=150&src=pr&token=1r45138NfQ&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)](https://app.codecov.io/gh/apache/groovy/pull/2252?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)
   
   ```diff
   @@                Coverage Diff                 @@
   ##               master      #2252        +/-   ##
   ==================================================
   + Coverage     69.0420%   69.0431%   +0.0011%     
     Complexity      29711      29711                
   ==================================================
     Files            1423       1423                
     Lines          114413     114417         +4     
     Branches        19844      19845         +1     
   ==================================================
   + Hits            78993      78997         +4     
     Misses          28787      28787                
     Partials         6633       6633                
   ```
   
   | [Files with missing lines](https://app.codecov.io/gh/apache/groovy/pull/2252?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Coverage Δ | |
   |---|---|---|
   | [...oovy/ginq/provider/collection/GinqAstWalker.groovy](https://app.codecov.io/gh/apache/groovy/pull/2252?src=pr&el=tree&filepath=subprojects%2Fgroovy-ginq%2Fsrc%2Fmain%2Fgroovy%2Forg%2Fapache%2Fgroovy%2Fginq%2Fprovider%2Fcollection%2FGinqAstWalker.groovy&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-c3VicHJvamVjdHMvZ3Jvb3Z5LWdpbnEvc3JjL21haW4vZ3Jvb3Z5L29yZy9hcGFjaGUvZ3Jvb3Z5L2dpbnEvcHJvdmlkZXIvY29sbGVjdGlvbi9HaW5xQXN0V2Fsa2VyLmdyb292eQ==) | `92.6509% <80.0000%> (-0.0931%)` | :arrow_down: |
   
   ... and [1 file with indirect coverage changes](https://app.codecov.io/gh/apache/groovy/pull/2252/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)
   
   </details>
   <details><summary> :rocket: New features to boost your workflow: </summary>
   
   - :snowflake: [Test Analytics](https://docs.codecov.com/docs/test-analytics): Detect flaky tests, report on failures, and find test suite problems.
   - :package: [JS Bundle Analysis](https://docs.codecov.com/docs/javascript-bundle-analysis): Save yourself from yourself by tracking and limiting bundle sizes in JS merges.
   </details>




> [GINQ] `groupby` alias can not be found in `switch` expression of `orderby`
> ---------------------------------------------------------------------------
>
>                 Key: GROOVY-11695
>                 URL: https://issues.apache.org/jira/browse/GROOVY-11695
>             Project: Groovy
>          Issue Type: Bug
>    Affects Versions: 5.0.0-beta-1, 4.0.27
>            Reporter: Daniel Sun
>            Assignee: Daniel Sun
>            Priority: Major
>             Fix For: 4.0.28, 5.0.0-beta-2
>
>
> No such property: g for class: TestScript0
> {code:java}
>             assert [[4, 1], [1, 2], [3, 1]] == GQ {
>                 from n in [1, 1, 3, 4]
>                 groupby n as g
>                 orderby switch (g) {
>                     case 4 -> 0
>                     default -> g
>                 }
>                 select g, count()
>             }.toList()
> {code}



--
This message was sent by Atlassian Jira
(v8.20.10#820010)