diff --git a/packer/config_file_test.go b/packer/config_file_test.go
index ba275793b..98fa67d99 100644
--- a/packer/config_file_test.go
+++ b/packer/config_file_test.go
@@ -42,7 +42,7 @@ func TestExpandUser_Empty(t *testing.T) {
 	var path, expected string
 
 	// Try an invalid user
-	path, err := ExpandUser("~invalid-user-that-should-not-exist")
+	_, err := ExpandUser("~invalid-user-that-should-not-exist")
 	if err == nil {
 		t.Fatalf("expected failure")
 	}
diff --git a/packer/core.go b/packer/core.go
index cae4c39cc..45912f070 100644
--- a/packer/core.go
+++ b/packer/core.go
@@ -407,7 +407,7 @@ func (c *Core) init() error {
 		}
 	}
 
-	if (changed == false) && (shouldRetry == true) {
+	if !changed && shouldRetry {
 		return fmt.Errorf("Failed to interpolate %s: Please make sure that "+
 			"the variable you're referencing has been defined; Packer treats "+
 			"all variables used to interpolate other user varaibles as "+
diff --git a/packer/core_test.go b/packer/core_test.go
index 6df55a47a..bf65f0ffa 100644
--- a/packer/core_test.go
+++ b/packer/core_test.go
@@ -670,17 +670,6 @@ func TestSensitiveVars(t *testing.T) {
 	}
 }
 
-func testComponentFinder() *ComponentFinder {
-	builderFactory := func(n string) (Builder, error) { return new(MockBuilder), nil }
-	ppFactory := func(n string) (PostProcessor, error) { return new(MockPostProcessor), nil }
-	provFactory := func(n string) (Provisioner, error) { return new(MockProvisioner), nil }
-	return &ComponentFinder{
-		Builder:       builderFactory,
-		PostProcessor: ppFactory,
-		Provisioner:   provFactory,
-	}
-}
-
 func testCoreTemplate(t *testing.T, c *CoreConfig, p string) {
 	tpl, err := template.ParseFile(p)
 	if err != nil {
diff --git a/packer/rpc/build_test.go b/packer/rpc/build_test.go
index aa19313de..9656c6227 100644
--- a/packer/rpc/build_test.go
+++ b/packer/rpc/build_test.go
@@ -21,7 +21,6 @@ type testBuild struct {
 	setDebugCalled   bool
 	setForceCalled   bool
 	setOnErrorCalled bool
-	cancelCalled     bool
 
 	errRunResult bool
 }
diff --git a/packer/rpc/post_processor_test.go b/packer/rpc/post_processor_test.go
index 1066c8f8b..3338915d2 100644
--- a/packer/rpc/post_processor_test.go
+++ b/packer/rpc/post_processor_test.go
@@ -112,6 +112,9 @@ func TestPostProcessorRPC_cancel(t *testing.T) {
 	// Test Configure
 	config := 42
 	err := ppClient.Configure(config)
+	if err != nil {
+		t.Fatalf("error configuring post-processor client: %s", err)
+	}
 
 	// Test PostProcess
 	a := &packer.MockArtifact{