chore: Remove GetSourceByName

- Introduced in 5455605342 and removed in
the same commit. Usage was purely testing code.

.deadcode-out

@@ -23,7 +23,6 @@ code.gitea.io/gitea/models/actions
 	ScheduleList.LoadRepos
 
 code.gitea.io/gitea/models/auth
-	GetSourceByName
 	WebAuthnCredentials
 
 code.gitea.io/gitea/models/db

models/auth/source.go

@@ -299,17 +299,6 @@ func GetSourceByID(ctx context.Context, id int64) (*Source, error) {
 	return source, nil
 }
 
-func GetSourceByName(ctx context.Context, name string) (*Source, error) {
-	source := &Source{}
-	has, err := db.GetEngine(ctx).Where("name = ?", name).Get(source)
-	if err != nil {
-		return nil, err
-	} else if !has {
-		return nil, ErrSourceNotExist{}
-	}
-	return source, nil
-}
-
 // UpdateSource updates a Source record in DB.
 func UpdateSource(ctx context.Context, source *Source) error {
 	var originalSource *Source

tests/integration/integration_test.go

@@ -298,9 +298,7 @@ func addAuthSource(t *testing.T, payload map[string]string) *auth.Source {
 	payload["_csrf"] = GetCSRF(t, session, "/admin/auths/new")
 	req := NewRequestWithValues(t, "POST", "/admin/auths/new", payload)
 	session.MakeRequest(t, req, http.StatusSeeOther)
-	source, err := auth.GetSourceByName(context.Background(), payload["name"])
+	return unittest.AssertExistsAndLoadBean(t, &auth.Source{Name: payload["name"]})
-	require.NoError(t, err)
-	return source
 }
 
 func authSourcePayloadOAuth2(name string) map[string]string {
@@ -358,9 +356,7 @@ func createRemoteAuthSource(t *testing.T, name, url, matchingSource string) *aut
 			MatchingSource: matchingSource,
 		},
 	}))
-	source, err := auth.GetSourceByName(context.Background(), name)
+	return unittest.AssertExistsAndLoadBean(t, &auth.Source{Name: name})
-	require.NoError(t, err)
-	return source
 }
 
 func createUser(ctx context.Context, t testing.TB, user *user_model.User) func() {