Skip to content

feat: generate golden files for notification templates #14537

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 24 commits into from
Sep 4, 2024
+97 −15
Merged

feat: generate golden files for notification templates #14537

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
24 commits
Select commit Hold shift + click to select a range
b2523b1
WIP
mtojek Sep 3, 2024
9559138
WIP2
mtojek Sep 3, 2024
39d80eb
Template body
mtojek Sep 3, 2024
d9fef24
fmt
mtojek Sep 3, 2024
994f484
adjust payload
mtojek Sep 4, 2024
c55a688
fix
mtojek Sep 4, 2024
796da33
no index
mtojek Sep 4, 2024
c23da28
fix range
mtojek Sep 4, 2024
53dc29f
fix: base_url
mtojek Sep 4, 2024
1753beb
test
mtojek Sep 4, 2024
e200056
WIP
mtojek Sep 4, 2024
1118d67
WIP
mtojek Sep 4, 2024
7c7e0f9
WIP
mtojek Sep 4, 2024
ca0d9ee
s/bobby/Bobby/g
mtojek Sep 4, 2024
d91742c
gen
mtojek Sep 4, 2024
9fe5def
back to false
mtojek Sep 4, 2024
4d95288
fix
mtojek Sep 4, 2024
b12dcfc
Address PR feedback
mtojek Sep 4, 2024
412e9b2
db=ci
mtojek Sep 4, 2024
419c56a
golden
mtojek Sep 4, 2024
14ad9e8
fix
mtojek Sep 4, 2024
7587abc
split
mtojek Sep 4, 2024
f537d26
done
mtojek Sep 4, 2024
47c387b
fmt
mtojek Sep 4, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
59 changes: 44 additions & 15 deletions coderd/notifications/notifications_test.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,15 +5,19 @@ import (
"context"
_ "embed"
"encoding/json"
"flag"
"fmt"
"go/ast"
"go/parser"
"go/token"
"net/http"
"net/http/httptest"
"net/url"
"os"
"path/filepath"
"slices"
"sort"
"strings"
"sync"
"sync/atomic"
"testing"
Expand Down Expand Up @@ -46,6 +50,9 @@ import (
"github.com/coder/coder/v2/testutil"
)

// updateGoldenFiles is a flag that can be set to update golden files.
var updateGoldenFiles = flag.Bool("update", false, "Update golden files")

func TestMain(m *testing.M) {
goleak.VerifyTestMain(m)
}
Expand Down Expand Up @@ -693,7 +700,7 @@ func TestNotificationTemplatesCanRender(t *testing.T) {
name: "TemplateWorkspaceDeleted",
id: notifications.TemplateWorkspaceDeleted,
payload: types.MessagePayload{
UserName: "bobby",
UserName: "Bobby",
Labels: map[string]string{
"name": "bobby-workspace",
"reason": "autodeleted due to dormancy",
Expand All @@ -705,7 +712,7 @@ func TestNotificationTemplatesCanRender(t *testing.T) {
name: "TemplateWorkspaceAutobuildFailed",
id: notifications.TemplateWorkspaceAutobuildFailed,
payload: types.MessagePayload{
UserName: "bobby",
UserName: "Bobby",
Labels: map[string]string{
"name": "bobby-workspace",
"reason": "autostart",
Expand All @@ -716,7 +723,7 @@ func TestNotificationTemplatesCanRender(t *testing.T) {
name: "TemplateWorkspaceDormant",
id: notifications.TemplateWorkspaceDormant,
payload: types.MessagePayload{
UserName: "bobby",
UserName: "Bobby",
Labels: map[string]string{
"name": "bobby-workspace",
"reason": "breached the template's threshold for inactivity",
Expand All @@ -730,7 +737,7 @@ func TestNotificationTemplatesCanRender(t *testing.T) {
name: "TemplateWorkspaceAutoUpdated",
id: notifications.TemplateWorkspaceAutoUpdated,
payload: types.MessagePayload{
UserName: "bobby",
UserName: "Bobby",
Labels: map[string]string{
"name": "bobby-workspace",
"template_version_name": "1.0",
Expand All @@ -742,7 +749,7 @@ func TestNotificationTemplatesCanRender(t *testing.T) {
name: "TemplateWorkspaceMarkedForDeletion",
id: notifications.TemplateWorkspaceMarkedForDeletion,
payload: types.MessagePayload{
UserName: "bobby",
UserName: "Bobby",
Labels: map[string]string{
"name": "bobby-workspace",
"reason": "template updated to new dormancy policy",
Expand All @@ -755,7 +762,7 @@ func TestNotificationTemplatesCanRender(t *testing.T) {
name: "TemplateUserAccountCreated",
id: notifications.TemplateUserAccountCreated,
payload: types.MessagePayload{
UserName: "bobby",
UserName: "Bobby",
Labels: map[string]string{
"created_account_name": "bobby",
},
Expand All @@ -765,7 +772,7 @@ func TestNotificationTemplatesCanRender(t *testing.T) {
name: "TemplateUserAccountDeleted",
id: notifications.TemplateUserAccountDeleted,
payload: types.MessagePayload{
UserName: "bobby",
UserName: "Bobby",
Labels: map[string]string{
"deleted_account_name": "bobby",
},
Expand All @@ -775,7 +782,7 @@ func TestNotificationTemplatesCanRender(t *testing.T) {
name: "TemplateUserAccountSuspended",
id: notifications.TemplateUserAccountSuspended,
payload: types.MessagePayload{
UserName: "bobby",
UserName: "Bobby",
Labels: map[string]string{
"suspended_account_name": "bobby",
},
Expand All @@ -785,7 +792,7 @@ func TestNotificationTemplatesCanRender(t *testing.T) {
name: "TemplateUserAccountActivated",
id: notifications.TemplateUserAccountActivated,
payload: types.MessagePayload{
UserName: "bobby",
UserName: "Bobby",
Labels: map[string]string{
"activated_account_name": "bobby",
},
Expand All @@ -795,7 +802,7 @@ func TestNotificationTemplatesCanRender(t *testing.T) {
name: "TemplateYourAccountSuspended",
id: notifications.TemplateYourAccountSuspended,
payload: types.MessagePayload{
UserName: "bobby",
UserName: "Bobby",
Labels: map[string]string{
"suspended_account_name": "bobby",
},
Expand All @@ -805,7 +812,7 @@ func TestNotificationTemplatesCanRender(t *testing.T) {
name: "TemplateYourAccountActivated",
id: notifications.TemplateYourAccountActivated,
payload: types.MessagePayload{
UserName: "bobby",
UserName: "Bobby",
Labels: map[string]string{
"activated_account_name": "bobby",
},
Expand All @@ -815,7 +822,7 @@ func TestNotificationTemplatesCanRender(t *testing.T) {
name: "TemplateTemplateDeleted",
id: notifications.TemplateTemplateDeleted,
payload: types.MessagePayload{
UserName: "bobby",
UserName: "Bobby",
Labels: map[string]string{
"name": "bobby-template",
"initiator": "rob",
Expand All @@ -826,7 +833,7 @@ func TestNotificationTemplatesCanRender(t *testing.T) {
name: "TemplateWorkspaceManualBuildFailed",
id: notifications.TemplateWorkspaceManualBuildFailed,
payload: types.MessagePayload{
UserName: "bobby",
UserName: "Bobby",
Labels: map[string]string{
"name": "bobby-workspace",
"template_name": "bobby-template",
Expand Down Expand Up @@ -869,14 +876,36 @@ func TestNotificationTemplatesCanRender(t *testing.T) {
Scan(&titleTmpl, &bodyTmpl)
require.NoError(t, err, "failed to query body template for template:", tc.id)

title, err := render.GoTemplate(titleTmpl, tc.payload, nil)
title, err := render.GoTemplate(titleTmpl, tc.payload, defaultHelpers())
require.NotContainsf(t, title, render.NoValue, "template %q is missing a label value", tc.name)
require.NoError(t, err, "failed to render notification title template")
require.NotEmpty(t, title, "title should not be empty")

body, err := render.GoTemplate(bodyTmpl, tc.payload, nil)
body, err := render.GoTemplate(bodyTmpl, tc.payload, defaultHelpers())
require.NoError(t, err, "failed to render notification body template")
require.NotEmpty(t, body, "body should not be empty")

partialName := strings.Split(t.Name(), "/")[1]
bodyGoldenFile := filepath.Join("testdata", "rendered-templates", partialName+"-body.md.golden")
titleGoldenFile := filepath.Join("testdata", "rendered-templates", partialName+"-title.md.golden")

if *updateGoldenFiles {
err = os.MkdirAll(filepath.Dir(bodyGoldenFile), 0o755)
require.NoError(t, err, "want no error creating golden file directory")
err = os.WriteFile(bodyGoldenFile, []byte(body), 0o600)
require.NoError(t, err, "want no error writing body golden file")
err = os.WriteFile(titleGoldenFile, []byte(title), 0o600)
require.NoError(t, err, "want no error writing title golden file")
return
}

wantBody, err := os.ReadFile(bodyGoldenFile)
require.NoError(t, err, "open golden file, run \"DB=ci make update-golden-files\" and commit the changes")
wantTitle, err := os.ReadFile(titleGoldenFile)
require.NoError(t, err, "open golden file, run \"DB=ci make update-golden-files\" and commit the changes")

require.Equal(t, string(wantBody), body, "body should be equal")
require.Equal(t, string(wantTitle), title, "title should be equal")
})
}
}
Expand Down
3 changes: 3 additions & 0 deletions coderd/notifications/testdata/rendered-templates/TemplateTemplateDeleted-body.md.golden
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,3 @@
Hi Bobby

The template **bobby-template** was deleted by **rob**.
1 change: 1 addition & 0 deletions coderd/notifications/testdata/rendered-templates/TemplateTemplateDeleted-title.md.golden
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
Template "bobby-template" deleted
2 changes: 2 additions & 0 deletions coderd/notifications/testdata/rendered-templates/TemplateUserAccountActivated-body.md.golden
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
Hi Bobby,
User account **bobby** has been activated.
1 change: 1 addition & 0 deletions ...rd/notifications/testdata/rendered-templates/TemplateUserAccountActivated-title.md.golden
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
User account "bobby" activated
3 changes: 3 additions & 0 deletions coderd/notifications/testdata/rendered-templates/TemplateUserAccountCreated-body.md.golden
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,3 @@
Hi Bobby,

New user account **bobby** has been created.
1 change: 1 addition & 0 deletions coderd/notifications/testdata/rendered-templates/TemplateUserAccountCreated-title.md.golden
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
User account "bobby" created
3 changes: 3 additions & 0 deletions coderd/notifications/testdata/rendered-templates/TemplateUserAccountDeleted-body.md.golden
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,3 @@
Hi Bobby,

User account **bobby** has been deleted.
1 change: 1 addition & 0 deletions coderd/notifications/testdata/rendered-templates/TemplateUserAccountDeleted-title.md.golden
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
User account "bobby" deleted
2 changes: 2 additions & 0 deletions coderd/notifications/testdata/rendered-templates/TemplateUserAccountSuspended-body.md.golden
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
Hi Bobby,
User account **bobby** has been suspended.
1 change: 1 addition & 0 deletions ...rd/notifications/testdata/rendered-templates/TemplateUserAccountSuspended-title.md.golden
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
User account "bobby" suspended
3 changes: 3 additions & 0 deletions coderd/notifications/testdata/rendered-templates/TemplateWorkspaceAutoUpdated-body.md.golden
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,3 @@
Hi Bobby
Your workspace **bobby-workspace** has been updated automatically to the latest template version (1.0).
Reason for update: **template now includes catnip**
1 change: 1 addition & 0 deletions ...rd/notifications/testdata/rendered-templates/TemplateWorkspaceAutoUpdated-title.md.golden
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
Workspace "bobby-workspace" updated automatically
3 changes: 3 additions & 0 deletions ...notifications/testdata/rendered-templates/TemplateWorkspaceAutobuildFailed-body.md.golden
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,3 @@
Hi Bobby
Automatic build of your workspace **bobby-workspace** failed.
The specified reason was "**autostart**".
1 change: 1 addition & 0 deletions ...otifications/testdata/rendered-templates/TemplateWorkspaceAutobuildFailed-title.md.golden
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
Workspace "bobby-workspace" autobuild failed
4 changes: 4 additions & 0 deletions coderd/notifications/testdata/rendered-templates/TemplateWorkspaceDeleted-body.md.golden
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@
Hi Bobby

Your workspace **bobby-workspace** was deleted.
The specified reason was "**autodeleted due to dormancy (autobuild)**".
1 change: 1 addition & 0 deletions coderd/notifications/testdata/rendered-templates/TemplateWorkspaceDeleted-title.md.golden
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
Workspace "bobby-workspace" deleted
5 changes: 5 additions & 0 deletions coderd/notifications/testdata/rendered-templates/TemplateWorkspaceDormant-body.md.golden
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
Hi Bobby

Your workspace **bobby-workspace** has been marked as [**dormant**](https://coder.com/docs/templates/schedule#dormancy-threshold-enterprise) because of breached the template's threshold for inactivity.
Dormant workspaces are [automatically deleted](https://coder.com/docs/templates/schedule#dormancy-auto-deletion-enterprise) after 24 hours of inactivity.
To prevent deletion, use your workspace with the link below.
1 change: 1 addition & 0 deletions coderd/notifications/testdata/rendered-templates/TemplateWorkspaceDormant-title.md.golden
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
Workspace "bobby-workspace" marked as dormant
4 changes: 4 additions & 0 deletions ...tifications/testdata/rendered-templates/TemplateWorkspaceManualBuildFailed-body.md.golden
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@
Hi Bobby,

A manual build of the workspace **bobby-workspace** using the template **bobby-template** failed (version: **bobby-template-version**).
The workspace build was initiated by **joe**.
1 change: 1 addition & 0 deletions ...ifications/testdata/rendered-templates/TemplateWorkspaceManualBuildFailed-title.md.golden
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
Workspace "bobby-workspace" manual build failed
4 changes: 4 additions & 0 deletions ...tifications/testdata/rendered-templates/TemplateWorkspaceMarkedForDeletion-body.md.golden
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@
Hi Bobby

Your workspace **bobby-workspace** has been marked for **deletion** after 24 hours of [dormancy](https://coder.com/docs/templates/schedule#dormancy-auto-deletion-enterprise) because of template updated to new dormancy policy.
To prevent deletion, use your workspace with the link below.
1 change: 1 addition & 0 deletions ...ifications/testdata/rendered-templates/TemplateWorkspaceMarkedForDeletion-title.md.golden
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
Workspace "bobby-workspace" marked for deletion
2 changes: 2 additions & 0 deletions coderd/notifications/testdata/rendered-templates/TemplateYourAccountActivated-body.md.golden
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
Hi Bobby,
Your account **bobby** has been activated.
1 change: 1 addition & 0 deletions ...rd/notifications/testdata/rendered-templates/TemplateYourAccountActivated-title.md.golden
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
Your account "bobby" has been activated
2 changes: 2 additions & 0 deletions coderd/notifications/testdata/rendered-templates/TemplateYourAccountSuspended-body.md.golden
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
Hi Bobby,
Your account **bobby** has been suspended.
1 change: 1 addition & 0 deletions ...rd/notifications/testdata/rendered-templates/TemplateYourAccountSuspended-title.md.golden
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
Your account "bobby" has been suspended
Loading