diff --git a/cmd/common_test.go b/cmd/common_test.go index 60fb23473a084b3ace3392524853bbfe66b67775..048ed4b679d91b783587d2161a6a30d2f1db5cd8 100644 --- a/cmd/common_test.go +++ b/cmd/common_test.go @@ -195,17 +195,19 @@ func TestIsMaster(t *testing.T) { defer restore() for _, tc := range testCases { - cfgFile = tc.cfgFile - initConfig() - - oldGetBinariesFunc := getBinariesFunc - getBinariesFunc = tc.getBinariesFunc - defer func() { - getBinariesFunc = oldGetBinariesFunc - cfgFile = "" + func() { + cfgFile = tc.cfgFile + initConfig() + + oldGetBinariesFunc := getBinariesFunc + getBinariesFunc = tc.getBinariesFunc + defer func() { + getBinariesFunc = oldGetBinariesFunc + cfgFile = "" + }() + + assert.Equal(t, tc.isMaster, isMaster(), tc.name) }() - - assert.Equal(t, tc.isMaster, isMaster(), tc.name) } } @@ -506,17 +508,19 @@ func TestIsEtcd(t *testing.T) { defer restore() for _, tc := range testCases { - cfgFile = tc.cfgFile - initConfig() - - oldGetBinariesFunc := getBinariesFunc - getBinariesFunc = tc.getBinariesFunc - defer func() { - getBinariesFunc = oldGetBinariesFunc - cfgFile = "" + func() { + cfgFile = tc.cfgFile + initConfig() + + oldGetBinariesFunc := getBinariesFunc + getBinariesFunc = tc.getBinariesFunc + defer func() { + getBinariesFunc = oldGetBinariesFunc + cfgFile = "" + }() + + assert.Equal(t, tc.isEtcd, isEtcd(), tc.name) }() - - assert.Equal(t, tc.isEtcd, isEtcd(), tc.name) } } diff --git a/cmd/run.go b/cmd/run.go index 20199c5bc03f7ee2af376461f22c03f08b63c808..16b6641deab14e51cd9c924f1782b6d0822caf0e 100644 --- a/cmd/run.go +++ b/cmd/run.go @@ -52,7 +52,10 @@ var runCmd = &cobra.Command{ // Merge version-specific config if any. path := filepath.Join(cfgDir, bv) - mergeConfig(path) + err = mergeConfig(path) + if err != nil { + fmt.Printf("Error in mergeConfig: %v\n", err) + } err = run(targets, bv) if err != nil {