Remove redundant nil check in sys_test.go

[Copilot]

Staticcheck S1009 flagged a redundant nil check before calling len() on a slice. In Go, len() returns 0 for nil slices, making the explicit nil check unnecessary.

Changes

• Removed redundant tt.serverIDs != nil check in TestNewSysServer

// Before
if tt.serverIDs != nil && len(tt.serverIDs) > 0 {
    assert.Equal(t, tt.serverIDs, server.serverIDs, "Server IDs should match")
}

// After
if len(tt.serverIDs) > 0 {
    assert.Equal(t, tt.serverIDs, server.serverIDs, "Server IDs should match")
}

The test behavior remains identical - len() on a nil slice evaluates to 0, so the condition is false for both nil and empty slices.

---

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.