Merge lp:~saviq/unity8/mock-customizations into lp:unity8

Proposed by Michał Sawicz
Status: Merged
Approved by: Michal Hruby
Approved revision: 991
Merged at revision: 989
Proposed branch: lp:~saviq/unity8/mock-customizations
Merge into: lp:unity8
Diff against target: 54 lines (+9/-2)
4 files modified
debian/control (+1/-1)
tests/mocks/Unity/fake_categories.cpp (+1/-1)
tests/mocks/Unity/fake_scope.cpp (+5/-0)
tests/mocks/Unity/fake_scope.h (+2/-0)
To merge this branch: bzr merge lp:~saviq/unity8/mock-customizations
Reviewer Review Type Date Requested Status
PS Jenkins bot (community) continuous-integration Needs Fixing
Michal Hruby (community) Approve
Review via email: mp+224757@code.launchpad.net

Commit message

Adapt scope mock to new api and quiet unused variable warnings.

Description of the change

 * Are there any related MPs required for this MP to build/function as expected? Please list.
No.

 * Did you perform an exploratory manual test run of your code change and any related functionality?
Yes. Builds!

 * Did you make sure that your branch does not contain spurious tags?
Yes.

 * If you changed the packaging (debian), did you subscribe the ubuntu-unity team to this MP?
N/A

 * If you changed the UI, has there been a design review?
N/A

To post a comment you must log in.
Revision history for this message
Michal Hruby (mhr3) wrote :

+1, actually compiles now ;)

review: Approve
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'debian/control'
2--- debian/control 2014-06-19 11:17:28 +0000
3+++ debian/control 2014-06-27 08:28:30 +0000
4@@ -17,7 +17,7 @@
5 libpulse-dev,
6 libqmenumodel-dev (>= 0.2.7),
7 libqt5xmlpatterns5-dev,
8- libunity-api-dev (>= 7.82),
9+ libunity-api-dev (>= 7.83),
10 libunity-mir-dev,
11 libusermetricsoutput1-dev,
12 libxcb1-dev,
13
14=== modified file 'tests/mocks/Unity/fake_categories.cpp'
15--- tests/mocks/Unity/fake_categories.cpp 2014-05-27 07:46:14 +0000
16+++ tests/mocks/Unity/fake_categories.cpp 2014-06-27 08:28:30 +0000
17@@ -50,7 +50,7 @@
18 }
19 }
20
21-bool Categories::overrideCategoryJson(QString const& categoryId, QString const& json)
22+bool Categories::overrideCategoryJson(QString const& /* categoryId */, QString const& /* json */)
23 {
24 qFatal("Using un-implemented Categories::overrideCategoryJson");
25 }
26
27=== modified file 'tests/mocks/Unity/fake_scope.cpp'
28--- tests/mocks/Unity/fake_scope.cpp 2014-05-29 10:25:21 +0000
29+++ tests/mocks/Unity/fake_scope.cpp 2014-06-27 08:28:30 +0000
30@@ -155,6 +155,11 @@
31 return true;
32 }
33
34+QVariantMap Scope::customizations() const
35+{
36+ return QVariantMap();
37+}
38+
39 unity::shell::scopes::DepartmentInterface* Scope::getDepartment(const QString& id)
40 {
41 if (id.isEmpty())
42
43=== modified file 'tests/mocks/Unity/fake_scope.h'
44--- tests/mocks/Unity/fake_scope.h 2014-05-29 10:25:21 +0000
45+++ tests/mocks/Unity/fake_scope.h 2014-06-27 08:28:30 +0000
46@@ -64,6 +64,8 @@
47 Q_INVOKABLE unity::shell::scopes::DepartmentInterface* getDepartment(const QString& id) override;
48 Q_INVOKABLE void loadDepartment(const QString& id) override;
49
50+ QVariantMap customizations() const override;
51+
52 protected:
53
54 QString m_id;

Subscribers

People subscribed via source and target branches