@ossy/timesheets 3.8.0 → 3.10.0

package/src/timesheet-reminder.spec.js

@@ -0,0 +1,75 @@
+import { describe, expect, it } from '@jest/globals'
+import {
+  formatPeriodLabel,
+  resolveTimesheetReminderRecipient,
+  shouldNotifyTimesheetCreated,
+  timesheetAssignedUrl,
+} from './timesheet-reminder.js'
+
+describe('shouldNotifyTimesheetCreated', () => {
+  it('notifies only auto-generated creates', () => {
+    expect(shouldNotifyTimesheetCreated({
+      resourceId: 'sheet-1',
+      payload: { content: { autoGenerated: true } },
+    })).toBe(true)
+
+    expect(shouldNotifyTimesheetCreated({
+      resourceId: 'sheet-2',
+      payload: { content: { autoGenerated: false } },
+    })).toBe(false)
+
+    expect(shouldNotifyTimesheetCreated({
+      resourceId: 'sheet-3',
+      payload: { content: {} },
+    })).toBe(false)
+
+    expect(shouldNotifyTimesheetCreated({
+      payload: { content: { autoGenerated: true } },
+    })).toBe(false)
+  })
+})
+
+describe('resolveTimesheetReminderRecipient', () => {
+  it('requires a linked employee with a valid email', () => {
+    const employees = [
+      { id: 'emp-1', content: { name: 'Ada', email: 'ada@example.com' } },
+      { id: 'emp-2', content: { name: 'Bob', email: 'not-an-email' } },
+    ]
+
+    expect(resolveTimesheetReminderRecipient(employees, {
+      content: { employeeId: 'emp-1' },
+    })).toEqual({ email: 'ada@example.com', name: 'Ada' })
+
+    expect(resolveTimesheetReminderRecipient(employees, {
+      content: { employeeId: 'emp-2' },
+    })).toBe(null)
+
+    expect(resolveTimesheetReminderRecipient(employees, {
+      content: {},
+    })).toBe(null)
+  })
+})
+
+describe('timesheetAssignedUrl', () => {
+  it('builds the Swedish home URL with an optional sheet query', () => {
+    expect(timesheetAssignedUrl(undefined, 'https://app.ossy.se/')).toBe(
+      'https://app.ossy.se/tidrapporter',
+    )
+    expect(timesheetAssignedUrl('sheet-1', 'https://app.ossy.se')).toBe(
+      'https://app.ossy.se/tidrapporter?r=sheet-1',
+    )
+    expect(timesheetAssignedUrl('sheet-1', '')).toBe(null)
+  })
+})
+
+describe('formatPeriodLabel', () => {
+  it('formats inclusive local dates', () => {
+    const label = formatPeriodLabel(
+      new Date(2026, 7, 1).getTime(),
+      new Date(2026, 7, 31).getTime(),
+      'en-US',
+    )
+    expect(label).toContain('2026')
+    expect(label).toContain('–')
+  })
+})

package/src/timesheet-resources.js

@@ -1,5 +1,11 @@
 import { ResourcesQueries } from '@ossy/resources/server'
 
+export {
+  optionalReference,
+  findTimesheetForPeriod,
+  toTimesheetDto,
+} from './timesheet-dto.js'
+
 /**
  * List timesheet resources by schema id (and optional workspace).
  */
@@ -8,21 +14,3 @@ export function getTimesheetResources ({ schemaId, belongsTo } = {}) {
   if (belongsTo) query.belongsTo = belongsTo
   return ResourcesQueries.GetResources(query)
 }
-
-export function findTimesheetForPeriod (resources, periodStart) {
-  return (resources || []).find((r) => r.content?.periodStart === periodStart) ?? null
-}
-
-export function toTimesheetDto (resource) {
-  if (!resource) return null
-  return {
-    id: resource.id,
-    periodStart: resource.content?.periodStart,
-    periodEnd: resource.content?.periodEnd,
-    status: resource.content?.status ?? 'draft',
-    holidayLocale: resource.content?.holidayLocale ?? null,
-    lines: resource.content?.lines ?? [],
-    createdAt: resource.createdAt,
-    updatedAt: resource.updatedAt,
-  }
-}

package/src/timesheet-resources.spec.js

@@ -0,0 +1,59 @@
+import { describe, expect, it } from '@jest/globals'
+import {
+  findTimesheetForPeriod,
+  optionalReference,
+  toTimesheetDto,
+} from './timesheet-dto.js'
+
+describe('optionalReference', () => {
+  it('normalizes ids, objects, and empty values', () => {
+    expect(optionalReference(' emp-1 ')).toEqual({ resourceId: 'emp-1' })
+    expect(optionalReference({ resourceId: 'emp-1' })).toEqual({ resourceId: 'emp-1' })
+    expect(optionalReference('')).toBe(null)
+    expect(optionalReference(null)).toBe(null)
+    expect(optionalReference({})).toBe(null)
+  })
+})
+
+describe('toTimesheetDto', () => {
+  it('includes optional employee and contract refs', () => {
+    expect(toTimesheetDto({
+      id: 'sheet-1',
+      content: {
+        periodStart: 1,
+        periodEnd: 2,
+        status: 'draft',
+        holidayLocale: 'SE',
+        employeeId: 'emp-1',
+        contractId: { resourceId: 'con-1' },
+        lines: [],
+      },
+      createdAt: 10,
+      updatedAt: 20,
+    })).toEqual({
+      id: 'sheet-1',
+      periodStart: 1,
+      periodEnd: 2,
+      status: 'draft',
+      holidayLocale: 'SE',
+      employeeId: { resourceId: 'emp-1' },
+      contractId: { resourceId: 'con-1' },
+      lines: [],
+      createdAt: 10,
+      updatedAt: 20,
+    })
+  })
+})
+
+describe('findTimesheetForPeriod', () => {
+  it('returns the newest sheet when several share a period', () => {
+    const resources = [
+      { id: 'old', createdAt: 1, content: { periodStart: 100 } },
+      { id: 'new', createdAt: 3, content: { periodStart: 100 } },
+      { id: 'mid', createdAt: 2, content: { periodStart: 100 } },
+      { id: 'other', createdAt: 9, content: { periodStart: 200 } },
+    ]
+    expect(findTimesheetForPeriod(resources, 100)?.id).toBe('new')
+    expect(findTimesheetForPeriod(resources, 999)).toBe(null)
+  })
+})

package/src/timesheet.schema.js

@@ -1,6 +1,8 @@
+import { TimesheetRefs, TimesheetsSchema } from './schema-ids.js'
+
 export default {
   name: 'Timesheet',
-  id: '@ossy/timesheets/schema/timesheet',
+  id: TimesheetsSchema.timesheet,
   categoryName: 'Timesheets',
   icon: 'calendar',
   fields: [
@@ -24,6 +26,23 @@ export default {
       type: 'text',
       description: 'ISO 3166-1 alpha-2 country used for workday/public-holiday prefill',
     },
+    {
+      name: 'employeeId',
+      type: 'reference',
+      of: TimesheetRefs.employee,
+      description: 'Optional employee this timesheet belongs to',
+    },
+    {
+      name: 'contractId',
+      type: 'reference',
+      of: TimesheetRefs.contract,
+      description: 'Optional contract this timesheet belongs to',
+    },
+    {
+      name: 'autoGenerated',
+      type: 'boolean',
+      description: 'Set when the daily auto-generate cron created this sheet',
+    },
     // `lines` is stored in content (array of day rows) — same nested-content pattern as booking availability.
   ],
 }