♻️ Move strange parser workaround tests to yaml

Author: nevans

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

@@ -0,0 +1,9 @@
+---
+:tests:
+  test_invalid_noop_response_is_ignored:
+    :comment: |
+      This should probably use UntaggedResponse, perhaps with an
+      UnparsedResponseData object for its #data.
+    :response: "* NOOP\r\n"
+    :expected: !ruby/struct:Net::IMAP::IgnoredResponse
+      raw_data: "* NOOP\r\n"

test/net/imap/test_imap_response_parser.rb

@@ -69,6 +69,10 @@ def teardown
   # RFC 5256: THREAD response
   generate_tests_from fixture_file: "thread_responses.yml"
 
+  ############################################################################
+  # Workarounds or unspecified extensions:
+  generate_tests_from fixture_file: "quirky_behaviors.yml"
+
   ############################################################################
   # More interesting tests about the behavior, either of the test or of the
   # response data, should still use normal tests, below
@@ -82,15 +86,8 @@ def test_enable
     assert_equal("SMTPUTF8", response.data.first)
   end
 
-  def test_ignored_response
-    parser = Net::IMAP::ResponseParser.new
-    response = nil
-    assert_nothing_raised do
-      response = parser.parse("* NOOP\r\n")
-    end
-    assert_instance_of(Net::IMAP::IgnoredResponse, response)
-  end
-
+  # todo: move this to response data tests file
+  # it's testing the mapping fn, not the parsing.
   def test_uidplus_copyuid__uid_mapping
     parser = Net::IMAP::ResponseParser.new
     response = parser.parse(