Skip to content

Override completions do not automatically set selection #5652

Closed

Description

@DanTup

Since today's release, this test is failing:

  1) completion_item_provider
       inserts full text for overrides:

      AssertionError [ERR_ASSERTION]: actual: 8:52-8:52 () vs expected: 8:25-8:51 (throw UnimplementedError())

false !== true

      + expected - actual

      -false
      +true
      
      at ensureTestSelection (src/test/helpers.ts:993:9)
      at ensureTestContentWithSelection (src/test/helpers.ts:1004:2)
      at Context.<anonymous> (src/test/dart/providers/completion_item_provider.test.ts:136:3)

It appears that after completing this code, the selection is just at the end of the throw UnimplementedError() instead of selecting it:

Image

Activity

  1. added this to the v3.118.0 milestone on Aug 13, 2025
  2. added a commit that references this issue on Aug 13, 2025
  3. modified the milestones: v3.118.0, v3.120.0 on Aug 28, 2025
  4. DanTup commented on Sep 2, 2025

    @DanTup
    MemberAuthor

    The issue was in the server, fix open at https://dart-review.googlesource.com/c/sdk/+/448180

  5. added
    in lsp/analysis serverSomething to be fixed in the Dart analysis server
    relies on sdk changesSomething that requires changes in the Dart/Flutter SDK to ship before it will become available
    on Sep 2, 2025
  6. changed the title [-]Fix/unskip failing test[/-] [+]Override completions do not automatically set selection[/+] on Sep 2, 2025
  7. DanTup commented on Sep 15, 2025

    @DanTup
    MemberAuthor
  8. DanTup commented on Sep 15, 2025

    @DanTup
    MemberAuthor

    We need to update the skipped test.

  9. reopened this on Sep 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    in editorRelates to code editing or language featuresin lsp/analysis serverSomething to be fixed in the Dart analysis serveris bugrelies on sdk changesSomething that requires changes in the Dart/Flutter SDK to ship before it will become available

    Type

    No type

    Projects

    No projects

      Milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions