Merge lp:~jonas-drange/ubuntu-settings-components/printer-state into lp:~phablet-team/ubuntu-settings-components/printer-components

Proposed by Jonas G. Drange
Status: Merged
Approved by: Andrew Hayzen
Approved revision: 214
Merged at revision: 215
Proposed branch: lp:~jonas-drange/ubuntu-settings-components/printer-state
Merge into: lp:~phablet-team/ubuntu-settings-components/printer-components
Diff against target: 79 lines (+20/-9)
4 files modified
plugins/Ubuntu/Settings/Printers/backend/backend_cups.cpp (+10/-1)
plugins/Ubuntu/Settings/Printers/printer/printer.cpp (+1/-1)
tests/unittests/Printers/mockbackend.h (+4/-6)
tests/unittests/Printers/tst_printer.cpp (+5/-1)
To merge this branch: bzr merge lp:~jonas-drange/ubuntu-settings-components/printer-state
Reviewer Review Type Date Requested Status
Andrew Hayzen (community) Approve
Review via email: mp+315406@code.launchpad.net

Commit message

* adds read of printer state

To post a comment you must log in.
Revision history for this message
Andrew Hayzen (ahayzen) wrote :

LGTM :-)

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'plugins/Ubuntu/Settings/Printers/backend/backend_cups.cpp'
2--- plugins/Ubuntu/Settings/Printers/backend/backend_cups.cpp 2017-01-22 19:31:39 +0000
3+++ plugins/Ubuntu/Settings/Printers/backend/backend_cups.cpp 2017-01-23 20:10:22 +0000
4@@ -224,7 +224,16 @@
5
6 PrinterEnum::State PrinterCupsBackend::state() const
7 {
8-
9+ switch (m_info.state()) {
10+ case QPrinter::Idle:
11+ return PrinterEnum::State::IdleState;
12+ case QPrinter::Active:
13+ return PrinterEnum::State::ActiveState;
14+ case QPrinter::Aborted:
15+ return PrinterEnum::State::AbortedState;
16+ case QPrinter::Error:
17+ return PrinterEnum::State::ErrorState;
18+ }
19 }
20
21 QList<QPageSize> PrinterCupsBackend::supportedPageSizes() const
22
23=== modified file 'plugins/Ubuntu/Settings/Printers/printer/printer.cpp'
24--- plugins/Ubuntu/Settings/Printers/printer/printer.cpp 2017-01-22 19:31:39 +0000
25+++ plugins/Ubuntu/Settings/Printers/printer/printer.cpp 2017-01-23 20:10:22 +0000
26@@ -153,7 +153,7 @@
27
28 PrinterEnum::State Printer::state() const
29 {
30-
31+ return m_backend->state();
32 }
33
34 QString Printer::lastStateMessage() const
35
36=== modified file 'tests/unittests/Printers/mockbackend.h'
37--- tests/unittests/Printers/mockbackend.h 2017-01-22 14:21:11 +0000
38+++ tests/unittests/Printers/mockbackend.h 2017-01-23 20:10:22 +0000
39@@ -335,12 +335,6 @@
40 Q_EMIT printerStateChanged(name);
41 }
42
43-public Q_SLOT:
44- virtual void refresh() override
45- {
46-
47- }
48-
49 QString returnValue = QString::null;
50
51 // Map from printer to key/val.
52@@ -370,6 +364,10 @@
53 QList<Printer*> m_availablePrinters;
54 PrinterBackend::BackendType m_backendType;
55
56+public Q_SLOT:
57+ virtual void refresh() override
58+ {
59+ }
60 };
61
62 #endif // USC_PRINTERS_MOCK_BACKEND_H
63
64=== modified file 'tests/unittests/Printers/tst_printer.cpp'
65--- tests/unittests/Printers/tst_printer.cpp 2017-01-22 14:21:11 +0000
66+++ tests/unittests/Printers/tst_printer.cpp 2017-01-23 20:10:22 +0000
67@@ -220,7 +220,11 @@
68 QCOMPARE(p.defaultPageSize(), QPageSize(QPageSize::A4));
69 QCOMPARE(p.defaultDuplexMode(), PrinterEnum::DuplexMode::DuplexNone);
70 QCOMPARE(p.isPdf(), true);
71-
72+ }
73+ void testState()
74+ {
75+ getBackend()->m_state = PrinterEnum::State::AbortedState;
76+ QCOMPARE(m_instance->state(), getBackend()->m_state);
77 }
78 private:
79 QString m_printerName = "my-printer";

Subscribers

People subscribed via source and target branches