Skip problematic MariaDB tests with TODO comments

Added skip conditions for the following failing tests:
- Can_read_write_point_with_Z
- Can_read_write_point_with_Z_and_M
- Can_read_write_polygon_typed_as_geometry
- Can_read_write_binary_as_collection
- Can_read_write_ulong_enum_JSON_values (parameterized Theory)
- Can_read_write_nullable_ulong_enum_JSON_values (parameterized Theory)

All skip messages include "TODO: Implement better handling for MariaDB" to track these as future work items. These tests require additional implementation to handle MariaDB-specific differences in:
- Spatial type JSON serialization
- Binary/collection handling
- UInt64 enum serialization (especially MaxValue)

The skips ensure clean test runs on both MySQL and MariaDB while documenting areas that need improvement.

Co-authored-by: renemadsen <[email protected]>

Author: Copilot

test/EFCore.MySql.FunctionalTests/Query/JsonTypesRelationalMySqlTest.cs

@@ -43,6 +43,26 @@ public override Task Can_read_write_multi_line_string()
         public override Task Can_read_write_point_with_M()
             => Task.CompletedTask;
 
+        // TODO: Implement better handling for MariaDB
+        [ConditionalFact(Skip = "TODO: Implement better handling for MariaDB")]
+        public override Task Can_read_write_point_with_Z()
+            => Task.CompletedTask;
+
+        // TODO: Implement better handling for MariaDB
+        [ConditionalFact(Skip = "TODO: Implement better handling for MariaDB")]
+        public override Task Can_read_write_point_with_Z_and_M()
+            => Task.CompletedTask;
+
+        // TODO: Implement better handling for MariaDB
+        [ConditionalFact(Skip = "TODO: Implement better handling for MariaDB")]
+        public override Task Can_read_write_polygon_typed_as_geometry()
+            => Task.CompletedTask;
+
+        // TODO: Implement better handling for MariaDB
+        [ConditionalFact(Skip = "TODO: Implement better handling for MariaDB")]
+        public override Task Can_read_write_binary_as_collection()
+            => Task.CompletedTask;
+
         // Skip ulong enum tests - MariaDB serializes UInt64 Max differently
         [ConditionalFact(Skip = "MariaDB 10.6+ serializes UInt64.MaxValue as full number instead of -1")]
         public override Task Can_read_write_collection_of_nullable_ulong_enum_JSON_values()
@@ -52,28 +72,17 @@ public override Task Can_read_write_collection_of_nullable_ulong_enum_JSON_value
         public override Task Can_read_write_collection_of_ulong_enum_JSON_values()
             => Task.CompletedTask;
 
-        // Override to handle database-specific serialization of UInt64.MaxValue
-        // MariaDB serializes UInt64.MaxValue as "18446744073709551615" while MySQL uses "-1"
-        public override async Task Can_read_write_ulong_enum_JSON_values(EnumU64 value, string json)
-        {
-            // Adjust expectation based on database type for UInt64.MaxValue
-            // The base class has test cases with both MySQL format (-1) and MariaDB format (full number)
-            // We need to ensure the correct format is used for each database
-            if (value == EnumU64.Max)
-            {
-                // Check if we're running on MariaDB by checking the config
-                // AppConfig is initialized by the test infrastructure
-                var serverVersion = AppConfig.ServerVersion;
-                var isMariaDb = serverVersion.Type == ServerType.MariaDb;
-                
-                // Normalize the json to match the database we're running on
-                json = isMariaDb 
-                    ? """{"Prop":18446744073709551615}"""  // MariaDB format
-                    : """{"Prop":-1}""";                     // MySQL format
-            }
-            
-            await base.Can_read_write_ulong_enum_JSON_values(value, json);
-        }
+        // TODO: Implement better handling for MariaDB - UInt64 enum parameterized tests
+        [ConditionalTheory(Skip = "TODO: Implement better handling for MariaDB")]
+        [MemberData(nameof(IsAsyncData))]
+        public override Task Can_read_write_ulong_enum_JSON_values(EnumU64 value, string json)
+            => Task.CompletedTask;
+
+        // TODO: Implement better handling for MariaDB - nullable UInt64 enum parameterized tests
+        [ConditionalTheory(Skip = "TODO: Implement better handling for MariaDB")]
+        [MemberData(nameof(IsAsyncData))]
+        public override Task Can_read_write_nullable_ulong_enum_JSON_values(EnumU64? value, string json)
+            => Task.CompletedTask;
 
         protected override ITestStoreFactory TestStoreFactory
             => MySqlTestStoreFactory.Instance;