diff --git a/command/config_file_unix.go b/command/config_file_unix.go new file mode 100644 index 000000000..b1eba3aed --- /dev/null +++ b/command/config_file_unix.go @@ -0,0 +1,7 @@ +// +build darwin freebsd linux netbsd openbsd solaris + +package command + +const ( + defaultConfigDir = ".packer.d" +) diff --git a/command/config_file_windows.go b/command/config_file_windows.go new file mode 100644 index 000000000..5d8350a63 --- /dev/null +++ b/command/config_file_windows.go @@ -0,0 +1,7 @@ +// +build windows + +package command + +const ( + defaultConfigDir = "packer.d" +) diff --git a/command/init_test.go b/command/init_test.go index 27ddf0e54..3d8296b4b 100644 --- a/command/init_test.go +++ b/command/init_test.go @@ -1,14 +1,17 @@ package command import ( + "context" "fmt" "log" "os" + "path/filepath" "runtime" "sort" "testing" "github.com/google/go-cmp/cmp" + "github.com/hashicorp/go-getter/v2" "github.com/hashicorp/packer-plugin-sdk/acctest" "golang.org/x/mod/sumdb/dirhash" ) @@ -220,6 +223,53 @@ func TestInitCommand_Run(t *testing.T) { "h1:47DEQpj8HBSa+/TImW+5JCeuQeRkm5NMpJWZG3hSuFU=", nil, }, + { + "manually-installed-single-component-plugin-works", + []func(t *testing.T, tc testCaseInit){ + skipInitTestUnlessEnVar(acctest.TestEnvVar).fn, + initTestGoGetPlugin{ + Src: "https://github.com/azr/packer-provisioner-comment/releases/download/v1.0.0/" + + "packer-provisioner-comment_v1.0.0_" + runtime.GOOS + "_" + runtime.GOARCH + ".zip", + Dst: filepath.Join(cfg.dir("5_pkr_config"), defaultConfigDir, "plugins"), + }.fn, + }, + testMetaFile(t), + nil, + map[string]string{ + "darwin": "h1:nVebbXToeehPUASRbvV9M4qaA9+UgoR5AMp7LjTrSBk=", + "linux": "h1:/U5vdeMtOpRKNu0ld8+qf4t6WC+BsfCQ6JRo9Dh/khI=", + "windows": "h1:0nkdNCjtTHTgBNkzVKG++/VYmWAvq/o236GGTxrIf/Q=", + }[runtime.GOOS], + map[string]string{ + `source.pkr.hcl`: ` + source "null" "test" { + communicator = "none" + } + `, + `build.pkr.hcl`: ` + build { + sources = ["source.null.test"] + provisioner "comment" { + comment = "Begin ยก" + ui = true + bubble_text = true + } + } + `, + }, + cfg.dir("5_pkr_config"), + cfg.dir("5_pkr_user_folder"), + 0, + nil, + map[string]string{ + "darwin": "h1:nVebbXToeehPUASRbvV9M4qaA9+UgoR5AMp7LjTrSBk=", + "linux": "h1:/U5vdeMtOpRKNu0ld8+qf4t6WC+BsfCQ6JRo9Dh/khI=", + "windows": "h1:0nkdNCjtTHTgBNkzVKG++/VYmWAvq/o236GGTxrIf/Q=", + }[runtime.GOOS], + []func(*testing.T, testCaseInit){ + testBuild{want: 0}.fn, + }, + }, } for _, tt := range tests { t.Run(tt.name, func(t *testing.T) { @@ -231,6 +281,7 @@ func TestInitCommand_Run(t *testing.T) { t.Cleanup(func() { _ = os.RemoveAll(tt.packerUserFolder) }) + os.Setenv("PACKER_CONFIG_DIR", tt.packerConfigDir) for _, init := range tt.init { init(t, tt) if t.Skipped() { @@ -245,7 +296,7 @@ func TestInitCommand_Run(t *testing.T) { t.Fatalf("HashDir: %v", err) } if diff := cmp.Diff(tt.expectedPackerConfigDirHashBeforeInit, hash); diff != "" { - t.Errorf("unexpected dir hash before init: %s", diff) + t.Errorf("unexpected dir hash before init: +found -expected %s", diff) } args := []string{tt.packerUserFolder} @@ -254,6 +305,10 @@ func TestInitCommand_Run(t *testing.T) { Meta: tt.Meta, } + if err := c.CoreConfig.Components.PluginConfig.Discover(); err != nil { + t.Fatalf("Failed to discover plugins: %s", err) + } + c.CoreConfig.Components.PluginConfig.KnownPluginFolders = []string{tt.packerConfigDir} if got := c.Run(args); got != tt.want { t.Errorf("InitCommand.Run() = %v, want %v", got, tt.want) @@ -291,8 +346,19 @@ func TestInitCommand_Run(t *testing.T) { type skipInitTestUnlessEnVar string func (key skipInitTestUnlessEnVar) fn(t *testing.T, tc testCaseInit) { - return // always run acc tests for now - if os.Getenv(string(key)) == "" { - t.Skipf("Acceptance test skipped unless env '%s' set", key) + // always run acc tests for now + // if os.Getenv(string(key)) == "" { + // t.Skipf("Acceptance test skipped unless env '%s' set", key) + // } +} + +type initTestGoGetPlugin struct { + Src string + Dst string +} + +func (opts initTestGoGetPlugin) fn(t *testing.T, _ testCaseInit) { + if _, err := getter.Get(context.Background(), opts.Dst, opts.Src); err != nil { + t.Fatalf("get: %v", err) } } diff --git a/packer/plugin.go b/packer/plugin.go index 8f7397934..1d8bcd7ba 100644 --- a/packer/plugin.go +++ b/packer/plugin.go @@ -106,6 +106,7 @@ func (c *PluginConfig) Discover() error { func (c *PluginConfig) discoverExternalComponents(path string) error { var err error + log.Printf("[TRACE] discovering plugins in %s", path) if !filepath.IsAbs(path) { path, err = filepath.Abs(path)