more types/state tests

chartlets.js/packages/lib/src/types/state/component.test.ts

@@ -0,0 +1,22 @@
+import { describe, it, expect } from "vitest";
+import { isComponentState, isContainerState } from "./component";
+
+describe("isComponentState", () => {
+  it("should work", () => {
+    expect(isComponentState({ type: "Button" })).toBe(true);
+    expect(isComponentState({ type: "Button", children: [] })).toBe(true);
+    expect(isComponentState({})).toBe(false);
+    expect(isComponentState("Button")).toBe(false);
+    expect(isComponentState({ type: 2 })).toBe(false);
+  });
+});
+
+describe("isContainerState", () => {
+  it("should work", () => {
+    expect(isContainerState({ type: "Button" })).toBe(false);
+    expect(isContainerState({ type: "Button", children: [] })).toBe(true);
+    expect(isContainerState({})).toBe(false);
+    expect(isContainerState("Button")).toBe(false);
+    expect(isContainerState({ type: 2 })).toBe(false);
+  });
+});

chartlets.js/packages/lib/src/types/state/options.test.ts

@@ -0,0 +1,28 @@
+import { describe, it, expect } from "vitest";
+import { isMutableHostStore } from "./options";
+
+const hostStore = {
+  get: (name: string) => name,
+  subscribe: (_cb: () => void) => {},
+};
+
+const mutableHostStore = {
+  ...hostStore,
+  set: (_name: string, _value: unknown) => {},
+};
+
+describe("isHostStore", () => {
+  it("should work", () => {
+    expect(isMutableHostStore({})).toBe(false);
+    expect(isMutableHostStore(hostStore)).toBe(true);
+    expect(isMutableHostStore(mutableHostStore)).toBe(true);
+  });
+});
+
+describe("isMutableHostStore", () => {
+  it("should work", () => {
+    expect(isMutableHostStore({})).toBe(false);
+    expect(isMutableHostStore(hostStore)).toBe(false);
+    expect(isMutableHostStore(mutableHostStore)).toBe(true);
+  });
+});

chartlets.js/packages/lib/src/types/state/options.ts

@@ -3,6 +3,8 @@ import type { ComponentType } from "react";
 import type { ApiOptions } from "@/types/api";
 import type { LoggingOptions } from "@/actions/helpers/configureLogging";
 import type { ComponentProps } from "@/component/Component";
+import { isObject } from "@/utils/isObject";
+import { isFunction } from "@/utils/isFunction";
 
 /**
  * The host store represents an interface to the state of
@@ -52,10 +54,14 @@ export interface MutableHostStore extends HostStore {
   set: (property: string, value: unknown) => void;
 }
 
-export function isMutableHostStore(
-  hostStore: HostStore | undefined,
-): hostStore is MutableHostStore {
-  return !!hostStore && typeof hostStore.set === "function";
+export function isHostStore(value: unknown): value is HostStore {
+  return (
+    isObject(value) && isFunction(value.get) && isFunction(value.subscribe)
+  );
+}
+
+export function isMutableHostStore(value: unknown): value is MutableHostStore {
+  return isHostStore(value) && isFunction(value.set);
 }
 
 /**