Changeset 11185

Timestamp:

08/18/2021 03:23:40 PM (3 years ago)

Author:

iandunn

Message:

Patterns: Test for single terms until phrase matching is implemented.

File:

• sites/trunk/api.wordpress.org/public_html/patterns/1.0/tests/test-index.php (modified) (5 diffs)

sites/trunk/api.wordpress.org/public_html/patterns/1.0/tests/test-index.php

@@ -25 +25 @@
     }
 
+    /**
+     * Asserts that the results contain the search term.
+     *
+     * @param array $patterns
+     * @param string $search_term
+     */
     public function assertAllPatternsMatchSearchTerm( $patterns, $search_term ) {
         $all_patterns_include_query = true;
+        if ( false !== strpos( $search_term, ' ' ) ) {
+            $this->markTestIncomplete( "This doesn't support phrase-matching quoted terms yet." );
+        }
 
         foreach ( $patterns as $pattern ) {
@@ -174 +183 @@
      */
     public function test_search_patterns( $search_term, $locale, $match_expected, $expected_post_ids ) : void {
-        // wrap term in double quotes to match exact phrase.
-        $response = send_request( '/patterns/1.0/?&search="' . $search_term . '"&pattern-keywords=11&locale=' . $locale );
+        $response = send_request( "/patterns/1.0/?search=$search_term&pattern-keywords=11&locale=$locale" );
 
         if ( $match_expected ) {
@@ -202 +210 @@
             // Should find posts that have the term in the title, but _not_ in the description.
             'match title only' => array(
-                'search_term'       => 'side by side',
+                'search_term'       => 'side',
                 'locale'            => 'en_US',
                 'match_expected'    => true,
@@ -210 +218 @@
             // Should find posts that have the term in the description, but _not_ in the title.
             'match description only' => array(
-                'search_term'       => 'quote on top',
+                'search_term'       => 'quote',
                 'locale'            => 'en_US',
                 'match_expected'    => true,
@@ -270 +278 @@
             // @link https://translate.wordpress.org/projects/patterns/core/
             "untranslated locales should have en_US posts as a fallback" => array(
-                'search_term'       => 'offset images', // Post ID 2226.
+                'search_term'       => 'offset',
                 'locale'            => 'la',
                 'match_expected'    => true,