optimized unit tests

Author: kerryjiang

src/SuperSocket.MySQL/Packets/QueryResultPacket.cs

@@ -44,7 +44,7 @@ public class QueryResultPacket : MySQLPacket
         /// <summary>
         /// Gets the number of columns in the result set.
         /// </summary>
-        public int ColumnCount { get; set; }
+        public int ColumnCount => Columns?.Count ?? 0;
 
         /// <summary>
         /// Initializes a new instance of the QueryResultPacket class.
@@ -81,8 +81,8 @@ public static QueryResultPacket FromResultSet(IReadOnlyList<ColumnDefinitionPack
             return new QueryResultPacket
             {
                 ErrorCode = 0,
-                Columns = columns,
-                Rows = rows
+                Columns = columns ?? Array.Empty<ColumnDefinitionPacket>(),
+                Rows = rows ?? Array.Empty<IReadOnlyList<string>>()
             };
         }
 

tests/SuperSocket.MySQL.Test/MainTest.cs

@@ -191,8 +191,22 @@ public async Task ExecuteQueryAsync_WithAuthentication_ShouldNotThrow()
 
                 // Assert
                 Assert.NotNull(result);
-                // Note: Since ExecuteQueryAsync is a placeholder implementation, 
-                // we're just verifying it doesn't throw when authenticated
+                // Verify that the result has a proper structure
+                Assert.True(result.IsSuccess || result.ErrorCode != 0, 
+                    "Result should either be successful or have a proper error code");
+                
+                if (result.IsSuccess)
+                {
+                    Assert.Equal(0, result.ErrorCode);
+                    Assert.Null(result.ErrorMessage);
+                    Assert.True(result.ColumnCount >= 0, "Column count should be non-negative");
+                    Assert.True(result.RowCount >= 0, "Row count should be non-negative");
+                }
+                else
+                {
+                    Assert.NotEqual(0, result.ErrorCode);
+                    Assert.NotNull(result.ErrorMessage);
+                }
             }
             finally
             {