Update some QML tests and temporarily skip them

These tests should be skipped until the next V8 update is landed into
QtJSBackend. The expected results of these tests currently check wrong
behavior. These bugs have been already fixed in the official V8 thus we
need to update and skip them until the fix is landed into QtJSBackend.

Change-Id: I77d8ee50b45cd6599cbb5735ddef7d1461aeceab
Reviewed-by: Simon Hausmann <simon.hausmann@digia.com>

1 files changed, 4 insertions, 7 deletions

diff --git a/tests/auto/qml/qjsengine/tst_qjsengine.cpp b/tests/auto/qml/qjsengine/tst_qjsengine.cpp
index 87d2673c2f..bbde5f4dae 100644
--- a/tests/auto/qml/qjsengine/tst_qjsengine.cpp
+++ b/tests/auto/qml/qjsengine/tst_qjsengine.cpp
@@ -2226,15 +2226,12 @@ void tst_QJSEngine::jsContinueInSwitch()
 
 void tst_QJSEngine::jsShadowReadOnlyPrototypeProperty()
 {
-    // SpiderMonkey has different behavior than JSC and V8; it disallows
-    // creating a property on the instance if there's a property with the
-    // same name in the prototype, and that property is read-only. We
-    // adopted that behavior in the old (4.5) QtScript back-end, but it
-    // just seems weird -- and non-compliant. Adopt the JSC behavior instead.
+    QSKIP("Temporarily skip this test case until the next V8 update is landed into QtJSBackend.");
+
     QJSEngine eng;
     QVERIFY(eng.evaluate("o = {}; o.__proto__ = parseInt; o.length").isNumber());
-    QCOMPARE(eng.evaluate("o.length = 123; o.length").toInt(), 123);
-    QVERIFY(eng.evaluate("o.hasOwnProperty('length')").toBool());
+    QVERIFY(eng.evaluate("o.length = 123; o.length").toInt() != 123);
+    QVERIFY(!eng.evaluate("o.hasOwnProperty('length')").toBool());
 }
 
 void tst_QJSEngine::jsReservedWords_data()