Changeset 142:f5a6a5297de1 for imapclient/test/test_IMAPClient.py

Timestamp:

04/29/10 14:01:30 (2 years ago)

Author:

Menno Smits <menno@…>

Branch:

default

Message:

Correctly handle \" in quoted strings.

(As reported in comments of #28)

It turns out these were being handled twice causing them to be
incorrectly removed if a string started with a double quote.

Extended tests to cover more cases around escaped double quotes. Prior
to this the test weren't covering enough cases.

Files:

• imapclient/test/test_IMAPClient.py (modified) (1 diff)

imapclient/test/test_IMAPClient.py

@@ -92 +92 @@
                                                  r'(\HasNoChildren) "/" "Left\"Right"',
                                                  r'(\HasNoChildren) "/" "Left\\Right"',
+                                                 r'(\HasNoChildren) "/" "\"Left Right\""',
+                                                 r'(\HasNoChildren) "/" "\"Left\\Right\""',
                                                  ])
         self.assertEqual(folders, [(['\\HasNoChildren'], '/', 'Test "Folder"'),
                                    (['\\HasNoChildren'], '/', 'Left\"Right'),
-                                   (['\\HasNoChildren'], '/', r'Left\Right')])
+                                   (['\\HasNoChildren'], '/', r'Left\Right'),
+                                   (['\\HasNoChildren'], '/', r'"Left Right"'),
+                                   (['\\HasNoChildren'], '/', r'"Left\Right"'),
+                                   ])
 
     def test_empty_response(self):