Merge lp://staging/~achiang/savilerow/core-v-api into lp://staging/~savilerow-team/savilerow/trunk-1.0

Proposed by Alex Chiang
Status: Merged
Merged at revision: 67
Proposed branch: lp://staging/~achiang/savilerow/core-v-api
Merge into: lp://staging/~savilerow-team/savilerow/trunk-1.0
Diff against target: 397 lines (+194/-129)
9 files modified
doc/README (+54/-4)
doc/api.rst (+64/-0)
doc/conf.py (+2/-2)
doc/core.rst (+34/-0)
doc/index.rst (+2/-2)
doc/tests.rst (+0/-102)
tests/api/__init__.py (+19/-0)
tests/core/__init__.py (+19/-0)
tests/tests/__init__.py (+0/-19)
To merge this branch: bzr merge lp://staging/~achiang/savilerow/core-v-api
Reviewer Review Type Date Requested Status
Chris Wayne (community) Approve
Review via email: mp+205049@code.staging.launchpad.net

Description of the change

Split tests into core vs api.

Note that we need to actually test this before approving the merge.

To post a comment you must log in.
Revision history for this message
Chris Wayne (cwayne) wrote :

Tested locally by pushing the tests onto an N4 (to /home/phablet/autopilot/customization), then running 'autopilot run customization', and they seemed to run just fine, although it would be good to check and see if this would affect how it's run in CI (i.e. do they assume any particular dir structure?)

Revision history for this message
Chris Wayne (cwayne) wrote :

SO in lp:ubuntu-test-cases/touch , it just pulls lp:savilerow/tests, so this should be good to go

review: Approve
Revision history for this message
Alex Chiang (achiang) wrote :

That is just the branch command.

We still need to consider how the tests are actually executed.

Notice the runes in tests/customization/tsauto.list in ubuntu-test-cases, and the substitution that's in there...

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