Some checks failed
CI / Run tests and lints (push) Failing after 29s
CI / Build CLI docs (push) Has been skipped
CI / Run Playwright end-to-end tests (mysql) (push) Failing after 3s
CI / Run Playwright end-to-end tests (postgres) (push) Failing after 2s
CI / Run Playwright end-to-end tests (sqlite) (push) Failing after 3s
CodeQL / Analyze (go) (push) Failing after 4s
CodeQL / Analyze (javascript) (push) Failing after 3s
Docker Image Scanners / scanners (push) Failing after 4s
Synchronize Issue Labels / Synchronize Issue Labels (push) Has been skipped
CI / Run end-to-end tests (postgres) (push) Failing after 19s
CI / Run end-to-end tests (sqlite) (push) Failing after 2s
CI / Run Playwright end-to-end tests (cockroach) (push) Failing after 2s
CI / Draft newsletter (push) Has been skipped
CI / Generate release (push) Has been skipped
CI / Pending approval Slack notification (push) Has been skipped
CI / Send newsletter (push) Has been skipped
177 lines
5.5 KiB
Go
177 lines
5.5 KiB
Go
// Copyright © 2023 Ory Corp
|
|
// SPDX-License-Identifier: Apache-2.0
|
|
|
|
package courier_test
|
|
|
|
import (
|
|
"fmt"
|
|
"net/http"
|
|
"net/http/httptest"
|
|
"slices"
|
|
"testing"
|
|
|
|
"github.com/gofrs/uuid"
|
|
"github.com/stretchr/testify/assert"
|
|
"github.com/stretchr/testify/require"
|
|
"github.com/tidwall/gjson"
|
|
"go.opentelemetry.io/otel/attribute"
|
|
sdktrace "go.opentelemetry.io/otel/sdk/trace"
|
|
"go.opentelemetry.io/otel/sdk/trace/tracetest"
|
|
|
|
"github.com/ory/kratos/courier"
|
|
templates "github.com/ory/kratos/courier/template/email"
|
|
"github.com/ory/kratos/driver/config"
|
|
"github.com/ory/kratos/pkg"
|
|
"github.com/ory/kratos/pkg/testhelpers"
|
|
"github.com/ory/kratos/x/events"
|
|
"github.com/ory/x/configx"
|
|
"github.com/ory/x/otelx"
|
|
"github.com/ory/x/otelx/semconv"
|
|
)
|
|
|
|
func queueNewMessage(t *testing.T, c courier.Courier) uuid.UUID {
|
|
t.Helper()
|
|
id, err := c.QueueEmail(t.Context(), templates.NewTestStub(&templates.TestStubModel{
|
|
To: "test-recipient-1@example.org",
|
|
Subject: "test-subject-1",
|
|
Body: "test-body-1",
|
|
}))
|
|
require.NoError(t, err)
|
|
return id
|
|
}
|
|
|
|
func TestDispatchMessageWithInvalidSMTP(t *testing.T) {
|
|
_, reg := pkg.NewRegistryDefaultWithDSN(t, "", configx.WithValues(map[string]any{
|
|
config.ViperKeyCourierMessageRetries: 5,
|
|
config.ViperKeyCourierSMTPURL: "http://foo.url",
|
|
}))
|
|
|
|
c, err := reg.Courier(t.Context())
|
|
require.NoError(t, err)
|
|
|
|
t.Run("case=failed sending", func(t *testing.T) {
|
|
id := queueNewMessage(t, c)
|
|
message, err := reg.CourierPersister().LatestQueuedMessage(t.Context())
|
|
require.NoError(t, err)
|
|
require.Equal(t, id, message.ID)
|
|
|
|
err = c.DispatchMessage(t.Context(), *message)
|
|
// sending the email fails, because there is no SMTP server at foo.url
|
|
require.Error(t, err)
|
|
|
|
messages, err := reg.CourierPersister().NextMessages(t.Context(), 10)
|
|
require.NoError(t, err)
|
|
require.Len(t, messages, 1)
|
|
})
|
|
}
|
|
|
|
func TestDispatchMessage(t *testing.T) {
|
|
_, reg := pkg.NewRegistryDefaultWithDSN(t, "", configx.WithValues(map[string]any{
|
|
config.ViperKeyCourierMessageRetries: 5,
|
|
config.ViperKeyCourierSMTPURL: "http://foo.url",
|
|
}))
|
|
|
|
c, err := reg.Courier(t.Context())
|
|
require.NoError(t, err)
|
|
t.Run("case=invalid channel", func(t *testing.T) {
|
|
message := courier.Message{
|
|
Channel: "invalid-channel",
|
|
Status: courier.MessageStatusQueued,
|
|
Type: courier.MessageTypeEmail,
|
|
Recipient: testhelpers.RandomEmail(),
|
|
Subject: "test-subject-1",
|
|
Body: "test-body-1",
|
|
TemplateType: "stub",
|
|
}
|
|
require.NoError(t, reg.CourierPersister().AddMessage(t.Context(), &message))
|
|
assert.ErrorContains(t, c.DispatchMessage(t.Context(), message), "no courier channels configured for: invalid-channel")
|
|
})
|
|
}
|
|
|
|
func TestDispatchQueue(t *testing.T) {
|
|
_, reg := pkg.NewRegistryDefaultWithDSN(t, "", configx.WithValue(config.ViperKeyCourierMessageRetries, 1))
|
|
|
|
c, err := reg.Courier(t.Context())
|
|
require.NoError(t, err)
|
|
c.FailOnDispatchError()
|
|
|
|
id := queueNewMessage(t, c)
|
|
require.NotEqual(t, uuid.Nil, id)
|
|
|
|
// Fails to deliver the first time
|
|
err = c.DispatchQueue(t.Context())
|
|
require.Error(t, err)
|
|
|
|
// Retry once, as we set above - still fails
|
|
err = c.DispatchQueue(t.Context())
|
|
require.Error(t, err)
|
|
|
|
// Now it has been retried once, which means 2 > 1 is true and it is no longer tried
|
|
err = c.DispatchQueue(t.Context())
|
|
require.NoError(t, err)
|
|
|
|
var message courier.Message
|
|
err = reg.Persister().GetConnection(t.Context()).
|
|
Where("status = ?", courier.MessageStatusAbandoned).
|
|
Eager("Dispatches").
|
|
First(&message)
|
|
|
|
require.NoError(t, err)
|
|
require.Equal(t, id, message.ID)
|
|
|
|
require.Len(t, message.Dispatches, 2)
|
|
require.Contains(t, gjson.GetBytes(message.Dispatches[0].Error, "reason").String(), "failed to send email via smtp")
|
|
require.Contains(t, gjson.GetBytes(message.Dispatches[1].Error, "reason").String(), "failed to send email via smtp")
|
|
}
|
|
|
|
func TestDispatchMessageEmitsEventWithNID(t *testing.T) {
|
|
t.Parallel()
|
|
|
|
srv := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
|
|
w.WriteHeader(http.StatusOK)
|
|
}))
|
|
t.Cleanup(srv.Close)
|
|
|
|
requestConfig := fmt.Sprintf(`{"url": "%s", "method": "POST", "body": "file://./stub/request.config.mailer.jsonnet"}`, srv.URL)
|
|
|
|
_, reg := pkg.NewFastRegistryWithMocks(t, configx.WithValues(map[string]any{
|
|
config.ViperKeyCourierDeliveryStrategy: "http",
|
|
config.ViperKeyCourierHTTPRequestConfig: requestConfig,
|
|
}))
|
|
|
|
recorder := tracetest.NewSpanRecorder()
|
|
provider := sdktrace.NewTracerProvider(sdktrace.WithSpanProcessor(recorder))
|
|
reg.SetTracer(otelx.NewNoop().WithOTLP(provider.Tracer("test")))
|
|
|
|
c, err := reg.Courier(t.Context())
|
|
require.NoError(t, err)
|
|
|
|
_, err = c.QueueEmail(t.Context(), templates.NewTestStub(&templates.TestStubModel{
|
|
To: testhelpers.RandomEmail(),
|
|
Subject: "test-subject",
|
|
Body: "test-body",
|
|
}))
|
|
require.NoError(t, err)
|
|
|
|
require.NoError(t, c.DispatchQueue(t.Context()))
|
|
|
|
ended := recorder.Ended()
|
|
i := slices.IndexFunc(ended, func(sp sdktrace.ReadOnlySpan) bool {
|
|
return sp.Name() == "courier.DispatchMessage"
|
|
})
|
|
require.GreaterOrEqual(t, i, 0, "courier.DispatchMessage span not found")
|
|
|
|
evs := ended[i].Events()
|
|
j := slices.IndexFunc(evs, func(ev sdktrace.Event) bool {
|
|
return ev.Name == events.CourierMessageDispatched.String()
|
|
})
|
|
require.GreaterOrEqual(t, j, 0, "CourierMessageDispatched event not found on span")
|
|
|
|
attrs := evs[j].Attributes
|
|
k := slices.IndexFunc([]attribute.KeyValue(attrs), func(a attribute.KeyValue) bool {
|
|
return string(a.Key) == semconv.AttributeKeyNID.String()
|
|
})
|
|
require.GreaterOrEqual(t, k, 0, "NID attribute not found on event")
|
|
assert.NotEmpty(t, attrs[k].Value.AsString())
|
|
}
|