removed a semicolon and the placeholder for the error which we used to locate it. Seems like SecItemCopyMatching is giving an error for some reason- no error on local machine

Signed-off-by: avaid96 <[email protected]>

Author: avaid96

osxkeychain/osxkeychain_darwin.go

@@ -104,7 +104,6 @@ func (h Osxkeychain) List() ([]string, []string, error) {
 	if errMsg != nil {
 		defer C.free(unsafe.Pointer(errMsg))
 		goMsg := C.GoString(errMsg)
-		goMsg = "Error is here"
 		return nil, nil, errors.New(goMsg)
 	}
 	var listLen int

osxkeychain/osxkeychain_darwin_test.go

@@ -37,18 +37,18 @@ func TestOSXKeychainHelper(t *testing.T) {
 	if err := helper.Delete(creds.ServerURL); err != nil {
 		t.Fatal(err)
 	}
-	helper.Add(creds);
+	helper.Add(creds)
 	defer helper.Delete(creds.ServerURL)
 	paths, accts, err := helper.List()
 	if err != nil || len(paths) == 0 || len(accts) == 0 {
 		t.Fatal(err)
 	}
 	helper.Add(creds1)
+	defer helper.Delete(creds1.ServerURL)
 	newpaths, newaccts, err := helper.List()
 	if len(newpaths)-len(paths) != 1 || len(newaccts)-len(accts) != 1 {
 		t.Fatal(err)
 	}
-	helper.Delete(creds1.ServerURL)
 }
 
 func TestMissingCredentials(t *testing.T) {