Autopilot

Autopilot doesn't follow pep8

Bug #1201057 reported by Leo Arias
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Autopilot
Fix Released
Undecided
Leo Arias
Autopilot 1.4
autopilot (Ubuntu)
Fix Released
Undecided
Unassigned

Bug Description

One of the best things of python code is that it is pretty. If we follow pep8 [1], it's even prettier. Also, it easier to read for people used to follow those conventions.

[1] http://www.python.org/dev/peps/pep-0008/

Related branches

lp:~elopio/autopilot/fix1201057-pep8-1
Thomi Richards (community): Approve
PS Jenkins bot: Approve (continuous-integration)
Diff: 598 lines (+145/-117)
3 files modified
autopilot/input/__init__.py (+86/-78)
autopilot/input/_common.py (+21/-10)
autopilot/input/_uinput.py (+38/-29)
lp:~elopio/autopilot/fix1201057-pep8-2
Thomi Richards (community): Approve
PS Jenkins bot: Approve (continuous-integration)
Diff: 619 lines (+160/-120)
3 files modified
autopilot/matchers/__init__.py (+25/-18)
autopilot/process/__init__.py (+49/-47)
autopilot/process/_bamf.py (+86/-55)
lp:~elopio/autopilot/fix1201057-pep8-3
Thomi Richards (community): Approve
PS Jenkins bot: Approve (continuous-integration)
Diff: 389 lines (+77/-58)
8 files modified
autopilot/display/_X11.py (+8/-5)
autopilot/display/__init__.py (+16/-11)
autopilot/display/_upa.py (+1/-0)
autopilot/vis/__init__.py (+1/-1)
autopilot/vis/bus_enumerator.py (+10/-12)
autopilot/vis/main_window.py (+17/-11)
autopilot/vis/objectproperties.py (+16/-10)
autopilot/vis/resources.py (+8/-8)
lp:~elopio/autopilot/fix1201057-pep8-4
Thomi Richards (community): Needs Fixing
PS Jenkins bot: Approve (continuous-integration)
Diff: 1036 lines (+252/-180)
5 files modified
autopilot/introspection/__init__.py (+91/-55)
autopilot/introspection/backends.py (+28/-23)
autopilot/introspection/dbus.py (+109/-82)
autopilot/introspection/gtk.py (+2/-4)
autopilot/introspection/qt.py (+22/-16)
lp:~elopio/autopilot/fix1201057-pep8-5
Thomi Richards (community): Approve
PS Jenkins bot: Approve (continuous-integration)
Diff: 1441 lines (+373/-230)
10 files modified
autopilot/tests/functional/test_ap_apps.py (+13/-10)
autopilot/tests/functional/test_application_mixin.py (+26/-13)
autopilot/tests/functional/test_autopilot_functional.py (+256/-164)
autopilot/tests/functional/test_custom_assertions.py (+33/-16)
autopilot/tests/functional/test_dbus_query.py (+2/-1)
autopilot/tests/functional/test_input_stack.py (+5/-2)
autopilot/tests/functional/test_introspection_features.py (+3/-4)
autopilot/tests/functional/test_mouse_emulator.py (+23/-13)
autopilot/tests/functional/test_open_window.py (+2/-1)
autopilot/tests/functional/test_process_emulator.py (+10/-6)
lp:~elopio/autopilot/fix1201057-pep8-6
Thomi Richards (community): Approve
PS Jenkins bot: Approve (continuous-integration)
Diff: 376 lines (+119/-70)
4 files modified
autopilot/introspection/__init__.py (+1/-1)
autopilot/tests/unit/test_proxy_objects.py (+103/-61)
autopilot/tests/unit/test_start_final_events.py (+1/-1)
autopilot/tests/unit/test_version_utility_fns.py (+14/-7)
lp:~elopio/autopilot/fix1201057-pep8-7
Thomi Richards (community): Approve
PS Jenkins bot: Approve (continuous-integration)
Diff: 406 lines (+86/-49)
8 files modified
autopilot/tests/unit/test_backend.py (+17/-8)
autopilot/tests/unit/test_command_line_args.py (+4/-2)
autopilot/tests/unit/test_custom_exceptions.py (+9/-4)
autopilot/tests/unit/test_introspection_features.py (+4/-0)
autopilot/tests/unit/test_matchers.py (+11/-6)
autopilot/tests/unit/test_out_of_test_addcleanup.py (+4/-3)
autopilot/tests/unit/test_pick_backend.py (+22/-15)
autopilot/tests/unit/test_platform.py (+15/-11)
lp:~elopio/autopilot/fix1201057-pep8-8
Thomi Richards (community): Approve
PS Jenkins bot: Approve (continuous-integration)
Diff: 1100 lines (+276/-198)
12 files modified
autopilot/__init__.py (+46/-43)
autopilot/clipboard.py (+2/-2)
autopilot/dbus_handler.py (+1/-1)
autopilot/emulators.py (+5/-7)
autopilot/gestures.py (+6/-4)
autopilot/globals.py (+20/-11)
autopilot/ibus.py (+14/-12)
autopilot/keybindings.py (+29/-19)
autopilot/platform.py (+13/-11)
autopilot/testcase.py (+98/-66)
autopilot/testresult.py (+2/-1)
autopilot/utilities.py (+40/-21)
lp:ubuntu/trusty/autopilot
Leo Arias (elopio)
Changed in autopilot:
status: New → In Progress
assignee: nobody → Leo Arias (elopio)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

Fix committed into lp:autopilot at revision None, scheduled for release in autopilot, milestone 1.3.2

Changed in autopilot:
status: In Progress → Fix Committed
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

Fix committed into lp:autopilot at revision 269, scheduled for release in autopilot, milestone 1.3.2

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

Fix committed into lp:autopilot at revision 270, scheduled for release in autopilot, milestone 1.3.2

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

Fix committed into lp:autopilot at revision 271, scheduled for release in autopilot, milestone 1.3.2

Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package autopilot - 1.3.1+13.10.20130715-0ubuntu1

---------------
autopilot (1.3.1+13.10.20130715-0ubuntu1) saucy; urgency=low

  [ Leo Arias ]
  * Fixed pep8 errors in autopilot/input. (LP: #1201057)
  * Fixed pep8 errors in autopilot/process and autopilot/matchers. (LP:
    #1201057)
  * Fixed pep8 errors in autopilto/vis and autopilot/display. (LP:
    #1201057)
  * Fixed pep8 errors in autopilot/introspection. (LP: #1201057)
  * Fixed pep8 errors in autopilot/functional. (LP: #1201057)

  [ Ubuntu daily release ]
  * Automatic snapshot from revision 271
 -- Ubuntu daily release <email address hidden> Mon, 15 Jul 2013 00:01:56 +0000

Changed in autopilot (Ubuntu):
status: New → Fix Released
Leo Arias (elopio)
Changed in autopilot:
status: Fix Committed → In Progress
Changed in autopilot (Ubuntu):
status: Fix Released → In Progress
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

Fix committed into lp:autopilot at revision 278, scheduled for release in autopilot, milestone 1.3.2

Changed in autopilot:
status: In Progress → Fix Committed
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

Fix committed into lp:autopilot at revision 281, scheduled for release in autopilot, milestone 1.3.2

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

Fix committed into lp:autopilot at revision 282, scheduled for release in autopilot, milestone 1.3.2

Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package autopilot - 1.3.1+13.10.20130724.1-0ubuntu1

---------------
autopilot (1.3.1+13.10.20130724.1-0ubuntu1) saucy; urgency=low

  [ Thomi Richards ]
  * Add mouse log message. (LP: #1203613)
  * Update Pointer wrapper coordinates when performing drag operation on
    a touch device. (LP: #1203808)
  * Fix failing tests. (LP: #1204031)

  [ Leo Arias ]
  * Fixed pep8 errors in autopilot/tests/unit, part 1. (LP: #1201057)
  * Fixed pep8 errors in autopilot/tests/unit, part 2. (LP: #1201057)
  * Fixed pep8 errors in the autopilot folder. (LP: #1201057)

  [ Christopher Lee ]
  * Fixes bug 1203716 where patch_environment didn't cleanup after
    itself. (LP: #1203716)

  [ Ubuntu daily release ]
  * Automatic snapshot from revision 284
 -- Ubuntu daily release <email address hidden> Wed, 24 Jul 2013 07:35:34 +0000

Changed in autopilot (Ubuntu):
status: In Progress → Fix Released
Thomi Richards (thomir-deactivatedaccount)
Changed in autopilot:
milestone: none → 1.4
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.