aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
-rw-r--r--templates/admin/emails/list.tmpl4
-rw-r--r--templates/admin/packages/list.tmpl4
-rw-r--r--templates/admin/user/list.tmpl2
-rw-r--r--tests/integration/auth_ldap_test.go45
-rw-r--r--web_src/css/admin.css4
-rw-r--r--web_src/css/helpers.css1
6 files changed, 16 insertions, 44 deletions
diff --git a/templates/admin/emails/list.tmpl b/templates/admin/emails/list.tmpl
index 67e9cb05e8..153877174b 100644
--- a/templates/admin/emails/list.tmpl
+++ b/templates/admin/emails/list.tmpl
@@ -47,8 +47,8 @@
{{range .Emails}}
<tr>
<td><a href="{{AppSubUrl}}/{{.Name | PathEscape}}">{{.Name}}</a></td>
- <td><span class="text truncate">{{.FullName}}</span></td>
- <td><span class="text email">{{.Email}}</span></td>
+ <td class="gt-ellipsis gt-max-width-12rem">{{.FullName}}</td>
+ <td class="gt-ellipsis gt-max-width-12rem">{{.Email}}</td>
<td>{{if .IsPrimary}}{{svg "octicon-check"}}{{else}}{{svg "octicon-x"}}{{end}}</td>
<td>
{{if .CanChange}}
diff --git a/templates/admin/packages/list.tmpl b/templates/admin/packages/list.tmpl
index a0b6fd9bb0..9aa1d933f6 100644
--- a/templates/admin/packages/list.tmpl
+++ b/templates/admin/packages/list.tmpl
@@ -56,8 +56,8 @@
{{end}}
</td>
<td>{{.Package.Type.Name}}</td>
- <td class="text truncate email">{{.Package.Name}}</td>
- <td><a href="{{.FullWebLink}}" class="text truncate email">{{.Version.Version}}</a></td>
+ <td class="gt-ellipsis gt-max-width-12rem">{{.Package.Name}}</td>
+ <td class="gt-ellipsis gt-max-width-12rem"><a href="{{.FullWebLink}}">{{.Version.Version}}</a></td>
<td><a href="{{.Creator.HomeLink}}">{{.Creator.Name}}</a></td>
<td>
{{if .Repository}}
diff --git a/templates/admin/user/list.tmpl b/templates/admin/user/list.tmpl
index 3742b4ae85..45fed87a40 100644
--- a/templates/admin/user/list.tmpl
+++ b/templates/admin/user/list.tmpl
@@ -85,7 +85,7 @@
<tr>
<td>{{.ID}}</td>
<td><a href="{{.HomeLink}}">{{.Name}}</a></td>
- <td><span class="text truncate email">{{.Email}}</span></td>
+ <td class="gt-ellipsis gt-max-width-12rem">{{.Email}}</td>
<td>{{if .IsActive}}{{svg "octicon-check"}}{{else}}{{svg "octicon-x"}}{{end}}</td>
<td>{{if .IsAdmin}}{{svg "octicon-check"}}{{else}}{{svg "octicon-x"}}{{end}}</td>
<td>{{if .IsRestricted}}{{svg "octicon-check"}}{{else}}{{svg "octicon-x"}}{{end}}</td>
diff --git a/tests/integration/auth_ldap_test.go b/tests/integration/auth_ldap_test.go
index d2160a3dbe..25395f5721 100644
--- a/tests/integration/auth_ldap_test.go
+++ b/tests/integration/auth_ldap_test.go
@@ -226,45 +226,20 @@ func TestLDAPUserSync(t *testing.T) {
addAuthSourceLDAP(t, "", "")
auth.SyncExternalUsers(context.Background(), true)
- session := loginUser(t, "user1")
// Check if users exists
- for _, u := range gitLDAPUsers {
- req := NewRequest(t, "GET", "/admin/users?q="+u.UserName)
- resp := session.MakeRequest(t, req, http.StatusOK)
-
- htmlDoc := NewHTMLParser(t, resp.Body)
-
- tr := htmlDoc.doc.Find("table.table tbody tr")
- if !assert.True(t, tr.Length() == 1) {
- continue
- }
- tds := tr.Find("td")
- if !assert.True(t, tds.Length() > 0) {
- continue
- }
- assert.Equal(t, u.UserName, strings.TrimSpace(tds.Find("td:nth-child(2) a").Text()))
- assert.Equal(t, u.Email, strings.TrimSpace(tds.Find("td:nth-child(3) span").Text()))
- if u.IsAdmin {
- assert.True(t, tds.Find("td:nth-child(5) svg").HasClass("octicon-check"))
- } else {
- assert.True(t, tds.Find("td:nth-child(5) svg").HasClass("octicon-x"))
- }
- if u.IsRestricted {
- assert.True(t, tds.Find("td:nth-child(6) svg").HasClass("octicon-check"))
- } else {
- assert.True(t, tds.Find("td:nth-child(6) svg").HasClass("octicon-x"))
- }
+ for _, gitLDAPUser := range gitLDAPUsers {
+ dbUser, err := user_model.GetUserByName(db.DefaultContext, gitLDAPUser.UserName)
+ assert.NoError(t, err)
+ assert.Equal(t, gitLDAPUser.UserName, dbUser.Name)
+ assert.Equal(t, gitLDAPUser.Email, dbUser.Email)
+ assert.Equal(t, gitLDAPUser.IsAdmin, dbUser.IsAdmin)
+ assert.Equal(t, gitLDAPUser.IsRestricted, dbUser.IsRestricted)
}
// Check if no users exist
- for _, u := range otherLDAPUsers {
- req := NewRequest(t, "GET", "/admin/users?q="+u.UserName)
- resp := session.MakeRequest(t, req, http.StatusOK)
-
- htmlDoc := NewHTMLParser(t, resp.Body)
-
- tr := htmlDoc.doc.Find("table.table tbody tr")
- assert.True(t, tr.Length() == 0)
+ for _, otherLDAPUser := range otherLDAPUsers {
+ _, err := user_model.GetUserByName(db.DefaultContext, otherLDAPUser.UserName)
+ assert.True(t, user_model.IsErrUserNotExist(err))
}
}
diff --git a/web_src/css/admin.css b/web_src/css/admin.css
index 8c9d1e201c..ec3cfbf22a 100644
--- a/web_src/css/admin.css
+++ b/web_src/css/admin.css
@@ -26,10 +26,6 @@
margin: 12px -1rem -1rem;
}
-.admin.user table.table .email {
- max-width: 200px;
-}
-
.admin dl.admin-dl-horizontal {
padding: 1em;
margin: 0;
diff --git a/web_src/css/helpers.css b/web_src/css/helpers.css
index 73f2ebe79b..8a6c2c6539 100644
--- a/web_src/css/helpers.css
+++ b/web_src/css/helpers.css
@@ -52,6 +52,7 @@ Gitea's private styles use `g-` prefix.
text-overflow: ellipsis;
}
+.gt-max-width-12rem { max-width: 12rem !important; }
.gt-max-width-24rem { max-width: 24rem !important; }
/* below class names match Tailwind CSS */