Skip to content

Commit

Permalink
test: exclude TestPHPOverrides on Colima and TestProcessHooks on Wind…
Browse files Browse the repository at this point in the history
…ows (#5883)
  • Loading branch information
rfay committed Feb 22, 2024
1 parent 18842b6 commit d7163e7
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 2 deletions.
4 changes: 2 additions & 2 deletions pkg/ddevapp/config_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -753,8 +753,8 @@ func TestConfigOverrideDetection(t *testing.T) {

// TestPHPOverrides tests to make sure that PHP overrides work in all webservers.
func TestPHPOverrides(t *testing.T) {
if nodeps.IsAppleSilicon() {
t.Skip("Skipping on mac M1 to ignore problems with 'connection reset by peer'")
if nodeps.IsAppleSilicon() || dockerutil.IsColima() {
t.Skip("Skipping on Apple Silicon and Colima to ignore problems with 'connection reset by peer or connection refused'")
}

assert := asrt.New(t)
Expand Down
4 changes: 4 additions & 0 deletions pkg/ddevapp/hooks_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@ import (
"fmt"
"os"
"path/filepath"
"runtime"
"testing"

"github.com/ddev/ddev/pkg/ddevapp"
Expand All @@ -15,6 +16,9 @@ import (

// TestProcessHooks tests execution of commands defined in config.yaml
func TestProcessHooks(t *testing.T) {
if runtime.GOOS == "windows" {
t.Skip("Skipping on Windows, as it always hangs")
}
assert := asrt.New(t)

site := TestSites[0]
Expand Down

0 comments on commit d7163e7

Please sign in to comment.