symfony-cli
diff --git a/‎.golangci.yml
Copy file name to clipboardExpand all lines: .golangci.yml
+14Lines changed: 14 additions & 0 deletions b/‎.golangci.yml
Copy file name to clipboardExpand all lines: .golangci.yml
+14Lines changed: 14 additions & 0 deletions
diff --git a/‎local/php/executor_test.go
Copy file name to clipboardExpand all lines: local/php/executor_test.go
+17-5Lines changed: 17 additions & 5 deletions b/‎local/php/executor_test.go
Copy file name to clipboardExpand all lines: local/php/executor_test.go
+17-5Lines changed: 17 additions & 5 deletions
diff --git a/‎local/proxy/proxy_test.go
Copy file name to clipboardExpand all lines: local/proxy/proxy_test.go
+4-3Lines changed: 4 additions & 3 deletions b/‎local/proxy/proxy_test.go
Copy file name to clipboardExpand all lines: local/proxy/proxy_test.go
+4-3Lines changed: 4 additions & 3 deletions
@@ -27,6 +27,20 @@ linters:
           - github.com/symfony-cli/terminal.Eprintln
           - github.com/symfony-cli/terminal.Eprintfln
           - github.com/symfony-cli/terminal.Eprint
+          - github.com/symfony-cli/terminal.SetLogLevel
           - fmt.Fprintln
           - fmt.Fprintf
           - fmt.Fprint
+  exclusions:
+    presets:
+      - std-error-handling
+      - common-false-positives
+    rules:
+      - path: _test\.go
+        linters:
+          - errcheck
+        text: "(?i)Error return value of .(os\\.(Chdir|Rename)|io\\.Copy). is not checked"
+      - path: _test\.go
+        linters:
+          - errcheck
+        text: "(?i)Error return value of .flags\\.Set. is not checked"
@@ -59,8 +59,12 @@ func testStdoutCapture(c *C, dst io.Writer) func() {
 
 	return func() {
 		// Close the writer end of the pipe
-		w.Sync()
-		w.Close()
+		if err := w.Sync(); err != nil {
+			c.Fatalf("err: %s", err)
+		}
+		if err := w.Close(); err != nil {
+			c.Fatalf("err: %s", err)
+		}
 
 		// Reset stdout
 		os.Stdout = old
@@ -213,7 +217,11 @@ func (s *ExecutorSuite) TestEnvInjection(c *C) {
 	os.Chdir(filepath.Join(home, "project"))
 
 	os.Rename("git", ".git")
-	defer os.Rename(".git", "git")
+	defer func() {
+		// handling error is not really worth it here: we could not really recover it anyway and the original directory
+		// is commited
+		_ = os.Rename(".git", "git")
+	}()
 	defer cleanupExecutorTempFiles()
 
 	var output bytes.Buffer
@@ -232,8 +240,12 @@ func (s *ExecutorSuite) TestEnvInjection(c *C) {
 	projectFile := filepath.Join(".platform", "local", "project.yaml")
 	contents, err := os.ReadFile(projectFile)
 	c.Assert(err, IsNil)
-	defer os.WriteFile(projectFile, contents, 0644)
-	os.WriteFile(projectFile, bytes.Replace(contents, []byte("bew7pfa7t2ut2"), []byte("aew7pfa7t2ut2"), 1), 0644)
+	defer func() {
+		// handling error is not really worth it here: we could not really recover it and anyway the original file
+		// content is commited
+		_ = os.WriteFile(projectFile, contents, 0644)
+	}()
+	c.Assert(os.WriteFile(projectFile, bytes.Replace(contents, []byte("bew7pfa7t2ut2"), []byte("aew7pfa7t2ut2"), 1), 0644), IsNil)
 
 	output.Reset()
 	outCloser = testStdoutCapture(c, &output)
 
@@ -59,7 +59,7 @@ func (s *ProxySuite) TestProxy(c *C) {
 		TLD:  "wip",
 		path: "testdata/.symfony5/proxy.json",
 	}, ca, log.New(zerolog.New(os.Stderr), "", 0), true)
-	os.MkdirAll("testdata/.symfony5", 0755)
+	c.Assert(os.MkdirAll("testdata/.symfony5", 0755), IsNil)
 	err = p.Save()
 	c.Assert(err, IsNil)
 
@@ -172,15 +172,16 @@ func (s *ProxySuite) TestProxy(c *C) {
 	{
 		backend := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
 			w.WriteHeader(200)
-			w.Write([]byte(`http://symfony-no-tls.wip`))
+			_, err := w.Write([]byte(`http://symfony-no-tls.wip`))
+			c.Assert(err, IsNil)
 		}))
 		defer backend.Close()
 		backendURL, err := url.Parse(backend.URL)
 		c.Assert(err, IsNil)
 
 		p := pid.New("symfony_com_no_tls", nil)
 		port, _ := strconv.Atoi(backendURL.Port())
-		p.Write(os.Getpid(), port, "http")
+		c.Assert(p.Write(os.Getpid(), port, "http"), IsNil)
 
 		req, _ := http.NewRequest("GET", "http://symfony-no-tls.wip/", nil)
 		req.Close = true