From 90572c5a22ab753e8e019d276426df7b1bf6d5b2 Mon Sep 17 00:00:00 2001 From: Gusted Date: Wed, 21 Dec 2022 02:22:23 +0100 Subject: [PATCH] Specify ID in `TestAPITeam` (#22192) - There have been [CI failures](https://codeberg.org/forgejo/forgejo/issues/111) in this specific test function. The code on itself looks good, the CI failures are likely caused by not specifying any field in `TeamUser`, which might have caused to unittest to return another `TeamUser` than the code expects. Co-authored-by: KN4CK3R --- tests/integration/api_team_test.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/integration/api_team_test.go b/tests/integration/api_team_test.go index 46207203b..f54c286fc 100644 --- a/tests/integration/api_team_test.go +++ b/tests/integration/api_team_test.go @@ -24,7 +24,7 @@ import ( func TestAPITeam(t *testing.T) { defer tests.PrepareTestEnv(t)() - teamUser := unittest.AssertExistsAndLoadBean(t, &organization.TeamUser{}) + teamUser := unittest.AssertExistsAndLoadBean(t, &organization.TeamUser{ID: 1}) team := unittest.AssertExistsAndLoadBean(t, &organization.Team{ID: teamUser.TeamID}) user := unittest.AssertExistsAndLoadBean(t, &user_model.User{ID: teamUser.UID})