@automattic/vip 4.1.5 → 4.1.6

package/internal/devenv/compose/view_test.go

@@ -1,120 +0,0 @@
-package compose
-
-import (
-	"encoding/json"
-	"testing"
-
-	"github.com/Automattic/vip/internal/devenv/instancedata"
-)
-
-func TestNewViewInterpretsMultisiteAndDefaults(t *testing.T) {
-	data := &instancedata.InstanceData{
-		SiteSlug:  "example",
-		WPTitle:   "Example",
-		Multisite: json.RawMessage("false"),
-		WordPress: instancedata.WordPressConfig{Mode: "image", Tag: "trunk"},
-		MuPlugins: instancedata.ComponentConfig{Mode: "image"},
-		AppCode:   instancedata.ComponentConfig{Mode: "local", Dir: "/srv/example"},
-		PHP:       "ghcr.io/automattic/vip-container-images/php-fpm:8.2",
-	}
-	v := NewView(data, Options{})
-
-	if v.SiteSlug != "example" {
-		t.Fatalf("SiteSlug = %q", v.SiteSlug)
-	}
-	if v.Domain != DefaultDomain {
-		t.Fatalf("default Domain = %q, want DefaultDomain %q", v.Domain, DefaultDomain)
-	}
-	if v.MultisiteEnabled {
-		t.Fatalf("multisite should be disabled for false")
-	}
-	if v.AdminPassword != "password" {
-		t.Fatalf("default AdminPassword = %q, want password", v.AdminPassword)
-	}
-	if !v.AppCodeLocal || v.AppCodeDir != "/srv/example" {
-		t.Fatalf("appCode local/dir wrong: %+v", v)
-	}
-	if v.MuPluginsLocal {
-		t.Fatalf("muPlugins should be image mode")
-	}
-	if v.DatabaseImage != "mysql:8.4" {
-		t.Fatalf("default db image = %q, want mysql:8.4", v.DatabaseImage)
-	}
-}
-
-func TestNewViewSubdomainMultisite(t *testing.T) {
-	data := &instancedata.InstanceData{
-		SiteSlug:  "ms",
-		Multisite: json.RawMessage(`"subdomain"`),
-		WordPress: instancedata.WordPressConfig{Mode: "image", Tag: "trunk"},
-		MariaDB:   "10.11",
-	}
-	v := NewView(data, Options{})
-	if !v.MultisiteEnabled || !v.MultisiteSubdomain {
-		t.Fatalf("expected subdomain multisite enabled: %+v", v)
-	}
-	if v.DatabaseImage != "mariadb:10.11" {
-		t.Fatalf("mariadb image = %q", v.DatabaseImage)
-	}
-}
-
-// TestNewViewBoolTrueMultisite locks the parity-critical branch: a bool `true`
-// multisite must enable subdomain routing (EJS `multisite === true` => --subdomain).
-func TestNewViewBoolTrueMultisite(t *testing.T) {
-	data := &instancedata.InstanceData{
-		SiteSlug:  "ms2",
-		Multisite: json.RawMessage("true"),
-		WordPress: instancedata.WordPressConfig{Mode: "image", Tag: "trunk"},
-	}
-	v := NewView(data, Options{})
-	if !v.MultisiteEnabled || !v.MultisiteSubdomain {
-		t.Fatalf("bool true multisite should be enabled+subdomain: %+v", v)
-	}
-}
-
-func TestNewViewCopiesEnvVars(t *testing.T) {
-	d := &instancedata.InstanceData{SiteSlug: "e", Multisite: json.RawMessage("false"), EnvVars: map[string]string{"A": "1"}}
-	v := NewView(d, Options{})
-	if v.EnvVars["A"] != "1" {
-		t.Fatalf("NewView did not copy EnvVars: %+v", v.EnvVars)
-	}
-}
-
-// TestNewViewSubdirectoryMultisite: a non-subdomain string enables multisite
-// but NOT subdomain routing.
-func TestNewViewSubdirectoryMultisite(t *testing.T) {
-	data := &instancedata.InstanceData{
-		SiteSlug:  "ms3",
-		Multisite: json.RawMessage(`"subdirectory"`),
-		WordPress: instancedata.WordPressConfig{Mode: "image", Tag: "trunk"},
-	}
-	v := NewView(data, Options{})
-	if !v.MultisiteEnabled {
-		t.Fatalf("subdirectory multisite should be enabled: %+v", v)
-	}
-	if v.MultisiteSubdomain {
-		t.Fatalf("subdirectory multisite must NOT be subdomain: %+v", v)
-	}
-}
-
-func TestDefaultDomainIsVipdevSite(t *testing.T) {
-	if DefaultDomain != "vipdev.site" {
-		t.Fatalf("DefaultDomain = %q, want vipdev.site", DefaultDomain)
-	}
-}
-
-func TestNewViewResolvesPHPImage(t *testing.T) {
-	base := func(php string) *instancedata.InstanceData {
-		return &instancedata.InstanceData{SiteSlug: "e", Multisite: json.RawMessage("false"), PHP: php}
-	}
-	cases := []struct{ php, want string }{
-		{"", "ghcr.io/automattic/vip-container-images/php-fpm:8.4"},                                                    // empty -> recommended default
-		{"8.4", "ghcr.io/automattic/vip-container-images/php-fpm:8.4"},                                                 // bare version -> mapped image
-		{"ghcr.io/automattic/vip-container-images/php-fpm:8.3", "ghcr.io/automattic/vip-container-images/php-fpm:8.3"}, // explicit image -> as-is
-	}
-	for _, c := range cases {
-		if got := NewView(base(c.php), Options{}).PHPImage; got != c.want {
-			t.Errorf("phpImage(%q) => %q, want %q", c.php, got, c.want)
-		}
-	}
-}

package/internal/devenv/configfile.go

@@ -1,300 +0,0 @@
-package devenv
-
-import (
-	"fmt"
-	"os"
-	"path/filepath"
-	"regexp"
-	"strings"
-
-	"gopkg.in/yaml.v3"
-)
-
-// Ports src/lib/dev-environment/dev-environment-configuration-file.ts.
-//
-// Node walks UP from the working directory looking for a dev-env configuration
-// file and, when one is found, uses its `slug` as the environment every
-// dev-env command targets (getEnvironmentName, dev-environment-cli.ts:166).
-// vip-next ignored the file entirely, so in a configured repo `dev-env destroy`
-// tore down whatever environment happened to be the only one on disk instead of
-// the configured one (register item 2.21).
-
-const (
-	// configFolder is Node's CONFIGURATION_FOLDER (dev-environment-cli.ts:50).
-	configFolder = ".wpvip"
-	// configFileName / configTemplateFileName are CONFIGURATION_FILE_NAME and
-	// CONFIGURATION_TEMPLATE_FILE_NAME.
-	configFileName         = "vip-dev-env.yml"
-	configTemplateFileName = "vip-dev-env.yml.ejs"
-	// configWalkMaxDepth is Node's `maxDepth` sanity bound on the upward walk.
-	configWalkMaxDepth = 64
-)
-
-// configFileVersions is Node's CONFIGURATION_FILE_VERSIONS.
-var configFileVersions = []string{"1"}
-
-// ConfigFile is a sanitized dev-env configuration file — the Go equivalent of
-// Node's ConfigurationFileOptions plus its `meta['configuration-path']`.
-type ConfigFile struct {
-	// Path is the file the values came from; Node prints it in the
-	// "Using environment X from Y" line.
-	Path                string
-	Version             string
-	Slug                string
-	Title               string
-	Multisite           string // "", "subdomain", "subdirectory", "true"/"false"
-	PHP                 string
-	WordPress           string
-	MuPlugins           string
-	AppCode             string
-	MediaRedirectDomain string
-	Overrides           string
-	Elasticsearch       *bool
-	PHPMyAdmin          *bool
-	Xdebug              *bool
-	Mailpit             *bool
-	Photon              *bool
-	Cron                *bool
-}
-
-// LoadConfigFile runs the discovery from the process working directory, which
-// is what Node uses (findConfigurationFile starts at process.cwd()).
-func LoadConfigFile() (*ConfigFile, error) {
-	wd, err := os.Getwd()
-	if err != nil {
-		return nil, nil //nolint:nilerr // no cwd => no configuration, same as Node's read failure
-	}
-	return FindConfigFile(wd)
-}
-
-// FindConfigFile walks up from startDir looking for a dev-env configuration
-// file, returning nil when there is none. A file that IS found but cannot be
-// parsed is a hard error — Node calls exit.withError() there, and silently
-// continuing would let a destructive command target the wrong environment.
-func FindConfigFile(startDir string) (*ConfigFile, error) {
-	for _, cand := range configFileCandidates(startDir) {
-		b, err := os.ReadFile(cand.file) // #nosec G304 -- walked, user-owned repo path
-		if err != nil {
-			// Node debug-logs and moves to the next candidate.
-			continue
-		}
-		contents := string(b)
-		if cand.template {
-			rendered, rerr := renderConfigTemplate(contents, cand.dir)
-			if rerr != nil {
-				return nil, fmt.Errorf("Configuration file %s could not be loaded:\n%s", cand.file, rerr)
-			}
-			contents = rendered
-		}
-		return sanitizeConfigFile(contents, cand.file)
-	}
-	return nil, nil
-}
-
-type configCandidate struct {
-	dir      string
-	file     string
-	template bool
-}
-
-// configFileCandidates reproduces findConfigurationFile's location list: for
-// each directory from startDir upward (stopping AT — not including — the
-// filesystem root, and after 64 directories), four candidates in this order:
-//
-//	<dir>/.wpvip/vip-dev-env.yml.ejs   (template)
-//	<dir>/.wpvip/vip-dev-env.yml
-//	<dir>/.vip-dev-env.yml.ejs         (template)
-//	<dir>/.vip-dev-env.yml
-//
-// The whole list is built first and then probed in order, so every candidate in
-// a nearer directory beats every candidate in an ancestor.
-func configFileCandidates(startDir string) []configCandidate {
-	current := filepath.Clean(startDir)
-	root := filepath.Dir(current)
-	for root != filepath.Dir(root) {
-		root = filepath.Dir(root)
-	}
-
-	var out []configCandidate
-	for depth := 0; current != root && depth < configWalkMaxDepth; depth++ {
-		wpvip := filepath.Join(current, configFolder)
-		out = append(out,
-			configCandidate{dir: wpvip, file: filepath.Join(wpvip, configTemplateFileName), template: true},
-			configCandidate{dir: wpvip, file: filepath.Join(wpvip, configFileName), template: false},
-			configCandidate{dir: current, file: filepath.Join(current, "."+configTemplateFileName), template: true},
-			configCandidate{dir: current, file: filepath.Join(current, "."+configFileName), template: false},
-		)
-		parent := filepath.Dir(current)
-		if parent == current {
-			break
-		}
-		current = parent
-	}
-	return out
-}
-
-// ejsTagRE matches any remaining EJS tag after configDir substitution.
-var ejsTagRE = regexp.MustCompile(`<%[-=_]?[\s\S]*?%>`)
-
-// configDirTagRE matches the one expression Node's template context provides:
-// `configDir` (ejs.render(contents, { configDir: dir })).
-var configDirTagRE = regexp.MustCompile(`<%[-=]\s*configDir\s*-?%>`)
-
-// renderConfigTemplate renders a .ejs configuration template. Node runs a full
-// EJS engine with exactly one variable in scope (configDir), which is all the
-// documented templates use. Anything richer is refused rather than silently
-// mis-rendered: a template we cannot evaluate would otherwise resolve to a
-// DIFFERENT environment than the Node CLI targets, on commands that delete data.
-func renderConfigTemplate(contents, configDir string) (string, error) {
-	rendered := configDirTagRE.ReplaceAllLiteralString(contents, configDir)
-	if tag := ejsTagRE.FindString(rendered); tag != "" {
-		return "", fmt.Errorf("unsupported EJS expression %q (only <%%= configDir %%> is supported)", tag)
-	}
-	return rendered, nil
-}
-
-// sanitizeConfigFile ports sanitizeConfiguration + adjustRelativePaths,
-// including the exact error wording Node exits with.
-func sanitizeConfigFile(contents, path string) (*ConfigFile, error) {
-	// yaml.v3 decodes scalars into `any` as typed values; decoding into
-	// map[string]string-ish accessors below keeps them verbatim, matching
-	// Node's FAILSAFE_SCHEMA ("Only allow strings, arrays, and objects").
-	var node yaml.Node
-	if err := yaml.Unmarshal([]byte(contents), &node); err != nil {
-		return nil, fmt.Errorf("Configuration file %s could not be loaded:\n%s", path, err)
-	}
-	raw := failsafeMapping(&node)
-	if raw == nil {
-		return nil, configGenericError(path)
-	}
-
-	version, hasVersion := raw["configuration-version"]
-	slug, hasSlug := raw["slug"]
-	if !hasVersion || version == "" || !hasSlug {
-		return nil, configGenericError(path)
-	}
-	if !isValidConfigVersion(version) {
-		return nil, fmt.Errorf(
-			"Configuration file %s has an invalid configuration-version key. "+
-				"Update to a supported version. For example:\n\n%s\nSupported configuration versions: %s.\n",
-			path, configFileExample(), strings.Join(configFileVersions, ", "))
-	}
-
-	cfg := &ConfigFile{
-		Path:                path,
-		Version:             version,
-		Slug:                slug,
-		Title:               raw["title"],
-		Multisite:           raw["multisite"],
-		PHP:                 raw["php"],
-		WordPress:           raw["wordpress"],
-		MuPlugins:           raw["mu-plugins"],
-		AppCode:             raw["app-code"],
-		MediaRedirectDomain: raw["media-redirect-domain"],
-		Overrides:           raw["overrides"],
-		Elasticsearch:       stringToBoolIfDefined(raw, "elasticsearch"),
-		PHPMyAdmin:          stringToBoolIfDefined(raw, "phpmyadmin"),
-		Xdebug:              stringToBoolIfDefined(raw, "xdebug"),
-		Mailpit:             stringToBoolIfDefined(raw, "mailpit"),
-		Photon:              stringToBoolIfDefined(raw, "photon"),
-		Cron:                stringToBoolIfDefined(raw, "cron"),
-	}
-	adjustConfigRelativePaths(cfg)
-	return cfg, nil
-}
-
-// failsafeMapping returns the document's top-level mapping as string->string,
-// or nil when the document is not a mapping (Node: `Array.isArray(configuration)
-// || typeof configuration !== 'object'` -> generic error). Nested values are
-// not used by any consumer, so only scalars are collected.
-func failsafeMapping(doc *yaml.Node) map[string]string {
-	if doc.Kind != yaml.DocumentNode || len(doc.Content) == 0 {
-		return nil
-	}
-	m := doc.Content[0]
-	if m.Kind != yaml.MappingNode {
-		return nil
-	}
-	out := make(map[string]string, len(m.Content)/2)
-	for i := 0; i+1 < len(m.Content); i += 2 {
-		key, val := m.Content[i], m.Content[i+1]
-		if key.Kind != yaml.ScalarNode {
-			continue
-		}
-		if val.Kind == yaml.ScalarNode {
-			// Scalar Value is the verbatim source text, which is exactly what
-			// FAILSAFE_SCHEMA yields (php: 8.10 stays "8.10", not 8.1).
-			out[key.Value] = val.Value
-		}
-	}
-	return out
-}
-
-// stringToBoolIfDefined ports Node's stringToBooleanIfDefined: only the exact
-// strings "true"/"false" produce a value; everything else (including a missing
-// key) stays undefined.
-func stringToBoolIfDefined(raw map[string]string, key string) *bool {
-	v, ok := raw[key]
-	if !ok {
-		return nil
-	}
-	switch v {
-	case "true":
-		t := true
-		return &t
-	case "false":
-		f := false
-		return &f
-	}
-	return nil
-}
-
-// adjustConfigRelativePaths ports adjustRelativePaths: app-code and mu-plugins
-// are resolved against the configuration file's directory unless they are one
-// of the image keywords.
-func adjustConfigRelativePaths(cfg *ConfigFile) {
-	dir := filepath.Dir(cfg.Path)
-	fix := func(v string) string {
-		if v == "" || v == "demo" || v == "image" || filepath.IsAbs(v) {
-			return v
-		}
-		return filepath.Join(dir, v)
-	}
-	cfg.AppCode = fix(cfg.AppCode)
-	cfg.MuPlugins = fix(cfg.MuPlugins)
-}
-
-func isValidConfigVersion(v string) bool {
-	for _, known := range configFileVersions {
-		if known == v {
-			return true
-		}
-	}
-	return false
-}
-
-func configGenericError(path string) error {
-	return fmt.Errorf(
-		"Configuration file %s is available but couldn't be loaded. "+
-			"Ensure there is a configuration-version and slug configured. For example:\n\n%s",
-		path, configFileExample())
-}
-
-// configFileExample ports getConfigurationFileExample().
-func configFileExample() string {
-	return fmt.Sprintf(`configuration-version: %s
-slug: dev-site
-title: Dev Site
-php: 8.2
-wordpress: 6.2
-app-code: ../
-mu-plugins: image
-multisite: false
-phpmyadmin: false
-elasticsearch: false
-xdebug: false
-mailpit: false
-photon: false
-cron: false
-`, configFileVersions[len(configFileVersions)-1])
-}

package/internal/devenv/configfile_test.go

@@ -1,246 +0,0 @@
-package devenv
-
-import (
-	"os"
-	"path/filepath"
-	"strings"
-	"testing"
-)
-
-func writeFile(t *testing.T, path, content string) {
-	t.Helper()
-	if err := os.MkdirAll(filepath.Dir(path), 0o755); err != nil {
-		t.Fatal(err)
-	}
-	if err := os.WriteFile(path, []byte(content), 0o600); err != nil {
-		t.Fatal(err)
-	}
-}
-
-const minimalConfig = "configuration-version: 1\nslug: configured-site\n"
-
-// Register 2.21. Node walks UP from the cwd looking for the dev-env
-// configuration file (dev-environment-configuration-file.ts findConfigurationFile).
-// vip-next ignored it entirely, so `destroy` in a configured repo targeted
-// whatever environment happened to be the only/selected one.
-func TestFindConfigFileInCurrentDirectory(t *testing.T) {
-	dir := t.TempDir()
-	writeFile(t, filepath.Join(dir, ".wpvip", "vip-dev-env.yml"), minimalConfig)
-
-	cfg, err := FindConfigFile(dir)
-	if err != nil {
-		t.Fatal(err)
-	}
-	if cfg == nil {
-		t.Fatal("no configuration file found")
-	}
-	if cfg.Slug != "configured-site" {
-		t.Errorf("Slug = %q, want configured-site", cfg.Slug)
-	}
-	if want := filepath.Join(dir, ".wpvip", "vip-dev-env.yml"); cfg.Path != want {
-		t.Errorf("Path = %q, want %q", cfg.Path, want)
-	}
-}
-
-// Node pushes candidates for the cwd, then its parent, then its parent's
-// parent — first readable file wins, so a deep working directory still finds
-// the repo-root configuration.
-func TestFindConfigFileWalksUp(t *testing.T) {
-	root := t.TempDir()
-	writeFile(t, filepath.Join(root, ".wpvip", "vip-dev-env.yml"), minimalConfig)
-	deep := filepath.Join(root, "a", "b", "c")
-	if err := os.MkdirAll(deep, 0o755); err != nil {
-		t.Fatal(err)
-	}
-
-	cfg, err := FindConfigFile(deep)
-	if err != nil {
-		t.Fatal(err)
-	}
-	if cfg == nil || cfg.Slug != "configured-site" {
-		t.Fatalf("walk-up did not find the repo-root configuration: %+v", cfg)
-	}
-}
-
-// The nearest directory wins over an ancestor.
-func TestFindConfigFileNearestWins(t *testing.T) {
-	root := t.TempDir()
-	writeFile(t, filepath.Join(root, ".wpvip", "vip-dev-env.yml"), minimalConfig)
-	nested := filepath.Join(root, "child")
-	writeFile(t, filepath.Join(nested, ".wpvip", "vip-dev-env.yml"),
-		"configuration-version: 1\nslug: nested-site\n")
-
-	cfg, err := FindConfigFile(nested)
-	if err != nil {
-		t.Fatal(err)
-	}
-	if cfg == nil || cfg.Slug != "nested-site" {
-		t.Fatalf("nearest configuration must win, got %+v", cfg)
-	}
-}
-
-// Node also accepts the dotfile form at each level, but `.wpvip/` is checked
-// first at the SAME level (locations are pushed in that order).
-func TestFindConfigFileDotfileForm(t *testing.T) {
-	dir := t.TempDir()
-	writeFile(t, filepath.Join(dir, ".vip-dev-env.yml"), "configuration-version: 1\nslug: dotfile-site\n")
-
-	cfg, err := FindConfigFile(dir)
-	if err != nil {
-		t.Fatal(err)
-	}
-	if cfg == nil || cfg.Slug != "dotfile-site" {
-		t.Fatalf("dotfile form not found: %+v", cfg)
-	}
-}
-
-func TestFindConfigFileWpvipBeatsDotfileAtSameLevel(t *testing.T) {
-	dir := t.TempDir()
-	writeFile(t, filepath.Join(dir, ".wpvip", "vip-dev-env.yml"), minimalConfig)
-	writeFile(t, filepath.Join(dir, ".vip-dev-env.yml"), "configuration-version: 1\nslug: dotfile-site\n")
-
-	cfg, err := FindConfigFile(dir)
-	if err != nil {
-		t.Fatal(err)
-	}
-	if cfg == nil || cfg.Slug != "configured-site" {
-		t.Fatalf(".wpvip/ must win at the same level, got %+v", cfg)
-	}
-}
-
-// Node's walk is bounded: `depth < maxDepth` with maxDepth = 64, and it stops
-// at the filesystem root (the root directory itself is never inspected).
-func TestFindConfigFileStopsAfter64Levels(t *testing.T) {
-	root := t.TempDir()
-	writeFile(t, filepath.Join(root, ".wpvip", "vip-dev-env.yml"), minimalConfig)
-
-	// 64 directories below root: the walk visits the start dir plus 63
-	// ancestors, so root itself is out of reach.
-	deep := root
-	for i := 0; i < 64; i++ {
-		deep = filepath.Join(deep, "d")
-	}
-	if err := os.MkdirAll(deep, 0o755); err != nil {
-		t.Fatal(err)
-	}
-	cfg, err := FindConfigFile(deep)
-	if err != nil {
-		t.Fatal(err)
-	}
-	if cfg != nil {
-		t.Errorf("walk must stop after 64 levels, found %q", cfg.Path)
-	}
-
-	// One level shallower is still in range — pins both sides of the bound so
-	// the test cannot pass just because the walk is broken.
-	inRange := filepath.Dir(deep)
-	cfg, err = FindConfigFile(inRange)
-	if err != nil {
-		t.Fatal(err)
-	}
-	if cfg == nil {
-		t.Error("63 levels below the configuration must still find it")
-	}
-}
-
-func TestFindConfigFileNoneReturnsNil(t *testing.T) {
-	cfg, err := FindConfigFile(t.TempDir())
-	if err != nil {
-		t.Fatal(err)
-	}
-	if cfg != nil {
-		t.Errorf("expected no configuration file, got %+v", cfg)
-	}
-}
-
-// Node's sanitizeConfiguration exits with an error when the file lacks
-// configuration-version or slug — a broken config must never silently fall
-// through to "some other environment" on a destructive command.
-func TestFindConfigFileMissingKeysIsFatal(t *testing.T) {
-	for name, body := range map[string]string{
-		"no version": "slug: x\n",
-		"no slug":    "configuration-version: 1\n",
-		"a list":     "- configuration-version: 1\n",
-	} {
-		t.Run(name, func(t *testing.T) {
-			dir := t.TempDir()
-			writeFile(t, filepath.Join(dir, ".wpvip", "vip-dev-env.yml"), body)
-			_, err := FindConfigFile(dir)
-			if err == nil {
-				t.Fatal("want an error, got nil")
-			}
-			if !strings.Contains(err.Error(), "couldn't be loaded") {
-				t.Errorf("error = %q, want Node's \"couldn't be loaded\" wording", err)
-			}
-		})
-	}
-}
-
-func TestFindConfigFileUnsupportedVersionIsFatal(t *testing.T) {
-	dir := t.TempDir()
-	writeFile(t, filepath.Join(dir, ".wpvip", "vip-dev-env.yml"), "configuration-version: 2\nslug: x\n")
-	_, err := FindConfigFile(dir)
-	if err == nil || !strings.Contains(err.Error(), "invalid configuration-version") {
-		t.Fatalf("error = %v, want Node's invalid configuration-version message", err)
-	}
-}
-
-func TestFindConfigFileMalformedYAMLIsFatal(t *testing.T) {
-	dir := t.TempDir()
-	writeFile(t, filepath.Join(dir, ".wpvip", "vip-dev-env.yml"), "configuration-version: 1\n\tslug: [\n")
-	_, err := FindConfigFile(dir)
-	if err == nil || !strings.Contains(err.Error(), "could not be loaded") {
-		t.Fatalf("error = %v, want Node's could-not-be-loaded message", err)
-	}
-}
-
-// FAILSAFE_SCHEMA: Node loads the YAML with the failsafe schema so `php: 8.1`
-// parses as the STRING "8.1", not the number 8.1 (which would stringify to
-// "8.1" here but "8.10" -> "8.1" elsewhere). Values must stay verbatim.
-func TestFindConfigFileKeepsNumberLikeValuesAsStrings(t *testing.T) {
-	dir := t.TempDir()
-	writeFile(t, filepath.Join(dir, ".wpvip", "vip-dev-env.yml"),
-		"configuration-version: 1\nslug: x\nphp: 8.10\nwordpress: 6.40\n")
-	cfg, err := FindConfigFile(dir)
-	if err != nil {
-		t.Fatal(err)
-	}
-	if cfg.PHP != "8.10" {
-		t.Errorf("PHP = %q, want the verbatim string 8.10", cfg.PHP)
-	}
-	if cfg.WordPress != "6.40" {
-		t.Errorf("WordPress = %q, want the verbatim string 6.40", cfg.WordPress)
-	}
-}
-
-// adjustRelativePaths resolves app-code / mu-plugins relative to the
-// configuration file's directory, leaving the "demo"/"image" keywords alone.
-func TestFindConfigFileResolvesRelativeComponentPaths(t *testing.T) {
-	dir := t.TempDir()
-	writeFile(t, filepath.Join(dir, ".wpvip", "vip-dev-env.yml"),
-		"configuration-version: 1\nslug: x\napp-code: ../\nmu-plugins: image\n")
-	cfg, err := FindConfigFile(dir)
-	if err != nil {
-		t.Fatal(err)
-	}
-	if want := filepath.Join(dir, ".wpvip", ".."); cfg.AppCode != want {
-		t.Errorf("AppCode = %q, want %q", cfg.AppCode, want)
-	}
-	if cfg.MuPlugins != "image" {
-		t.Errorf("MuPlugins = %q, want the image keyword untouched", cfg.MuPlugins)
-	}
-}
-
-// Node's slug is used verbatim (configuration.slug.toString()); unlike --slug
-// it does NOT go through processSlug, so it is not lowercased.
-func TestFindConfigFileSlugIsNotLowercased(t *testing.T) {
-	dir := t.TempDir()
-	writeFile(t, filepath.Join(dir, ".wpvip", "vip-dev-env.yml"), "configuration-version: 1\nslug: MixedCase\n")
-	cfg, err := FindConfigFile(dir)
-	if err != nil {
-		t.Fatal(err)
-	}
-	if cfg.Slug != "MixedCase" {
-		t.Errorf("Slug = %q, want MixedCase (Node does not processSlug the config value)", cfg.Slug)
-	}
-}