New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Enable async completion #33795
Enable async completion #33795
Conversation
5b1add8
to
e55e15c
Compare
src/VisualStudio/IntegrationTest/IntegrationTests/AbstractIntegrationTest.cs
Show resolved
Hide resolved
src/VisualStudio/IntegrationTest/IntegrationTests/CSharp/CSharpIntelliSense.cs
Outdated
Show resolved
Hide resolved
src/VisualStudio/IntegrationTest/IntegrationTests/CSharp/CSharpIntelliSense.cs
Show resolved
Hide resolved
e55e15c
to
e2b11da
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@@ -99,7 +101,7 @@ public void CtrlAltSpace() | |||
VisualStudio.Editor.Verify.CurrentLineText("System.Console.WriteLine();$$", assertCaretPosition: true); | |||
VisualStudio.Editor.SendKeys(VirtualKey.Home, Shift(VirtualKey.End), VirtualKey.Delete); | |||
|
|||
VisualStudio.ExecuteCommand(WellKnownCommandNames.Edit_ToggleCompletionMode); | |||
VisualStudio.Editor.SendKeys(new KeyPress(VirtualKey.Space, ShiftState.Ctrl | ShiftState.Alt)); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Change is fine but just checking here -- did we break the command?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No, the command works, and we use the command in other test contexts. This was supposed to be a test for Ctrl+Alt+Space though.
Tests impacted by the following bugs are disabled until we can address them:
Fixes #33797