Merge lp:~michihenning/storage-framework/add-mcloud-scope into lp:storage-framework/devel
- add-mcloud-scope
- Merge into devel
Status: | Merged |
---|---|
Approved by: | Michi Henning |
Approved revision: | 48 |
Merged at revision: | 50 |
Proposed branch: | lp:~michihenning/storage-framework/add-mcloud-scope |
Merge into: | lp:storage-framework/devel |
Diff against target: |
68 lines (+17/-8) 3 files modified
src/qt/client/internal/remote_client/RuntimeImpl.cpp (+12/-5) tests/remote-client/remote-client_test.cpp (+3/-3) tests/utils/fake-online-accounts-daemon.py (+2/-0) |
To merge this branch: | bzr merge lp:~michihenning/storage-framework/add-mcloud-scope |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
unity-api-1-bot | continuous-integration | Approve | |
James Henstridge | Approve | ||
Review via email: mp+302619@code.launchpad.net |
Commit message
Added fake mcloud scope.
Description of the change
Added fake mcloud scope.
unity-api-1-bot (unity-api-1-bot) wrote : | # |
unity-api-1-bot (unity-api-1-bot) wrote : | # |
FAILED: Autolanding.
Merging failed. More details in the following jenkins job:
https:/
Executed test runs:
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
unity-api-1-bot (unity-api-1-bot) wrote : | # |
FAILED: Continuous integration, rev:47
https:/
Executed test runs:
FAILURE: https:/
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
FAILURE: https:/
FAILURE: https:/
FAILURE: https:/
FAILURE: https:/
FAILURE: https:/
FAILURE: https:/
FAILURE: https:/
FAILURE: https:/
FAILURE: https:/
Click here to trigger a rebuild:
https:/
unity-api-1-bot (unity-api-1-bot) wrote : | # |
PASSED: Continuous integration, rev:48
https:/
Executed test runs:
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
Click here to trigger a rebuild:
https:/
unity-api-1-bot (unity-api-1-bot) wrote : | # |
FAILED: Autolanding.
More details in the following jenkins job:
https:/
Executed test runs:
FAILURE: https:/
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
FAILURE: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
unity-api-1-bot (unity-api-1-bot) : | # |
Preview Diff
1 | === modified file 'src/qt/client/internal/remote_client/RuntimeImpl.cpp' |
2 | --- src/qt/client/internal/remote_client/RuntimeImpl.cpp 2016-08-03 06:29:48 +0000 |
3 | +++ src/qt/client/internal/remote_client/RuntimeImpl.cpp 2016-08-11 06:12:08 +0000 |
4 | @@ -123,15 +123,22 @@ |
5 | } |
6 | |
7 | timer_.stop(); |
8 | + |
9 | + static QString const service_ids[] = { "com.canonical.scopes.mcloud_mcloud_mcloud", "google-drive-scope" }; |
10 | + |
11 | try |
12 | { |
13 | QVector<Account::SPtr> accounts; |
14 | - for (auto const& a : manager_->availableAccounts("google-drive-scope")) |
15 | + for (auto const service_id : service_ids) |
16 | { |
17 | - auto impl = new AccountImpl(public_instance_, a->id(), "", a->serviceId(), a->displayName()); |
18 | - Account::SPtr acc(new Account(impl)); |
19 | - impl->set_public_instance(acc); |
20 | - accounts.append(acc); |
21 | + for (auto const& a : manager_->availableAccounts(service_id)) |
22 | + { |
23 | + qDebug() << "got account:" << a->displayName() << a->serviceId() << a->id(); |
24 | + auto impl = new AccountImpl(public_instance_, a->id(), "", a->serviceId(), a->displayName()); |
25 | + Account::SPtr acc(new Account(impl)); |
26 | + impl->set_public_instance(acc); |
27 | + accounts.append(acc); |
28 | + } |
29 | } |
30 | accounts_ = accounts; |
31 | make_ready_future(qf_, accounts); |
32 | |
33 | === modified file 'tests/remote-client/remote-client_test.cpp' |
34 | --- tests/remote-client/remote-client_test.cpp 2016-08-05 05:37:23 +0000 |
35 | +++ tests/remote-client/remote-client_test.cpp 2016-08-11 06:12:08 +0000 |
36 | @@ -52,7 +52,7 @@ |
37 | void SetUp() override |
38 | { |
39 | dbus_.reset(new DBusEnvironment); |
40 | - dbus_->add_demo_provider("google-drive-scope"); |
41 | + dbus_->add_demo_provider("com.canonical.scopes.mcloud_mcloud_mcloud"); |
42 | dbus_->start_services(); |
43 | } |
44 | |
45 | @@ -188,8 +188,8 @@ |
46 | auto acc = get_account(runtime); |
47 | EXPECT_EQ(runtime, acc->runtime()); |
48 | EXPECT_EQ("", acc->owner()); |
49 | - EXPECT_EQ("google-drive-scope", acc->owner_id()) << acc->owner_id().toStdString(); |
50 | - EXPECT_EQ("Fake google account", acc->description()) << acc->description().toStdString(); |
51 | + EXPECT_EQ("com.canonical.scopes.mcloud_mcloud_mcloud", acc->owner_id()); |
52 | + EXPECT_EQ("Fake mcloud account", acc->description()); |
53 | } |
54 | |
55 | TEST_F(RuntimeTest, roots) |
56 | |
57 | === modified file 'tests/utils/fake-online-accounts-daemon.py' |
58 | --- tests/utils/fake-online-accounts-daemon.py 2016-07-25 07:14:18 +0000 |
59 | +++ tests/utils/fake-online-accounts-daemon.py 2016-08-11 06:12:08 +0000 |
60 | @@ -156,6 +156,8 @@ |
61 | Password("user", "pass")), |
62 | Account(42, "Fake google account", "google-drive-scope", |
63 | OAuth2("fake-google-access-token", 0, [])), |
64 | + Account(99, "Fake mcloud account", "com.canonical.scopes.mcloud_mcloud_mcloud", |
65 | + OAuth2("fake-mcloud-access-token", 0, [])), |
66 | ] |
67 | server = Server(accounts) |
68 | server.run() |
PASSED: Continuous integration, rev:46 /jenkins. canonical. com/unity- api-1/job/ lp-storage- framework- ci/62/ /jenkins. canonical. com/unity- api-1/job/ build/346 /jenkins. canonical. com/unity- api-1/job/ build-0- fetch/352 /jenkins. canonical. com/unity- api-1/job/ build-1- sourcepkg/ release= vivid+overlay/ 273 /jenkins. canonical. com/unity- api-1/job/ build-1- sourcepkg/ release= xenial+ overlay/ 273 /jenkins. canonical. com/unity- api-1/job/ build-1- sourcepkg/ release= yakkety/ 273 /jenkins. canonical. com/unity- api-1/job/ build-2- binpkg/ arch=amd64, release= vivid+overlay/ 203 /jenkins. canonical. com/unity- api-1/job/ build-2- binpkg/ arch=amd64, release= vivid+overlay/ 203/artifact/ output/ *zip*/output. zip /jenkins. canonical. com/unity- api-1/job/ build-2- binpkg/ arch=amd64, release= xenial+ overlay/ 203 /jenkins. canonical. com/unity- api-1/job/ build-2- binpkg/ arch=amd64, release= xenial+ overlay/ 203/artifact/ output/ *zip*/output. zip /jenkins. canonical. com/unity- api-1/job/ build-2- binpkg/ arch=amd64, release= yakkety/ 203 /jenkins. canonical. com/unity- api-1/job/ build-2- binpkg/ arch=amd64, release= yakkety/ 203/artifact/ output/ *zip*/output. zip /jenkins. canonical. com/unity- api-1/job/ build-2- binpkg/ arch=armhf, release= vivid+overlay/ 203 /jenkins. canonical. com/unity- api-1/job/ build-2- binpkg/ arch=armhf, release= vivid+overlay/ 203/artifact/ output/ *zip*/output. zip /jenkins. canonical. com/unity- api-1/job/ build-2- binpkg/ arch=armhf, release= xenial+ overlay/ 203 /jenkins. canonical. com/unity- api-1/job/ build-2- binpkg/ arch=armhf, release= xenial+ overlay/ 203/artifact/ output/ *zip*/output. zip /jenkins. canonical. com/unity- api-1/job/ build-2- binpkg/ arch=armhf, release= yakkety/ 203 /jenkins. canonical. com/unity- api-1/job/ build-2- binpkg/ arch=armhf, release= yakkety/ 203/artifact/ output/ *zip*/output. zip /jenkins. canonical. com/unity- api-1/job/ build-2- binpkg/ arch=i386, release= vivid+overlay/ 203 /jenkins. canonical. com/unity- api-1/job/ build-2- binpkg/ arch=i386, release= vivid+overlay/ 203/artifact/ output/ *zip*/output. zip /jenkins. canonical. com/unity- api-1/job/ build-2- binpkg/ arch=i386, release= xenial+ overlay/ 203 /jenkins. canonical. com/unity- api-1/job/ build-2- binpkg/ arch=i386, release= xenial+ overlay/ 203/artifact/ output/ *zip*/output. zip /jenkins. canonical. com/unity- api-1/job/ build-2- binpkg/ arch=i386, release= yakkety/ 203 /jenkins. canonical. com/unity- api-1/job/ build-2- binpkg/ arch=i386, release= yakkety/ 203/artifact/ output/ *zip*/output. zip
https:/
Executed test runs:
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
Click here to trigger a rebuild: /jenkins. canonical. com/unity- api-1/job/ lp-storage- framework- ci/62/rebuild
https:/