Skip to content

Fix testing for pipeline consumer branch #1588

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.

Sign up for GitHub

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

Merged
rjmholt merged 17 commits into from
Oct 26, 2021
Merged

Fix testing for pipeline consumer branch #1588

Changes from 1 commit
Commits
Show all changes
17 commits
Select commit Hold shift + click to select a range
dfee772
Initial test build
rjmholt Oct 14, 2021
61ae8f2
Add Host factory
rjmholt Oct 14, 2021
def819e
Enable host with no repl
rjmholt Oct 19, 2021
e6f21dc
Comment out LSP unit tests
rjmholt Oct 19, 2021
2daa962
Bump CI
rjmholt Oct 19, 2021
702636c
Add timeout to test
rjmholt Oct 20, 2021
d5aebd6
Ensure CI is triggered for PRs not for master
rjmholt Oct 20, 2021
d1acc64
Route startup exceptions through and ensure startup has completed bef…
rjmholt Oct 20, 2021
5d94ad6
Fix test compile error
rjmholt Oct 21, 2021
b0edca9
Fix process exit throwing
rjmholt Oct 25, 2021
2846bef
Fix bad output when console repl is disabled
rjmholt Oct 25, 2021
0681d37
Add debug logging for integration test streams
rjmholt Oct 25, 2021
196ec3b
Fix overrides
rjmholt Oct 25, 2021
d22b57c
Fix variable test with scope change
rjmholt Oct 25, 2021
5699e48
Fix assembly log exception
rjmholt Oct 25, 2021
ca7c815
Only set pipeline thread apartment state on Windows
rjmholt Oct 26, 2021
fcc4cd8
Remove debug code
rjmholt Oct 26, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Add timeout to test
  • Loading branch information
@rjmholt
rjmholt committed Oct 20, 2021
commit 702636c23f8fe2ae5fb64db36275b060a62c9800
4 changes: 3 additions & 1 deletion test/PowerShellEditorServices.Test.E2E/LanguageServerProtocolMessageTests.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1153,6 +1153,8 @@ await PsesLanguageClient
[Fact]
public async Task CanSendEvaluateRequestAsync()
{
using var cancellationSource = new CancellationTokenSource(millisecondsDelay: 5000);

EvaluateResponseBody evaluateResponseBody =
await PsesLanguageClient
.SendRequest<EvaluateRequestArguments>(
Expand All @@ -1161,7 +1163,7 @@ await PsesLanguageClient
{
Expression = "Get-ChildItem"
})
.Returning<EvaluateResponseBody>(CancellationToken.None).ConfigureAwait(false);
.Returning<EvaluateResponseBody>(cancellationSource.Token).ConfigureAwait(false);

// These always gets returned so this test really just makes sure we get _any_ response.
Assert.Equal("", evaluateResponseBody.Result);
Expand Down