From 5296755b9ef6c83ba89669874ac63577189a3513 Mon Sep 17 00:00:00 2001
From: lightclient <lightclient@protonmail.com>
Date: Thu, 15 Aug 2024 09:31:40 -0600
Subject: [PATCH] beacon/light/sync: cleanup TestRangeLock

---
 beacon/light/sync/update_sync_test.go | 18 ++++++------------
 1 file changed, 6 insertions(+), 12 deletions(-)

diff --git a/beacon/light/sync/update_sync_test.go b/beacon/light/sync/update_sync_test.go
index 2f494d6aed..88d573165e 100644
--- a/beacon/light/sync/update_sync_test.go
+++ b/beacon/light/sync/update_sync_test.go
@@ -201,34 +201,28 @@ func TestUpdateSyncDifferentHeads(t *testing.T) {
 	chain.ExpNextSyncPeriod(t, 17)
 }
 
-func Test_lock(t *testing.T) {
-	r := rangeLock{}
-	// locking from 0 to 99
+func TestRangeLock(t *testing.T) {
+	r := make(rangeLock)
+
+	// Lock from 0 to 99.
 	r.lock(0, 100, 1)
-	// check that elements have values > 0
 	for i := 0; i < 100; i++ {
 		if v, ok := r[uint64(i)]; v <= 0 || !ok {
 			t.Fatalf("integer space: %d not locked", i)
 		}
 	}
 
-	// unlocking from 0 to 99
+	// Unlock from 0 to 99.
 	r.lock(0, 100, -1)
-	// check that elements have values <= 0
 	for i := 0; i < 100; i++ {
 		if v, ok := r[uint64(i)]; v > 0 || ok {
 			t.Fatalf("integer space: %d is locked", i)
 		}
 	}
-}
 
-func Test_firstUnlocked(t *testing.T) {
-	r := rangeLock{}
-	// locking from 0 to 99
+	// Lock from 0 to 99 then unlock from 10 to 59.
 	r.lock(0, 100, 1)
-	// unlocking from 10 to 59
 	r.lock(10, 50, -1)
-	// check first element unlocked
 	first, count := r.firstUnlocked(0, 100)
 	if first != 10 || count != 50 {
 		t.Fatalf("unexpected first: %d or count: %d", first, count)