From 7eb3f30d502c195663f3e4723e64b0698ae7e324 Mon Sep 17 00:00:00 2001 From: Andrew Pryde Date: Mon, 13 Feb 2017 12:51:01 +0000 Subject: [PATCH] Fix oracle-bmcs tests panicing instead of failing Replace use of t.Errorf with t.Fatalf to prevent subsequent panic on nil pointer access. --- builder/oracle/bmcs/config_test.go | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/builder/oracle/bmcs/config_test.go b/builder/oracle/bmcs/config_test.go index 135828d13..8c48ae50c 100644 --- a/builder/oracle/bmcs/config_test.go +++ b/builder/oracle/bmcs/config_test.go @@ -75,7 +75,7 @@ func TestConfig(t *testing.T) { _, errs := NewConfig(raw) if errs != nil { - t.Errorf("err: %+v", errs) + t.Fatalf("err: %+v", errs) } }) @@ -109,7 +109,7 @@ func TestConfig(t *testing.T) { _, errs := NewConfig(raw) if errs != nil { - t.Errorf("err: %+v", errs) + t.Fatalf("err: %+v", errs) } }) @@ -118,7 +118,7 @@ func TestConfig(t *testing.T) { raw := testConfig(cfgFile) c, errs := NewConfig(raw) if errs != nil { - t.Errorf("err: %+v", errs) + t.Fatalf("err: %+v", errs) } expected := "ocid1.tenancy.oc1..aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa" @@ -164,7 +164,7 @@ func TestConfig(t *testing.T) { c, errs := NewConfig(raw) if errs != nil { - t.Errorf("err: %+v", errs) + t.Fatalf("err: %+v", errs) } accessVal := getField(c.AccessCfg, v)