Add Tests for Components

[aaronbrethorst]

No description provided.

[coveralls]

coverage: 74.359% (+58.3%) from 16.097%
when pulling 111fef7 on tests
into d4b0cd3 on main.

[Copilot]

Pull Request Overview

This PR adds comprehensive testing infrastructure and component tests for the Svelte application.

• Introduces a global Vitest setup (vitest-setup.js) with mocks for browser APIs, SvelteKit stores, environment variables, and i18n.
• Updates vite.config.js to include the Svelte testing plugin, configure coverage reporters, thresholds, and setup files.
• Adds MSW server setup, mock providers, shared test utilities, fixtures, and dozens of new component tests.

Reviewed Changes

Copilot reviewed 32 out of 33 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
vitest-setup.js	Adds global mocks for browser APIs, env vars, i18n
vite.config.js	Includes @testing-library/svelte plugin and coverage config
src/tests/setup/msw-server.js	Configures MSW server lifecycle for tests
src/tests/mocks/mapProviders.js	Provides mock implementations for map providers
src/tests/helpers/test-utils.js	Shared render and store mock utilities
src/components/search/SearchField.svelte	Trim and validate search input with error handling
src/components/search/SearchPane.svelte	Wraps route fetch in try/catch
src/lib/keybinding.js	Refactors keybinding action to support updates

Comments suppressed due to low confidence (3)

src/components/search/SearchField.svelte:13

• Add a check for response.ok before parsing JSON. If the response is not OK, handle or forward the error (e.g., call handleSearchResults({ error: ... })) instead of always calling response.json().

	async function handleSearch() {

src/tests/mocks/handlers.js:65

• [nitpick] Using Date.now() directly in handlers leads to non-deterministic test data. Consider using a fixed timestamp or mocking Date.now() with vi.useFakeTimers() for stable and reproducible tests.

							serviceDate: Date.now(),

src/lib/keybinding.js:15

• Ensure that setHandler() is invoked on mount (without arguments) so the initial keydown listener is registered. If omitted, the action will never listen for key events until an update occurs.

	const setHandler = (newParams) => {

[Copilot]

Add if (!response.ok) handling after this fetch to avoid unhandled JSON parsing errors. Consider notifying the user or falling back gracefully instead of only logging to console.

Suggested change

	const response = await fetch(`/api/oba/stops-for-route/${route.id}`);
	const response = await fetch(`/api/oba/stops-for-route/${route.id}`);
	if (!response.ok) {
	throw new Error(`Failed to fetch route data: ${response.status} ${response.statusText}`);
	}

[Copilot]

[nitpick] Tests import $env/dynamic/public in some modules. Consider adding a mock for $env/dynamic/public here to ensure consistent environment variable behavior across tests.