From c1f4f8e626f3d5c2a2d6112e727ae0d15248cf19 Mon Sep 17 00:00:00 2001
From: Milian Wolff <milian.wolff@kdab.com>
Date: Fri, 10 Jan 2014 11:38:55 +0100
Subject: Properly convert JSON data to target type of function parameter.

This used to work but broke in one of the last few commits apparently.
Now we add a proper unit test to ensure it stays working.

The issue was that JSON only knows e.g. numeric types stored as double.
When we then try to call a method taking an int with the VariantArgument
that tries to convert the double to int, we failed and produced an
invalid QVariant which then converts to 0.

Now we use the appropriate API to convert the JSON data to the correct
target type before calling the method. Furthermore, it became clear
that we can greatly cleanup the VariantArgument thanks to that. It now
is reduced to just a QVariant wrapper class with an implicit cast
operator to QGenericArgument.

Change-Id: Ieaf60f548ea9584e7d760f9cd935da455787f376
Reviewed-by: Simon Hausmann <simon.hausmann@digia.com>
---
 tests/webchannel/tst_webchannel.cpp | 44 +++++++++++++++++++++++++++++++++++++
 tests/webchannel/tst_webchannel.h   | 11 ++++++++++
 2 files changed, 55 insertions(+)

(limited to 'tests/webchannel')

diff --git a/tests/webchannel/tst_webchannel.cpp b/tests/webchannel/tst_webchannel.cpp
index 7fbe5f9..c8f212c 100644
--- a/tests/webchannel/tst_webchannel.cpp
+++ b/tests/webchannel/tst_webchannel.cpp
@@ -50,6 +50,8 @@
 
 TestWebChannel::TestWebChannel(QObject *parent)
     : QObject(parent)
+    , m_lastInt(0)
+    , m_lastDouble(0)
 {
 }
 
@@ -58,6 +60,21 @@ TestWebChannel::~TestWebChannel()
 
 }
 
+void TestWebChannel::setInt(int i)
+{
+    m_lastInt = i;
+}
+
+void TestWebChannel::setDouble(double d)
+{
+    m_lastDouble = d;
+}
+
+void TestWebChannel::setVariant(const QVariant &v)
+{
+    m_lastVariant = v;
+}
+
 void TestWebChannel::testInitChannel()
 {
     QWebChannel channel;
@@ -214,6 +231,33 @@ void TestWebChannel::testInfoForObject()
     }
 }
 
+void TestWebChannel::testInvokeMethodConversion()
+{
+    QWebChannel channel;
+
+    QJsonArray args;
+    args.append(QJsonValue(1000));
+
+    {
+        int method = metaObject()->indexOfMethod("setInt(int)");
+        QVERIFY(method != -1);
+        QVERIFY(channel.d->publisher->invokeMethod(this, method, args, QJsonValue()));
+        QCOMPARE(m_lastInt, args.at(0).toInt());
+    }
+    {
+        int method = metaObject()->indexOfMethod("setDouble(double)");
+        QVERIFY(method != -1);
+        QVERIFY(channel.d->publisher->invokeMethod(this, method, args, QJsonValue()));
+        QCOMPARE(m_lastDouble, args.at(0).toDouble());
+    }
+    {
+        int method = metaObject()->indexOfMethod("setVariant(QVariant)");
+        QVERIFY(method != -1);
+        QVERIFY(channel.d->publisher->invokeMethod(this, method, args, QJsonValue()));
+        QCOMPARE(m_lastVariant, args.at(0).toVariant());
+    }
+}
+
 static QHash<QString, QObject*> createObjects(QObject *parent)
 {
     const int num = 100;
diff --git a/tests/webchannel/tst_webchannel.h b/tests/webchannel/tst_webchannel.h
index 173921b..003f921 100644
--- a/tests/webchannel/tst_webchannel.h
+++ b/tests/webchannel/tst_webchannel.h
@@ -44,6 +44,7 @@
 #define TST_WEBCHANNEL_H
 
 #include <QObject>
+#include <QVariant>
 
 class TestObject : public QObject
 {
@@ -192,15 +193,25 @@ public:
     explicit TestWebChannel(QObject *parent = 0);
     virtual ~TestWebChannel();
 
+    Q_INVOKABLE void setInt(int i);
+    Q_INVOKABLE void setDouble(double d);
+    Q_INVOKABLE void setVariant(const QVariant &v);
+
 private slots:
     void testInitChannel();
     void testRegisterObjects();
     void testInfoForObject();
+    void testInvokeMethodConversion();
 
     void benchClassInfo();
     void benchInitializeClients();
     void benchPropertyUpdates();
     void benchRegisterObjects();
+
+private:
+    int m_lastInt;
+    double m_lastDouble;
+    QVariant m_lastVariant;
 };
 
 #endif // TST_WEBCHANNEL_H
-- 
cgit v1.2.1