Merge lp://staging/~canonical-ci-engineering/ubuntu-test-cases/testing-docs into lp://staging/ubuntu-test-cases/touch

Proposed by Francis Ginther
Status: Merged
Approved by: Francis Ginther
Approved revision: 378
Merged at revision: 376
Proposed branch: lp://staging/~canonical-ci-engineering/ubuntu-test-cases/testing-docs
Merge into: lp://staging/ubuntu-test-cases/touch
Diff against target: 120 lines (+84/-1)
1 file modified
README-cli.rst (+84/-1)
To merge this branch: bzr merge lp://staging/~canonical-ci-engineering/ubuntu-test-cases/testing-docs
Reviewer Review Type Date Requested Status
Francis Ginther Approve
Paul Larson Approve
Thomi Richards (community) Approve
Review via email: mp+251507@code.staging.launchpad.net

This proposal supersedes a proposal from 2015-03-02.

Commit message

Add README documentation for developer centric testing of a modified debian or click application.

Description of the change

Add README documentation for developer centric testing of a modified debian or click application.

To post a comment you must log in.
Revision history for this message
Francis Ginther (fginther) wrote : Posted in a previous version of this proposal

I've also been playing with a modified version oh phablet-click-test-setup (from phablet-tools) which takes care of installation of the test sources and would allow removing the following:

33 +The above phablet-click-test-setup command will install the standard test
34 +dependencies and the click application's test sources as specified in the
35 +manifest. This is usually the application's trunk branch. To override the test
36 +sources with local changes, replace the test sources that were copied to the
37 +device. This example assumes the application code is checked out under the
38 +'dropping-letters' directory with the test sources under 'tests/autopilot'::
39 +
40 + adb shell rm -rf /home/phablet/autopilot/dropping_letters_app
41 + adb push dropping-letters/tests/autopilot \
42 + /home/phablet/autopilot

Revision history for this message
Paul Larson (pwlars) wrote : Posted in a previous version of this proposal

is pkcon preferred over click install?

Revision history for this message
Francis Ginther (fginther) wrote : Posted in a previous version of this proposal

> is pkcon preferred over click install?

I've been unable to find info in any of the wiki's but the 'click' command provides this:
"
phablet@ubuntu-phablet:~$ click install --help
Usage: click install [options] PACKAGE-FILE

This is a low-level tool; to install a package as an ordinary user
you should generally use "pkcon install-local PACKAGE-FILE"
instead.
"

Will keep looking.

Revision history for this message
Colin Watson (cjwatson) wrote : Posted in a previous version of this proposal

The click documentation should be considered authoritative on this.

377. By Francis Ginther

A couple of minor corrections and updates.

Revision history for this message
Thomi Richards (thomir-deactivatedaccount) wrote :

Found a typo

review: Needs Fixing
Revision history for this message
Francis Ginther (fginther) wrote :

> Found a typo

plars fixed this in rev 377.

Revision history for this message
Thomi Richards (thomir-deactivatedaccount) wrote :

That looks better then.

review: Approve
378. By Paul Larson

Add a section on running tests for a modified debian package

Revision history for this message
Paul Larson (pwlars) wrote :

This now has the debian package testing instructions merged, and I think that's all we were intending to add. So there was something else, I'm +1 to go ahead and merge this in.

review: Approve
Revision history for this message
Francis Ginther (fginther) wrote :

Also approve.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches