Autopilot

Merge lp://staging/~javier.collado/autopilot/bug1285575 into lp://staging/autopilot

bug1285575
Merge into trunk

Proposed by Javier Collado on 2014-02-27

Status:

Needs review

Proposed branch:

lp://staging/~javier.collado/autopilot/bug1285575

Merge into:

lp://staging/autopilot

Diff against target:

263 lines (+195/-1)

2 files modified

autopilot/introspection/__init__.py (+76/-1)
autopilot/tests/unit/test_introspection_features.py (+119/-0)

To merge this branch:

bzr merge lp://staging/~javier.collado/autopilot/bug1285575

Low

In Progress

Link a bug report

Reviewer	Review Type	Date Requested	Status
PS Jenkins bot	continuous-integration		Needs Fixing on 2014-03-03
Autopilot Hackers		2014-02-27	Pending
Review via email: mp+208638@code.staging.launchpad.net

Commit message

Get proxy object faster if process information is given

Process name and command line can be passed to filter processes based on the information from /proc/<pid>/ to get a proxy object faster (no need to query multiple connections through dbus and wait for long time because of timeouts)

Description of the change

This change adds the process_name and process_cmdline options to
autopilot.introspection.get_proxy_object_for_existing_process.

This can be used to get a proxy object faster than with only the
application_name since there's no need to query each dbus connection one by one
to find the one for the application being searched.

Revision history for this message

PS Jenkins bot (ps-jenkins) wrote on 2014-02-27:

FAILED: Continuous integration, rev:446
No commit message was specified in the merge proposal. Click on the following link and set the commit message (if you want a jenkins rebuild you need to trigger it yourself):
https://code.launchpad.net/~javier.collado/autopilot/bug1285575/+merge/208638/+edit-commit-message

http://jenkins.qa.ubuntu.com/job/autopilot-ci/534/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/autopilot-trusty-amd64-ci/260/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/autopilot-trusty-armhf-ci/260/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/autopilot-trusty-i386-ci/169/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-trusty/3565/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-amd64/3569/console

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/autopilot-ci/534/rebuild

review: Needs Fixing (continuous-integration)

lp://staging/~javier.collado/autopilot/bug1285575 updated on 2014-03-03

447. By Javier Collado on 2014-03-03

Fixed test_makes_child_calls

autopilot.tests.unit.test_introspection_features.ProxyObjectGenerationTests.test_makes_child_calls

448. By Javier Collado on 2014-03-03

Fixed test_raises_ProcessSearchError

autopilot.tests.unit.test_introspection_features.ProxyObjectGenerationTests.test_raises_ProcessSearchError

Revision history for this message

PS Jenkins bot (ps-jenkins) wrote on 2014-03-03:

FAILED: Continuous integration, rev:448
No commit message was specified in the merge proposal. Click on the following link and set the commit message (if you want a jenkins rebuild you need to trigger it yourself):
https://code.launchpad.net/~javier.collado/autopilot/bug1285575/+merge/208638/+edit-commit-message

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/autopilot-ci/535/rebuild

review: Needs Fixing (continuous-integration)

lp://staging/~javier.collado/autopilot/bug1285575 updated on 2014-03-03

449. By Javier Collado on 2014-03-03: Added a few test cases for _check_proc_fs

Revision history for this message

PS Jenkins bot (ps-jenkins) wrote on 2014-03-03:

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/autopilot-ci/536/rebuild

review: Needs Fixing (continuous-integration)

Unmerged revisions

449. By Javier Collado on 2014-03-03

Added a few test cases for _check_proc_fs

448. By Javier Collado on 2014-03-03

Fixed test_raises_ProcessSearchError

autopilot.tests.unit.test_introspection_features.ProxyObjectGenerationTests.test_raises_ProcessSearchError

447. By Javier Collado on 2014-03-03

Fixed test_makes_child_calls

autopilot.tests.unit.test_introspection_features.ProxyObjectGenerationTests.test_makes_child_calls

446. By Javier Collado on 2014-02-27

Added _check_proc_fs function

This function helps to search a process for a given process name and/or process
command line. The name and command line are extracted from /proc/<pid>/stat and
/proc/<pid>/cmdline by the psutil library.