Skip to content

Conversation

@briangregoryholmes
Copy link
Contributor

@briangregoryholmes briangregoryholmes commented Jan 20, 2026

Summary

Adds e2e test coverage for rendering of the primary timeseries line.

This PR follows up on the timezone-related fixes in:

Changes

E2E Tests

  • Adds tests to verify correct point and annotation placement on timeseries charts
  • Validates that timezone handling works correctly across different dashboard configurations

Import Fixes

Fixed runtime-client imports across 27 files. The types were previously imported from a barrel file that also exports svelte-query related modules. Since svelte-query has Svelte dependencies that cannot be resolved in the Playwright test environment, the imports needed to be updated to reference the type definitions directly, bypassing the barrel file.

Checklist:

  • Covered by tests
  • Ran it and it works as intended
  • Reviewed the diff before requesting a review
  • Checked for unhandled edge cases
  • Linked the issues it closes
  • Checked if the docs need to be updated. If so, create a separate Linear DOCS issue
  • Intend to cherry-pick into the release branch
  • I'm proud of this work!

@briangregoryholmes briangregoryholmes self-assigned this Jan 20, 2026
@briangregoryholmes briangregoryholmes changed the title chore: timeseries e2e test chore: timeseries e2e test coverage Jan 20, 2026
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.

2 participants