Fix test naming conventions and add unaligned data test

- Rename test functions to follow Go conventions for unexported functions
- Add TestHash128_UnalignedInput to verify handling of unaligned data
- Address feedback from gemini-code-assist and Copilot

Signed-off-by: manmathbh <[email protected]>

Author: manmathbh

pkg/utils/hash/hash_test.go

@@ -146,7 +146,7 @@ func TestHash128_AllTailLengths(t *testing.T) {
 	}
 }
 
-func TestRotl64(t *testing.T) {
+func Test_rotl64(t *testing.T) {
 	tests := []struct {
 		name     string
 		x        uint64
@@ -202,7 +202,7 @@ func TestRotl64(t *testing.T) {
 	}
 }
 
-func TestFmix64(t *testing.T) {
+func Test_fmix64(t *testing.T) {
 	tests := []struct {
 		name     string
 		input    uint64
@@ -241,7 +241,7 @@ func TestFmix64(t *testing.T) {
 	}
 }
 
-func TestFmix64_Avalanche(t *testing.T) {
+func Test_fmix64_Avalanche(t *testing.T) {
 	// Test that changing a single bit in input changes many bits in output
 	input1 := uint64(0x0123456789abcdef)
 	input2 := uint64(0x0123456789abcdee) // flip last bit
@@ -298,4 +298,21 @@ func BenchmarkHash128_Large(b *testing.B) {
 	for i := 0; i < b.N; i++ {
 		Hash128(data, seed)
 	}
+}
+
+func TestHash128_UnalignedInput(t *testing.T) {
+    // Create a buffer and then a sub-slice that is not 8-byte aligned.
+    buf := make([]byte, 33)
+    for i := range buf {
+        buf[i] = byte(i)
+    }
+    unalignedData := buf[1:] // len=32
+
+    defer func() {
+        if r := recover(); r != nil {
+            t.Errorf("Hash128 panicked on unaligned data: %v", r)
+        }
+    }()
+
+    Hash128(unalignedData, 0)
 }