diff --git a/templates/org/team/new.tmpl b/templates/org/team/new.tmpl index 5c29b9a293..9f1db98227 100644 --- a/templates/org/team/new.tmpl +++ b/templates/org/team/new.tmpl @@ -52,7 +52,7 @@ {{ctx.Locale.Tr "org.teams.admin_access_helper"}} diff --git a/tests/integration/org_test.go b/tests/integration/org_test.go index 4035818e1a..cc1664d93a 100644 --- a/tests/integration/org_test.go +++ b/tests/integration/org_test.go @@ -297,3 +297,39 @@ func TestOrgNewMigrationButton(t *testing.T) { htmlDoc.AssertElement(t, migrateSelector, true) }) } + +func TestTeamWithoutPermissionToShowTable(t *testing.T) { + defer tests.PrepareTestEnv(t)() + + user := unittest.AssertExistsAndLoadBean(t, &user_model.User{ID: 2}) + org := unittest.AssertExistsAndLoadBean(t, &user_model.User{ID: 3, Type: user_model.UserTypeOrganization}) + team := unittest.AssertExistsAndLoadBean(t, &organization.Team{ID: 2}) + session := loginUser(t, user.Name) + + // set all units to "No access" + req := NewRequestWithValues(t, "POST", fmt.Sprintf("/org/%s/teams/%s/edit", org.Name, team.Name), map[string]string{ + "_csrf": GetCSRF(t, session, fmt.Sprintf("/org/%s/teams/%s/edit", org.Name, team.Name)), + "team_name": team.Name, + "description": "", + "repo_access": "all", + "permission": "read", + "unit_1": "0", + "unit_2": "0", + "unit_3": "0", + "unit_4": "0", + "unit_5": "0", + "unit_8": "0", + "unit_9": "0", + "unit_10": "0", + }) + session.MakeRequest(t, req, http.StatusSeeOther) + + req = NewRequestWithValues(t, "GET", fmt.Sprintf("/org/%s/teams/%s/edit", org.Name, team.Name), map[string]string{ + "_csrf": GetCSRF(t, session, fmt.Sprintf("/org/%s/teams/%s/edit", org.Name, team.Name)), + }) + resp := session.MakeRequest(t, req, http.StatusOK) + htmlDoc := NewHTMLParser(t, resp.Body) + + _, checked := htmlDoc.Find(`input[name="permission"][value="read"]`).Attr("checked") + assert.True(t, checked) +}