Add phpManager type to enforce OS-specific interface

internal/legacy/legacy.go

@@ -108,9 +108,8 @@ func (c *CLIWrapper) init() error {
 		}
 		return nil
 	})
-	g.Go(func() error {
-		return c.copyPHP(cacheDir)
-	})
+
+	g.Go(newPHPManager(cacheDir).copy)
 
 	if err := g.Wait(); err != nil {
 		return err
@@ -176,14 +175,15 @@ func (c *CLIWrapper) Exec(ctx context.Context, args ...string) error {
 
 // makeCmd makes a legacy CLI command with the given context and arguments.
 func (c *CLIWrapper) makeCmd(ctx context.Context, args []string, cacheDir string) *exec.Cmd {
-	iniSettings := c.phpSettings(cacheDir)
-	var cmdArgs = make([]string, 0, len(args)+2+len(iniSettings)*2)
-	for _, s := range iniSettings {
+	phpMgr := newPHPManager(cacheDir)
+	settings := phpMgr.iniSettings()
+	var cmdArgs = make([]string, 0, len(args)+2+len(settings)*2)
+	for _, s := range settings {
 		cmdArgs = append(cmdArgs, "-d", s)
 	}
 	cmdArgs = append(cmdArgs, c.pharPath(cacheDir))
 	cmdArgs = append(cmdArgs, args...)
-	return exec.CommandContext(ctx, c.phpPath(cacheDir), cmdArgs...) //nolint:gosec
+	return exec.CommandContext(ctx, phpMgr.binPath(), cmdArgs...) //nolint:gosec
 }
 
 // PharPath returns the path to the legacy CLI's Phar file.

internal/legacy/php_manager.go

@@ -0,0 +1,20 @@
+package legacy
+
+type phpManager interface {
+	// copy writes embedded PHP files to temporary files.
+	copy() error
+
+	// binPath returns the path to the temporary PHP binary.
+	binPath() string
+
+	// iniSettings returns PHP INI entries (key=value format).
+	iniSettings() []string
+}
+
+type phpManagerPerOS struct {
+	cacheDir string
+}
+
+func newPHPManager(cacheDir string) phpManager {
+	return &phpManagerPerOS{cacheDir}
+}

internal/legacy/php_manager_test.go

@@ -0,0 +1,16 @@
+package legacy
+
+import (
+	"testing"
+
+	"github.com/stretchr/testify/assert"
+)
+
+func TestPHPManager(t *testing.T) {
+	tempDir := t.TempDir()
+
+	pm := newPHPManager(tempDir)
+	assert.NoError(t, pm.copy())
+
+	assert.FileExists(t, pm.binPath())
+}

internal/legacy/php_manager_unix.go

@@ -0,0 +1,21 @@
+//go:build darwin || linux
+
+package legacy
+
+import (
+	"path/filepath"
+
+	"github.com/platformsh/cli/internal/file"
+)
+
+func (m *phpManagerPerOS) copy() error {
+	return file.WriteIfNeeded(m.binPath(), phpCLI, 0o755)
+}
+
+func (m *phpManagerPerOS) binPath() string {
+	return filepath.Join(m.cacheDir, "php")
+}
+
+func (m *phpManagerPerOS) iniSettings() []string {
+	return nil
+}

internal/legacy/php_windows.go → internal/legacy/php_manager_windows.go

@@ -22,9 +22,8 @@ var phpCLI []byte
 //go:embed archives/cacert.pem
 var caCert []byte
 
-// copyPHP to destination, if it does not exist
-func (c *CLIWrapper) copyPHP(cacheDir string) error {
-	destDir := filepath.Join(cacheDir, "php")
+func (m *phpManagerPerOS) copy() error {
+	destDir := filepath.Join(m.cacheDir, "php")
 
 	r, err := zip.NewReader(bytes.NewReader(phpCLI), int64(len(phpCLI)))
 	if err != nil {
@@ -49,9 +48,16 @@ func (c *CLIWrapper) copyPHP(cacheDir string) error {
 	return nil
 }
 
-// phpPath returns the path to the temporary PHP-CLI binary
-func (c *CLIWrapper) phpPath(cacheDir string) string {
-	return filepath.Join(cacheDir, "php", "php.exe")
+func (m *phpManagerPerOS) binPath() string {
+	return filepath.Join(m.cacheDir, "php", "php.exe")
+}
+
+func (m *phpManagerPerOS) iniSettings() []string {
+	return []string{
+		"extension=" + filepath.Join(m.cacheDir, "php", "ext", "php_curl.dll"),
+		"extension=" + filepath.Join(m.cacheDir, "php", "ext", "php_openssl.dll"),
+		"openssl.cafile=" + filepath.Join(m.cacheDir, "php", "extras", "cacert.pem"),
+	}
 }
 
 // copyZipFile extracts a file from the Zip to the destination directory.
@@ -89,16 +95,8 @@ func copyZipFile(f *zip.File, destDir string) error {
 	}
 
 	if err := file.Write(destPath, b, f.Mode()); err != nil {
-		return fmt.Errorf("could not write extracted file %s: %w", destPath, err)
+		return fmt.Errorf("could not copy extracted file %s: %w", destPath, err)
 	}
 
 	return nil
 }
-
-func (c *CLIWrapper) phpSettings(cacheDir string) []string {
-	return []string{
-		"extension=" + filepath.Join(cacheDir, "php", "ext", "php_curl.dll"),
-		"extension=" + filepath.Join(cacheDir, "php", "ext", "php_openssl.dll"),
-		"openssl.cafile=" + filepath.Join(cacheDir, "php", "extras", "cacert.pem"),
-	}
-}