feat: spark routes shows "Required Filters"

codeigniter4 · Nov 20, 2023 · baa9777 · baa9777
1 parent c168b16
commit baa9777
Show file tree

Hide file tree

Showing 2 changed files with 34 additions and 0 deletions.
diff --git a/system/Commands/Utilities/Routes.php b/system/Commands/Utilities/Routes.php
@@ -192,5 +192,30 @@ public function run(array $params)
  }
 
  CLI::table($tbody, $thead);
+
+ $this->showRequiredFilters();
+ }
+
+ private function showRequiredFilters(): void
+ {
+ $filterCollector = new FilterCollector();
+
+ $required = $filterCollector->getRequiredFilters();
+
+ $filters = [];
+
+ foreach ($required['before'] as $filter) {
+ $filters[] = CLI::color($filter, 'yellow');
+ }
+
+ CLI::write('Required Before Filters: ' . implode(', ', $filters));
+
+ $filters = [];
+
+ foreach ($required['after'] as $filter) {
+ $filters[] = CLI::color($filter, 'yellow');
+ }
+
+ CLI::write(' Required After Filters: ' . implode(', ', $filters));
  }
 }
diff --git a/tests/system/Commands/RoutesTest.php b/tests/system/Commands/RoutesTest.php
@@ -76,6 +76,15 @@ public function testRoutesCommand(): void
  +---------+---------+---------------+----------------------------------------+----------------+---------------+
  EOL;
  $this->assertStringContainsString($expected, $this->getBuffer());
+
+ $expected = <<<'EOL'
+ Required Before Filters: forcehttps, pagecache
+ Required After Filters: pagecache, performance, toolbar
+ EOL;
+ $this->assertStringContainsString(
+ $expected,
+ preg_replace('/\033\[.+?m/u', '', $this->getBuffer())
+ );
  }
 
  public function testRoutesCommandSortByHandler(): void