fix test

Author: yp05327

tests/integration/api_repo_tags_test.go

@@ -10,6 +10,7 @@ import (
 
 	"code.gitea.io/gitea/models/actions"
 	auth_model "code.gitea.io/gitea/models/auth"
+	repo_model "code.gitea.io/gitea/models/repo"
 	"code.gitea.io/gitea/models/unittest"
 	user_model "code.gitea.io/gitea/models/user"
 	"code.gitea.io/gitea/modules/setting"
@@ -23,13 +24,12 @@ import (
 func TestAPIRepoTags(t *testing.T) {
 	defer tests.PrepareTestEnv(t)()
 	user := unittest.AssertExistsAndLoadBean(t, &user_model.User{ID: 2})
+	repo1 := unittest.AssertExistsAndLoadBean(t, &repo_model.Repository{ID: 1})
 	// Login as User2.
 	session := loginUser(t, user.Name)
 	token := getTokenForLoggedInUser(t, session, auth_model.AccessTokenScopeWriteRepository)
 
-	repoName := "repo1"
-
-	req := NewRequestf(t, "GET", "/api/v1/repos/%s/%s/tags", user.Name, repoName).
+	req := NewRequestf(t, "GET", "/api/v1/repos/%s/%s/tags", user.Name, repo1.Name).
 		AddTokenAuth(token)
 	resp := MakeRequest(t, req, http.StatusOK)
 
@@ -44,7 +44,7 @@ func TestAPIRepoTags(t *testing.T) {
 	assert.Equal(t, setting.AppURL+"user2/repo1/archive/v1.1.zip", tags[0].ZipballURL)
 	assert.Equal(t, setting.AppURL+"user2/repo1/archive/v1.1.tar.gz", tags[0].TarballURL)
 
-	newTag := createNewTagUsingAPI(t, session, token, user.Name, repoName, "gitea/22", "", "nice!\nand some text")
+	newTag := createNewTagUsingAPI(t, session, token, user.Name, repo1.Name, "gitea/22", "", "nice!\nand some text")
 	resp = MakeRequest(t, req, http.StatusOK)
 	DecodeJSON(t, resp, &tags)
 	assert.Len(t, tags, 2)
@@ -58,15 +58,15 @@ func TestAPIRepoTags(t *testing.T) {
 	}
 
 	// get created tag
-	req = NewRequestf(t, "GET", "/api/v1/repos/%s/%s/tags/%s", user.Name, repoName, newTag.Name).
+	req = NewRequestf(t, "GET", "/api/v1/repos/%s/%s/tags/%s", user.Name, repo1.Name, newTag.Name).
 		AddTokenAuth(token)
 	resp = MakeRequest(t, req, http.StatusOK)
 	var tag *api.Tag
 	DecodeJSON(t, resp, &tag)
 	assert.EqualValues(t, newTag, tag)
 
 	// delete tag
-	delReq := NewRequestf(t, "DELETE", "/api/v1/repos/%s/%s/tags/%s", user.Name, repoName, newTag.Name).
+	delReq := NewRequestf(t, "DELETE", "/api/v1/repos/%s/%s/tags/%s", user.Name, repo1.Name, newTag.Name).
 		AddTokenAuth(token)
 	MakeRequest(t, delReq, http.StatusNoContent)
 
@@ -76,14 +76,20 @@ func TestAPIRepoTags(t *testing.T) {
 	// check action run
 	tests := []struct {
 		id    int64
+		ref   string
+		SHA   string
 		event webhook.HookEventType
 	}{
 		{
 			id:    793,
+			ref:   fmt.Sprintf("refs/tags/%s", newTag.Name),
+			SHA:   newTag.Commit.SHA,
 			event: webhook.HookEventCreate,
 		},
 		{
 			id:    794,
+			ref:   repo1.DefaultBranch,
+			SHA:   "94dc45d6ec51090b36c4d5bd5d9e6f5a70da9edf",
 			event: webhook.HookEventDelete,
 		},
 	}
@@ -92,9 +98,9 @@ func TestAPIRepoTags(t *testing.T) {
 		assert.NotNil(t, run)
 		assert.Equal(t, int64(1), run.RepoID)
 		assert.Equal(t, "test.yaml", run.WorkflowID)
-		assert.Equal(t, newTag.Commit.SHA, run.CommitSHA)
+		assert.Equal(t, tt.SHA, run.CommitSHA)
 		assert.Equal(t, "add workflow", run.Title)
-		assert.Equal(t, fmt.Sprintf("refs/tags/%s", newTag.Name), run.Ref)
+		assert.Equal(t, tt.ref, run.Ref)
 		assert.Equal(t, tt.event, run.Event)
 	}
 }