Skip to content

Commit 780386d

Browse files
committed
fix: correct test assertion on CLI incoming
1 parent 107ceea commit 780386d

2 files changed

Lines changed: 6 additions & 2 deletions

File tree

examples/advanced/KedroSpaceflights.Custom/Program.cs

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -94,6 +94,8 @@ private static void ConfigureServices(IServiceCollection services, string basePa
9494
flowthru
9595
.RegisterFlow(label: "Reporting", flow: ReportingFlow.Create)
9696
.WithDescription("Generates reports and visualizations");
97+
98+
flowthru.ConfigureExecution(opts => opts.MaxDegreeOfParallelism = 8);
9799
});
98100

99101
services.AddLogging(logging =>

tests/Flowthru.Tests/05_Cli/ArgumentParserParallelismTests.cs

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -53,11 +53,13 @@ public void Parse_ParallelismAuto_SetsMaxDegreeToProcessorCount(string token)
5353
// ─────────────────────────────────────────────────────────────────────────
5454

5555
[Test]
56-
public void Parse_NoParallelismFlag_DefaultsToOne()
56+
public void Parse_NoParallelismFlag_LeavesNull()
5757
{
5858
var parsed = ArgumentParser.Parse([], NoFlows);
5959

60-
Assert.That(parsed.Options!.MaxDegreeOfParallelism, Is.EqualTo(1));
60+
// Null means "unspecified" — the priority chain in FlowthruService resolves it
61+
// to the service-level default, or 1 if none is configured.
62+
Assert.That(parsed.Options!.MaxDegreeOfParallelism, Is.Null);
6163
}
6264

6365
// ─────────────────────────────────────────────────────────────────────────

0 commit comments

Comments
 (0)