From 049fb2d9c1468d78700b5fb4e93320ffa1e70b90 Mon Sep 17 00:00:00 2001 From: Ricard Clau Date: Thu, 7 Jul 2016 20:01:23 +0100 Subject: [PATCH] actually check for errors in functional tests --- provisioner/ansible/provisioner_test.go | 15 ++++++++++++--- 1 file changed, 12 insertions(+), 3 deletions(-) diff --git a/provisioner/ansible/provisioner_test.go b/provisioner/ansible/provisioner_test.go index 582c28907..12a0f4e9b 100644 --- a/provisioner/ansible/provisioner_test.go +++ b/provisioner/ansible/provisioner_test.go @@ -252,7 +252,10 @@ func TestAnsibleGetVersion(t *testing.T) { var p Provisioner p.config.Command = "ansible-playbook" - p.getVersion() + err := p.getVersion() + if err != nil { + t.Fatalf("err: %s", err) + } } func TestAnsibleLongMessages(t *testing.T) { @@ -263,7 +266,10 @@ func TestAnsibleLongMessages(t *testing.T) { var p Provisioner p.config.Command = "ansible-playbook" p.config.PlaybookFile = "./test-fixtures/long-debug-message.yml" - p.Prepare() + err := p.Prepare() + if err != nil { + t.Fatalf("err: %s", err) + } comm := &packer.MockCommunicator{} ui := &packer.BasicUi{ @@ -271,5 +277,8 @@ func TestAnsibleLongMessages(t *testing.T) { Writer: new(bytes.Buffer), } - p.Provision(ui, comm) + err = p.Provision(ui, comm) + if err != nil { + t.Fatalf("err: %s", err) + } }