Skip to content

Commit 3007e96

Browse files
committed
Fixed the MonitorEventsAsync_Succeeds test.
1 parent 9da0ced commit 3007e96

1 file changed

Lines changed: 2 additions & 1 deletion

File tree

test/Docker.DotNet.Tests/ISystemOperations.Tests.cs

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -111,7 +111,7 @@ public async Task MonitorEventsAsync_Succeeds()
111111

112112
await _client.Images.CreateImageAsync(new ImagesCreateParameters { FromImage = "hello-world" }, null, progressJSONMessage);
113113

114-
var task = Task.Run(() => _client.System.MonitorEventsAsync(new ContainerEventsParameters(), progressMessage, cts.Token));
114+
var task = _client.System.MonitorEventsAsync(new ContainerEventsParameters(), progressMessage, cts.Token);
115115

116116
await _client.Images.TagImageAsync(repository, new ImageTagParameters { RepositoryName = repository, Tag = newTag });
117117

@@ -120,6 +120,7 @@ public async Task MonitorEventsAsync_Succeeds()
120120
bool taskIsCancelled = false;
121121
try
122122
{
123+
Thread.Sleep(500); // Let's wait some time before cancelling the monitor stream to make sure we got some progress back.
123124
await task;
124125
}
125126
catch (OperationCanceledException)

0 commit comments

Comments
 (0)