projax/web/system_views_test.go

package web_test

import (
	"context"
	"strings"
	"testing"
	"time"

	"github.com/m/projax/web"
)

// TestSystemViewLookup verifies the code-resident lookup returns the
// expected slugs in display order, and that LookupSystemView round-trips
// each entry.
func TestSystemViewLookup(t *testing.T) {
	all := web.AllSystemViews()
	wantSlugs := []string{"tree", "dashboard", "calendar", "timeline", "graph"}
	if len(all) != len(wantSlugs) {
		t.Fatalf("AllSystemViews len = %d, want %d", len(all), len(wantSlugs))
	}
	for i, sv := range all {
		if sv.Slug != wantSlugs[i] {
			t.Errorf("position %d: slug = %q, want %q", i, sv.Slug, wantSlugs[i])
		}
		if sv.URL != "/views/"+sv.Slug {
			t.Errorf("position %d: URL = %q, want /views/%s", i, sv.URL, sv.Slug)
		}
		round := web.LookupSystemView(sv.Slug)
		if round == nil || round.Slug != sv.Slug {
			t.Errorf("LookupSystemView(%q) round-trip failed", sv.Slug)
		}
	}
	if web.LookupSystemView("not-a-system-slug") != nil {
		t.Error("LookupSystemView should return nil for unknown slugs")
	}
}

// TestLegacyRedirects verifies the slice C URL migration: each legacy
// route 301-redirects to its /views/{slug} counterpart with chip params
// preserved.
func TestLegacyRedirects(t *testing.T) {
	srv, pool := mustServer(t)
	defer pool.Close()
	h := srv.Routes()
	cases := []struct {
		path, want string
	}{
		{"/", "/views/tree"},
		{"/dashboard", "/views/dashboard"},
		{"/calendar", "/views/calendar"},
		{"/timeline", "/views/timeline"},
		{"/graph", "/views/graph"},
		// chip params survive the redirect:
		{"/dashboard?tag=work", "/views/dashboard?tag=work"},
		{"/timeline?from=2026-05-01", "/views/timeline?from=2026-05-01"},
	}
	for _, tc := range cases {
		code, body := get(t, h, tc.path)
		if code != 301 {
			t.Errorf("GET %s status=%d body=%q, want 301", tc.path, code, body)
		}
		if !strings.Contains(body, `href="`+tc.want+`"`) {
			t.Errorf("GET %s body=%q, want redirect to %q", tc.path, body, tc.want)
		}
	}
}

// TestLegacyViewUUIDRedirect — when a legacy URL carries the 5i overlay
// `?view=<uuid>` param, the redirect resolves the uuid to the current
// slug (per m's Q3 pick), so old bookmarks land on the right user view.
func TestLegacyViewUUIDRedirect(t *testing.T) {
	srv, pool := mustServer(t)
	defer pool.Close()
	h := srv.Routes()
	ctx := context.Background()
	stamp := strings.ReplaceAll(time.Now().UTC().Format("150405.000"), ".", "")
	slug := "p5j-c-legacy-" + stamp
	defer pool.Exec(context.Background(), `DELETE FROM projax.views WHERE slug = $1`, slug)
	var id string
	if err := pool.QueryRow(ctx, `
INSERT INTO projax.views (slug, name, filter_json)
VALUES ($1, 'Legacy', '{"view_type":"list"}'::jsonb)
RETURNING id`, slug).Scan(&id); err != nil {
		t.Fatalf("seed view: %v", err)
	}
	// Old-style URL: /?view=<uuid>
	code, body := get(t, h, "/?view="+id)
	if code != 301 {
		t.Fatalf("GET /?view=<uuid> status=%d body=%q want 301", code, body)
	}
	if !strings.Contains(body, "/views/"+slug) {
		t.Errorf("redirect should resolve uuid → slug; got body=%q", body)
	}
}