Remove potentially spuriously failing tests

Author: kunitoki

tests/yup_data_model/yup_DataTreeQuery.cpp

@@ -929,64 +929,3 @@ TEST_F (DataTreeQueryTests, XPathComplexExpressions)
     auto result3 = DataTreeQuery::xpath (testTree, "//Button[not(@enabled='false')]").nodes();
     EXPECT_GT (static_cast<int> (result3.size()), 0);
 }
-
-//==============================================================================
-// Performance Tests
-
-TEST_F (DataTreeQueryTests, LargeDatasetPerformance)
-{
-    // Create large dataset
-    DataTree largeTree ("LargeRoot");
-    {
-        auto transaction = largeTree.beginTransaction ("Build large dataset");
-
-        for (int i = 0; i < 1000; ++i)
-        {
-            DataTree item ("Item");
-            {
-                auto itemTrans = item.beginTransaction ("Setup item");
-                itemTrans.setProperty ("id", i);
-                itemTrans.setProperty ("category", i % 10);
-                itemTrans.setProperty ("active", i % 2 == 0);
-                itemTrans.setProperty ("score", i * 1.5);
-            }
-            transaction.addChild (item);
-        }
-    }
-
-    // Test performance doesn't degrade severely
-    auto start = std::chrono::high_resolution_clock::now();
-
-    auto result = DataTreeQuery::from (largeTree)
-                      .descendants ("Item")
-                      .propertyWhere<int> ("category", [] (int cat)
-    {
-        return cat < 5;
-    }).propertyWhere<bool> ("active", [] (bool active)
-    {
-        return active;
-    }).take (100)
-                      .nodes();
-
-    auto end = std::chrono::high_resolution_clock::now();
-    auto duration = std::chrono::duration_cast<std::chrono::milliseconds> (end - start);
-
-    // Should complete within reasonable time (adjust threshold as needed)
-    EXPECT_LT (duration.count(), 100); // Less than 100ms
-    EXPECT_GT (static_cast<int> (result.size()), 0);
-    EXPECT_LE (static_cast<int> (result.size()), 100);
-}
-
-/*
-TEST_F(DataTreeQueryTests, OutOfBoundsAccess)
-{
-    auto result = DataTreeQuery::from(testTree).children();
-
-    // Test that accessing beyond bounds returns invalid node
-    auto node = result.getNode(-1);
-    // This should trigger an assertion in debug builds, but not crash in release
-
-    // In release builds, should handle gracefully
-    SUCCED();
-}
-*/