Snider/Mining
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

ax(node): rename loop variable p to peer in peer_test.go
Some checks are pending
Security Scan / security (push) Waiting to run
Details
Test / test (push) Waiting to run
Details

Browse source 
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 <charon@lethean.io>
This commit is contained in:
Claude 2026-04-02 16:21:34 +01:00
parent cdf9525afd
commit e263fe7f1e
No known key found for this signature in database
GPG key ID: AF404715446AEB41
1 changed files with 4 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
pkg/node/peer_test.go
View file

@ -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)