6bf8bebf51
CI / Test and Build (push) Failing after 1s
CI / Migrate Dev DB (push) Has been skipped
CI / Migrate DB (push) Has been skipped
CodeQL / Analyze actions (push) Has been cancelled
CodeQL / Analyze javascript-typescript (push) Has been cancelled
CI / Detect Version (push) Has been cancelled
CI / Detect Desktop Changes (push) Has been cancelled
CI / Build AMD64 (blacksmith-2vcpu-ubuntu-2404, ./docker/cron.Dockerfile, ubuntu-latest, ghcr.io/simstudioai/cron) (push) Has been cancelled
CI / Build AMD64 (blacksmith-2vcpu-ubuntu-2404, ./docker/db.Dockerfile, ECR_MIGRATIONS, ubuntu-latest, ghcr.io/simstudioai/migrations) (push) Has been cancelled
CI / Build AMD64 (blacksmith-4vcpu-ubuntu-2404, ./docker/pii.Dockerfile, ECR_PII, ubuntu-latest, ghcr.io/simstudioai/pii) (push) Has been cancelled
CI / Build AMD64 (blacksmith-4vcpu-ubuntu-2404, ./docker/realtime.Dockerfile, ECR_REALTIME, ubuntu-latest, ghcr.io/simstudioai/realtime) (push) Has been cancelled
CI / Build AMD64 (blacksmith-8vcpu-ubuntu-2404, ./docker/app.Dockerfile, ECR_APP, linux-x64-8-core, ghcr.io/simstudioai/simstudio) (push) Has been cancelled
CI / Build ARM64 (GHCR Only) (blacksmith-4vcpu-ubuntu-2404-arm, ./docker/cron.Dockerfile, ubuntu-24.04-arm, ghcr.io/simstudioai/cron) (push) Has been cancelled
CI / Build ARM64 (GHCR Only) (blacksmith-4vcpu-ubuntu-2404-arm, ./docker/db.Dockerfile, ubuntu-24.04-arm, ghcr.io/simstudioai/migrations) (push) Has been cancelled
CI / Build ARM64 (GHCR Only) (blacksmith-4vcpu-ubuntu-2404-arm, ./docker/pii.Dockerfile, ubuntu-24.04-arm, ghcr.io/simstudioai/pii) (push) Has been cancelled
CI / Build ARM64 (GHCR Only) (blacksmith-4vcpu-ubuntu-2404-arm, ./docker/realtime.Dockerfile, ubuntu-24.04-arm, ghcr.io/simstudioai/realtime) (push) Has been cancelled
CI / Build ARM64 (GHCR Only) (blacksmith-8vcpu-ubuntu-2404-arm, ./docker/app.Dockerfile, linux-arm64-8-core, ghcr.io/simstudioai/simstudio) (push) Has been cancelled
CI / Check Docs Changes (push) Has been cancelled
Publish CLI Package / publish-npm (push) Has been cancelled
Publish Python SDK / publish-pypi (push) Has been cancelled
CI / Deploy Trigger.dev (Dev) (push) Has been cancelled
Helm Chart / Lint, test, and validate chart (push) Has been cancelled
Helm Chart / Chart version bumped (push) Has been cancelled
Publish TypeScript SDK / publish-npm (push) Has been cancelled
CI / Build Dev ECR (blacksmith-8vcpu-ubuntu-2404, ./docker/app.Dockerfile, ECR_APP, linux-x64-8-core) (push) Has been cancelled
CI / Promote Images (push) Has been cancelled
CI / Create GHCR Manifests (ghcr.io/simstudioai/cron) (push) Has been cancelled
CI / Create GHCR Manifests (ghcr.io/simstudioai/migrations) (push) Has been cancelled
CI / Create GHCR Manifests (ghcr.io/simstudioai/pii) (push) Has been cancelled
CI / Create GHCR Manifests (ghcr.io/simstudioai/realtime) (push) Has been cancelled
CI / Build Dev ECR (blacksmith-2vcpu-ubuntu-2404, ./docker/db.Dockerfile, ECR_MIGRATIONS, ubuntu-latest) (push) Has been cancelled
CI / Build Dev ECR (blacksmith-4vcpu-ubuntu-2404, ./docker/pii.Dockerfile, ECR_PII, ubuntu-latest) (push) Has been cancelled
CI / Build Dev ECR (blacksmith-4vcpu-ubuntu-2404, ./docker/realtime.Dockerfile, ECR_REALTIME, ubuntu-latest) (push) Has been cancelled
CI / Create GHCR Manifests (ghcr.io/simstudioai/simstudio) (push) Has been cancelled
CI / Process Docs (push) Has been cancelled
CI / Create GitHub Release (push) Has been cancelled
CI / Check Desktop Signing Secrets (push) Has been cancelled
CI / Desktop Release (push) Has been cancelled
CI / Create Desktop Prerelease (push) Has been cancelled
CI / Desktop Prerelease Build (push) Has been cancelled
CI / Publish Desktop Prerelease (push) Has been cancelled
CI / Prune Desktop Prereleases (push) Has been cancelled
Helm Chart / Install on kind and run helm test (push) Has been cancelled
145 lines
4.8 KiB
TypeScript
145 lines
4.8 KiB
TypeScript
/**
|
|
* Covers the `active -> disabled` edge detection that drives the auto-disable
|
|
* email. `applyScheduleFailureUpdate` is the exported seam over the shared
|
|
* guarded UPDATE, so asserting on it also pins the behavior of the trigger.dev
|
|
* call sites that go through the same helper.
|
|
*
|
|
* @vitest-environment node
|
|
*/
|
|
import { databaseMock, dbChainMockFns, resetDbChainMock, schemaMock } from '@sim/testing'
|
|
import { afterAll, beforeEach, describe, expect, it, vi } from 'vitest'
|
|
|
|
const { notifyMock } = vi.hoisted(() => ({
|
|
notifyMock: vi.fn(() => Promise.resolve()),
|
|
}))
|
|
|
|
vi.mock('@/lib/workflows/schedules/disable-notifications', () => ({
|
|
notifyScheduleAutoDisabled: notifyMock,
|
|
}))
|
|
|
|
// This module imports its tables from `@sim/db` directly, which the global mock
|
|
// does not re-export. Widen it rather than rewriting the source's imports.
|
|
vi.mock('@sim/db', () => ({ ...databaseMock, ...schemaMock }))
|
|
|
|
import {
|
|
applyScheduleFailureUpdate,
|
|
buildScheduleCancellationUpdate,
|
|
classifyScheduleExecutionResult,
|
|
releaseScheduleLock,
|
|
} from '@/background/schedule-execution'
|
|
|
|
const BASE = {
|
|
scheduleId: 'schedule-1',
|
|
now: new Date('2025-01-01T00:00:00.000Z'),
|
|
nextRunAt: new Date('2025-01-01T01:00:00.000Z'),
|
|
expectedLastQueuedAt: new Date('2024-12-31T23:00:00.000Z'),
|
|
requestId: 'req-1',
|
|
context: 'test context',
|
|
}
|
|
|
|
afterAll(() => {
|
|
resetDbChainMock()
|
|
})
|
|
|
|
describe('applyScheduleFailureUpdate', () => {
|
|
beforeEach(() => {
|
|
vi.clearAllMocks()
|
|
resetDbChainMock()
|
|
})
|
|
|
|
it('reports disabled when the write returns a disabled row', async () => {
|
|
dbChainMockFns.returning.mockResolvedValueOnce([{ id: 'schedule-1', status: 'disabled' }])
|
|
|
|
const result = await applyScheduleFailureUpdate(BASE)
|
|
|
|
expect(result).toEqual({ updated: true, disabled: true })
|
|
})
|
|
|
|
it('reports not disabled below the threshold (the 99-of-100 case)', async () => {
|
|
dbChainMockFns.returning.mockResolvedValueOnce([{ id: 'schedule-1', status: 'active' }])
|
|
|
|
const result = await applyScheduleFailureUpdate(BASE)
|
|
|
|
expect(result).toEqual({ updated: true, disabled: false })
|
|
})
|
|
|
|
it('reports neither updated nor disabled when the claim guard matches no row', async () => {
|
|
dbChainMockFns.returning.mockResolvedValueOnce([])
|
|
|
|
const result = await applyScheduleFailureUpdate(BASE)
|
|
|
|
expect(result).toEqual({ updated: false, disabled: false })
|
|
})
|
|
|
|
it('defers the email so an in-transaction caller can send it after commit', async () => {
|
|
dbChainMockFns.returning.mockResolvedValueOnce([{ id: 'schedule-1', status: 'disabled' }])
|
|
|
|
await applyScheduleFailureUpdate(BASE)
|
|
|
|
expect(notifyMock).not.toHaveBeenCalled()
|
|
})
|
|
|
|
it('excludes already-disabled rows so a disable is reported at most once', async () => {
|
|
dbChainMockFns.returning.mockResolvedValueOnce([{ id: 'schedule-1', status: 'disabled' }])
|
|
|
|
await applyScheduleFailureUpdate(BASE)
|
|
|
|
const condition = dbChainMockFns.where.mock.calls.at(-1)?.[0]
|
|
expect(JSON.stringify(condition)).toContain('disabled')
|
|
})
|
|
})
|
|
|
|
describe('releaseScheduleLock', () => {
|
|
beforeEach(() => {
|
|
vi.clearAllMocks()
|
|
resetDbChainMock()
|
|
})
|
|
|
|
it('never notifies, even when the row it releases is already disabled', async () => {
|
|
dbChainMockFns.returning.mockResolvedValueOnce([{ id: 'schedule-1', status: 'disabled' }])
|
|
|
|
const released = await releaseScheduleLock('schedule-1', 'req-1', new Date(), 'release')
|
|
|
|
expect(released).toBe(true)
|
|
expect(notifyMock).not.toHaveBeenCalled()
|
|
})
|
|
})
|
|
|
|
describe('schedule cancellation accounting', () => {
|
|
it('keeps a user cancellation distinct from a workflow failure', () => {
|
|
expect(classifyScheduleExecutionResult({ success: false, status: 'cancelled' }, false)).toBe(
|
|
'cancelled'
|
|
)
|
|
})
|
|
|
|
it('continues to classify timeout cancellation as failure', () => {
|
|
expect(classifyScheduleExecutionResult({ success: false, status: 'cancelled' }, true)).toBe(
|
|
'failure'
|
|
)
|
|
})
|
|
|
|
it('uses the persisted cancellation CAS as the authoritative terminal outcome', () => {
|
|
expect(
|
|
classifyScheduleExecutionResult({ success: true, status: 'completed' }, false, 'cancelled')
|
|
).toBe('cancelled')
|
|
expect(
|
|
classifyScheduleExecutionResult({ success: false, status: 'cancelled' }, true, 'cancelled')
|
|
).toBe('cancelled')
|
|
})
|
|
|
|
it('advances cadence and releases the claim without changing failure accounting', () => {
|
|
const now = new Date('2026-08-03T12:00:00.000Z')
|
|
const nextRunAt = new Date('2026-08-03T13:00:00.000Z')
|
|
|
|
expect(buildScheduleCancellationUpdate(now, nextRunAt)).toEqual({
|
|
lastRanAt: now,
|
|
updatedAt: now,
|
|
nextRunAt,
|
|
lastQueuedAt: null,
|
|
infraRetryCount: 0,
|
|
})
|
|
expect(buildScheduleCancellationUpdate(now, nextRunAt)).not.toHaveProperty('failedCount')
|
|
expect(buildScheduleCancellationUpdate(now, nextRunAt)).not.toHaveProperty('lastFailedAt')
|
|
})
|
|
})
|