Add tests

src/lib/data.ts

@@ -15,7 +15,7 @@ export interface AbstractLocale {
 }
 
 export const US_REMOTE_LOCALE = 'us';
-export const CA_REMOTE_LOCALE = 'ca';
+export const CA_REMOTE_LOCALE = 'canada';
 export const OTHER_LOCALE = 'other';
 
 export class UnknownLocale implements AbstractLocale {
@@ -61,7 +61,6 @@ export class AllCanadaLocale implements AbstractLocale {
 }
 
 export class Locale implements AbstractLocale {
-	effectiveDate?: string;
 	country!: string;
 	state!: string;
 	stateCode!: string;

src/tests/checking.test.ts

@@ -5,9 +5,11 @@ import {
 	Strength,
 	OTHER_LOCALE,
 	US_REMOTE_LOCALE,
+	CA_REMOTE_LOCALE,
 	type AbstractLocale,
 	UnknownLocale,
-	AllUSLocale
+	AllUSLocale,
+	AllCanadaLocale
 } from '$lib/data';
 import { describe, expect, it } from 'vitest';
 
@@ -119,6 +121,19 @@ const locales: Record<string, Locale> = [
 		who: {
 			officeSupervisorInLocale: true
 		}
+	}),
+	new Locale({
+		country: 'Canada',
+		state: 'British Columbia',
+		stateCode: 'BC',
+		strength: Strength.Strong,
+		what: {
+			salary: true
+		},
+		when: [{ situation: Situation.Interested }],
+		who: {
+			officeSupervisorInLocale: true
+		}
 	})
 ].reduce((map, locale) => {
 	map[locale.id] = locale;
@@ -128,6 +143,7 @@ const locales: Record<string, Locale> = [
 const allLocales: Record<string, AbstractLocale> = {
 	[OTHER_LOCALE]: new UnknownLocale(),
 	[US_REMOTE_LOCALE]: new AllUSLocale(),
+	[CA_REMOTE_LOCALE]: new AllCanadaLocale(),
 	...locales
 };
 
@@ -301,12 +317,42 @@ describe('matches with placeholder role location', () => {
 	it('does not include non-matching location', () => {
 		expect(matches.filter((m) => m.locale.id === 'nevada').length).toBe(0);
 	});
+	it('does not include non-matching Canada location', () => {
+		expect(matches.filter((m) => m.locale.id === 'canada-british-columbia').length).toBe(0);
+	});
 	it('does not include abstract location matches', () => {
 		expect(matches.filter((m) => m.locale.id === 'us').length).toBe(0);
+		expect(matches.filter((m) => m.locale.id === 'canada').length).toBe(0);
 		expect(matches.filter((m) => m.locale.id === 'other').length).toBe(0);
 	});
 });
 
+describe('handles Canadian and cross-border roles', () => {
+	const matches = findMatchingLaws(
+		{
+			situation: Situation.Interested,
+			userLocation: 'canada-british-columbia',
+			companyLocation: 'colorado',
+			officeSupervisorLocation: 'other',
+			employeeInLocation: true,
+			totalEmployees: 50,
+			roleLocation: ['us', 'canada', 'other']
+		},
+		locales,
+		allLocales
+	);
+
+	it('includes matching user location in Canada', () => {
+		expect(matches).toContainEqual({
+			locale: locales['canada-british-columbia'],
+			earliestDisclosurePoint: Situation.Interested,
+			minEmployeesInLocale: 0,
+			what: { salary: true },
+			isGeoMatch: true
+		});
+	});
+});
+
 describe('matches with placeholder user location', () => {
 	const matches = findMatchingLaws(
 		{