Merge lp:~ahayzen/ubuntu-ui-extras/fix-copies-not-set-updateFrom into lp:ubuntu-ui-extras/staging

[Andrew Hayzen]

* Set m_copies when using updateFrom and add test for updateFrom

* Set m_copies when using updateFrom and add test for updateFrom

[lp:~ahayzen/ubuntu-ui-extras/fix-copies-not-set-updateFrom]

135. By Andrew Hayzen on 2017-03-31

* Remove debug line

Unmerged revisions

=== modified file 'debian/changelog'
--- debian/changelog	2017-03-22 13:20:18 +0000
+++ debian/changelog	2017-03-31 10:54:20 +0000
@@ -1,4 +1,4 @@
-ubuntu-ui-extras (0.3+17.04.20170301.2-0ubuntu2) UNRELEASED; urgency=medium
+ubuntu-ui-extras (0.3+17.04.20170323-0ubuntu1) zesty; urgency=medium
 
   [ Andrew Hayzen, Jonas G. Drange ]
   * Adds devices as an model on Printers, and re-instates remote printer model on Printers.
@@ -63,7 +63,7 @@
   * Rename translation domain to ubuntu-ui-extras
   * Fix for printsupport being build-depends rather than depends
 
- -- Jonas G. Drange <[e-mail]>  Mon, 20 Mar 2017 21:04:05 +0100
+ -- Andrew Hayzen <[e-mail]>  Thu, 23 Mar 2017 12:54:43 +0000
 
 ubuntu-ui-extras (0.2+17.04.20170301.2-0ubuntu1) zesty; urgency=medium
 

=== modified file 'modules/Ubuntu/Components/Extras/Printers/printer/printer.cpp'
--- modules/Ubuntu/Components/Extras/Printers/printer/printer.cpp	2017-03-21 13:01:42 +0000
+++ modules/Ubuntu/Components/Extras/Printers/printer/printer.cpp	2017-03-31 10:54:20 +0000
@@ -410,6 +410,7 @@
     // Note: do not use loadAttributes otherwise can cause UI block
     m_acceptJobs = other->m_acceptJobs;
     m_backend = other->m_backend;
+    m_copies = other->m_copies;
     m_defaultColorModel = other->m_defaultColorModel;
     m_defaultPrintQuality = other->m_defaultPrintQuality;
     m_deviceUri = other->m_deviceUri;

=== modified file 'tests/unittests/Printers/tst_printer.cpp'
--- tests/unittests/Printers/tst_printer.cpp	2017-03-20 13:31:55 +0000
+++ tests/unittests/Printers/tst_printer.cpp	2017-03-31 10:54:20 +0000
@@ -36,14 +36,17 @@
     void init()
     {
         m_backend = new MockPrinterBackend(m_printerName);
-        m_instance = new Printer(m_backend);
+        m_instance = QSharedPointer<Printer>(new Printer(m_backend));
     }
     void cleanup()
     {
-        QSignalSpy destroyedSpy(m_instance, SIGNAL(destroyed(QObject*)));
-        m_instance->deleteLater();
+        QSignalSpy destroyedSpy(m_instance.data(), SIGNAL(destroyed(QObject*)));
+        m_instance.clear();
         QTRY_COMPARE(destroyedSpy.count(), 1);
-        delete m_backend;
+
+        if (m_backend) {
+            delete m_backend;
+        }
     }
     void testName()
     {
@@ -370,10 +373,92 @@
         // Ideally, all QObjects in the Printer API needs to be tested here.
         QCOMPARE(p.jobs()->thread(), &thread);
     }
+
+    void testUpdateFrom()
+    {
+        // Setup any variables for flipped values
+        bool newAcceptJobs = !m_instance->acceptJobs();
+        int newCopies = m_instance->copies() + 1;
+        QString newDeviceUri = m_instance->deviceUri() + "/test";
+        bool newShared = !m_instance->shared();
+        QString newLastMessage = m_instance->lastMessage() + "test";
+
+        // Setup color model and quality models
+        ColorModel colorA;
+        colorA.text = "ColorA";
+
+        ColorModel defaultColorModel;
+        defaultColorModel.text = "ColorB";
+        QList<ColorModel> colorModels({colorA, defaultColorModel});
+
+        PrintQuality qualityA;
+        qualityA.name = "QualityA";
+
+        PrintQuality defaultPrintQuality;
+        defaultPrintQuality.name = "QualityB";
+        QList<PrintQuality> qualities({qualityA, defaultPrintQuality});
+
+        QList<PrinterEnum::DuplexMode> duplexModes({
+            PrinterEnum::DuplexMode::DuplexNone,
+            PrinterEnum::DuplexMode::DuplexLongSide,
+            PrinterEnum::DuplexMode::DuplexShortSide,
+            PrinterEnum::DuplexMode::DuplexShortSide
+        });
+
+        auto newDefaultDuplexMode = m_instance->defaultDuplexMode() == PrinterEnum::DuplexMode::DuplexNone
+                ? PrinterEnum::DuplexMode::DuplexShortSide
+                : PrinterEnum::DuplexMode::DuplexNone;
+
+        // Create a printer that has different settings to the default
+        QString printerName = "test-printer-b";
+
+        MockPrinterBackend *backend = new MockPrinterBackend(printerName);
+        backend->printerOptions[printerName].insert("AcceptJobs", newAcceptJobs);
+        backend->printerOptions[printerName].insert("Copies", QString::number(newCopies));
+        backend->printerOptions[printerName].insert("DefaultColorModel", QVariant::fromValue(defaultColorModel));
+        backend->printerOptions[printerName].insert("DefaultPrintQuality", QVariant::fromValue(defaultPrintQuality));
+        backend->printerOptions[printerName].insert(
+            "DeviceUri", newDeviceUri);
+        backend->printerOptions[printerName].insert("Shared", newShared);
+        backend->printerOptions[printerName].insert(
+            "StateMessage", newLastMessage);
+        backend->printerOptions[printerName].insert(
+            "SupportedColorModels", QVariant::fromValue(colorModels));
+        backend->printerOptions[printerName].insert(
+            "SupportedPrintQualities", QVariant::fromValue(qualities));
+
+        backend->m_supportedDuplexModes = duplexModes;
+        backend->printerOptions[printerName].insert(
+            "Duplex", Utils::duplexModeToPpdChoice(newDefaultDuplexMode));
+
+        QSharedPointer<Printer> p = QSharedPointer<Printer>(new Printer(backend));
+
+        qDebug() << p->deviceUri();
+
+        // Update the default printer from this
+        m_instance->updateFrom(p);
+
+        // Check we have the new values
+        QCOMPARE(m_instance->acceptJobs(), newAcceptJobs);
+        QCOMPARE(m_instance->copies(), newCopies);
+        QCOMPARE(m_instance->defaultColorModel(), defaultColorModel);
+        QCOMPARE(m_instance->defaultPrintQuality(), defaultPrintQuality);
+        QCOMPARE(m_instance->deviceUri(), newDeviceUri);
+        QCOMPARE(m_instance->shared(), newShared);
+        QCOMPARE(m_instance->lastMessage(), newLastMessage);
+        QCOMPARE(m_instance->supportedColorModels(), colorModels);
+        QCOMPARE(m_instance->supportedPrintQualities(), qualities);
+
+        // We test that the duplexModes have changed to check that the backend
+        // has changed, as m_backend is private we can't check directly
+        QCOMPARE(m_instance->defaultDuplexMode(), newDefaultDuplexMode);
+        QCOMPARE(m_instance->supportedDuplexModes(), duplexModes);
+    }
+
 private:
     QString m_printerName = "my-printer";
     MockPrinterBackend *m_backend = nullptr;
-    Printer *m_instance = nullptr;
+    QSharedPointer<Printer> m_instance;
 };
 
 QTEST_GUILESS_MAIN(TestPrinter)