config_test: properly detect failed config writes

This patch fixes the setup stage of the config tests to notice when
the writes fail (eg $PWD is a read-only filesystem) and to correctly
skip the entire test function as a result.

1 files changed, 17 insertions, 4 deletions

diff --git a/config_test.go b/config_test.go
index 8c2decc..fea8d8a 100644
--- a/config_test.go
+++ b/config_test.go
@@ -18,10 +18,22 @@ func setupConfig() (*Config, error) {
 		return nil, err
 	}
 
-	c.SetString("foo.bar", "baz")
-	c.SetBool("foo.bool", true)
-	c.SetInt32("foo.int32", 32)
-	c.SetInt64("foo.int64", 64)
+	err = c.SetString("foo.bar", "baz")
+	if err != nil {
+		return nil, err
+	}
+	err = c.SetBool("foo.bool", true)
+	if err != nil {
+		return nil, err
+	}
+	err = c.SetInt32("foo.int32", 32)
+	if err != nil {
+		return nil, err
+	}
+	err = c.SetInt64("foo.int64", 64)
+	if err != nil {
+		return nil, err
+	}
 
 	return c, err
 }
@@ -86,6 +98,7 @@ func TestConfigLookups(t *testing.T) {
 
 	if err != nil {
 		t.Errorf("Setup error: '%v'. Expected none\n", err)
+		return
 	}
 	defer c.Free()