fix: tests

Author: bryanmylee

src/lib/columns.newDataColumn.test.ts

@@ -28,11 +28,12 @@ it('falls back on the string accessor as id', () => {
 	expect(actual.id).toBe('firstName');
 });
 
-it('throws if id is undefined without string accessor', () => {
+it('throws if id is undefined without string accessor or header', () => {
 	expect(() => {
 		new DataColumn<User>({
-			header: 'First Name',
-			accessor: (u) => u.firstName,
-		});
-	}).toThrowError('A column id or string accessor is required');
+			// eslint-disable-next-line @typescript-eslint/no-explicit-any
+			accessor: (u: any) => u.firstName,
+			// eslint-disable-next-line @typescript-eslint/no-explicit-any
+		} as any);
+	}).toThrowError('A column id, string accessor, or header is required');
 });

src/lib/columns.ts

@@ -136,7 +136,8 @@ export class DataColumn<
 		if (id === undefined && this.accessorKey === undefined && header === undefined) {
 			throw new Error('A column id, string accessor, or header is required');
 		}
-		this.id = (id ?? this.accessorKey ? String(this.accessorKey) : String(header)) as Id;
+		const accessorKeyId = typeof this.accessorKey === 'string' ? this.accessorKey : null;
+		this.id = (id ?? accessorKeyId ?? String(header)) as Id;
 	}
 
 	// eslint-disable-next-line @typescript-eslint/no-explicit-any