Add "noindex,follow" for /Search

pradoslaw · Jan 25, 2024 · 8943885 · 8943885
1 parent 5bd1c2c
commit 8943885
Show file tree

Hide file tree

Showing 3 changed files with 18 additions and 2 deletions.
diff --git a/app/Providers/SeoServiceProvider.php b/app/Providers/SeoServiceProvider.php
@@ -29,7 +29,10 @@ private function metaRobots(): string
         if ($request->getHost() === '4programmers.dev') {
             return 'noindex,nofollow';
         }
-        if ($request->getRequestUri() === '/Forum/Interesting') {
+        if ($request->getPathInfo() === '/Search') {
+            return 'noindex,follow';
+        }
+        if ($request->getPathInfo() === '/Forum/Interesting') {
             return 'noindex,nofollow';
         }
         return 'index,follow';

diff --git a/tests/Unit/Seo/Meta/Fixture/Assertion.php b/tests/Unit/Seo/Meta/Fixture/Assertion.php
@@ -18,6 +18,11 @@ function assertNoIndexable(string $uri): void
         $this->assertMetaRobots($uri, 'noindex,nofollow');
     }
 
+    function assertCrawlable(string $uri): void
+    {
+        $this->assertMetaRobots($uri, 'noindex,follow');
+    }
+
     function assertMetaRobots(string $uri, string $metaRobots): void
     {
         Assert::assertSame($metaRobots, $this->metaProperty('robots', $uri));

diff --git a/tests/Unit/Seo/Meta/MetaRobotsTest.php b/tests/Unit/Seo/Meta/MetaRobotsTest.php
@@ -22,7 +22,7 @@ public function homepage()
      */
     public function userTags()
     {
-        $this->assertNoIndexable('/Forum/Interesting');
+        $this->assertNoIndexable('/Forum/Interesting?query');
     }
 
     /**
@@ -40,4 +40,12 @@ public function developerEnvironment()
     {
         $this->assertNoIndexable('http://4programmers.dev/');
     }
+
+    /**
+     * @test
+     */
+    public function search()
+    {
+        $this->assertCrawlable('/Search?query');
+    }
 }