Merge lp:~osomon/notes-app/fix-pep8-warnings into lp:notes-app

Proposed by Olivier Tilloy
Status: Merged
Approved by: Olivier Tilloy
Approved revision: 195
Merged at revision: 202
Proposed branch: lp:~osomon/notes-app/fix-pep8-warnings
Merge into: lp:notes-app
Diff against target: 119 lines (+20/-14)
5 files modified
tests/autopilot/notes_app/tests/test_delete.py (+2/-1)
tests/autopilot/notes_app/tests/test_expand_collapse.py (+12/-10)
tests/autopilot/notes_app/tests/test_images.py (+2/-1)
tests/autopilot/notes_app/tests/test_parts.py (+2/-1)
tests/autopilot/notes_app/tests/test_quit.py (+2/-1)
To merge this branch: bzr merge lp:~osomon/notes-app/fix-pep8-warnings
Reviewer Review Type Date Requested Status
PS Jenkins bot continuous-integration Approve
Ugo Riboni (community) Approve
Review via email: mp+188523@code.launchpad.net

Commit message

Fix pep8 warnings.

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Ugo Riboni (uriboni) wrote :

Code looks ok for me, so comment-approving until CI gets its stuff together.

review: Approve
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'tests/autopilot/notes_app/tests/test_delete.py'
--- tests/autopilot/notes_app/tests/test_delete.py 2013-07-29 20:53:21 +0000
+++ tests/autopilot/notes_app/tests/test_delete.py 2013-10-01 07:45:21 +0000
@@ -40,7 +40,8 @@
40 conn = sqlite3.connect(path)40 conn = sqlite3.connect(path)
41 cursor = conn.cursor()41 cursor = conn.cursor()
42 cursor.execute("DELETE FROM notes")42 cursor.execute("DELETE FROM notes")
43 cursor.execute("INSERT INTO notes (date, note) VALUES ('2013-04-07', '" + note_data + "')")43 cursor.execute("INSERT INTO notes (date, note) "
44 "VALUES ('2013-04-07', '" + note_data + "')")
44 conn.commit()45 conn.commit()
45 conn.close()46 conn.close()
4647
4748
=== modified file 'tests/autopilot/notes_app/tests/test_expand_collapse.py'
--- tests/autopilot/notes_app/tests/test_expand_collapse.py 2013-08-06 12:47:59 +0000
+++ tests/autopilot/notes_app/tests/test_expand_collapse.py 2013-10-01 07:45:21 +0000
@@ -46,7 +46,8 @@
46 cursor = conn.cursor()46 cursor = conn.cursor()
47 cursor.execute("DELETE FROM notes")47 cursor.execute("DELETE FROM notes")
48 for note in notes:48 for note in notes:
49 cursor.execute("INSERT INTO notes (date, note) VALUES ('2013-04-07', '" + note + "')")49 cursor.execute("INSERT INTO notes (date, note) "
50 "VALUES ('2013-04-07', '" + note + "')")
50 conn.commit()51 conn.commit()
51 conn.close()52 conn.close()
5253
@@ -60,16 +61,17 @@
60 self.pointing_device.click_object(first)61 self.pointing_device.click_object(first)
61 self.assertThat(first.isExpanded, Eventually(Equals(True)))62 self.assertThat(first.isExpanded, Eventually(Equals(True)))
6263
63 # Here and in further tests we have to wait for height to equal actualExpandedHeight64 # Here and in further tests we have to wait for height to equal
64 # because that is the point when the expanding animation is finished and the note65 # actualExpandedHeight because that is the point when the expanding
65 # reaches its final expanded geometry. Just testing for isExpanded does not work66 # animation is finished and the note reaches its final expanded
66 # as the property changes immediately and then the animation is triggered.67 # geometry. Just testing for isExpanded does not work as the property
67 self.assertThat(first.height, Eventually(Equals(first.actualExpandedHeight)))68 # changes immediately and then the animation is triggered.
69 first.height.wait_for(first.actualExpandedHeight)
6870
69 self.pointing_device.click_object(second)71 self.pointing_device.click_object(second)
70 self.assertThat(first.isExpanded, Eventually(Equals(False)))72 self.assertThat(first.isExpanded, Eventually(Equals(False)))
71 self.assertThat(second.isExpanded, Eventually(Equals(True)))73 self.assertThat(second.isExpanded, Eventually(Equals(True)))
72 self.assertThat(second.height, Eventually(Equals(second.actualExpandedHeight)))74 second.height.wait_for(second.actualExpandedHeight)
7375
74 self.pointing_device.click_object(first)76 self.pointing_device.click_object(first)
75 self.assertThat(first.isExpanded, Eventually(Equals(True)))77 self.assertThat(first.isExpanded, Eventually(Equals(True)))
@@ -81,7 +83,7 @@
8183
82 self.pointing_device.click_object(first)84 self.pointing_device.click_object(first)
83 self.assertThat(first.isExpanded, Eventually(Equals(True)))85 self.assertThat(first.isExpanded, Eventually(Equals(True)))
84 self.assertThat(first.height, Eventually(Equals(first.actualExpandedHeight)))86 first.height.wait_for(first.actualExpandedHeight)
8587
86 self.pointing_device.click_object(header)88 self.pointing_device.click_object(header)
87 self.assertThat(first.isExpanded, Eventually(Equals(False)))89 self.assertThat(first.isExpanded, Eventually(Equals(False)))
@@ -92,7 +94,7 @@
9294
93 self.pointing_device.click_object(first)95 self.pointing_device.click_object(first)
94 self.assertThat(first.isExpanded, Eventually(Equals(True)))96 self.assertThat(first.isExpanded, Eventually(Equals(True)))
95 self.assertThat(first.height, Eventually(Equals(first.actualExpandedHeight)))97 first.height.wait_for(first.actualExpandedHeight)
9698
97 # click in the empty space after all the notes99 # click in the empty space after all the notes
98 note_x, note_y, note_w, note_h = second.globalRect100 note_x, note_y, note_w, note_h = second.globalRect
@@ -106,7 +108,7 @@
106108
107 self.pointing_device.click_object(first)109 self.pointing_device.click_object(first)
108 self.assertThat(first.isExpanded, Eventually(Equals(True)))110 self.assertThat(first.isExpanded, Eventually(Equals(True)))
109 self.assertThat(first.height, Eventually(Equals(first.actualExpandedHeight)))111 first.height.wait_for(first.actualExpandedHeight)
110112
111 # click in the empty space between notes113 # click in the empty space between notes
112 note_x, note_y, note_w, note_h = first.globalRect114 note_x, note_y, note_w, note_h = first.globalRect
113115
=== modified file 'tests/autopilot/notes_app/tests/test_images.py'
--- tests/autopilot/notes_app/tests/test_images.py 2013-09-26 17:55:31 +0000
+++ tests/autopilot/notes_app/tests/test_images.py 2013-10-01 07:45:21 +0000
@@ -37,7 +37,8 @@
37 conn = sqlite3.connect(path)37 conn = sqlite3.connect(path)
38 cursor = conn.cursor()38 cursor = conn.cursor()
39 cursor.execute("DELETE FROM notes")39 cursor.execute("DELETE FROM notes")
40 cursor.execute("INSERT INTO notes (date, note) VALUES ('2013-04-07', '" + note_data + "')")40 cursor.execute("INSERT INTO notes (date, note) "
41 "VALUES ('2013-04-07', '" + note_data + "')")
41 conn.commit()42 conn.commit()
42 conn.close()43 conn.close()
4344
4445
=== modified file 'tests/autopilot/notes_app/tests/test_parts.py'
--- tests/autopilot/notes_app/tests/test_parts.py 2013-07-29 20:53:21 +0000
+++ tests/autopilot/notes_app/tests/test_parts.py 2013-10-01 07:45:21 +0000
@@ -38,7 +38,8 @@
38 conn = sqlite3.connect(path)38 conn = sqlite3.connect(path)
39 cursor = conn.cursor()39 cursor = conn.cursor()
40 cursor.execute("DELETE FROM notes")40 cursor.execute("DELETE FROM notes")
41 cursor.execute("INSERT INTO notes (date, note) VALUES ('2013-04-07', '" + note_data + "')")41 cursor.execute("INSERT INTO notes (date, note) "
42 "VALUES ('2013-04-07', '" + note_data + "')")
42 conn.commit()43 conn.commit()
43 conn.close()44 conn.close()
4445
4546
=== modified file 'tests/autopilot/notes_app/tests/test_quit.py'
--- tests/autopilot/notes_app/tests/test_quit.py 2013-09-26 17:55:31 +0000
+++ tests/autopilot/notes_app/tests/test_quit.py 2013-10-01 07:45:21 +0000
@@ -76,7 +76,8 @@
76 conn = sqlite3.connect(path)76 conn = sqlite3.connect(path)
77 cursor = conn.cursor()77 cursor = conn.cursor()
78 cursor.execute("DELETE FROM notes")78 cursor.execute("DELETE FROM notes")
79 cursor.execute("INSERT INTO notes (date, note) VALUES ('2013-04-07', '" + note_data + "')")79 cursor.execute("INSERT INTO notes (date, note) "
80 "VALUES ('2013-04-07', '" + note_data + "')")
80 conn.commit()81 conn.commit()
81 conn.close()82 conn.close()
8283

Subscribers

People subscribed via source and target branches