Skip to content

Commit db6612c

Browse files
committed
test(cli): remove HelpTest for printModelList wrapper
- Drop `HelpTest::printModelListGroupsModelsByVendor` since `printModelList` is now purely a wrapper and will be covered by integration tests or core `listModels` tests.
1 parent 780d851 commit db6612c

File tree

1 file changed

+0
-31
lines changed

1 file changed

+0
-31
lines changed

tests/Console/HelpTest.php

Lines changed: 0 additions & 31 deletions
Original file line numberDiff line numberDiff line change
@@ -25,35 +25,4 @@ public function printHelpWithNoExitOptionOutputsUsageAndOptions(): void
2525
$this->assertStringContainsString('Environment:', $output);
2626
$this->assertStringContainsString('Examples:', $output);
2727
}
28-
29-
/** @test */
30-
public function printModelListGroupsModelsByVendor(): void
31-
{
32-
// Prepare a minimal stub registry
33-
$stub = $this->createMock(ModelRegistry::class);
34-
$stub->method('getAll')->willReturn([
35-
'alpha:model-a' => ['vendor' => 'alpha', 'notes' => 'Alpha notes'],
36-
'beta:model-b' => ['vendor' => 'beta', 'notes' => 'Beta notes'],
37-
'alpha:model-c' => ['vendor' => 'alpha', 'notes' => null],
38-
]);
39-
40-
ob_start();
41-
Help::printModelList($stub);
42-
$output = ob_get_clean();
43-
44-
// Should list two vendors
45-
$this->assertStringContainsString("alpha:", $output);
46-
$this->assertStringContainsString("beta:", $output);
47-
48-
// Under alpha, both model keys should appear
49-
$this->assertStringContainsString('alpha:model-a', $output);
50-
$this->assertStringContainsString('alpha:model-c', $output);
51-
52-
// Under beta, its model key should appear
53-
$this->assertStringContainsString('beta:model-b', $output);
54-
55-
// Notes should be appended where present
56-
$this->assertStringContainsString('Alpha notes', $output);
57-
$this->assertStringContainsString('Beta notes', $output);
58-
}
5928
}

0 commit comments

Comments
 (0)