fix: 根据代码审查反馈改进6处测试质量

Co-authored-by: nnhy <506367+nnhy@users.noreply.github.com>

Author: Copilot

XUnitTestRocketMQ/HelperTests.cs XUnitTestRocketMQ/MessageExtendedTests.csXUnitTestRocketMQ/HelperTests.cs renamed to XUnitTestRocketMQ/MessageExtendedTests.cs

@@ -210,6 +210,7 @@ public void GetProperties_EmptyProperties()
         var props = msg.GetProperties();
 
         Assert.NotNull(props);
+        Assert.Equal(String.Empty, props);
     }
     #endregion
 

XUnitTestRocketMQ/MessageQueueTests.cs

@@ -84,15 +84,16 @@ public void GetHashCode_SameProperties_SameHash()
     }
 
     [Fact]
-    [DisplayName("GetHashCode_不同属性通常不同哈希")]
-    public void GetHashCode_DifferentProperties_DifferentHash()
+    [DisplayName("GetHashCode_同一实例多次调用相同哈希")]
+    public void GetHashCode_SameInstance_MultipleCalls_SameHash()
     {
-        var q1 = new MessageQueue { Topic = "test1", BrokerName = "broker-a", QueueId = 0 };
-        var q2 = new MessageQueue { Topic = "test2", BrokerName = "broker-a", QueueId = 0 };
+        var q = new MessageQueue { Topic = "test1", BrokerName = "broker-a", QueueId = 0 };
+
+        // 同一实例在一次执行过程中多次调用 GetHashCode，结果应保持一致
+        var h1 = q.GetHashCode();
+        var h2 = q.GetHashCode();
 
-        // 不同对象的哈希值不保证不同，但通常不同
-        // 这里只验证相同属性哈希一致
-        Assert.Equal(q1.GetHashCode(), q1.GetHashCode());
+        Assert.Equal(h1, h2);
     }
 
     [Fact]
@@ -105,11 +106,11 @@ public void GetHashCode_WorksAsDictionaryKey()
         var dict = new Dictionary<MessageQueue, String>();
         dict[q1] = "value";
 
-        // q2 与 q1 相等，应能查到
-        // 注意：Dictionary 使用 GetHashCode + Equals
-        // MessageQueue.Equals 返回 true 但默认 Dictionary 用引用比较
-        // 除非重写了 Equals 和 GetHashCode
-        Assert.True(dict.ContainsKey(q1));
+        // q2 与 q1 属性相同且相等，应能作为键查到
+        // 注意：Dictionary 使用 GetHashCode + Equals 处理键的相等性
+        // 前提是 MessageQueue 正确重写了 Equals 和 GetHashCode
+        Assert.True(dict.ContainsKey(q2));
+        Assert.Equal("value", dict[q2]);
     }
     #endregion
 

XUnitTestRocketMQ/RequestHeaderTests.cs

@@ -164,8 +164,10 @@ public void EndTransactionRequestHeader_BooleanSerialization()
 
         var dic = header.GetProperties();
 
-        // 值是原始 Boolean 对象（非字符串）
+        // 值是原始 Boolean 对象（非字符串），且为 true
         Assert.True(dic.ContainsKey("fromTransactionCheck"));
+        var fromTransactionCheck = Assert.IsType<Boolean>(dic["fromTransactionCheck"]);
+        Assert.True(fromTransactionCheck);
     }
     #endregion
 }

XUnitTestRocketMQ/WeightRoundRobinTests.cs

@@ -35,16 +35,30 @@ public void Set_ValidWeights_SetsReadyTrue()
     [DisplayName("Set_相同权重不重复设置")]
     public void Set_SameWeights_NoReset()
     {
-        var lb = new WeightRoundRobin();
-        lb.Set([1, 2, 3]);
+        var lb1 = new WeightRoundRobin();
+        var lb2 = new WeightRoundRobin();
 
-        // 先选一次改变状态
-        lb.Get();
+        // 初次设置相同权重
+        lb1.Set([1, 2, 3]);
+        lb2.Set([1, 2, 3]);
 
-        // 再次设置相同权重，不应重置状态
-        lb.Set([1, 2, 3]);
+        // 先各选一次改变状态
+        lb1.Get();
+        lb2.Get();
 
-        Assert.True(lb.Ready);
+        // 对 lb1 再次设置相同权重，不应重置状态
+        lb1.Set([1, 2, 3]);
+
+        // Ready 仍应为 true
+        Assert.True(lb1.Ready);
+
+        // 后续多次 Get 的返回序列应与未再次 Set 的 lb2 完全一致
+        for (var i = 0; i < 10; i++)
+        {
+            var expected = lb2.Get();
+            var actual = lb1.Get();
+            Assert.Equal(expected, actual);
+        }
     }
 
     [Fact]