mix/selfpost
1
0
Fork 0
Code Issues Pull Requests Actions 3 Packages Projects Releases Wiki Activity

panel: pair domain detail cards like Status (SPF | DMARC)
test / test (push) Has been cancelled
Details

Browse Source 
Co-authored-by: Cursor <cursoragent@cursor.com>
This commit is contained in:
Микс
2026-08-12 15:33:41 +03:00
parent 1d30605533
commit 44e79c4120
3 changed files with 264 additions and 245 deletions
Download Patch File Download Diff File Expand all files Collapse all files
internal/web/view/templates_test.go
+27 -1
View File
@@ -264,7 +264,7 @@ func TestOnlyThePagesMadeOfDataDeclareThemselvesWide(t *testing.T) {
if err != nil {
t.Fatalf("New: %v", err)
}
wide := map[string]bool{"settings": true, "deliveries": true, "delivery": true, "mail_queue": true, "status": true, "system_log": true}
wide := map[string]bool{"settings": true, "deliveries": true, "delivery": true, "mail_queue": true, "status": true, "system_log": true, "domain_detail": true}
for name, page := range engine.Pages() {
var buf bytes.Buffer
if err := page.ExecuteTemplate(&buf, "wide", nil); err != nil {
@@ -280,6 +280,32 @@ func TestOnlyThePagesMadeOfDataDeclareThemselvesWide(t *testing.T) {
}
}
// The domain page pairs cards the same way Status does: four .split rows
// (DKIM|DNS, SPF|DMARC, settings|add-app, rate-limit|export) with Applications
// and Danger full-width. Losing a row silently stacks the page again.
func TestDomainDetailPageHasPairedCards(t *testing.T) {
body, err := fs.ReadFile(assetsFS, "templates/domain_detail.html")
if err != nil {
t.Fatalf("read domain_detail: %v", err)
}
src := string(body)
if got := strings.Count(src, `class="split"`); got != 4 {
t.Errorf("domain detail has %d .split rows, want 4", got)
}
for _, id := range []string{
`id="dkim"`, `id="dns-status"`, `id="spf"`, `id="dmarc"`,
`id="settings"`, `id="add-application"`, `id="applications"`,
`id="rate-limit"`, `id="export"`, `id="danger"`,
} {
if !strings.Contains(src, id) {
t.Errorf("domain detail is missing %s", id)
}
}
if strings.Contains(src, `id="spf-dmarc"`) {
t.Error("domain detail still has the combined spf-dmarc card; SPF and DMARC are separate")
}
}
// Drill-down pages carry an up-link directly under the heading and above the
// cards. A link at the bottom of a form is easy to miss and drifts from the
// rest of the panel, so the shared back_link template is mandatory on those