Add comments and better variable names

peer.go

@@ -346,6 +346,7 @@ func (p *Peer) GetConnection(ctx context.Context) (*Connection, error) {
 		return activeConn, nil
 	}
 
+	// Lock here to restrict new connection creation attempts to one goroutine
 	p.newConnLock.Lock()
 	defer p.newConnLock.Unlock()
 

peer_test.go

@@ -201,11 +201,11 @@ func TestPeerGetConnectionWithNoActiveConnections(t *testing.T) {
 			wg          sync.WaitGroup
 			lock        sync.Mutex
 			conn        *Connection
-			numRequests = 10
+			concurrency = 10
 			p           = client.Peers().Add(hostPort)
 		)
 
-		for i := 0; i < numRequests; i++ {
+		for i := 0; i < concurrency; i++ {
 			wg.Add(1)
 			go func() {
 				defer wg.Done()