Skip to content

test(calendar): stabilize date-sensitive page fixtures - #213

Merged
h4yfans merged 1 commit into
mainfrom
feat/fix-calendar-test-fixtures
Apr 15, 2026
Merged

test(calendar): stabilize date-sensitive page fixtures#213
h4yfans merged 1 commit into
mainfrom
feat/fix-calendar-test-fixtures

Conversation

@h4yfans

@h4yfans h4yfans commented Apr 15, 2026

Copy link
Copy Markdown
Collaborator

What

Stabilize the CalendarPage renderer fixtures so they follow the active local day instead of hard-coded April 2026 timestamps.

Why

All open tech-debt PRs are currently blocked by the same failing renderer test in calendar-page.test.tsx. The test data was anchored to stale fixed dates while CalendarPage derives its initial day view from the current date, so CI started failing once the calendar fixtures drifted out of sync with the runtime day.

How

Use a small local-date fixture helper in calendar-page.test.tsx.
Keep the day-view Memry items on the current local day.
Move the imported calendar event to the next local day so the month-view assertions still render both Memry and imported items without overflowing one cell.

Type

  • feat — new feature
  • fix — bug fix
  • refactor — restructure without behavior change
  • style — visual/UI only
  • test — adding or updating tests
  • chore — tooling, deps, config
  • docs — documentation only
  • ci — CI/CD changes

Test plan

  • Unit tests added/updated
  • Integration tests added/updated
  • Manual testing (describe below)

pnpm exec vitest run --config config/vitest.config.ts src/renderer/src/components/calendar/calendar-page.test.tsx
pnpm lint

Checklist

  • Self-reviewed the diff
  • No hardcoded secrets or credentials
  • Files stay under ~500 LOC
  • Follows immutable data patterns

@h4yfans
h4yfans merged commit 9c7004c into main Apr 15, 2026
2 checks passed
@h4yfans
h4yfans deleted the feat/fix-calendar-test-fixtures branch April 15, 2026 20:10
h4yfans added a commit that referenced this pull request May 6, 2026
test(calendar): stabilize date-sensitive page fixtures
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant