Make use of the new varadic template feature + cosmetic changes.

Change constructor to take the a package manager argument. Add a
package manager member to the operation. Remove setter function.

Change-Id: Iffb860e24089beb6496ac4ee479d547a29463728
Reviewed-by: Kai Koehne <kai.koehne@theqtcompany.com>

1 files changed, 2 insertions, 3 deletions

diff --git a/tests/auto/installer/consumeoutputoperationtest/tst_consumeoutputoperationtest.cpp b/tests/auto/installer/consumeoutputoperationtest/tst_consumeoutputoperationtest.cpp
index c0f250435..1322303ff 100644
--- a/tests/auto/installer/consumeoutputoperationtest/tst_consumeoutputoperationtest.cpp
+++ b/tests/auto/installer/consumeoutputoperationtest/tst_consumeoutputoperationtest.cpp
@@ -68,7 +68,7 @@ private slots:
 
     void testMissingArguments()
     {
-        ConsumeOutputOperation operation;
+        ConsumeOutputOperation operation(0);
 
         QVERIFY(operation.testOperation());
         QVERIFY(!operation.performOperation());
@@ -89,8 +89,7 @@ private slots:
     {
         QString testOutput = getOutputFrom(QUOTE(QMAKE_BINARY), QStringList("-query"));
 
-        ConsumeOutputOperation operation;
-        operation.setValue(QLatin1String("installer"), QVariant::fromValue(&m_core));
+        ConsumeOutputOperation operation(&m_core);
 
         operation.setArguments(QStringList() << "testConsumeOutputKey" << QUOTE(QMAKE_BINARY) << "-query");
         QVERIFY2(operation.performOperation(), qPrintable(operation.errorString()));