From 21819254ddb7d48f3483cadcecb20a63d1db8cd7 Mon Sep 17 00:00:00 2001 From: Paul Holzinger Date: Wed, 15 Jun 2022 19:50:39 +0200 Subject: golangci-lint: update to v1.46.2 Update to the latest golangci-lint version. v1.46 added new linters. I disabled nonamedreturns and exhaustruct since they enforce a certain code style and using them would require big changes to the code base. The nosprintfhostport is new and I fixed one problem in the tests. While the test itself is fine because it uses ipv4 only the linter still looks good because the sprintf use will fail for ipv6 addresses. Signed-off-by: Paul Holzinger --- test/e2e/systemd_activate_test.go | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) (limited to 'test') diff --git a/test/e2e/systemd_activate_test.go b/test/e2e/systemd_activate_test.go index c50e43b8d..240139b98 100644 --- a/test/e2e/systemd_activate_test.go +++ b/test/e2e/systemd_activate_test.go @@ -8,6 +8,7 @@ import ( "os" "os/exec" "path/filepath" + "strconv" "syscall" "time" @@ -61,9 +62,10 @@ var _ = Describe("Systemd activate", func() { host := "127.0.0.1" port, err := podmanUtils.GetRandomPort() Expect(err).ToNot(HaveOccurred()) + addr := net.JoinHostPort(host, strconv.Itoa(port)) activateSession := testUtils.StartSystemExec(activate, []string{ - fmt.Sprintf("--listen=%s:%d", host, port), + "--listen", addr, podmanTest.PodmanBinary, "--root=" + filepath.Join(tempDir, "server_root"), "system", "service", @@ -73,7 +75,7 @@ var _ = Describe("Systemd activate", func() { // Curried functions for specialized podman calls podmanRemote := func(args ...string) *testUtils.PodmanSession { - args = append([]string{"--url", fmt.Sprintf("tcp://%s:%d", host, port)}, args...) + args = append([]string{"--url", "tcp://" + addr}, args...) return testUtils.SystemExec(podmanTest.RemotePodmanBinary, args) } @@ -111,7 +113,7 @@ var _ = Describe("Systemd activate", func() { port, err := podmanUtils.GetRandomPort() Expect(err).ToNot(HaveOccurred()) - addr := fmt.Sprintf("%s:%d", host, port) + addr := net.JoinHostPort(host, strconv.Itoa(port)) // start systemd activation with datagram socket activateSession := testUtils.StartSystemExec(activate, []string{ -- cgit v1.2.3-54-g00ecf