Merge lp:~canonical-platform-qa/cupstream2distro-config/address_book_service_integration_testing_with_clients into lp:cupstream2distro-config

Proposed by Omer Akram
Status: Merged
Approved by: Francis Ginther
Approved revision: 1168
Merged at revision: 1167
Proposed branch: lp:~canonical-platform-qa/cupstream2distro-config/address_book_service_integration_testing_with_clients
Merge into: lp:cupstream2distro-config
Diff against target: 13 lines (+3/-0)
1 file modified
stacks/head/phone.cfg (+3/-0)
To merge this branch: bzr merge lp:~canonical-platform-qa/cupstream2distro-config/address_book_service_integration_testing_with_clients
Reviewer Review Type Date Requested Status
PS Jenkins bot continuous-integration Approve
Francis Ginther Approve
Review via email: mp+227787@code.launchpad.net

Commit message

run integration tests for the service in CI

Description of the change

run integration tests for the service in CI

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
Francis Ginther (fginther) wrote :

For running the autopilot test suite, all you need to add are the following:

9 + standard_jobs: generic-mediumtests-utopic generic-deb-autopilot-utopic-touch
20 + test_suite: address_book_service
21 + test_packages: address-book-service address-book-service-testability

The changes to the other parameters have no impact on running of the integration tests, but these do result in disabling code coverage builds. I don't think disabling code coverage was your intent.

review: Needs Fixing
Revision history for this message
Omer Akram (om26er) wrote :

> For running the autopilot test suite, all you need to add are the following:
>
> 9 + standard_jobs: generic-mediumtests-utopic generic-deb-
> autopilot-utopic-touch
> 20 + test_suite: address_book_service
> 21 + test_packages: address-book-service address-book-service-
> testability
>
> The changes to the other parameters have no impact on running of the
> integration tests, but these do result in disabling code coverage builds. I
> don't think disabling code coverage was your intent.

Ok thanks. Fixed that.

1167. By Omer Akram

fix per suggests

1168. By Omer Akram

more

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
Francis Ginther (fginther) wrote :

Thanks for addressing the comment.

review: Approve
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
1=== modified file 'stacks/head/phone.cfg'
2--- stacks/head/phone.cfg 2014-06-09 20:48:31 +0000
3+++ stacks/head/phone.cfg 2014-07-23 17:43:56 +0000
4@@ -25,6 +25,9 @@
5 daily_release: False
6 autolanding_template: False
7 use_stack_ppa: False
8+ standard_jobs: generic-mediumtests-utopic generic-deb-autopilot-utopic-touch
9+ test_suite: address_book_service_testability
10+ test_packages: address-book-service address-book-service-testability
11 address-book-app:
12 daily_release: False
13 autolanding_template: False

Subscribers

People subscribed via source and target branches

to all changes: