Compare commits
4 Commits
| Author | SHA1 | Date | |
|---|---|---|---|
| 85ad2d65ee | |||
| f75ae60e74 | |||
| 4740687b96 | |||
| 1cb3fc3d14 |
@@ -0,0 +1,32 @@
|
|||||||
|
name: CI
|
||||||
|
|
||||||
|
# The release workflow gates a tag, which is late: a broken commit sits green
|
||||||
|
# until somebody decides to publish. This runs the same checks on the way in.
|
||||||
|
#
|
||||||
|
# push is scoped to main so that a branch pushed as part of a pull request is
|
||||||
|
# not checked twice.
|
||||||
|
on:
|
||||||
|
push:
|
||||||
|
branches: [main]
|
||||||
|
pull_request:
|
||||||
|
|
||||||
|
# A rapid series of pushes only needs the last one checked.
|
||||||
|
concurrency:
|
||||||
|
group: ci-${{ github.ref }}
|
||||||
|
cancel-in-progress: true
|
||||||
|
|
||||||
|
jobs:
|
||||||
|
test:
|
||||||
|
runs-on: ubuntu-latest
|
||||||
|
steps:
|
||||||
|
- uses: actions/checkout@v4
|
||||||
|
|
||||||
|
- uses: actions/setup-go@v5
|
||||||
|
with:
|
||||||
|
go-version-file: go.mod
|
||||||
|
|
||||||
|
- name: Vet
|
||||||
|
run: go vet ./...
|
||||||
|
|
||||||
|
- name: Test
|
||||||
|
run: go test ./...
|
||||||
@@ -73,6 +73,7 @@ go build -ldflags="-X main.version=v0.1.0" -o terdut-tui .
|
|||||||
## Sections
|
## Sections
|
||||||
|
|
||||||
`Incidents` (the queue, and the default) · `Alerts` (raw read-only feed) ·
|
`Incidents` (the queue, and the default) · `Alerts` (raw read-only feed) ·
|
||||||
|
`Stats` (MTTA/MTTR and alert frequency charts) ·
|
||||||
`Archived` (archived incidents) · `Schedule` · `Users`
|
`Archived` (archived incidents) · `Schedule` · `Users`
|
||||||
|
|
||||||
## Development stages
|
## Development stages
|
||||||
|
|||||||
@@ -8,11 +8,11 @@ Written in Go using [Bubbletea](https://github.com/charmbracelet/bubbletea).
|
|||||||
|
|
||||||
- **Incident queue** — open incidents with severity, status, assignee and age, auto-refreshing
|
- **Incident queue** — open incidents with severity, status, assignee and age, auto-refreshing
|
||||||
- **Incident actions** — acknowledge, assign, snooze, note, resolve and archive
|
- **Incident actions** — acknowledge, assign, snooze, note, resolve and archive
|
||||||
- **Timeline** — the full history of an incident, system events and notes together
|
- **Timeline** — the full history of an incident, system events, pages and notes together
|
||||||
- **Alert feed** — the raw read-only alerts underneath, each linked to its incident
|
- **Alert feed** — the raw read-only alerts underneath, each linked to its incident
|
||||||
- **On-call schedule** — visual calendar of who is on duty, assign and remove entries
|
- **On-call schedule** — visual calendar of who is on duty, assign and remove entries
|
||||||
- **Statistics** — MTTA and MTTR, plus alert frequency by name, hour and day
|
- **Statistics** — MTTA and MTTR, plus alert frequency by name, hour and day
|
||||||
- **User management** — add and remove users, manage API keys
|
- **User management** — add and remove users, manage API keys, set each user's ntfy topic
|
||||||
|
|
||||||
> Requires terdut-server **v0.4.0 or later**. Earlier servers have no incidents API;
|
> Requires terdut-server **v0.4.0 or later**. Earlier servers have no incidents API;
|
||||||
> use terdut-tui v0.3.x with those.
|
> use terdut-tui v0.3.x with those.
|
||||||
@@ -37,6 +37,22 @@ Two behaviours worth knowing before you press a key:
|
|||||||
- **Snooze is the "not now" button.** It hides an incident from the default queue
|
- **Snooze is the "not now" button.** It hides an incident from the default queue
|
||||||
without closing it, and expires on its own.
|
without closing it, and expires on its own.
|
||||||
|
|
||||||
|
## Push notifications
|
||||||
|
|
||||||
|
When the server is configured for ntfy, an incident that opens pages whoever is
|
||||||
|
on call. Each user has their own topic, shown as a column in the Users section
|
||||||
|
and edited with `t`. A user with no topic falls back to the server's shared
|
||||||
|
fallback topic, which carries **no Acknowledge button** — the topic is shared, so
|
||||||
|
a button on it would let any subscriber acknowledge as somebody else.
|
||||||
|
|
||||||
|
Every delivery lands on the incident's timeline: `Notified <user> (triggered)`
|
||||||
|
when ntfy accepted the page, and `Notification to <user> failed` when it ran out
|
||||||
|
of retries. That second one is the one to look for when nobody's phone rang.
|
||||||
|
|
||||||
|
Editing topics needs terdut-server **v0.6.0 or later**; the timeline entries need
|
||||||
|
**v0.7.0 or later**. Against an older server the topic column stays empty and
|
||||||
|
editing one reports the server's 404.
|
||||||
|
|
||||||
## Installation
|
## Installation
|
||||||
|
|
||||||
Download the latest release binary for your platform from the [releases page](https://github.com/yeniklas/terdut-tui/releases), or build from source:
|
Download the latest release binary for your platform from the [releases page](https://github.com/yeniklas/terdut-tui/releases), or build from source:
|
||||||
@@ -78,9 +94,10 @@ Global:
|
|||||||
| `esc` | Go back |
|
| `esc` | Go back |
|
||||||
| `r` | Refresh |
|
| `r` | Refresh |
|
||||||
| `f` | Cycle filter |
|
| `f` | Cycle filter |
|
||||||
| `S` | Statistics |
|
|
||||||
| `q` | Quit |
|
| `q` | Quit |
|
||||||
|
|
||||||
|
The sections, in `tab` order: Incidents · Alerts · Stats · Archived · Schedule · Users.
|
||||||
|
|
||||||
Incidents section:
|
Incidents section:
|
||||||
|
|
||||||
| Key | Action |
|
| Key | Action |
|
||||||
@@ -108,6 +125,12 @@ Alerts section (read-only):
|
|||||||
| `f` | Cycle: firing → resolved → all → archived |
|
| `f` | Cycle: firing → resolved → all → archived |
|
||||||
| `i` | In detail: jump to the alert's incident |
|
| `i` | In detail: jump to the alert's incident |
|
||||||
|
|
||||||
|
Stats section:
|
||||||
|
|
||||||
|
| Key | Action |
|
||||||
|
|-----|--------|
|
||||||
|
| `j` / `k`, `pgup` / `pgdn` | Scroll |
|
||||||
|
|
||||||
Schedule section:
|
Schedule section:
|
||||||
|
|
||||||
| Key | Action |
|
| Key | Action |
|
||||||
@@ -121,5 +144,6 @@ Users section:
|
|||||||
| Key | Action |
|
| Key | Action |
|
||||||
|-----|--------|
|
|-----|--------|
|
||||||
| `n` | Create a user |
|
| `n` | Create a user |
|
||||||
|
| `t` | Edit the user's ntfy topic — submit empty to clear it |
|
||||||
| `d` | Delete a user |
|
| `d` | Delete a user |
|
||||||
| `k` | API keys for the selected user |
|
| `k` | API keys for the selected user |
|
||||||
|
|||||||
@@ -401,6 +401,23 @@ func (c *Client) CreateUser(username, email string) (*User, error) {
|
|||||||
return &user, c.do(req, &user)
|
return &user, c.do(req, &user)
|
||||||
}
|
}
|
||||||
|
|
||||||
|
// SetUserNotifyTarget points a user's push notifications at an ntfy topic.
|
||||||
|
//
|
||||||
|
// An empty topic clears it: the server stores NULL, and that user's incidents
|
||||||
|
// page the shared fallback topic instead — which carries no Acknowledge button,
|
||||||
|
// because anyone subscribed to it could otherwise acknowledge as somebody else.
|
||||||
|
func (c *Client) SetUserNotifyTarget(userID int64, topic string) (*User, error) {
|
||||||
|
body := struct {
|
||||||
|
NtfyTopic string `json:"ntfy_topic"`
|
||||||
|
}{NtfyTopic: topic}
|
||||||
|
req, err := c.newRequestWithBody(http.MethodPut, fmt.Sprintf("/api/users/%d/notify", userID), body)
|
||||||
|
if err != nil {
|
||||||
|
return nil, err
|
||||||
|
}
|
||||||
|
var user User
|
||||||
|
return &user, c.do(req, &user)
|
||||||
|
}
|
||||||
|
|
||||||
func (c *Client) DeleteUser(id int64) error {
|
func (c *Client) DeleteUser(id int64) error {
|
||||||
req, err := c.newRequest(http.MethodDelete, fmt.Sprintf("/api/users/%d", id))
|
req, err := c.newRequest(http.MethodDelete, fmt.Sprintf("/api/users/%d", id))
|
||||||
if err != nil {
|
if err != nil {
|
||||||
|
|||||||
@@ -83,6 +83,8 @@ func TestClient_IncidentEndpoints(t *testing.T) {
|
|||||||
http.MethodDelete, "/api/incidents/7/notes/12", ""},
|
http.MethodDelete, "/api/incidents/7/notes/12", ""},
|
||||||
{"stats", func(c *Client) error { _, err := c.GetIncidentStats(); return err },
|
{"stats", func(c *Client) error { _, err := c.GetIncidentStats(); return err },
|
||||||
http.MethodGet, "/api/stats/incidents", ""},
|
http.MethodGet, "/api/stats/incidents", ""},
|
||||||
|
{"set notify target", func(c *Client) error { _, err := c.SetUserNotifyTarget(7, "t"); return err },
|
||||||
|
http.MethodPut, "/api/users/7/notify", ""},
|
||||||
}
|
}
|
||||||
|
|
||||||
for _, tt := range tests {
|
for _, tt := range tests {
|
||||||
@@ -163,6 +165,43 @@ func TestClient_RequestBodies(t *testing.T) {
|
|||||||
t.Errorf("expected duration 90m, got %q", body.Duration)
|
t.Errorf("expected duration 90m, got %q", body.Duration)
|
||||||
}
|
}
|
||||||
})
|
})
|
||||||
|
|
||||||
|
t.Run("set notify target", func(t *testing.T) {
|
||||||
|
c, got := stub(t, http.StatusOK, `{}`)
|
||||||
|
if _, err := c.SetUserNotifyTarget(3, "terdut-niklas"); err != nil {
|
||||||
|
t.Fatalf("set notify target: %v", err)
|
||||||
|
}
|
||||||
|
if got.body != `{"ntfy_topic":"terdut-niklas"}` {
|
||||||
|
t.Errorf("unexpected body %q", got.body)
|
||||||
|
}
|
||||||
|
})
|
||||||
|
|
||||||
|
// Clearing has to put an explicit empty string on the wire: omitting the
|
||||||
|
// field would leave the topic untouched instead of removing it.
|
||||||
|
t.Run("clear notify target", func(t *testing.T) {
|
||||||
|
c, got := stub(t, http.StatusOK, `{}`)
|
||||||
|
if _, err := c.SetUserNotifyTarget(3, ""); err != nil {
|
||||||
|
t.Fatalf("clear notify target: %v", err)
|
||||||
|
}
|
||||||
|
if got.body != `{"ntfy_topic":""}` {
|
||||||
|
t.Errorf("expected an explicit empty topic, got %q", got.body)
|
||||||
|
}
|
||||||
|
})
|
||||||
|
}
|
||||||
|
|
||||||
|
func TestUser_TopicFlattensNilAndEmpty(t *testing.T) {
|
||||||
|
var users []User
|
||||||
|
if err := json.Unmarshal([]byte(
|
||||||
|
`[{"id":1,"username":"a"},{"id":2,"username":"b","ntfy_topic":""},
|
||||||
|
{"id":3,"username":"c","ntfy_topic":"terdut-c"}]`), &users); err != nil {
|
||||||
|
t.Fatalf("decode: %v", err)
|
||||||
|
}
|
||||||
|
want := []string{"", "", "terdut-c"}
|
||||||
|
for i, u := range users {
|
||||||
|
if got := u.Topic(); got != want[i] {
|
||||||
|
t.Errorf("user %d: expected topic %q, got %q", u.ID, want[i], got)
|
||||||
|
}
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
// The 409 on re-resolving is the server telling the user why nothing happened,
|
// The 409 on re-resolving is the server telling the user why nothing happened,
|
||||||
|
|||||||
@@ -95,6 +95,13 @@ const (
|
|||||||
EventUnsnoozed = "unsnoozed"
|
EventUnsnoozed = "unsnoozed"
|
||||||
EventResolved = "resolved"
|
EventResolved = "resolved"
|
||||||
EventNote = "note"
|
EventNote = "note"
|
||||||
|
|
||||||
|
// Written by the server's notifier from the delivery result, not at enqueue.
|
||||||
|
// Detail carries the notification kind ("triggered", "reminder", "resolved"),
|
||||||
|
// and on a failure the reason after it. An absent user means the page went to
|
||||||
|
// the shared fallback topic rather than to a person.
|
||||||
|
EventNotified = "notified"
|
||||||
|
EventNotifyFailed = "notify_failed"
|
||||||
)
|
)
|
||||||
|
|
||||||
// IncidentEvent is one entry in an incident's timeline. An empty Username means
|
// IncidentEvent is one entry in an incident's timeline. An empty Username means
|
||||||
@@ -158,6 +165,21 @@ type User struct {
|
|||||||
Username string `json:"username"`
|
Username string `json:"username"`
|
||||||
Email string `json:"email"`
|
Email string `json:"email"`
|
||||||
CreatedAt time.Time `json:"created_at"`
|
CreatedAt time.Time `json:"created_at"`
|
||||||
|
|
||||||
|
// NtfyTopic is where this user's push notifications go. Nil and empty mean
|
||||||
|
// the same thing — no topic of their own — because the server stores a blank
|
||||||
|
// string as NULL. Their incidents fall back to the server's shared fallback
|
||||||
|
// topic, which carries no Acknowledge button.
|
||||||
|
NtfyTopic *string `json:"ntfy_topic,omitempty"`
|
||||||
|
}
|
||||||
|
|
||||||
|
// Topic reads the user's ntfy topic, flattening the nil and empty cases the
|
||||||
|
// server treats alike.
|
||||||
|
func (u User) Topic() string {
|
||||||
|
if u.NtfyTopic == nil {
|
||||||
|
return ""
|
||||||
|
}
|
||||||
|
return *u.NtfyTopic
|
||||||
}
|
}
|
||||||
|
|
||||||
type APIKey struct {
|
type APIKey struct {
|
||||||
|
|||||||
+53
-10
@@ -21,11 +21,12 @@ const (
|
|||||||
// Incidents lead: they are the work. Alerts is the raw feed underneath.
|
// Incidents lead: they are the work. Alerts is the raw feed underneath.
|
||||||
sectionIncidents section = iota
|
sectionIncidents section = iota
|
||||||
sectionAlerts
|
sectionAlerts
|
||||||
|
sectionStats
|
||||||
sectionArchived
|
sectionArchived
|
||||||
sectionSchedule
|
sectionSchedule
|
||||||
sectionUsers
|
sectionUsers
|
||||||
|
|
||||||
sectionCount = 5
|
sectionCount = 6
|
||||||
)
|
)
|
||||||
|
|
||||||
type mode int
|
type mode int
|
||||||
@@ -37,9 +38,9 @@ const (
|
|||||||
modeNote
|
modeNote
|
||||||
modeSnooze
|
modeSnooze
|
||||||
modeConfirm
|
modeConfirm
|
||||||
modeStats
|
|
||||||
modeUserPicker
|
modeUserPicker
|
||||||
modeUserCreate
|
modeUserCreate
|
||||||
|
modeUserNotifyEdit
|
||||||
modeAPIKeyMenu
|
modeAPIKeyMenu
|
||||||
modeAPIKeyCreate
|
modeAPIKeyCreate
|
||||||
modeAPIKeyReveal
|
modeAPIKeyReveal
|
||||||
@@ -195,14 +196,14 @@ type Model struct {
|
|||||||
pendingDeleteEntry *api.ScheduleEntry
|
pendingDeleteEntry *api.ScheduleEntry
|
||||||
|
|
||||||
// Stats
|
// Stats
|
||||||
topAlerts []api.TopAlert
|
topAlerts []api.TopAlert
|
||||||
hourStats []api.HourStat
|
hourStats []api.HourStat
|
||||||
dayStats []api.DayStat
|
dayStats []api.DayStat
|
||||||
|
// statsLoaded tracks the first fetch separately from emptiness: a server with
|
||||||
|
// no alerts yet legitimately returns three empty slices.
|
||||||
|
statsLoaded bool
|
||||||
statsLoading bool
|
statsLoading bool
|
||||||
statsViewport viewport.Model
|
statsViewport viewport.Model
|
||||||
// statsReturnMode is where esc goes back to, since stats opens from both
|
|
||||||
// the dashboard and an incident.
|
|
||||||
statsReturnMode mode
|
|
||||||
|
|
||||||
// Schedule
|
// Schedule
|
||||||
scheduleWindow time.Time
|
scheduleWindow time.Time
|
||||||
@@ -224,6 +225,7 @@ type Model struct {
|
|||||||
selectedUser api.User
|
selectedUser api.User
|
||||||
userFormInputs [2]textinput.Model
|
userFormInputs [2]textinput.Model
|
||||||
userFormFocus int
|
userFormFocus int
|
||||||
|
ntfyTopicInput textinput.Model
|
||||||
apiKeyNameInput textinput.Model
|
apiKeyNameInput textinput.Model
|
||||||
apiKeyRevokeInput textinput.Model
|
apiKeyRevokeInput textinput.Model
|
||||||
revealedAPIKey api.APIKey
|
revealedAPIKey api.APIKey
|
||||||
@@ -253,6 +255,10 @@ func NewModel(client *api.Client, serverURL string, refreshInterval time.Duratio
|
|||||||
manageT := table.New(table.WithFocused(true))
|
manageT := table.New(table.WithFocused(true))
|
||||||
manageT.SetStyles(ts)
|
manageT.SetStyles(ts)
|
||||||
|
|
||||||
|
// Sized by the first tea.WindowSizeMsg; built here so it carries the default
|
||||||
|
// scroll keymap, which the zero value lacks.
|
||||||
|
statsVP := viewport.New(0, 0)
|
||||||
|
|
||||||
noteIn := textinput.New()
|
noteIn := textinput.New()
|
||||||
noteIn.Placeholder = "type your note…"
|
noteIn.Placeholder = "type your note…"
|
||||||
noteIn.CharLimit = 1000
|
noteIn.CharLimit = 1000
|
||||||
@@ -269,6 +275,10 @@ func NewModel(client *api.Client, serverURL string, refreshInterval time.Duratio
|
|||||||
emailIn.Placeholder = "email"
|
emailIn.Placeholder = "email"
|
||||||
emailIn.CharLimit = 128
|
emailIn.CharLimit = 128
|
||||||
|
|
||||||
|
topicIn := textinput.New()
|
||||||
|
topicIn.Placeholder = "ntfy topic — empty clears it"
|
||||||
|
topicIn.CharLimit = 128
|
||||||
|
|
||||||
keyNameIn := textinput.New()
|
keyNameIn := textinput.New()
|
||||||
keyNameIn.Placeholder = "key name (e.g. laptop)"
|
keyNameIn.Placeholder = "key name (e.g. laptop)"
|
||||||
keyNameIn.CharLimit = 64
|
keyNameIn.CharLimit = 64
|
||||||
@@ -298,6 +308,7 @@ func NewModel(client *api.Client, serverURL string, refreshInterval time.Duratio
|
|||||||
incidentTable: incidentT,
|
incidentTable: incidentT,
|
||||||
alertTable: alertT,
|
alertTable: alertT,
|
||||||
archivedTable: archivedT,
|
archivedTable: archivedT,
|
||||||
|
statsViewport: statsVP,
|
||||||
noteInput: noteIn,
|
noteInput: noteIn,
|
||||||
snoozeInput: snoozeIn,
|
snoozeInput: snoozeIn,
|
||||||
scheduleWindow: window,
|
scheduleWindow: window,
|
||||||
@@ -305,6 +316,7 @@ func NewModel(client *api.Client, serverURL string, refreshInterval time.Duratio
|
|||||||
userPickerTable: pickerT,
|
userPickerTable: pickerT,
|
||||||
userManageTable: manageT,
|
userManageTable: manageT,
|
||||||
userFormInputs: [2]textinput.Model{usernameIn, emailIn},
|
userFormInputs: [2]textinput.Model{usernameIn, emailIn},
|
||||||
|
ntfyTopicInput: topicIn,
|
||||||
apiKeyNameInput: keyNameIn,
|
apiKeyNameInput: keyNameIn,
|
||||||
apiKeyRevokeInput: revokeIn,
|
apiKeyRevokeInput: revokeIn,
|
||||||
help: help.New(),
|
help: help.New(),
|
||||||
@@ -366,7 +378,11 @@ func (m *Model) rebuildUserManageTable() {
|
|||||||
m.userManageTable.SetColumns(userManageColumns(m.width))
|
m.userManageTable.SetColumns(userManageColumns(m.width))
|
||||||
rows := make([]table.Row, len(m.users))
|
rows := make([]table.Row, len(m.users))
|
||||||
for i, u := range m.users {
|
for i, u := range m.users {
|
||||||
rows[i] = table.Row{u.Username, u.Email, u.CreatedAt.UTC().Format("2006-01-02")}
|
topic := u.Topic()
|
||||||
|
if topic == "" {
|
||||||
|
topic = "—"
|
||||||
|
}
|
||||||
|
rows[i] = table.Row{u.Username, u.Email, topic, u.CreatedAt.UTC().Format("2006-01-02")}
|
||||||
}
|
}
|
||||||
m.userManageTable.SetRows(rows)
|
m.userManageTable.SetRows(rows)
|
||||||
m.userManageTable.SetHeight(tableHeight(m.height, 10))
|
m.userManageTable.SetHeight(tableHeight(m.height, 10))
|
||||||
@@ -397,6 +413,14 @@ func (m *Model) refreshStatsContent() {
|
|||||||
buildStatsContent(m.incidentStats, m.topAlerts, m.hourStats, m.dayStats, m.width))
|
buildStatsContent(m.incidentStats, m.topAlerts, m.hourStats, m.dayStats, m.width))
|
||||||
}
|
}
|
||||||
|
|
||||||
|
func (m Model) statsViewportHeight() int {
|
||||||
|
h := m.height - 5
|
||||||
|
if h < 1 {
|
||||||
|
h = 1
|
||||||
|
}
|
||||||
|
return h
|
||||||
|
}
|
||||||
|
|
||||||
func (m Model) detailViewportHeight() int {
|
func (m Model) detailViewportHeight() int {
|
||||||
h := m.height - 5
|
h := m.height - 5
|
||||||
if m.mode == modeNote || m.mode == modeSnooze {
|
if m.mode == modeNote || m.mode == modeSnooze {
|
||||||
@@ -489,13 +513,16 @@ func userPickerColumns(width int) []table.Column {
|
|||||||
func userManageColumns(width int) []table.Column {
|
func userManageColumns(width int) []table.Column {
|
||||||
createdW := 12
|
createdW := 12
|
||||||
usernameW := 25
|
usernameW := 25
|
||||||
emailW := width - usernameW - createdW - 8
|
topicW := 22
|
||||||
|
// 8 = bubbles' Padding(0, 1) on each of the four cells.
|
||||||
|
emailW := width - usernameW - topicW - createdW - 8
|
||||||
if emailW < 15 {
|
if emailW < 15 {
|
||||||
emailW = 15
|
emailW = 15
|
||||||
}
|
}
|
||||||
return []table.Column{
|
return []table.Column{
|
||||||
{Title: "Username", Width: usernameW},
|
{Title: "Username", Width: usernameW},
|
||||||
{Title: "Email", Width: emailW},
|
{Title: "Email", Width: emailW},
|
||||||
|
{Title: "Ntfy Topic", Width: topicW},
|
||||||
{Title: "Created", Width: createdW},
|
{Title: "Created", Width: createdW},
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
@@ -896,6 +923,22 @@ func createUserCmd(client *api.Client, username, email string) tea.Cmd {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
// setUserNotifyTargetCmd points a user's pages at a topic, or clears it when
|
||||||
|
// topic is empty. It re-lists afterwards so the table shows what the server
|
||||||
|
// stored rather than what was typed.
|
||||||
|
func setUserNotifyTargetCmd(client *api.Client, userID int64, topic string) tea.Cmd {
|
||||||
|
return func() tea.Msg {
|
||||||
|
if _, err := client.SetUserNotifyTarget(userID, topic); err != nil {
|
||||||
|
return userActionErrMsg{err}
|
||||||
|
}
|
||||||
|
users, err := client.ListUsers()
|
||||||
|
if err != nil {
|
||||||
|
return userActionErrMsg{err}
|
||||||
|
}
|
||||||
|
return usersFetchedMsg{users: users}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
func deleteUserCmd(client *api.Client, userID int64) tea.Cmd {
|
func deleteUserCmd(client *api.Client, userID int64) tea.Cmd {
|
||||||
return func() tea.Msg {
|
return func() tea.Msg {
|
||||||
if err := client.DeleteUser(userID); err != nil {
|
if err := client.DeleteUser(userID); err != nil {
|
||||||
|
|||||||
@@ -172,6 +172,32 @@ func TestAlertRows_ShowIncidentLink(t *testing.T) {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
// A user with no ntfy topic gets no pages of their own — the row has to say so
|
||||||
|
// rather than leaving a blank that reads as "not loaded yet".
|
||||||
|
func TestUserManageRows_ShowMissingTopic(t *testing.T) {
|
||||||
|
topic := "terdut-niklas"
|
||||||
|
empty := ""
|
||||||
|
m := NewModel(nil, "http://test", time.Minute)
|
||||||
|
m.width, m.height = 120, 40
|
||||||
|
m.users = []api.User{
|
||||||
|
{ID: 1, Username: "niklas", NtfyTopic: &topic},
|
||||||
|
{ID: 2, Username: "alex"},
|
||||||
|
// The server stores a blank topic as NULL, but a stale client or an older
|
||||||
|
// server can still hand one back; it means the same thing.
|
||||||
|
{ID: 3, Username: "sam", NtfyTopic: &empty},
|
||||||
|
}
|
||||||
|
m.rebuildUserManageTable()
|
||||||
|
|
||||||
|
rows := m.userManageTable.Rows()
|
||||||
|
if rows[0][2] != "terdut-niklas" {
|
||||||
|
t.Errorf("expected the topic in the row, got %q", rows[0][2])
|
||||||
|
}
|
||||||
|
if rows[1][2] != "—" || rows[2][2] != "—" {
|
||||||
|
t.Errorf("expected an em dash for nil and empty topics, got %q and %q",
|
||||||
|
rows[1][2], rows[2][2])
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
// A previous release overflowed the terminal by two columns because the padding
|
// A previous release overflowed the terminal by two columns because the padding
|
||||||
// budget was wrong. Columns plus bubbles' per-cell padding must land exactly on
|
// budget was wrong. Columns plus bubbles' per-cell padding must land exactly on
|
||||||
// the window width.
|
// the window width.
|
||||||
@@ -191,6 +217,17 @@ func TestColumnWidthsFitTheTerminal(t *testing.T) {
|
|||||||
name, width, sum, padding, sum+padding)
|
name, width, sum, padding, sum+padding)
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
// The users table is four cells, so its padding budget differs.
|
||||||
|
sum := 0
|
||||||
|
for _, w := range widths(userManageColumns(width)) {
|
||||||
|
sum += w
|
||||||
|
}
|
||||||
|
const userPadding = 8
|
||||||
|
if sum+userPadding != width {
|
||||||
|
t.Errorf("user columns at width %d sum to %d+%d = %d",
|
||||||
|
width, sum, userPadding, sum+userPadding)
|
||||||
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -198,7 +235,9 @@ func TestColumnWidthsFitTheTerminal(t *testing.T) {
|
|||||||
// what must not happen is a negative or zero column.
|
// what must not happen is a negative or zero column.
|
||||||
func TestColumnWidthsStayPositiveWhenNarrow(t *testing.T) {
|
func TestColumnWidthsStayPositiveWhenNarrow(t *testing.T) {
|
||||||
for _, width := range []int{20, 40, 60} {
|
for _, width := range []int{20, 40, 60} {
|
||||||
for _, w := range append(widths(incidentColumns(width)), widths(alertColumns(width))...) {
|
cols := append(widths(incidentColumns(width)), widths(alertColumns(width))...)
|
||||||
|
cols = append(cols, widths(userManageColumns(width))...)
|
||||||
|
for _, w := range cols {
|
||||||
if w < 1 {
|
if w < 1 {
|
||||||
t.Errorf("width %d produced a non-positive column %d", width, w)
|
t.Errorf("width %d produced a non-positive column %d", width, w)
|
||||||
}
|
}
|
||||||
|
|||||||
+65
-41
@@ -24,7 +24,7 @@ func (m Model) Update(msg tea.Msg) (tea.Model, tea.Cmd) {
|
|||||||
m.detailViewport.Width = m.width
|
m.detailViewport.Width = m.width
|
||||||
m.detailViewport.Height = m.detailViewportHeight()
|
m.detailViewport.Height = m.detailViewportHeight()
|
||||||
m.statsViewport.Width = m.width
|
m.statsViewport.Width = m.width
|
||||||
m.statsViewport.Height = m.height - 5
|
m.statsViewport.Height = m.statsViewportHeight()
|
||||||
m.refreshDetailContent()
|
m.refreshDetailContent()
|
||||||
m.refreshStatsContent()
|
m.refreshStatsContent()
|
||||||
return m, nil
|
return m, nil
|
||||||
@@ -118,13 +118,16 @@ func (m Model) Update(msg tea.Msg) (tea.Model, tea.Cmd) {
|
|||||||
m.hourStats = msg.byHour
|
m.hourStats = msg.byHour
|
||||||
m.dayStats = msg.byDay
|
m.dayStats = msg.byDay
|
||||||
m.statsLoading = false
|
m.statsLoading = false
|
||||||
|
m.statsLoaded = true
|
||||||
m.refreshStatsContent()
|
m.refreshStatsContent()
|
||||||
return m, nil
|
return m, nil
|
||||||
|
|
||||||
case detailStatsErrMsg:
|
case detailStatsErrMsg:
|
||||||
m.statsLoading = false
|
m.statsLoading = false
|
||||||
|
// Mark it loaded even on failure, so tabbing back in does not re-fire the
|
||||||
|
// request every time. The tick and r still retry.
|
||||||
|
m.statsLoaded = true
|
||||||
m.statusMsg = "stats error: " + msg.err.Error()
|
m.statusMsg = "stats error: " + msg.err.Error()
|
||||||
m.mode = modeDashboard
|
|
||||||
return m, clearStatusCmd()
|
return m, clearStatusCmd()
|
||||||
|
|
||||||
// ── Schedule messages ─────────────────────────────────────────────────
|
// ── Schedule messages ─────────────────────────────────────────────────
|
||||||
@@ -205,6 +208,9 @@ func (m Model) refreshActiveSection() tea.Cmd {
|
|||||||
return tea.Batch(fetchIncidentsCmd(m.client, m.incidentFilter), fetchStatsCmd(m.client))
|
return tea.Batch(fetchIncidentsCmd(m.client, m.incidentFilter), fetchStatsCmd(m.client))
|
||||||
case sectionAlerts:
|
case sectionAlerts:
|
||||||
return tea.Batch(fetchAlertsCmd(m.client, m.alertFilter), fetchStatsCmd(m.client))
|
return tea.Batch(fetchAlertsCmd(m.client, m.alertFilter), fetchStatsCmd(m.client))
|
||||||
|
case sectionStats:
|
||||||
|
// Both: fetchStatsCmd feeds the Incident Response block, the other the charts.
|
||||||
|
return tea.Batch(fetchStatsCmd(m.client), fetchDetailStatsCmd(m.client))
|
||||||
case sectionArchived:
|
case sectionArchived:
|
||||||
return fetchArchivedIncidentsCmd(m.client)
|
return fetchArchivedIncidentsCmd(m.client)
|
||||||
case sectionSchedule:
|
case sectionSchedule:
|
||||||
@@ -240,12 +246,6 @@ func (m Model) routeKey(msg tea.KeyMsg) (Model, tea.Cmd) {
|
|||||||
m2, ourCmd := m.handleKey(msg)
|
m2, ourCmd := m.handleKey(msg)
|
||||||
return m2, tea.Batch(inputCmd, ourCmd)
|
return m2, tea.Batch(inputCmd, ourCmd)
|
||||||
|
|
||||||
case modeStats:
|
|
||||||
var vpCmd tea.Cmd
|
|
||||||
m.statsViewport, vpCmd = m.statsViewport.Update(msg)
|
|
||||||
m2, ourCmd := m.handleKey(msg)
|
|
||||||
return m2, tea.Batch(vpCmd, ourCmd)
|
|
||||||
|
|
||||||
case modeUserPicker:
|
case modeUserPicker:
|
||||||
var tableCmd tea.Cmd
|
var tableCmd tea.Cmd
|
||||||
m.userPickerTable, tableCmd = m.userPickerTable.Update(msg)
|
m.userPickerTable, tableCmd = m.userPickerTable.Update(msg)
|
||||||
@@ -258,6 +258,12 @@ func (m Model) routeKey(msg tea.KeyMsg) (Model, tea.Cmd) {
|
|||||||
m2, ourCmd := m.handleKey(msg)
|
m2, ourCmd := m.handleKey(msg)
|
||||||
return m2, tea.Batch(inputCmd, ourCmd)
|
return m2, tea.Batch(inputCmd, ourCmd)
|
||||||
|
|
||||||
|
case modeUserNotifyEdit:
|
||||||
|
var inputCmd tea.Cmd
|
||||||
|
m.ntfyTopicInput, inputCmd = m.ntfyTopicInput.Update(msg)
|
||||||
|
m2, ourCmd := m.handleKey(msg)
|
||||||
|
return m2, tea.Batch(inputCmd, ourCmd)
|
||||||
|
|
||||||
case modeAPIKeyCreate:
|
case modeAPIKeyCreate:
|
||||||
var inputCmd tea.Cmd
|
var inputCmd tea.Cmd
|
||||||
m.apiKeyNameInput, inputCmd = m.apiKeyNameInput.Update(msg)
|
m.apiKeyNameInput, inputCmd = m.apiKeyNameInput.Update(msg)
|
||||||
@@ -286,6 +292,11 @@ func (m Model) routeKey(msg tea.KeyMsg) (Model, tea.Cmd) {
|
|||||||
m.alertTable, tableCmd = m.alertTable.Update(msg)
|
m.alertTable, tableCmd = m.alertTable.Update(msg)
|
||||||
m2, ourCmd := m.handleKey(msg)
|
m2, ourCmd := m.handleKey(msg)
|
||||||
return m2, tea.Batch(tableCmd, ourCmd)
|
return m2, tea.Batch(tableCmd, ourCmd)
|
||||||
|
case sectionStats:
|
||||||
|
var vpCmd tea.Cmd
|
||||||
|
m.statsViewport, vpCmd = m.statsViewport.Update(msg)
|
||||||
|
m2, ourCmd := m.handleKey(msg)
|
||||||
|
return m2, tea.Batch(vpCmd, ourCmd)
|
||||||
case sectionArchived:
|
case sectionArchived:
|
||||||
var tableCmd tea.Cmd
|
var tableCmd tea.Cmd
|
||||||
m.archivedTable, tableCmd = m.archivedTable.Update(msg)
|
m.archivedTable, tableCmd = m.archivedTable.Update(msg)
|
||||||
@@ -319,12 +330,12 @@ func (m Model) handleKey(msg tea.KeyMsg) (Model, tea.Cmd) {
|
|||||||
return m.handleSnoozeKey(msg)
|
return m.handleSnoozeKey(msg)
|
||||||
case modeConfirm:
|
case modeConfirm:
|
||||||
return m.handleConfirmKey(msg)
|
return m.handleConfirmKey(msg)
|
||||||
case modeStats:
|
|
||||||
return m.handleStatsKey(msg)
|
|
||||||
case modeUserPicker:
|
case modeUserPicker:
|
||||||
return m.handleUserPickerKey(msg)
|
return m.handleUserPickerKey(msg)
|
||||||
case modeUserCreate:
|
case modeUserCreate:
|
||||||
return m.handleUserCreateKey(msg)
|
return m.handleUserCreateKey(msg)
|
||||||
|
case modeUserNotifyEdit:
|
||||||
|
return m.handleUserNotifyEditKey(msg)
|
||||||
case modeAPIKeyMenu:
|
case modeAPIKeyMenu:
|
||||||
return m.handleAPIKeyMenuKey(msg)
|
return m.handleAPIKeyMenuKey(msg)
|
||||||
case modeAPIKeyCreate:
|
case modeAPIKeyCreate:
|
||||||
@@ -482,12 +493,6 @@ func (m Model) handleDashboardKey(msg tea.KeyMsg) (Model, tea.Cmd) {
|
|||||||
}
|
}
|
||||||
return m, nil
|
return m, nil
|
||||||
|
|
||||||
case "S":
|
|
||||||
if !m.connected {
|
|
||||||
return m, nil
|
|
||||||
}
|
|
||||||
return m.openStats()
|
|
||||||
|
|
||||||
case "n":
|
case "n":
|
||||||
if m.activeSection != sectionUsers || !m.connected {
|
if m.activeSection != sectionUsers || !m.connected {
|
||||||
return m, nil
|
return m, nil
|
||||||
@@ -500,6 +505,23 @@ func (m Model) handleDashboardKey(msg tea.KeyMsg) (Model, tea.Cmd) {
|
|||||||
m.mode = modeUserCreate
|
m.mode = modeUserCreate
|
||||||
return m, nil
|
return m, nil
|
||||||
|
|
||||||
|
case "t":
|
||||||
|
if m.activeSection != sectionUsers || !m.connected || len(m.users) == 0 {
|
||||||
|
return m, nil
|
||||||
|
}
|
||||||
|
cursor := m.userManageTable.Cursor()
|
||||||
|
if cursor >= len(m.users) {
|
||||||
|
return m, nil
|
||||||
|
}
|
||||||
|
m.selectedUser = m.users[cursor]
|
||||||
|
// Prefilled with what they have, so editing a topic does not mean
|
||||||
|
// retyping it, and clearing one is a deliberate wipe.
|
||||||
|
m.ntfyTopicInput.SetValue(m.selectedUser.Topic())
|
||||||
|
m.ntfyTopicInput.CursorEnd()
|
||||||
|
m.ntfyTopicInput.Focus()
|
||||||
|
m.mode = modeUserNotifyEdit
|
||||||
|
return m, nil
|
||||||
|
|
||||||
case "k":
|
case "k":
|
||||||
if m.activeSection != sectionUsers || !m.connected || len(m.users) == 0 {
|
if m.activeSection != sectionUsers || !m.connected || len(m.users) == 0 {
|
||||||
return m, nil
|
return m, nil
|
||||||
@@ -524,6 +546,11 @@ func (m *Model) loadSectionIfEmpty() tea.Cmd {
|
|||||||
m.loading = true
|
m.loading = true
|
||||||
return fetchAlertsCmd(m.client, m.alertFilter)
|
return fetchAlertsCmd(m.client, m.alertFilter)
|
||||||
}
|
}
|
||||||
|
case sectionStats:
|
||||||
|
if !m.statsLoaded {
|
||||||
|
m.statsLoading = true
|
||||||
|
return tea.Batch(fetchStatsCmd(m.client), fetchDetailStatsCmd(m.client))
|
||||||
|
}
|
||||||
case sectionArchived:
|
case sectionArchived:
|
||||||
if len(m.archivedIncidents) == 0 {
|
if len(m.archivedIncidents) == 0 {
|
||||||
m.archivedLoading = true
|
m.archivedLoading = true
|
||||||
@@ -677,9 +704,6 @@ func (m Model) handleIncidentDetailKey(msg tea.KeyMsg) (Model, tea.Cmd) {
|
|||||||
m.mode = modeConfirm
|
m.mode = modeConfirm
|
||||||
return m, nil
|
return m, nil
|
||||||
|
|
||||||
case "S":
|
|
||||||
return m.openStats()
|
|
||||||
|
|
||||||
case "[":
|
case "[":
|
||||||
return m.moveNoteCursor(-1), nil
|
return m.moveNoteCursor(-1), nil
|
||||||
|
|
||||||
@@ -733,9 +757,6 @@ func (m Model) handleAlertDetailKey(msg tea.KeyMsg) (Model, tea.Cmd) {
|
|||||||
return m, clearStatusCmd()
|
return m, clearStatusCmd()
|
||||||
}
|
}
|
||||||
return m.openIncident(api.Incident{ID: *m.selectedAlert.IncidentID})
|
return m.openIncident(api.Incident{ID: *m.selectedAlert.IncidentID})
|
||||||
|
|
||||||
case "S":
|
|
||||||
return m.openStats()
|
|
||||||
}
|
}
|
||||||
|
|
||||||
return m, nil
|
return m, nil
|
||||||
@@ -835,25 +856,6 @@ func (m Model) handleConfirmKey(msg tea.KeyMsg) (Model, tea.Cmd) {
|
|||||||
return m, nil
|
return m, nil
|
||||||
}
|
}
|
||||||
|
|
||||||
// ── Stats ─────────────────────────────────────────────────────────────────
|
|
||||||
|
|
||||||
func (m Model) handleStatsKey(msg tea.KeyMsg) (Model, tea.Cmd) {
|
|
||||||
if msg.String() == "esc" {
|
|
||||||
m.mode = m.statsReturnMode
|
|
||||||
return m, nil
|
|
||||||
}
|
|
||||||
return m, nil
|
|
||||||
}
|
|
||||||
|
|
||||||
// openStats enters the statistics view, remembering where to go back to.
|
|
||||||
func (m Model) openStats() (Model, tea.Cmd) {
|
|
||||||
m.statsReturnMode = m.mode
|
|
||||||
m.mode = modeStats
|
|
||||||
m.statsLoading = true
|
|
||||||
m.statsViewport = viewport.New(m.width, m.height-5)
|
|
||||||
return m, fetchDetailStatsCmd(m.client)
|
|
||||||
}
|
|
||||||
|
|
||||||
// ── User picker ───────────────────────────────────────────────────────────
|
// ── User picker ───────────────────────────────────────────────────────────
|
||||||
|
|
||||||
func (m Model) handleUserPickerKey(msg tea.KeyMsg) (Model, tea.Cmd) {
|
func (m Model) handleUserPickerKey(msg tea.KeyMsg) (Model, tea.Cmd) {
|
||||||
@@ -939,6 +941,28 @@ func (m Model) handleUserCreateKey(msg tea.KeyMsg) (Model, tea.Cmd) {
|
|||||||
return m, nil
|
return m, nil
|
||||||
}
|
}
|
||||||
|
|
||||||
|
// handleUserNotifyEditKey edits one user's ntfy topic.
|
||||||
|
//
|
||||||
|
// Unlike the other forms here, an empty value is not a mistake to reject: it is
|
||||||
|
// how a topic is cleared, which the server accepts and treats as NULL.
|
||||||
|
func (m Model) handleUserNotifyEditKey(msg tea.KeyMsg) (Model, tea.Cmd) {
|
||||||
|
switch msg.String() {
|
||||||
|
case "esc":
|
||||||
|
m.ntfyTopicInput.Blur()
|
||||||
|
m.mode = modeDashboard
|
||||||
|
return m, nil
|
||||||
|
|
||||||
|
case "enter":
|
||||||
|
topic := strings.TrimSpace(m.ntfyTopicInput.Value())
|
||||||
|
m.ntfyTopicInput.Blur()
|
||||||
|
m.mode = modeDashboard
|
||||||
|
m.usersLoading = true
|
||||||
|
return m, setUserNotifyTargetCmd(m.client, m.selectedUser.ID, topic)
|
||||||
|
}
|
||||||
|
|
||||||
|
return m, nil
|
||||||
|
}
|
||||||
|
|
||||||
func (m Model) handleAPIKeyMenuKey(msg tea.KeyMsg) (Model, tea.Cmd) {
|
func (m Model) handleAPIKeyMenuKey(msg tea.KeyMsg) (Model, tea.Cmd) {
|
||||||
switch msg.String() {
|
switch msg.String() {
|
||||||
case "esc":
|
case "esc":
|
||||||
|
|||||||
+132
-24
@@ -309,13 +309,97 @@ func TestDeleteNote_ConfirmsThenActs(t *testing.T) {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
// ── Ntfy topic ────────────────────────────────────────────────────────────
|
||||||
|
|
||||||
|
// onUsers puts the model in the Users section with a loaded table.
|
||||||
|
func onUsers(users []api.User) Model {
|
||||||
|
m := sized()
|
||||||
|
m.activeSection = sectionUsers
|
||||||
|
m.users = users
|
||||||
|
m.rebuildUserManageTable()
|
||||||
|
return m
|
||||||
|
}
|
||||||
|
|
||||||
|
func userFixtures() []api.User {
|
||||||
|
topic := "terdut-niklas"
|
||||||
|
return []api.User{
|
||||||
|
{ID: 1, Username: "niklas", Email: "niklas@example.com", NtfyTopic: &topic},
|
||||||
|
{ID: 2, Username: "alex", Email: "alex@example.com"},
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
func TestNotifyTopic_EditPrefillsTheCurrentTopic(t *testing.T) {
|
||||||
|
m, _ := press(t, onUsers(userFixtures()), "t")
|
||||||
|
|
||||||
|
if m.mode != modeUserNotifyEdit {
|
||||||
|
t.Fatalf("expected the topic editor, got mode %v", m.mode)
|
||||||
|
}
|
||||||
|
if m.selectedUser.ID != 1 {
|
||||||
|
t.Errorf("expected the user under the cursor, got %d", m.selectedUser.ID)
|
||||||
|
}
|
||||||
|
// Prefilled, so editing a topic does not mean retyping it from scratch.
|
||||||
|
if got := m.ntfyTopicInput.Value(); got != "terdut-niklas" {
|
||||||
|
t.Errorf("expected the current topic prefilled, got %q", got)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
// A user with no topic opens an empty field rather than the previous user's.
|
||||||
|
func TestNotifyTopic_EditStartsEmptyWhenUnset(t *testing.T) {
|
||||||
|
m := onUsers(userFixtures())
|
||||||
|
m, _ = press(t, m, "t")
|
||||||
|
m, _ = press(t, m, "esc")
|
||||||
|
m.userManageTable.SetCursor(1)
|
||||||
|
|
||||||
|
m, _ = press(t, m, "t")
|
||||||
|
if got := m.ntfyTopicInput.Value(); got != "" {
|
||||||
|
t.Errorf("expected an empty field for a user with no topic, got %q", got)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
func TestNotifyTopic_EscapeAbandonsWithoutSaving(t *testing.T) {
|
||||||
|
m, _ := press(t, onUsers(userFixtures()), "t")
|
||||||
|
m, cmd := press(t, m, "esc")
|
||||||
|
|
||||||
|
if cmd != nil {
|
||||||
|
t.Error("expected escape to save nothing")
|
||||||
|
}
|
||||||
|
if m.mode != modeDashboard {
|
||||||
|
t.Errorf("expected a return to the dashboard, got mode %v", m.mode)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
// Clearing a topic is a real action, not a no-op: it is how a user is taken off
|
||||||
|
// their own topic and back onto the shared fallback. Contrast the snooze prompt,
|
||||||
|
// where an empty value means "I changed my mind".
|
||||||
|
func TestNotifyTopic_EmptyInputStillSubmits(t *testing.T) {
|
||||||
|
m, _ := press(t, onUsers(userFixtures()), "t")
|
||||||
|
m.ntfyTopicInput.SetValue("")
|
||||||
|
|
||||||
|
m, cmd := press(t, m, "enter")
|
||||||
|
if cmd == nil {
|
||||||
|
t.Fatal("expected clearing the topic to call the server")
|
||||||
|
}
|
||||||
|
if m.mode != modeDashboard {
|
||||||
|
t.Errorf("expected a return to the dashboard, got mode %v", m.mode)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
func TestNotifyTopic_IsUsersSectionOnly(t *testing.T) {
|
||||||
|
m := sized()
|
||||||
|
m.activeSection = sectionIncidents
|
||||||
|
if next, cmd := press(t, m, "t"); cmd != nil || next.mode != modeDashboard {
|
||||||
|
t.Error("expected t to do nothing outside the Users section")
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
func TestTab_CyclesEverySection(t *testing.T) {
|
func TestTab_CyclesEverySection(t *testing.T) {
|
||||||
m := sized()
|
m := sized()
|
||||||
if m.activeSection != sectionIncidents {
|
if m.activeSection != sectionIncidents {
|
||||||
t.Fatal("incidents is the section the client opens on")
|
t.Fatal("incidents is the section the client opens on")
|
||||||
}
|
}
|
||||||
|
|
||||||
want := []section{sectionAlerts, sectionArchived, sectionSchedule, sectionUsers, sectionIncidents}
|
want := []section{sectionAlerts, sectionStats, sectionArchived, sectionSchedule,
|
||||||
|
sectionUsers, sectionIncidents}
|
||||||
for i, expected := range want {
|
for i, expected := range want {
|
||||||
m, _ = press(t, m, "tab")
|
m, _ = press(t, m, "tab")
|
||||||
if m.activeSection != expected {
|
if m.activeSection != expected {
|
||||||
@@ -341,30 +425,54 @@ func TestFilter_CyclesPerSection(t *testing.T) {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
// Stats opens from both the queue and an incident, and esc has to go back to
|
// Stats is a section like any other: no key of its own, no mode of its own, and
|
||||||
// wherever it was opened from.
|
// it loads once on first visit rather than on every tab-in — the three empty
|
||||||
func TestStats_ReturnsWhereItWasOpenedFrom(t *testing.T) {
|
// slices a quiet server returns are a real answer, not a missing one.
|
||||||
t.Run("from the queue", func(t *testing.T) {
|
func TestStats_IsAnOrdinarySection(t *testing.T) {
|
||||||
m, _ := press(t, sized(), "S")
|
m := sized()
|
||||||
if m.mode != modeStats {
|
m.activeSection = sectionAlerts
|
||||||
t.Fatalf("expected stats, got mode %v", m.mode)
|
|
||||||
}
|
|
||||||
m, _ = press(t, m, "esc")
|
|
||||||
if m.mode != modeDashboard {
|
|
||||||
t.Errorf("expected the dashboard, got mode %v", m.mode)
|
|
||||||
}
|
|
||||||
})
|
|
||||||
|
|
||||||
t.Run("from an incident", func(t *testing.T) {
|
m, cmd := press(t, m, "tab")
|
||||||
m, _ := press(t, onIncident(openIncidentFixture(), nil), "S")
|
if m.activeSection != sectionStats {
|
||||||
if m.mode != modeStats {
|
t.Fatalf("expected the stats section, got %v", m.activeSection)
|
||||||
t.Fatalf("expected stats, got mode %v", m.mode)
|
}
|
||||||
}
|
if m.mode != modeDashboard {
|
||||||
m, _ = press(t, m, "esc")
|
t.Errorf("stats is a section, not a mode: got mode %v", m.mode)
|
||||||
if m.mode != modeIncidentDetail {
|
}
|
||||||
t.Errorf("expected the incident, got mode %v", m.mode)
|
if cmd == nil {
|
||||||
}
|
t.Error("the first visit should fetch")
|
||||||
})
|
}
|
||||||
|
|
||||||
|
m.statsLoaded = true
|
||||||
|
m.statsLoading = false
|
||||||
|
if cmd := m.loadSectionIfEmpty(); cmd != nil {
|
||||||
|
t.Error("a second visit should reuse what was already fetched")
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
// S used to open the stats overlay from anywhere. It is gone, and must not
|
||||||
|
// disturb the view it is pressed in.
|
||||||
|
func TestStats_KeyIsGone(t *testing.T) {
|
||||||
|
m, _ := press(t, sized(), "S")
|
||||||
|
if m.activeSection != sectionIncidents || m.mode != modeDashboard {
|
||||||
|
t.Errorf("S should do nothing on the queue, got section %v mode %v",
|
||||||
|
m.activeSection, m.mode)
|
||||||
|
}
|
||||||
|
|
||||||
|
m, _ = press(t, onIncident(openIncidentFixture(), nil), "S")
|
||||||
|
if m.mode != modeIncidentDetail {
|
||||||
|
t.Errorf("S should leave the incident open, got mode %v", m.mode)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
// The overlay never auto-refreshed, because the tick skipped every non-dashboard
|
||||||
|
// mode. As a section it rides the tick like the rest.
|
||||||
|
func TestStats_RefreshesOnTick(t *testing.T) {
|
||||||
|
m := sized()
|
||||||
|
m.activeSection = sectionStats
|
||||||
|
if m.refreshActiveSection() == nil {
|
||||||
|
t.Error("the stats section should refresh on the tick")
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
// Alerts carry no workflow state, so the detail view offers nothing but a way
|
// Alerts carry no workflow state, so the detail view offers nothing but a way
|
||||||
|
|||||||
+67
-15
@@ -10,7 +10,8 @@ import (
|
|||||||
"github.com/yeniklas/terdut-tui/internal/api"
|
"github.com/yeniklas/terdut-tui/internal/api"
|
||||||
)
|
)
|
||||||
|
|
||||||
var sectionNames = []string{"Incidents", "Alerts", "Archived", "Schedule", "Users"}
|
// Order must match the section constants — renderTabs indexes this by ordinal.
|
||||||
|
var sectionNames = []string{"Incidents", "Alerts", "Stats", "Archived", "Schedule", "Users"}
|
||||||
|
|
||||||
func (m Model) View() string {
|
func (m Model) View() string {
|
||||||
if m.width == 0 {
|
if m.width == 0 {
|
||||||
@@ -68,12 +69,12 @@ func (m Model) renderBody() string {
|
|||||||
default:
|
default:
|
||||||
return m.renderSchedule()
|
return m.renderSchedule()
|
||||||
}
|
}
|
||||||
case modeStats:
|
|
||||||
return m.renderStats()
|
|
||||||
case modeUserPicker:
|
case modeUserPicker:
|
||||||
return m.renderUserPicker()
|
return m.renderUserPicker()
|
||||||
case modeUserCreate:
|
case modeUserCreate:
|
||||||
return m.renderUserCreate()
|
return m.renderUserCreate()
|
||||||
|
case modeUserNotifyEdit:
|
||||||
|
return m.renderUserNotifyEdit()
|
||||||
case modeAPIKeyMenu:
|
case modeAPIKeyMenu:
|
||||||
return m.renderAPIKeyMenu()
|
return m.renderAPIKeyMenu()
|
||||||
case modeAPIKeyCreate:
|
case modeAPIKeyCreate:
|
||||||
@@ -99,12 +100,12 @@ func (m Model) renderFooter() string {
|
|||||||
switch m.mode {
|
switch m.mode {
|
||||||
case modeIncidentDetail:
|
case modeIncidentDetail:
|
||||||
if !m.selectedIncident.IsOpen() {
|
if !m.selectedIncident.IsOpen() {
|
||||||
return withStatus(" x·archive c·note [/]·select d·del S·stats esc·back")
|
return withStatus(" x·archive c·note [/]·select d·del esc·back")
|
||||||
}
|
}
|
||||||
return withStatus(" a·ack A·unack R·resolve s·assign z·snooze Z·unsnooze c·note [/]·select d·del S·stats esc·back")
|
return withStatus(" a·ack A·unack R·resolve s·assign z·snooze Z·unsnooze c·note [/]·select d·del esc·back")
|
||||||
|
|
||||||
case modeAlertDetail:
|
case modeAlertDetail:
|
||||||
return withStatus(" i·open incident S·stats esc·back")
|
return withStatus(" i·open incident esc·back")
|
||||||
|
|
||||||
case modeNote:
|
case modeNote:
|
||||||
return "\n" + styleFooter.Render(" enter·submit esc·cancel")
|
return "\n" + styleFooter.Render(" enter·submit esc·cancel")
|
||||||
@@ -115,9 +116,6 @@ func (m Model) renderFooter() string {
|
|||||||
case modeConfirm:
|
case modeConfirm:
|
||||||
return "\n" + styleError.Render(" "+m.confirmPrompt())
|
return "\n" + styleError.Render(" "+m.confirmPrompt())
|
||||||
|
|
||||||
case modeStats:
|
|
||||||
return withStatus(" esc·back")
|
|
||||||
|
|
||||||
case modeUserPicker:
|
case modeUserPicker:
|
||||||
if m.pickerTarget == pickerIncidentAssignee {
|
if m.pickerTarget == pickerIncidentAssignee {
|
||||||
return withStatus(" j/k·navigate enter·assign incident esc·cancel")
|
return withStatus(" j/k·navigate enter·assign incident esc·cancel")
|
||||||
@@ -131,6 +129,9 @@ func (m Model) renderFooter() string {
|
|||||||
case modeUserCreate:
|
case modeUserCreate:
|
||||||
return withStatus(" tab·next field enter·create esc·cancel")
|
return withStatus(" tab·next field enter·create esc·cancel")
|
||||||
|
|
||||||
|
case modeUserNotifyEdit:
|
||||||
|
return withStatus(" enter·save esc·cancel (empty clears the topic)")
|
||||||
|
|
||||||
case modeAPIKeyMenu:
|
case modeAPIKeyMenu:
|
||||||
return withStatus(" n·new key r·revoke by ID esc·back")
|
return withStatus(" n·new key r·revoke by ID esc·back")
|
||||||
|
|
||||||
@@ -146,15 +147,17 @@ func (m Model) renderFooter() string {
|
|||||||
default:
|
default:
|
||||||
switch m.activeSection {
|
switch m.activeSection {
|
||||||
case sectionIncidents:
|
case sectionIncidents:
|
||||||
return withStatus(" enter·detail x·archive f·filter S·stats r·refresh tab·section q·quit")
|
return withStatus(" enter·detail x·archive f·filter r·refresh tab·section q·quit")
|
||||||
case sectionAlerts:
|
case sectionAlerts:
|
||||||
return withStatus(" enter·detail f·filter S·stats r·refresh tab·section q·quit")
|
return withStatus(" enter·detail f·filter r·refresh tab·section q·quit")
|
||||||
|
case sectionStats:
|
||||||
|
return withStatus(" ↑/↓·scroll r·refresh tab·section q·quit")
|
||||||
case sectionArchived:
|
case sectionArchived:
|
||||||
return withStatus(" enter·detail x·unarchive r·refresh tab·section q·quit")
|
return withStatus(" enter·detail x·unarchive r·refresh tab·section q·quit")
|
||||||
case sectionSchedule:
|
case sectionSchedule:
|
||||||
return withStatus(" +·assign day W·assign week d·del ←/→·shift week tab·section r·refresh q·quit")
|
return withStatus(" +·assign day W·assign week d·del ←/→·shift week tab·section r·refresh q·quit")
|
||||||
case sectionUsers:
|
case sectionUsers:
|
||||||
return withStatus(" n·new user d·delete k·API keys r·refresh tab·section q·quit")
|
return withStatus(" n·new user t·topic d·delete k·API keys r·refresh tab·section q·quit")
|
||||||
}
|
}
|
||||||
return "\n" + styleFooter.Render(m.help.ShortHelpView(m.keys.ShortHelp()))
|
return "\n" + styleFooter.Render(m.help.ShortHelpView(m.keys.ShortHelp()))
|
||||||
}
|
}
|
||||||
@@ -187,6 +190,8 @@ func (m Model) renderDashboard() string {
|
|||||||
return m.renderIncidents()
|
return m.renderIncidents()
|
||||||
case sectionAlerts:
|
case sectionAlerts:
|
||||||
return m.renderAlerts()
|
return m.renderAlerts()
|
||||||
|
case sectionStats:
|
||||||
|
return m.renderStats()
|
||||||
case sectionArchived:
|
case sectionArchived:
|
||||||
return m.renderArchived()
|
return m.renderArchived()
|
||||||
case sectionSchedule:
|
case sectionSchedule:
|
||||||
@@ -359,7 +364,9 @@ func (m Model) renderPrompt(prompt string) string {
|
|||||||
// ── Stats ──────────────────────────────────────────────────────────────────
|
// ── Stats ──────────────────────────────────────────────────────────────────
|
||||||
|
|
||||||
func (m Model) renderStats() string {
|
func (m Model) renderStats() string {
|
||||||
if m.statsLoading {
|
// Only announce loading before the first result: a background refresh must not
|
||||||
|
// blank the page out from under whoever is reading it.
|
||||||
|
if m.statsLoading && !m.statsLoaded {
|
||||||
return "\n" + styleMuted.Render(" Loading statistics…")
|
return "\n" + styleMuted.Render(" Loading statistics…")
|
||||||
}
|
}
|
||||||
return m.statsViewport.View()
|
return m.statsViewport.View()
|
||||||
@@ -548,6 +555,15 @@ func eventLabel(e api.IncidentEvent) string {
|
|||||||
return " Resolved by " + who
|
return " Resolved by " + who
|
||||||
}
|
}
|
||||||
return " Resolved (all alerts stopped firing)"
|
return " Resolved (all alerts stopped firing)"
|
||||||
|
case api.EventNotified:
|
||||||
|
// An empty username here is not "the server acted": it means the page
|
||||||
|
// went to the shared fallback topic, so it belongs to nobody.
|
||||||
|
return fmt.Sprintf(" Notified %s%s", notifiedTarget(who), notifyKind(e.Detail))
|
||||||
|
case api.EventNotifyFailed:
|
||||||
|
// The detail is "<kind>: <reason>", and the reason is the point — it is
|
||||||
|
// the only thing that says why nobody's phone rang.
|
||||||
|
return truncate(fmt.Sprintf(" Notification to %s failed · %s",
|
||||||
|
notifiedTarget(who), e.Detail), 52)
|
||||||
default:
|
default:
|
||||||
label := " " + e.Type
|
label := " " + e.Type
|
||||||
if e.Detail != "" {
|
if e.Detail != "" {
|
||||||
@@ -557,6 +573,25 @@ func eventLabel(e api.IncidentEvent) string {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
// notifiedTarget names who a page reached. The server attaches no user when it
|
||||||
|
// published to the shared fallback topic, and saying so is the difference
|
||||||
|
// between "somebody was paged" and "the on-call rota was empty".
|
||||||
|
func notifiedTarget(username string) string {
|
||||||
|
if username == "" {
|
||||||
|
return "the fallback topic"
|
||||||
|
}
|
||||||
|
return username
|
||||||
|
}
|
||||||
|
|
||||||
|
// notifyKind renders the notification kind the server puts in Detail. It is an
|
||||||
|
// open set, so anything unrecognised is shown rather than dropped.
|
||||||
|
func notifyKind(detail string) string {
|
||||||
|
if detail == "" {
|
||||||
|
return ""
|
||||||
|
}
|
||||||
|
return " (" + detail + ")"
|
||||||
|
}
|
||||||
|
|
||||||
func buildAlertDetailContent(alert api.Alert, width int) string {
|
func buildAlertDetailContent(alert api.Alert, width int) string {
|
||||||
now := time.Now()
|
now := time.Now()
|
||||||
var b strings.Builder
|
var b strings.Builder
|
||||||
@@ -731,6 +766,16 @@ func (m Model) renderUserCreate() string {
|
|||||||
emailLabel + m.userFormInputs[1].View() + "\n"
|
emailLabel + m.userFormInputs[1].View() + "\n"
|
||||||
}
|
}
|
||||||
|
|
||||||
|
func (m Model) renderUserNotifyEdit() string {
|
||||||
|
header := fmt.Sprintf("\n Push notifications for %s\n", styleBold.Render(m.selectedUser.Username))
|
||||||
|
hint := line(styleMuted,
|
||||||
|
" The ntfy topic this user's pages go to. Leave it empty to clear it —\n"+
|
||||||
|
" their incidents then page the server's shared fallback topic, which\n"+
|
||||||
|
" carries no Acknowledge button.")
|
||||||
|
label := styleSelected.Render(" Topic: ")
|
||||||
|
return header + "\n" + hint + "\n" + label + m.ntfyTopicInput.View() + "\n"
|
||||||
|
}
|
||||||
|
|
||||||
func (m Model) renderAPIKeyMenu() string {
|
func (m Model) renderAPIKeyMenu() string {
|
||||||
header := fmt.Sprintf("\n API keys for %s\n", styleBold.Render(m.selectedUser.Username))
|
header := fmt.Sprintf("\n API keys for %s\n", styleBold.Render(m.selectedUser.Username))
|
||||||
warning := line(styleMuted, " Keys cannot be listed — only new keys can be created,\n or existing ones revoked by their integer ID.")
|
warning := line(styleMuted, " Keys cannot be listed — only new keys can be created,\n or existing ones revoked by their integer ID.")
|
||||||
@@ -806,12 +851,19 @@ func renderBarWidth(count, maxCount, maxWidth int) int {
|
|||||||
return w
|
return w
|
||||||
}
|
}
|
||||||
|
|
||||||
|
// truncate shortens s to max terminal cells, marking the cut with an ellipsis.
|
||||||
|
//
|
||||||
|
// Counted in runes rather than bytes: these strings are laid out against
|
||||||
|
// fixed-width columns, and a byte cut through a multi-byte rune would both
|
||||||
|
// mis-measure the column and emit a broken character. Server-supplied text —
|
||||||
|
// labels, annotations, delivery errors — is not guaranteed to be ASCII.
|
||||||
func truncate(s string, max int) string {
|
func truncate(s string, max int) string {
|
||||||
if max < 1 {
|
if max < 1 {
|
||||||
return ""
|
return ""
|
||||||
}
|
}
|
||||||
if len(s) <= max {
|
r := []rune(s)
|
||||||
|
if len(r) <= max {
|
||||||
return s
|
return s
|
||||||
}
|
}
|
||||||
return s[:max-1] + "…"
|
return string(r[:max-1]) + "…"
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -6,6 +6,7 @@ import (
|
|||||||
"testing"
|
"testing"
|
||||||
"time"
|
"time"
|
||||||
|
|
||||||
|
tea "github.com/charmbracelet/bubbletea"
|
||||||
"github.com/yeniklas/terdut-tui/internal/api"
|
"github.com/yeniklas/terdut-tui/internal/api"
|
||||||
)
|
)
|
||||||
|
|
||||||
@@ -174,6 +175,18 @@ func TestEventLabel_KnownTypes(t *testing.T) {
|
|||||||
{api.IncidentEvent{Type: api.EventResolved, Username: "bo"}, "Resolved by bo"},
|
{api.IncidentEvent{Type: api.EventResolved, Username: "bo"}, "Resolved by bo"},
|
||||||
// No user means the server closed it via the alert cascade.
|
// No user means the server closed it via the alert cascade.
|
||||||
{api.IncidentEvent{Type: api.EventResolved}, "all alerts stopped firing"},
|
{api.IncidentEvent{Type: api.EventResolved}, "all alerts stopped firing"},
|
||||||
|
{api.IncidentEvent{Type: api.EventNotified, Username: "bo", Detail: "triggered"},
|
||||||
|
"Notified bo (triggered)"},
|
||||||
|
{api.IncidentEvent{Type: api.EventNotified, Username: "bo", Detail: "reminder"},
|
||||||
|
"Notified bo (reminder)"},
|
||||||
|
// On a notification, no user means the shared fallback topic — not that
|
||||||
|
// the server acted on its own.
|
||||||
|
{api.IncidentEvent{Type: api.EventNotified, Detail: "triggered"},
|
||||||
|
"Notified the fallback topic (triggered)"},
|
||||||
|
{api.IncidentEvent{Type: api.EventNotifyFailed, Username: "bo", Detail: "triggered: ntfy returned 502"},
|
||||||
|
"Notification to bo failed"},
|
||||||
|
{api.IncidentEvent{Type: api.EventNotifyFailed, Detail: "triggered: no route to host"},
|
||||||
|
"Notification to the fallback topic failed"},
|
||||||
}
|
}
|
||||||
for _, tt := range tests {
|
for _, tt := range tests {
|
||||||
t.Run(tt.event.Type, func(t *testing.T) {
|
t.Run(tt.event.Type, func(t *testing.T) {
|
||||||
@@ -182,6 +195,32 @@ func TestEventLabel_KnownTypes(t *testing.T) {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
// The timeline is where a page that never landed becomes visible, so both
|
||||||
|
// outcomes have to survive into the rendered pane.
|
||||||
|
func TestIncidentDetail_RendersNotifications(t *testing.T) {
|
||||||
|
now := time.Now()
|
||||||
|
inc := api.Incident{ID: 1, Title: "DiskFull", Status: api.StatusTriggered, TriggeredAt: now}
|
||||||
|
timeline := []api.IncidentEvent{
|
||||||
|
{Type: api.EventTriggered, CreatedAt: now},
|
||||||
|
{Type: api.EventNotified, Username: "niklas", Detail: "triggered", CreatedAt: now},
|
||||||
|
{Type: api.EventNotifyFailed, Username: "niklas",
|
||||||
|
Detail: "reminder: ntfy returned 502", CreatedAt: now},
|
||||||
|
}
|
||||||
|
|
||||||
|
got := buildIncidentDetailContent(inc, timeline, -1, 120)
|
||||||
|
mustContain(t, got, "Notified niklas (triggered)", "Notification to niklas failed")
|
||||||
|
}
|
||||||
|
|
||||||
|
func TestUserNotifyEdit_SaysWhatAnEmptyValueDoes(t *testing.T) {
|
||||||
|
m := sized()
|
||||||
|
m.mode = modeUserNotifyEdit
|
||||||
|
m.selectedUser = api.User{ID: 1, Username: "niklas"}
|
||||||
|
|
||||||
|
mustContain(t, m.View(), "niklas", "empty to clear it", "fallback topic")
|
||||||
|
// The footer has to repeat it: that is where the reader looks for what a key does.
|
||||||
|
mustContain(t, m.renderFooter(), "empty clears the topic")
|
||||||
|
}
|
||||||
|
|
||||||
func TestAlertDetail_SaysItIsReadOnlyAndLinksTheIncident(t *testing.T) {
|
func TestAlertDetail_SaysItIsReadOnlyAndLinksTheIncident(t *testing.T) {
|
||||||
now := time.Now()
|
now := time.Now()
|
||||||
id := int64(7)
|
id := int64(7)
|
||||||
@@ -239,7 +278,7 @@ func TestView_TabsAndDashboardRender(t *testing.T) {
|
|||||||
m.rebuildIncidentTable()
|
m.rebuildIncidentTable()
|
||||||
|
|
||||||
mustContain(t, m.View(),
|
mustContain(t, m.View(),
|
||||||
"Incidents", "Alerts", "Archived", "Schedule", "Users",
|
"Incidents", "Alerts", "Stats", "Archived", "Schedule", "Users",
|
||||||
"Triggered: 1", "filter: open",
|
"Triggered: 1", "filter: open",
|
||||||
"DiskFull", "critical", "admin",
|
"DiskFull", "critical", "admin",
|
||||||
"enter·detail")
|
"enter·detail")
|
||||||
@@ -254,6 +293,27 @@ func TestView_EmptyStates(t *testing.T) {
|
|||||||
mustContain(t, m.View(), "No archived incidents.")
|
mustContain(t, m.View(), "No archived incidents.")
|
||||||
}
|
}
|
||||||
|
|
||||||
|
// The stats page renders inside the normal section chrome now, so it has to
|
||||||
|
// survive the real path: a window size message sizes the viewport and fills it.
|
||||||
|
func TestView_StatsSectionRendersInPlace(t *testing.T) {
|
||||||
|
m := NewModel(nil, "http://test", time.Minute)
|
||||||
|
m.connected = true
|
||||||
|
m.incidentStats = &api.IncidentStats{Total: 3, Triggered: 1}
|
||||||
|
m.topAlerts = []api.TopAlert{{Name: "DiskFull", Count: 4}}
|
||||||
|
m.statsLoaded = true
|
||||||
|
|
||||||
|
next, _ := m.Update(tea.WindowSizeMsg{Width: 120, Height: 40})
|
||||||
|
m = next.(Model)
|
||||||
|
m.activeSection = sectionStats
|
||||||
|
|
||||||
|
out := m.View()
|
||||||
|
mustContain(t, out, "Stats", "Incident Response", "Top Alerts", "DiskFull",
|
||||||
|
"tab·section")
|
||||||
|
if strings.Contains(plain(out), "Loading statistics") {
|
||||||
|
t.Error("loaded stats should not show the loading placeholder")
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
func TestView_ConnectionError(t *testing.T) {
|
func TestView_ConnectionError(t *testing.T) {
|
||||||
m := sized()
|
m := sized()
|
||||||
m.connected = false
|
m.connected = false
|
||||||
|
|||||||
Reference in New Issue
Block a user