Skip op-numeric-unary-minus-1 testcase because it crashes.

It seems to require a larger refactoring in order to solve this, so
we skip this for now in order for us to be able to run the remaining
tests in CI.

Change-Id: I2ae4860a92cdb7fb88ca89c4dd1f047bacbaaf9e
Reviewed-by: Simon Hausmann <simon.hausmann@theqtcompany.com>

2 files changed, 10 insertions, 1 deletions

diff --git a/src/xmlpatterns/expr/qunaryexpression.cpp b/src/xmlpatterns/expr/qunaryexpression.cpp
index 1c4524e..f99096b 100644
--- a/src/xmlpatterns/expr/qunaryexpression.cpp
+++ b/src/xmlpatterns/expr/qunaryexpression.cpp
@@ -59,7 +59,10 @@ Item UnaryExpression::evaluateSingleton(const DynamicContext::Ptr &context) cons
     {
         const Item item(m_operand2->evaluateSingleton(context));
 
-        if(item)
+        if (item)
+            // If the item is an untypedAtomic (or atomicString), the cast to Numeric will
+            // return a garbage object and will most likely cause a crash.
+            // There is simply no way to static cast from an instance of AtomicString to a Number.
             return item.as<Numeric>()->toNegated();
         else
             return Item();
diff --git a/tests/auto/xmlpatternssdk/TestCase.cpp b/tests/auto/xmlpatternssdk/TestCase.cpp
index a81cba9..bf8616c 100644
--- a/tests/auto/xmlpatternssdk/TestCase.cpp
+++ b/tests/auto/xmlpatternssdk/TestCase.cpp
@@ -77,6 +77,12 @@ TestResult::List TestCase::execute(const ExecutionStage stage,
             result.append(createTestResult(TestResult::Fail, QLatin1String("Skipped this test, we crash on it.")));
             return result;
     }
+    else if (name() == QLatin1String("op-numeric-unary-minus-1"))
+    {
+            TestResult::List result;
+            result.append(createTestResult(TestResult::Fail, QLatin1String("Skipped this test, we crash on it.")));
+            return result;
+    }
 
     qDebug() << "Running test case: " << name();