include vega in test coverage again and add typing for useSinalListeners

Author: clarasb

chartlets.js/packages/lib/src/plugins/vega/VegaChart.test.tsx

@@ -50,7 +50,7 @@ describe("VegaChart", () => {
 });
 
 const chart: TopLevelSpec = {
-  $schema: "https://vega.github.io/schema/vega-lite/v5.20.1.json",
+  $schema: "https://vega.github.io/schema/vega-lite/v6.json",
   config: { view: { continuousWidth: 300, continuousHeight: 300 } },
   data: { name: "data-0" },
   mark: { type: "bar" },

chartlets.js/packages/lib/src/plugins/vega/hooks/useSignalListeners.ts

@@ -25,6 +25,11 @@ type SelectionParameter = {
   select: "point" | "interval" | { type: "point" | "interval"; on: string };
 };
 
+type UseSignalListenersReturn = {
+  onEmbed: (result: VegaEmbedResult) => void;
+  signalListenerMap: Record<string, SignalHandler>;
+};
+
 const isSelectionParameter = (param: unknown): param is SelectionParameter =>
   isObject(param) &&
   (param.select === "point" ||
@@ -38,7 +43,7 @@ export function useSignalListeners(
   type: string,
   id: string | undefined,
   onChange: ComponentChangeHandler,
-) /*: Record<string, SignalHandler>*/ {
+): UseSignalListenersReturn {
   /*
    * Here, we create map of signals which will be then used to create the
    * map of signal-listeners because not all params are event-listeners, and we

chartlets.js/packages/lib/vite.config.ts

@@ -78,6 +78,5 @@ export default defineConfig({
         return false;
       }
     },
-    exclude: ["**/vega/index.test.ts", "**/vega/VegaChart.test.tsx"],
   },
 });