From 1831a0905525ce9ff148a106ca9eb1d7b81501e4 Mon Sep 17 00:00:00 2001 From: Chris Bednarski Date: Wed, 24 Jun 2015 17:47:00 -0700 Subject: [PATCH] Fix compress crash - Changed config from pointer to value to fix crash - Removed acceptance flag from compress tests since they would have caught this --- post-processor/compress/post-processor.go | 2 +- post-processor/compress/post-processor_test.go | 6 ------ 2 files changed, 1 insertion(+), 7 deletions(-) diff --git a/post-processor/compress/post-processor.go b/post-processor/compress/post-processor.go index 8b70bc456..72b85090b 100644 --- a/post-processor/compress/post-processor.go +++ b/post-processor/compress/post-processor.go @@ -35,7 +35,7 @@ type Config struct { } type PostProcessor struct { - config *Config + config Config } var ( diff --git a/post-processor/compress/post-processor_test.go b/post-processor/compress/post-processor_test.go index d7bca6c7a..db23cf3b1 100644 --- a/post-processor/compress/post-processor_test.go +++ b/post-processor/compress/post-processor_test.go @@ -9,7 +9,6 @@ import ( "testing" "github.com/mitchellh/packer/builder/file" - env "github.com/mitchellh/packer/helper/builder/testing" "github.com/mitchellh/packer/packer" "github.com/mitchellh/packer/template" ) @@ -187,11 +186,6 @@ func setup(t *testing.T) (packer.Ui, packer.Artifact, error) { } func testArchive(t *testing.T, config string) packer.Artifact { - if os.Getenv(env.TestEnvVar) == "" { - t.Skip(fmt.Sprintf( - "Acceptance tests skipped unless env '%s' set", env.TestEnvVar)) - } - ui, artifact, err := setup(t) if err != nil { t.Fatalf("Error bootstrapping test: %s", err)