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.
This commit is contained in:
parent
c646a2eb30
commit
f18ea412dc
|
@ -18,10 +18,22 @@ func setupConfig() (*Config, error) {
|
||||||
return nil, err
|
return nil, err
|
||||||
}
|
}
|
||||||
|
|
||||||
c.SetString("foo.bar", "baz")
|
err = c.SetString("foo.bar", "baz")
|
||||||
c.SetBool("foo.bool", true)
|
if err != nil {
|
||||||
c.SetInt32("foo.int32", 32)
|
return nil, err
|
||||||
c.SetInt64("foo.int64", 64)
|
}
|
||||||
|
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
|
return c, err
|
||||||
}
|
}
|
||||||
|
@ -86,6 +98,7 @@ func TestConfigLookups(t *testing.T) {
|
||||||
|
|
||||||
if err != nil {
|
if err != nil {
|
||||||
t.Errorf("Setup error: '%v'. Expected none\n", err)
|
t.Errorf("Setup error: '%v'. Expected none\n", err)
|
||||||
|
return
|
||||||
}
|
}
|
||||||
defer c.Free()
|
defer c.Free()
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue