Moved the expected value to first position in assertions because that's how PyCharm...

git-svn-id: http://www.winterrodeln.org/svn/wrpylib/trunk@2443 7aebc617-e5e2-0310-91dc-80fb5f6d2477

diff --git a/tests/test_wrvalidators.py b/tests/test_wrvalidators.py
index 42fc9cbf1d7216c11a72dc112ed2fbd4f4819015..5c3aafb8e5ac01cec1a1a3b893ad67cfb74d1014 100644 (file)
--- a/tests/test_wrvalidators.py
+++ b/tests/test_wrvalidators.py
@@ -28,14 +28,14 @@ class TestNoGermanConverter(unittest.TestCase):
 
 class TestOptNoGerman(unittest.TestCase):
     def test_from_str(self):
-        self.assertEqual(opt_no_german_from_str('abc'), (True, 'abc'))
-        self.assertEqual(opt_no_german_from_str('Nein'), (False, None))
-        self.assertEqual(opt_no_german_from_str(''), (None, None))
+        self.assertEqual((True, 'abc'), opt_no_german_from_str('abc'))
+        self.assertEqual((False, None), opt_no_german_from_str('Nein'))
+        self.assertEqual((None, None), opt_no_german_from_str(''))
 
     def test_to_str(self):
-        self.assertEqual(opt_no_german_to_str((True, 'abc')), 'abc')
-        self.assertEqual(opt_no_german_to_str((False, None)), 'Nein')
-        self.assertEqual(opt_no_german_to_str((None, None)), '')
+        self.assertEqual('abc', opt_no_german_to_str((True, 'abc')))
+        self.assertEqual('Nein', opt_no_german_to_str((False, None)))
+        self.assertEqual('', opt_no_german_to_str((None, None)))
 
 
 # TODO: choice converter
@@ -277,18 +277,18 @@ class TestOptTristateGermanComment(unittest.TestCase):
 
 class TestUrl(unittest.TestCase):
     def test_from_str(self):
-        self.assertEqual(url_from_str('http://www.winterrodeln.org/wiki/Arzler_Alm/'), 'http://www.winterrodeln.org/wiki/Arzler_Alm/')
-        self.assertEqual(url_from_str('http://www.winterrodeln.org/wiki/Nösslachhütte/'), 'http://www.winterrodeln.org/wiki/Nösslachhütte/')
-        self.assertEqual(url_from_str('https://www.winterrodeln.org/wiki/Nösslachhütte/'), 'https://www.winterrodeln.org/wiki/Nösslachhütte/')
+        self.assertEqual('http://www.winterrodeln.org/wiki/Arzler_Alm/', url_from_str('http://www.winterrodeln.org/wiki/Arzler_Alm/'))
+        self.assertEqual('http://www.winterrodeln.org/wiki/Nösslachhütte/', url_from_str('http://www.winterrodeln.org/wiki/Nösslachhütte/'))
+        self.assertEqual('https://www.winterrodeln.org/wiki/Nösslachhütte/', url_from_str('https://www.winterrodeln.org/wiki/Nösslachhütte/'))
         with self.assertRaises(ValueError):
             url_from_str('mailto:office@example.com')
         with self.assertRaises(ValueError):
             url_from_str('/wiki/Arzler_Alm/')
 
     def test_to_str(self):
-        self.assertEqual(url_to_str('http://www.winterrodeln.org/wiki/Arzler_Alm/'), 'http://www.winterrodeln.org/wiki/Arzler_Alm/')
-        self.assertEqual(url_to_str('http://www.winterrodeln.org/wiki/Nösslachhütte/'), 'http://www.winterrodeln.org/wiki/Nösslachhütte/')
-        self.assertEqual(url_to_str('https://www.winterrodeln.org/wiki/Nösslachhütte/'), 'https://www.winterrodeln.org/wiki/Nösslachhütte/')
+        self.assertEqual('http://www.winterrodeln.org/wiki/Arzler_Alm/', url_to_str('http://www.winterrodeln.org/wiki/Arzler_Alm/'))
+        self.assertEqual('http://www.winterrodeln.org/wiki/Nösslachhütte/', url_to_str('http://www.winterrodeln.org/wiki/Nösslachhütte/'))
+        self.assertEqual('https://www.winterrodeln.org/wiki/Nösslachhütte/', url_to_str('https://www.winterrodeln.org/wiki/Nösslachhütte/'))
 
 
 # webauskunft converter
@@ -296,14 +296,14 @@ class TestUrl(unittest.TestCase):
 
 class TestWebauskunft(unittest.TestCase):
     def test_from_str(self):
-        self.assertEqual(webauskunft_from_str('http://www.example.com/current'), (True, 'http://www.example.com/current'))
-        self.assertEqual(webauskunft_from_str(''), (None, None))
-        self.assertEqual(webauskunft_from_str('Nein'), (False, None))
+        self.assertEqual((True, 'http://www.example.com/current'), webauskunft_from_str('http://www.example.com/current'))
+        self.assertEqual((None, None), webauskunft_from_str(''))
+        self.assertEqual((False, None), webauskunft_from_str('Nein'))
 
     def test_to_str(self):
-        self.assertEqual(webauskunft_to_str((True, 'http://www.example.com/current')), 'http://www.example.com/current')
-        self.assertEqual(webauskunft_to_str((None, None)), '')
-        self.assertEqual(webauskunft_to_str((False, None)), 'Nein')
+        self.assertEqual('http://www.example.com/current', webauskunft_to_str((True, 'http://www.example.com/current')))
+        self.assertEqual('', webauskunft_to_str((None, None)))
+        self.assertEqual('Nein', webauskunft_to_str((False, None)))
 
 
 # wikipage converter
@@ -398,36 +398,36 @@ class TestEmails(unittest.TestCase):
 
 class TestPhoneNumber(unittest.TestCase):
     def test_from_str(self):
-        self.assertEqual(phone_number_from_str('+43-699-123456789'), '+43-699-123456789')
-        self.assertEqual(phone_number_from_str('+43-69945'), '+43-69945')
-        self.assertEqual(phone_number_from_str('+43-512-507-6418'), '+43-512-507-6418')
+        self.assertEqual('+43-699-123456789', phone_number_from_str('+43-699-123456789'))
+        self.assertEqual('+43-69945', phone_number_from_str('+43-69945'))
+        self.assertEqual('+43-512-507-6418', phone_number_from_str('+43-512-507-6418'))
         with self.assertRaises(ValueError):
             phone_number_from_str('+43-')
         with self.assertRaises(ValueError):
             phone_number_from_str('0512123456789')
 
     def test_to_str(self):
-        self.assertEqual(phone_number_to_str('+43-699-123456789'), '+43-699-123456789')
-        self.assertEqual(phone_number_to_str('+43-69945'), '+43-69945')
-        self.assertEqual(phone_number_to_str('+43-512-507-6418'), '+43-512-507-6418')
+        self.assertEqual('+43-699-123456789', phone_number_to_str('+43-699-123456789'))
+        self.assertEqual('+43-69945', phone_number_to_str('+43-69945'))
+        self.assertEqual('+43-512-507-6418', phone_number_to_str('+43-512-507-6418'))
 
 
 class TestOptPhoneCommentEnum(unittest.TestCase):
     def test_from_str(self):
         self.assertEqual(opt_phone_comment_enum_from_str(''), None)
-        self.assertEqual(opt_phone_comment_enum_from_str('Nein'), [])
-        self.assertEqual(opt_phone_comment_enum_from_str('+43-512-123456 (untertags)'), [('+43-512-123456', 'untertags')])
-        self.assertEqual(opt_phone_comment_enum_from_str('+43-512-1234 (untertags); +43-664-123456 (Alm)'), [('+43-512-1234', 'untertags'), ('+43-664-123456', 'Alm')])
+        self.assertEqual([], opt_phone_comment_enum_from_str('Nein'))
+        self.assertEqual([('+43-512-123456', 'untertags')], opt_phone_comment_enum_from_str('+43-512-123456 (untertags)'))
+        self.assertEqual([('+43-512-1234', 'untertags'), ('+43-664-123456', 'Alm')], opt_phone_comment_enum_from_str('+43-512-1234 (untertags); +43-664-123456 (Alm)'))
         with self.assertRaises(ValueError):
             opt_phone_comment_enum_from_str('+43-512-123456+ (untertags)')
         with self.assertRaises(ValueError):
             opt_phone_comment_enum_from_str('+43-512-123456')
 
     def test_to_str(self):
-        self.assertEqual(opt_phone_comment_enum_to_str(None), '')
-        self.assertEqual(opt_phone_comment_enum_to_str([]), 'Nein')
-        self.assertEqual(opt_phone_comment_enum_to_str([('+43-512-123456', 'untertags')]), '+43-512-123456 (untertags)')
-        self.assertEqual(opt_phone_comment_enum_to_str([('+43-512-1234', 'untertags'), ('+43-664-123456', 'Alm')]), '+43-512-1234 (untertags); +43-664-123456 (Alm)')
+        self.assertEqual('', opt_phone_comment_enum_to_str(None))
+        self.assertEqual('Nein', opt_phone_comment_enum_to_str([]))
+        self.assertEqual('+43-512-123456 (untertags)', opt_phone_comment_enum_to_str([('+43-512-123456', 'untertags')]))
+        self.assertEqual('+43-512-1234 (untertags); +43-664-123456 (Alm)', opt_phone_comment_enum_to_str([('+43-512-1234', 'untertags'), ('+43-664-123456', 'Alm')]))
 
 
 # longitude/latitude converter
@@ -494,8 +494,8 @@ class TestLiftGermanValidator(unittest.TestCase):
 
 class TestSingleCachet(unittest.TestCase):
     def test_from_str(self):
-        self.assertEqual(single_cachet_german_from_str('Tiroler Naturrodelbahn-Gütesiegel 2009 mittel'), ('Tiroler Naturrodelbahn-Gütesiegel', '2009', 'mittel'))
-        self.assertEqual(single_cachet_german_from_str('Tiroler Naturrodelbahn-Gütesiegel 2013 schwer'), ('Tiroler Naturrodelbahn-Gütesiegel', '2013', 'schwer'))
+        self.assertEqual(('Tiroler Naturrodelbahn-Gütesiegel', '2009', 'mittel'), single_cachet_german_from_str('Tiroler Naturrodelbahn-Gütesiegel 2009 mittel'))
+        self.assertEqual(('Tiroler Naturrodelbahn-Gütesiegel', '2013', 'schwer'), single_cachet_german_from_str('Tiroler Naturrodelbahn-Gütesiegel 2013 schwer'))
         with self.assertRaises(ValueError):
             single_cachet_german_from_str('')
         with self.assertRaises(ValueError):
@@ -508,28 +508,27 @@ class TestSingleCachet(unittest.TestCase):
             single_cachet_german_from_str('Tiroler Naturrodelbahn-Gütesiegel 2013 schwerer')
 
     def test_to_str(self):
-        self.assertEqual(single_cachet_german_to_str(('Tiroler Naturrodelbahn-Gütesiegel', '2009', 'mittel')), 'Tiroler Naturrodelbahn-Gütesiegel 2009 mittel')
-        self.assertEqual(single_cachet_german_to_str(('Tiroler Naturrodelbahn-Gütesiegel', '2013', 'schwer')), 'Tiroler Naturrodelbahn-Gütesiegel 2013 schwer')
+        self.assertEqual('Tiroler Naturrodelbahn-Gütesiegel 2009 mittel', single_cachet_german_to_str(('Tiroler Naturrodelbahn-Gütesiegel', '2009', 'mittel')))
+        self.assertEqual('Tiroler Naturrodelbahn-Gütesiegel 2013 schwer', single_cachet_german_to_str(('Tiroler Naturrodelbahn-Gütesiegel', '2013', 'schwer')))
 
 
 class TestCachetGerman(unittest.TestCase):
     def test_from_str(self):
         self.assertEqual(cachet_german_from_str(''), None)
-        self.assertEqual(cachet_german_from_str('Nein'), [])
-        self.assertEqual(cachet_german_from_str('Tiroler Naturrodelbahn-Gütesiegel 2009 mittel'), [('Tiroler Naturrodelbahn-Gütesiegel', '2009', 'mittel')])
-        self.assertEqual(cachet_german_from_str('Tiroler Naturrodelbahn-Gütesiegel 2013 schwer; Tiroler Naturrodelbahn-Gütesiegel 2009 mittel'),
-            [('Tiroler Naturrodelbahn-Gütesiegel', '2013', 'schwer'), ('Tiroler Naturrodelbahn-Gütesiegel', '2009', 'mittel')])
+        self.assertEqual([], cachet_german_from_str('Nein'))
+        self.assertEqual([('Tiroler Naturrodelbahn-Gütesiegel', '2009', 'mittel')], cachet_german_from_str('Tiroler Naturrodelbahn-Gütesiegel 2009 mittel'))
+        self.assertEqual([('Tiroler Naturrodelbahn-Gütesiegel', '2013', 'schwer'), ('Tiroler Naturrodelbahn-Gütesiegel', '2009', 'mittel')], 
+                         cachet_german_from_str('Tiroler Naturrodelbahn-Gütesiegel 2013 schwer; Tiroler Naturrodelbahn-Gütesiegel 2009 mittel'))
         with self.assertRaises(ValueError):
             cachet_german_from_str('Ja')
         with self.assertRaises(ValueError):
             cachet_german_from_str('Tiroler Naturrodelbahn-Gütesiegel 2013 schwer Tiroler Naturrodelbahn-Gütesiegel 2009 mittel')
 
     def test_to_str(self):
-        self.assertEqual(cachet_german_to_str(None), '')
-        self.assertEqual(cachet_german_to_str([]), 'Nein')
-        self.assertEqual(cachet_german_to_str([('Tiroler Naturrodelbahn-Gütesiegel', '2009', 'mittel')]), 'Tiroler Naturrodelbahn-Gütesiegel 2009 mittel')
-        self.assertEqual(cachet_german_to_str([('Tiroler Naturrodelbahn-Gütesiegel', '2013', 'schwer'), ('Tiroler Naturrodelbahn-Gütesiegel', '2009', 'mittel')]),
-            'Tiroler Naturrodelbahn-Gütesiegel 2013 schwer; Tiroler Naturrodelbahn-Gütesiegel 2009 mittel')
+        self.assertEqual('', cachet_german_to_str(None))
+        self.assertEqual('Nein', cachet_german_to_str([]))
+        self.assertEqual('Tiroler Naturrodelbahn-Gütesiegel 2009 mittel', cachet_german_to_str([('Tiroler Naturrodelbahn-Gütesiegel', '2009', 'mittel')]))
+        self.assertEqual('Tiroler Naturrodelbahn-Gütesiegel 2013 schwer; Tiroler Naturrodelbahn-Gütesiegel 2009 mittel', cachet_german_to_str([('Tiroler Naturrodelbahn-Gütesiegel', '2013', 'schwer'), ('Tiroler Naturrodelbahn-Gütesiegel', '2009', 'mittel')]))
 
 
 # night light days converter
@@ -561,22 +560,22 @@ class TestNightLightDays(unittest.TestCase):
 
 class TestOptStrOptCommentEnum(unittest.TestCase):
     def test_from_str(self):
-        self.assertEqual(opt_str_opt_comment_enum_from_str(''), None)
-        self.assertEqual(opt_str_opt_comment_enum_from_str('Nein'), [])
-        self.assertEqual(opt_str_opt_comment_enum_from_str('Talstation'), [('Talstation', None)])
-        self.assertEqual(opt_str_opt_comment_enum_from_str('Talstation (unten)'), [('Talstation', 'unten')])
-        self.assertEqual(opt_str_opt_comment_enum_from_str('Talstation (unten); Mittelstation'), [('Talstation', 'unten'), ('Mittelstation', None)])
+        self.assertEqual(None, opt_str_opt_comment_enum_from_str(''))
+        self.assertEqual([], opt_str_opt_comment_enum_from_str('Nein'))
+        self.assertEqual([('Talstation', None)], opt_str_opt_comment_enum_from_str('Talstation'))
+        self.assertEqual([('Talstation', 'unten')], opt_str_opt_comment_enum_from_str('Talstation (unten)'))
+        self.assertEqual([('Talstation', 'unten'), ('Mittelstation', None)], opt_str_opt_comment_enum_from_str('Talstation (unten); Mittelstation'))
         with self.assertRaises(ValueError):
             opt_str_opt_comment_enum_from_str('(unten)')
         with self.assertRaises(ValueError):
             opt_str_opt_comment_enum_from_str('Talstation (unten); ; Mittelstation')
 
     def test_to_str(self):
-        self.assertEqual(opt_str_opt_comment_enum_to_str(None), '')
-        self.assertEqual(opt_str_opt_comment_enum_to_str([]), 'Nein')
-        self.assertEqual(opt_str_opt_comment_enum_to_str([('Talstation', None)]), 'Talstation')
-        self.assertEqual(opt_str_opt_comment_enum_to_str([('Talstation', 'unten')]), 'Talstation (unten)')
-        self.assertEqual(opt_str_opt_comment_enum_to_str([('Talstation', 'unten'), ('Mittelstation', None)]), 'Talstation (unten); Mittelstation')
+        self.assertEqual('', opt_str_opt_comment_enum_to_str(None))
+        self.assertEqual('Nein', opt_str_opt_comment_enum_to_str([]))
+        self.assertEqual('Talstation', opt_str_opt_comment_enum_to_str([('Talstation', None)]))
+        self.assertEqual('Talstation (unten)', opt_str_opt_comment_enum_to_str([('Talstation', 'unten')]))
+        self.assertEqual('Talstation (unten); Mittelstation', opt_str_opt_comment_enum_to_str([('Talstation', 'unten'), ('Mittelstation', None)]))
 
 
 # wikibox converter
@@ -587,13 +586,13 @@ class TestWikibox(unittest.TestCase):
         value = '{{MyTemplate|apple=2|banana=5}}'
         converter_dict = OrderedDict([('apple', opt_uint_converter), ('banana', opt_uint_converter)])
         result = wikibox_from_str(value, 'MyTemplate', converter_dict)
-        self.assertEqual(result['apple'], 2)
-        self.assertEqual(result['banana'], 5)
+        self.assertEqual(2, result['apple'])
+        self.assertEqual(5, result['banana'])
 
         value = '{{MyTemplate\n | apple = 2 \n| banana = 5 }}'
         result = wikibox_from_str(value, 'MyTemplate', converter_dict)
-        self.assertEqual(result['apple'], 2)
-        self.assertEqual(result['banana'], 5)
+        self.assertEqual(2, result['apple'])
+        self.assertEqual(5, result['banana'])
 
         with self.assertRaises(ValueError):
             wikibox_from_str(value, 'myTemplate', converter_dict)
@@ -611,7 +610,7 @@ class TestWikibox(unittest.TestCase):
         value = OrderedDict([('apple', 2), ('banana', 5)])
         converter_dict = OrderedDict([('apple', opt_uint_converter), ('banana', opt_uint_converter)])
         result = wikibox_to_str(value, 'MyTemplate', converter_dict)
-        self.assertEqual(result, '{{MyTemplate|apple=2|banana=5}}')
+        self.assertEqual('{{MyTemplate|apple=2|banana=5}}', result)
 
 
 # Rodelbahnbox converter
@@ -649,29 +648,29 @@ class TestRodelbahnbox(unittest.TestCase):
 
     def test_from_str(self):
         value = rodelbahnbox_from_str(self.value)
-        self.assertEqual(value['Position'], LonLat(12.806522, 46.807218))
-        self.assertEqual(value['Position oben'], LonLat(12.818658, 46.799014))
-        self.assertEqual(value['Höhe oben'], 1046)
-        self.assertEqual(value['Position unten'], LonLat(None, None))
-        self.assertEqual(value['Höhe unten'], None)
-        self.assertEqual(value['Länge'], 3500)
-        self.assertEqual(value['Schwierigkeit'], 2)
-        self.assertEqual(value['Lawinen'], 1)
-        self.assertEqual(value['Betreiber'], 'Bringungsgemeinschaft Kreithof-Dolomitenhütte')
-        self.assertEqual(value['Öffentliche Anreise'], 4)
-        self.assertEqual(value['Aufstieg möglich'], True)
-        self.assertEqual(value['Aufstieg getrennt'], (0.5, None))
-        self.assertEqual(value['Gehzeit'], 75)
-        self.assertEqual(value['Aufstiegshilfe'], [('Taxi', None), ('Sonstige', 'PKW bis Kreithof')])
-        self.assertEqual(value['Beleuchtungsanlage'], (1.0, None))
-        self.assertEqual(value['Beleuchtungstage'], (7, None))
-        self.assertEqual(value['Rodelverleih'], [])
-        self.assertEqual(value['Gütesiegel'], [('Tiroler Naturrodelbahn-Gütesiegel', '2009', 'mittel')])
-        self.assertEqual(value['Webauskunft'], (True, 'http://www.lienzerdolomiten.info/at/tobogorpt.html'))
-        self.assertEqual(value['Telefonauskunft'], [('+43-664-2253782', 'Dolomitenhütte')])
-        self.assertEqual(value['Bild'], 'Dolomitenrodelbahn Tristach 2011-12-22 oberer Bereich.jpg')
-        self.assertEqual(value['In Übersichtskarte'], True)
-        self.assertEqual(value['Forumid'], 139)
+        self.assertEqual(LonLat(12.806522, 46.807218), value['Position'])
+        self.assertEqual(LonLat(12.818658, 46.799014), value['Position oben'])
+        self.assertEqual(1046, value['Höhe oben'])
+        self.assertEqual(LonLat(None, None), value['Position unten'])
+        self.assertEqual(None, value['Höhe unten'])
+        self.assertEqual(3500, value['Länge'])
+        self.assertEqual(2, value['Schwierigkeit'])
+        self.assertEqual(1, value['Lawinen'])
+        self.assertEqual('Bringungsgemeinschaft Kreithof-Dolomitenhütte', value['Betreiber'])
+        self.assertEqual(4, value['Öffentliche Anreise'])
+        self.assertEqual(True, value['Aufstieg möglich'])
+        self.assertEqual((0.5, None), value['Aufstieg getrennt'])
+        self.assertEqual(75, value['Gehzeit'])
+        self.assertEqual([('Taxi', None), ('Sonstige', 'PKW bis Kreithof')], value['Aufstiegshilfe'])
+        self.assertEqual((1.0, None), value['Beleuchtungsanlage'])
+        self.assertEqual((7, None), value['Beleuchtungstage'])
+        self.assertEqual([], value['Rodelverleih'])
+        self.assertEqual([('Tiroler Naturrodelbahn-Gütesiegel', '2009', 'mittel')], value['Gütesiegel'])
+        self.assertEqual((True, 'http://www.lienzerdolomiten.info/at/tobogorpt.html'), value['Webauskunft'])
+        self.assertEqual([('+43-664-2253782', 'Dolomitenhütte')], value['Telefonauskunft'])
+        self.assertEqual('Dolomitenrodelbahn Tristach 2011-12-22 oberer Bereich.jpg', value['Bild'])
+        self.assertEqual(True, value['In Übersichtskarte'])
+        self.assertEqual(139, value['Forumid'])
 
     def test_to_str(self):
         value = OrderedDict([
@@ -698,7 +697,7 @@ class TestRodelbahnbox(unittest.TestCase):
             ('Bild', 'Dolomitenrodelbahn Tristach 2011-12-22 oberer Bereich.jpg'),
             ('In Übersichtskarte', True),
             ('Forumid', 139)])
-        self.assertEqual(rodelbahnbox_to_str(value), self.value)
+        self.assertEqual(self.value, rodelbahnbox_to_str(value))
 
 
 # TODO: Gasthausbox converter