Skip to content

Commit 12bcee9

Browse files
committed
Honor fanout follower schema minimum
Fanout validation now relies on the scenario schema's followers minimum instead of imposing an extra five-follower floor. This keeps runner preflight consistent with accepted suite configuration. #802 (comment) Assisted-by: Codex:gpt-5.5
1 parent 1d3e9ce commit 12bcee9

2 files changed

Lines changed: 3 additions & 9 deletions

File tree

packages/cli/src/bench/scenarios/fanout.test.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -65,18 +65,18 @@ test("fanoutRunner - triggers benchmark hook and reports drain", async () => {
6565
assert.strictEqual(triggerRecipients, 5);
6666
});
6767

68-
test("fanoutRunner.validate - requires enough followers for fanout queue", () => {
68+
test("fanoutRunner.validate - accepts schema-minimum follower count", () => {
6969
const scenario = normalizeSuite({
7070
version: 1,
7171
target: "http://target.test/",
7272
scenarios: [{
7373
name: "fanout",
7474
type: "fanout",
7575
sender: "alice",
76-
followers: 4,
76+
followers: 1,
7777
}],
7878
}).scenarios[0];
79-
assert.throws(() => fanoutRunner.validate?.(scenario), /at least 5/);
79+
assert.doesNotThrow(() => fanoutRunner.validate?.(scenario));
8080
});
8181

8282
test("fanoutRunner.validate - rejects invalid sinkBase URLs", () => {

packages/cli/src/bench/scenarios/fanout.ts

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -38,12 +38,6 @@ export const fanoutRunner: ScenarioRunner = {
3838
`trigger.kind: "benchmark-hook".`,
3939
);
4040
}
41-
if ((scenario.followers ?? DEFAULT_FOLLOWERS) < 5) {
42-
throw new Error(
43-
`Scenario "${scenario.name}": fanout needs at least 5 followers to ` +
44-
"exercise Fedify's fanout queue.",
45-
);
46-
}
4741
resolveSinkBase(scenario.name, scenario.raw.sinkBase);
4842
},
4943

0 commit comments

Comments
 (0)