Fix tests

The previous merge managed to break repo_form.go

Signed-off-by: Andrew Thornton <[email protected]>

Author: zeripath

integrations/api_repo_test.go

@@ -325,7 +325,7 @@ func TestAPIRepoMigrate(t *testing.T) {
 		if resp.Code == http.StatusUnprocessableEntity {
 			respJSON := map[string]string{}
 			DecodeJSON(t, resp, &respJSON)
-			if assert.Equal(t, respJSON["message"], "Remote visit addressed rate limitation.") {
+			if assert.Equal(t, "Remote visit addressed rate limitation.", respJSON["message"]) {
 				t.Log("test hit github rate limitation")
 			}
 		} else {

modules/auth/repo_form.go

@@ -108,15 +108,15 @@ func ParseRemoteAddr(remoteAddr, authUsername, authPassword string, user *models
 		}
 	} else if !user.CanImportLocal() {
 		return "", models.ErrInvalidCloneAddr{IsPermissionDenied: true}
-	}
-
-	isDir, err := util.IsDir(remoteAddr)
-	if err != nil {
-		log.Error("Unable to check if %s is a directory: %v", remoteAddr, err)
-		return "", err
-	}
-	if !isDir {
-		return "", models.ErrInvalidCloneAddr{IsInvalidPath: true}
+	} else {
+		isDir, err := util.IsDir(remoteAddr)
+		if err != nil {
+			log.Error("Unable to check if %s is a directory: %v", remoteAddr, err)
+			return "", err
+		}
+		if !isDir {
+			return "", models.ErrInvalidCloneAddr{IsInvalidPath: true}
+		}
 	}
 
 	return remoteAddr, nil