Merge lp:~danilo/pygettextpo/pep8-fixes into lp:pygettextpo

Proposed by Данило Шеган
Status: Needs review
Proposed branch: lp:~danilo/pygettextpo/pep8-fixes
Merge into: lp:pygettextpo
Diff against target: 54 lines (+9/-1)
1 file modified
test_gettextpo.py (+9/-1)
To merge this branch: bzr merge lp:~danilo/pygettextpo/pep8-fixes
Reviewer Review Type Date Requested Status
Benji York (community) code Approve
Review via email: mp+92615@code.launchpad.net

Description of the change

Provide minimal PEP-8 whitespace fixes so pep8 tool doesn't complain.

Tested with "pep8 *.py".

To post a comment you must log in.
Revision history for this message
Benji York (benji) wrote :

Looks good. Long live consistent whitespace!

review: Approve (code)
Revision history for this message
Colin Watson (cjwatson) wrote :

I've converted pygettextpo to git and merged this branch (though I can't mark this MP as Merged):

  https://git.launchpad.net/pygettextpo/commit/?id=682497c4c61c29514cae59028fdfed0128e69ee9

Unmerged revisions

25. By Данило Шеган

Minimal PEP-8 fixes.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'test_gettextpo.py'
2--- test_gettextpo.py 2009-06-02 15:31:25 +0000
3+++ test_gettextpo.py 2012-02-11 11:17:17 +0000
4@@ -4,11 +4,14 @@
5 import unittest
6 import gettextpo
7
8+
9 class PoFileTestCase(unittest.TestCase):
10+
11 def testCreateEmpty(self):
12 # Test that we can create an empty pofile object
13 pofile = gettextpo.PoFile()
14 self.assertEquals(list(iter(pofile)), [])
15+
16 def testAddMessage(self):
17 # Test that we can add messages to a new pofile object
18 pofile = gettextpo.PoFile()
19@@ -18,6 +21,7 @@
20 poiter.insert(msg)
21
22 self.assertEquals(list(iter(pofile)), [msg])
23+
24 def testAddMessageTwice(self):
25 # A message object can only be added to one pofile object
26 pofile1 = gettextpo.PoFile()
27@@ -31,7 +35,9 @@
28 poiter = iter(pofile2)
29 self.assertRaises(ValueError, poiter.insert, msg)
30
31+
32 class PoMessageTestCase(unittest.TestCase):
33+
34 def testCreateMessage(self):
35 # Test that messages can be created.
36 msg = gettextpo.PoMessage()
37@@ -47,7 +53,7 @@
38 msg = gettextpo.PoMessage()
39 msg.set_msgstr('Hello World')
40 self.assertEquals(msg.msgstr, 'Hello World')
41-
42+
43 def testSetMsgStrPlural(self):
44 # Test handling of plural msgstrs. The PoMessage object can
45 # not hold plural msgstrs if the msgid does not have a plural.
46@@ -63,7 +69,9 @@
47 msg.set_msgstr_plural(2, 'Two')
48 self.assertEquals(msg.msgstr_plural, ['Zero', 'One', 'Two'])
49
50+
51 class CheckFormatTestCase(unittest.TestCase):
52+
53 def testGoodFormat(self):
54 # Check that no exception is raised on a good translation.
55

Subscribers

People subscribed via source and target branches