From af72231fe4a91226d945db9a526fb755332548eb Mon Sep 17 00:00:00 2001
From: Albert Astals Cid <albert.astals@canonical.com>
Date: Tue, 4 Apr 2017 16:26:20 +0200
Subject: Error out if TestCase.verify receives more than 2 parameters

Change-Id: I3299f602970067aaef290d4f1c7449d3ab03fb3c
Reviewed-by: Simon Hausmann <simon.hausmann@qt.io>
Reviewed-by: Mitch Curtis <mitch.curtis@qt.io>
---
 src/imports/testlib/TestCase.qml | 3 +++
 1 file changed, 3 insertions(+)

(limited to 'src/imports/testlib')

diff --git a/src/imports/testlib/TestCase.qml b/src/imports/testlib/TestCase.qml
index 3e6eb64311..8c1744a2b2 100644
--- a/src/imports/testlib/TestCase.qml
+++ b/src/imports/testlib/TestCase.qml
@@ -444,6 +444,9 @@ Item {
         or \c{QVERIFY2(condition, message)} in C++.
     */
     function verify(cond, msg) {
+        if (arguments.length > 2)
+            qtest_fail("More than two arguments given to verify(). Did you mean tryVerify() or tryCompare()?", 1)
+
         if (msg === undefined)
             msg = "";
         if (!qtest_results.verify(cond, msg, util.callerFile(), util.callerLine()))
-- 
cgit v1.2.3