Check properly if an error is nil

CenturyLinkCloud · Sep 21, 2015 · 5e3bcc4 · 5e3bcc4
1 parent f0f6444
commit 5e3bcc4
Showing 1 changed file with 6 additions and 2 deletions.
diff --git a/model_loader/model_loader_test.go b/model_loader/model_loader_test.go
@@ -250,8 +250,12 @@ func TestModelLoader(t *testing.T) {
 		t.Logf("Executing %d test case.", i+1)
 		res := testModel{}
 		err := model_loader.LoadModel(testCase.args, &res)
-		if (err != nil || testCase.err != "") && err.Error() != testCase.err {
-			t.Errorf("Invalid error.\n Expected: %s,\n obtained %s", testCase.err, err.Error())
+		errMsg := ""
+		if err != nil {
+			errMsg = err.Error()
+		}
+		if (err != nil || testCase.err != "") && errMsg != testCase.err {
+			t.Errorf("Invalid error.\n Expected: %s,\n obtained %s", testCase.err, errMsg)
 		}
 		if testCase.res != nil && !reflect.DeepEqual(testCase.res, res) {
 			t.Errorf("Invalid result.\n expected %#v,\n obtained %#v", testCase.res, res)