Skip to content

Commit 6387ad2

Browse files
committed
some fb
1 parent ef0961a commit 6387ad2

File tree

1 file changed

+9
-0
lines changed

1 file changed

+9
-0
lines changed

test/Grpc.IntegrationTests/LargePayloadTests.cs

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,9 @@
22
// Licensed under the MIT License.
33

44
using Microsoft.DurableTask.Client;
5+
using Microsoft.DurableTask.Client.Entities;
56
using Microsoft.DurableTask.Converters;
7+
using Microsoft.DurableTask.Entities;
68
using Microsoft.DurableTask.Worker;
79
using Microsoft.Extensions.DependencyInjection;
810
using Xunit.Abstractions;
@@ -11,6 +13,7 @@ namespace Microsoft.DurableTask.Grpc.Tests;
1113

1214
public class LargePayloadTests(ITestOutputHelper output, GrpcSidecarFixture sidecarFixture) : IntegrationTestBase(output, sidecarFixture)
1315
{
16+
// Validates client externalizes a large orchestration input and worker resolves it.
1417
[Fact]
1518
public async Task OrchestrationInput_IsExternalizedByClient_ResolvedByWorker()
1619
{
@@ -69,6 +72,7 @@ public async Task OrchestrationInput_IsExternalizedByClient_ResolvedByWorker()
6972
Assert.True(fakeStore.UploadCount >= 1);
7073
}
7174

75+
// Validates worker externalizes large activity input and delivers resolved payload to activity.
7276
[Fact]
7377
public async Task ActivityInput_IsExternalizedByWorker_ResolvedByActivity()
7478
{
@@ -110,6 +114,7 @@ public async Task ActivityInput_IsExternalizedByWorker_ResolvedByActivity()
110114
Assert.True(workerStore.DownloadCount >= 1);
111115
}
112116

117+
// Validates worker externalizes large activity output which is resolved by the orchestrator.
113118
[Fact]
114119
public async Task ActivityOutput_IsExternalizedByWorker_ResolvedByOrchestrator()
115120
{
@@ -151,6 +156,7 @@ public async Task ActivityOutput_IsExternalizedByWorker_ResolvedByOrchestrator()
151156
Assert.True(workerStore.DownloadCount >= 1);
152157
}
153158

159+
// Ensures querying a completed instance downloads and resolves an externalized output on the client.
154160
[Fact]
155161
public async Task QueryCompletedInstance_DownloadsExternalizedOutputOnClient()
156162
{
@@ -205,6 +211,7 @@ public async Task QueryCompletedInstance_DownloadsExternalizedOutputOnClient()
205211
Assert.True(clientStore.UploadCount == 1);
206212
}
207213

214+
// Ensures payloads below the threshold are not externalized by client or worker.
208215
[Fact]
209216
public async Task BelowThreshold_NotExternalized()
210217
{
@@ -255,6 +262,7 @@ public async Task BelowThreshold_NotExternalized()
255262
Assert.Equal(0, clientStore.DownloadCount);
256263
}
257264

265+
// Validates client externalizes a large external event payload and worker resolves it.
258266
[Fact]
259267
public async Task ExternalEventPayload_IsExternalizedByClient_ResolvedByWorker()
260268
{
@@ -306,6 +314,7 @@ public async Task ExternalEventPayload_IsExternalizedByClient_ResolvedByWorker()
306314
Assert.True(fakeStore.UploadCount >= 1);
307315
}
308316

317+
// Validates worker externalizes both output and custom status; client resolves them on query.
309318
[Fact]
310319
public async Task OutputAndCustomStatus_ExternalizedByWorker_ResolvedOnQuery()
311320
{

0 commit comments

Comments
 (0)