♻️ Move SEARCH response parser tests to yaml

🚧 Tests for modseq on SearchResult were commented out for now.  That PR
is coming soon.  :)

Author: nevans

test/net/imap/fixtures/response_parser/search_responses.yml

@@ -0,0 +1,76 @@
+---
+:tests:
+  test_search_response_empty:
+    :response: "* SEARCH\r\n"
+    :expected: !ruby/struct:Net::IMAP::UntaggedResponse
+      name: SEARCH
+      data: []
+      # data: !ruby/array:Net::IMAP::SearchResult
+      #   internal: []
+      #   ivars:
+      #     "@modseq":
+      raw_data: "* SEARCH\r\n"
+
+  test_search_response_single_seq_nums_returned:
+    :response: "* SEARCH 1\r\n"
+    :expected: !ruby/struct:Net::IMAP::UntaggedResponse
+      name: SEARCH
+      data: # !ruby/array:Net::IMAP::SearchResult
+      #   internal:
+        - 1
+      #   ivars:
+      #     "@modseq":
+      raw_data: "* SEARCH 1\r\n"
+
+  test_search_response_multiple_seq_nums_returned:
+    :response: "* SEARCH 1 2 3\r\n"
+    :expected: !ruby/struct:Net::IMAP::UntaggedResponse
+      name: SEARCH
+      data: # !ruby/array:Net::IMAP::SearchResult
+        # internal:
+        - 1
+        - 2
+        - 3
+        # ivars:
+          # "@modseq":
+      raw_data: "* SEARCH 1 2 3\r\n"
+
+  test_invalid_search_response_single_result_with_trailing_space:
+    :quirky_servers: Yahoo
+    :response: "* SEARCH 1 \r\n"
+    :expected: !ruby/struct:Net::IMAP::UntaggedResponse
+      name: SEARCH
+      data: # !ruby/array:Net::IMAP::SearchResult
+        # internal:
+        - 1
+        # ivars:
+          # "@modseq":
+      raw_data: "* SEARCH 1 \r\n"
+
+  test_invalid_search_response_multiple_result_with_trailing_space:
+    :quirky_servers: Yahoo
+    :response: "* SEARCH 1 2 3 \r\n"
+    :expected: !ruby/struct:Net::IMAP::UntaggedResponse
+      name: SEARCH
+      data: # !ruby/array:Net::IMAP::SearchResult
+        # internal:
+        - 1
+        - 2
+        - 3
+        # ivars:
+          # "@modseq":
+      raw_data: "* SEARCH 1 2 3 \r\n"
+
+  test_search_response_with_condstore_modseq:
+    :comment: |
+      [Bug #10112]
+    :response: "* SEARCH 87216 87221 (MODSEQ 7667567)\r\n"
+    :expected: !ruby/struct:Net::IMAP::UntaggedResponse
+      name: SEARCH
+      data: # !ruby/array:Net::IMAP::SearchResult
+        # internal:
+        - 87216
+        - 87221
+        # ivars:
+          # "@modseq": 7667567
+      raw_data: "* SEARCH 87216 87221 (MODSEQ 7667567)\r\n"

test/net/imap/test_imap_response_parser.rb

@@ -25,11 +25,14 @@ def teardown
   #
   # TODO: add instructions for how to quickly add or update yaml tests
   ############################################################################
-  # Core IMAP specs, by RFC9051 section (w/ obsolete listed last):
+  # Core IMAP, by RFC9051 section (w/obsolete in relative RFC3501 section):
 
   # §7.3.2: NAMESPACE response (also RFC2342)
   generate_tests_from fixture_file: "namespace_responses.yml"
 
+  # RFC3501 §7.2.5: SEARCH response (obsolete in IMAP4rev2):
+  generate_tests_from fixture_file: "search_responses.yml"
+
   # §7.5.2: FETCH response, BODYSTRUCTURE msg-att
   generate_tests_from fixture_file: "body_structure_responses.yml"
 
@@ -68,34 +71,6 @@ def test_resp_text_code
     assert_equal "CLOSED", response.data.code.name
   end
 
-  def test_search_response
-    parser = Net::IMAP::ResponseParser.new
-    response = parser.parse(<<EOF.gsub(/\n/, "\r\n"))
-* SEARCH
-EOF
-    assert_equal [], response.data
-    response = parser.parse(<<EOF.gsub(/\n/, "\r\n"))
-* SEARCH 1
-EOF
-    assert_equal [1], response.data
-    response = parser.parse(<<EOF.gsub(/\n/, "\r\n"))
-* SEARCH 1 2 3
-EOF
-    assert_equal [1, 2, 3], response.data
-  end
-
-  def test_search_response_of_yahoo
-    parser = Net::IMAP::ResponseParser.new
-    response = parser.parse(<<EOF.gsub(/\n/, "\r\n"))
-* SEARCH 1\s
-EOF
-    assert_equal [1], response.data
-    response = parser.parse(<<EOF.gsub(/\n/, "\r\n"))
-* SEARCH 1 2 3\s
-EOF
-    assert_equal [1, 2, 3], response.data
-  end
-
   def test_msg_att_extra_space
     parser = Net::IMAP::ResponseParser.new
     response = parser.parse(<<EOF.gsub(/\n/, "\r\n"))
@@ -194,14 +169,6 @@ def test_id
     assert_equal(nil, response.data.fetch("support-url"))
   end
 
-  # [Bug #10112]
-  def test_search_modseq
-    parser = Net::IMAP::ResponseParser.new
-    response = parser.parse("* SEARCH 87216 87221 (MODSEQ 7667567)\r\n")
-    assert_equal("SEARCH", response.name)
-    assert_equal([87216, 87221], response.data)
-  end
-
   # [Bug #13649]
   def test_status
     parser = Net::IMAP::ResponseParser.new