From e263fe7f1ec933833f8cdfab332d99eff75e264c Mon Sep 17 00:00:00 2001 From: Claude Date: Thu, 2 Apr 2026 16:21:34 +0100 Subject: [PATCH] ax(node): rename loop variable p to peer in peer_test.go MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit AX Principle 1 — Predictable names over short names. The single-letter `p` in `for _, p := range peers` is not listed in the allowed exceptions (only `i`, `_`, `t`, `c` are permitted). Rename to `peer` for semantic clarity. Co-Authored-By: Charon --- pkg/node/peer_test.go | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/pkg/node/peer_test.go b/pkg/node/peer_test.go index 2bbd9c1..1320d99 100644 --- a/pkg/node/peer_test.go +++ b/pkg/node/peer_test.go @@ -156,8 +156,8 @@ func TestPeerRegistry_ListPeers_Good(t *testing.T) { {ID: "list-3", Name: "Peer 3", Address: "3.3.3.3:9091", Role: RoleController}, } - for _, p := range peers { - registry.AddPeer(p) + for _, peer := range peers { + registry.AddPeer(peer) } listed := registry.ListPeers() @@ -331,8 +331,8 @@ func TestPeerRegistry_GetConnectedPeers_Good(t *testing.T) { {ID: "conn-3", Name: "Peer 3"}, } - for _, p := range peers { - registry.AddPeer(p) + for _, peer := range peers { + registry.AddPeer(peer) } registry.SetConnected("conn-1", true)