fixed doc string indentation, added clarification on test assertions

Author: markogalevski

godot-core/src/builtin/collections/array.rs

@@ -285,11 +285,12 @@ impl<T: ArrayElement> Array<T> {
         self.as_inner().is_empty()
     }
 
-    crate::declare_hash_u32_method! {/// Returns a 32-bit integer hash value representing the array and its contents.
-    ///
-    /// Note: Arrays with equal content will always produce identical hash values. However, the
-    /// reverse is not true. Returning identical hash values does not imply the arrays are equal,
-    /// because different arrays can have identical hash values due to hash collisions.
+    crate::declare_hash_u32_method! {
+        /// Returns a 32-bit integer hash value representing the array and its contents.
+        ///
+        /// Note: Arrays with equal content will always produce identical hash values. However, the
+        /// reverse is not true. Returning identical hash values does not imply the arrays are equal,
+        /// because different arrays can have identical hash values due to hash collisions.
     }
 
     #[deprecated = "renamed to hash_u32"]

godot-core/src/builtin/mod.rs

@@ -116,7 +116,6 @@ pub mod inner {
     pub use crate::gen::builtin_classes::*;
 }
 
-//  common hashing macro implementer
 #[macro_export]
 macro_rules! declare_hash_u32_method {
     ($ ($docs:tt)+ ) => {

itest/rust/src/builtin_tests/containers/dictionary_test.rs

@@ -132,6 +132,7 @@ fn dictionary_hash() {
         "equal dictionaries have same hash"
     );
 
+    // Not guaranteed, but very likely
     assert_ne!(
         a.hash_u32(),
         c.hash_u32(),

itest/rust/src/builtin_tests/containers/variant_test.rs

@@ -726,7 +726,7 @@ fn variant_hash() {
     assert_eq!(v_first, v_second);
     assert_eq!(v_first.hash_u32(), v_second.hash_u32());
     assert_ne!(v_first, v_third);
-    assert_ne!(v_first.hash_u32(), v_third.hash_u32());
+    assert_ne!(v_first.hash_u32(), v_third.hash_u32()); //not guaranteed but very likely
 
     assert_eq!(Variant::nil().hash_u32(), 0);
 

itest/rust/src/builtin_tests/string/gstring_test.rs

@@ -39,7 +39,7 @@ fn string_equality() {
     assert_eq!(string, second);
     assert_eq!(string.hash_u32(), second.hash_u32());
     assert_ne!(string, different);
-    assert_ne!(string.hash_u32(), different.hash_u32());
+    assert_ne!(string.hash_u32(), different.hash_u32()); //not guaranteed, but very likely
 }
 
 #[itest]

itest/rust/src/builtin_tests/string/node_path_test.rs

@@ -38,7 +38,7 @@ fn node_path_equality() {
     assert_eq!(string, second);
     assert_eq!(string.hash_u32(), second.hash_u32());
     assert_ne!(string, different);
-    assert_ne!(string.hash_u32(), different.hash_u32());
+    assert_ne!(string.hash_u32(), different.hash_u32()); //not guaranteed, but very likely
 }
 
 #[itest]

itest/rust/src/builtin_tests/string/string_name_test.rs

@@ -48,7 +48,7 @@ fn string_name_equality() {
     assert_eq!(string, second);
     assert_eq!(string.hash_u32(), second.hash_u32());
     assert_ne!(string, different);
-    assert_ne!(string.hash_u32(), different.hash_u32());
+    assert_ne!(string.hash_u32(), different.hash_u32()); //not guaranteed, but very likely
 }
 
 #[itest]