[9.x] Fix deprecation warnings from route:list when filtering on name…

… or domain (#41421)

* Add failing test for passing null to Str::contains

* Fix issue by casting route name and domain to string since they can be null

src/Illuminate/Foundation/Console/RouteListCommand.php

@@ -240,10 +240,10 @@ protected function isVendorRoute(Route $route)
      */
     protected function filterRoute(array $route)
     {
-        if (($this->option('name') && ! Str::contains($route['name'], $this->option('name'))) ||
+        if (($this->option('name') && ! Str::contains((string) $route['name'], $this->option('name'))) ||
             ($this->option('path') && ! Str::contains($route['uri'], $this->option('path'))) ||
             ($this->option('method') && ! Str::contains($route['method'], strtoupper($this->option('method')))) ||
-            ($this->option('domain') && ! Str::contains($route['domain'], $this->option('domain'))) ||
+            ($this->option('domain') && ! Str::contains((string) $route['domain'], $this->option('domain'))) ||
             ($this->option('except-vendor') && $route['vendor'])) {
             return;
         }

tests/Testing/Console/RouteListCommandTest.php

@@ -5,13 +5,16 @@
 use Illuminate\Contracts\Routing\Registrar;
 use Illuminate\Foundation\Auth\User;
 use Illuminate\Foundation\Console\RouteListCommand;
+use Illuminate\Foundation\Testing\Concerns\InteractsWithDeprecationHandling;
 use Illuminate\Http\RedirectResponse;
 use Illuminate\Routing\Controller;
 use Illuminate\Support\Facades\Facade;
 use Orchestra\Testbench\TestCase;
 
 class RouteListCommandTest extends TestCase
 {
+    use InteractsWithDeprecationHandling;
+
     /**
      * @var \Illuminate\Contracts\Routing\Registrar
      */
@@ -92,6 +95,24 @@ public function testDisplayRoutesForCliInVerboseMode()
             ->expectsOutput('');
     }
 
+    public function testRouteCanBeFilteredByName()
+    {
+        $this->withoutDeprecationHandling();
+
+        $this->router->get('/', function () {
+            //
+        });
+        $this->router->get('/foo', function () {
+            //
+        })->name('foo.show');
+
+        $this->artisan(RouteListCommand::class, ['--name' => 'foo'])
+            ->assertSuccessful()
+            ->expectsOutput('')
+            ->expectsOutput('  GET|HEAD       foo ...................................... foo.show')
+            ->expectsOutput('');
+    }
+
     protected function tearDown(): void
     {
         parent::tearDown();