diff options
author | zeripath <art27@cantab.net> | 2019-11-25 23:21:37 +0000 |
---|---|---|
committer | Lauris BH <lauris@nix.lv> | 2019-11-26 01:21:37 +0200 |
commit | 055f6d229672524e3933940f95aa28939d94961a (patch) | |
tree | 72d1d427ce694517b70b6b751e35fdf6d5874428 /integrations/api_repo_test.go | |
parent | f5bd0884d2adea8ef7ba1bbd9dacb240d7c6b0f7 (diff) | |
download | gitea-055f6d229672524e3933940f95aa28939d94961a.tar.gz gitea-055f6d229672524e3933940f95aa28939d94961a.zip |
Fix "data race" in testlogger (#9159)
* Fix data race in testlogger
* Update git_helper_for_declarative_test.go
Diffstat (limited to 'integrations/api_repo_test.go')
-rw-r--r-- | integrations/api_repo_test.go | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/integrations/api_repo_test.go b/integrations/api_repo_test.go index e021911afd..add546fb49 100644 --- a/integrations/api_repo_test.go +++ b/integrations/api_repo_test.go @@ -19,7 +19,7 @@ import ( ) func TestAPIUserReposNotLogin(t *testing.T) { - prepareTestEnv(t) + defer prepareTestEnv(t)() user := models.AssertExistsAndLoadBean(t, &models.User{ID: 2}).(*models.User) req := NewRequestf(t, "GET", "/api/v1/users/%s/repos", user.Name) @@ -37,7 +37,7 @@ func TestAPIUserReposNotLogin(t *testing.T) { } func TestAPISearchRepo(t *testing.T) { - prepareTestEnv(t) + defer prepareTestEnv(t)() const keyword = "test" req := NewRequestf(t, "GET", "/api/v1/repos/search?q=%s", keyword) @@ -207,7 +207,7 @@ func getRepo(t *testing.T, repoID int64) *models.Repository { } func TestAPIViewRepo(t *testing.T) { - prepareTestEnv(t) + defer prepareTestEnv(t)() req := NewRequest(t, "GET", "/api/v1/repos/user2/repo1") resp := MakeRequest(t, req, http.StatusOK) @@ -219,7 +219,7 @@ func TestAPIViewRepo(t *testing.T) { } func TestAPIOrgRepos(t *testing.T) { - prepareTestEnv(t) + defer prepareTestEnv(t)() user := models.AssertExistsAndLoadBean(t, &models.User{ID: 2}).(*models.User) user2 := models.AssertExistsAndLoadBean(t, &models.User{ID: 1}).(*models.User) user3 := models.AssertExistsAndLoadBean(t, &models.User{ID: 5}).(*models.User) @@ -265,7 +265,7 @@ func TestAPIOrgRepos(t *testing.T) { } func TestAPIGetRepoByIDUnauthorized(t *testing.T) { - prepareTestEnv(t) + defer prepareTestEnv(t)() user := models.AssertExistsAndLoadBean(t, &models.User{ID: 4}).(*models.User) session := loginUser(t, user.Name) token := getTokenForLoggedInUser(t, session) @@ -286,7 +286,7 @@ func TestAPIRepoMigrate(t *testing.T) { {ctxUserID: 2, userID: 6, cloneURL: "https://github.com/go-gitea/git.git", repoName: "git-bad-org", expectedStatus: http.StatusForbidden}, } - prepareTestEnv(t) + defer prepareTestEnv(t)() for _, testCase := range testCases { user := models.AssertExistsAndLoadBean(t, &models.User{ID: testCase.ctxUserID}).(*models.User) session := loginUser(t, user.Name) @@ -351,7 +351,7 @@ func TestAPIOrgRepoCreate(t *testing.T) { {ctxUserID: 28, orgName: "user6", repoName: "repo-not-creator", expectedStatus: http.StatusForbidden}, } - prepareTestEnv(t) + defer prepareTestEnv(t)() for _, testCase := range testCases { user := models.AssertExistsAndLoadBean(t, &models.User{ID: testCase.ctxUserID}).(*models.User) session := loginUser(t, user.Name) |