Add upper bound check (maxPort = 65535) to reject invalid port numbers. Add comprehensive test cases for port validation.
36 lines
1.0 KiB
Go
36 lines
1.0 KiB
Go
package handlers
|
|
|
|
import "testing"
|
|
|
|
func TestParsePortValues(t *testing.T) {
|
|
tests := []struct {
|
|
name string
|
|
host string
|
|
container string
|
|
wantHost int
|
|
wantCont int
|
|
wantValid bool
|
|
}{
|
|
{"valid ports", "8080", "80", 8080, 80, true},
|
|
{"port 1", "1", "1", 1, 1, true},
|
|
{"port 65535", "65535", "65535", 65535, 65535, true},
|
|
{"host port above 65535", "99999", "80", 0, 0, false},
|
|
{"container port above 65535", "80", "99999", 0, 0, false},
|
|
{"both ports above 65535", "70000", "70000", 0, 0, false},
|
|
{"zero port", "0", "80", 0, 0, false},
|
|
{"negative port", "-1", "80", 0, 0, false},
|
|
{"non-numeric", "abc", "80", 0, 0, false},
|
|
}
|
|
|
|
for _, tt := range tests {
|
|
t.Run(tt.name, func(t *testing.T) {
|
|
host, cont, valid := parsePortValues(tt.host, tt.container)
|
|
if host != tt.wantHost || cont != tt.wantCont || valid != tt.wantValid {
|
|
t.Errorf("parsePortValues(%q, %q) = (%d, %d, %v), want (%d, %d, %v)",
|
|
tt.host, tt.container, host, cont, valid,
|
|
tt.wantHost, tt.wantCont, tt.wantValid)
|
|
}
|
|
})
|
|
}
|
|
}
|