introduce String method on PortMapping

Signed-off-by: Nicolas De Loof <[email protected]>
docker · Jan 22, 2024 · a314a9d · a314a9d
1 parent fa09c95
commit a314a9d
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 0 deletions.
diff --git a/nat/nat.go b/nat/nat.go
@@ -150,6 +150,10 @@ type PortMapping struct {
 	Binding PortBinding
 }
 
+func (p *PortMapping) String() string {
+	return net.JoinHostPort(p.Binding.HostIP, fmt.Sprintf("%s:%s", p.Binding.HostPort, p.Port))
+}
+
 func splitParts(rawport string) (string, string, string) {
 	parts := strings.Split(rawport, ":")
 	n := len(parts)

diff --git a/nat/nat_test.go b/nat/nat_test.go
@@ -703,3 +703,16 @@ func TestParseNetworkOptsSctp(t *testing.T) {
 		}
 	}
 }
+
+func TestStringer(t *testing.T) {
+	spec := "192.168.1.100:8080:6000/udp"
+	mappings, err := ParsePortSpec(spec)
+	if err != nil {
+		t.Fatal(err)
+	}
+	for _, m := range mappings {
+		if m.String() != spec {
+			t.Fail()
+		}
+	}
+}