Clean up some minor issues; add ai? tests?

Author: switz

README.md

@@ -342,7 +342,7 @@ driver({
     isUploaded: match.demo_uploaded,
   },
   derived: {
-+   isDisabled: (_, stateEnums, activeEnum) => (activeEnum ?? 0) <= stateEnums.isUploading,
++   isDisabled: (_, enums, activeEnum) => (activeEnum ?? 0) <= enums.isUploading,
   }
 })
 ```

package.json

@@ -37,7 +37,7 @@
     "name": "Daniel Saewitz (switz)",
     "url": "https://saewitz.com"
   },
-  "version": "0.9.0",
+  "version": "0.10.0",
   "repository": {
     "type": "git",
     "url": "https://github.com/switz/driver.git"

src/types.ts

@@ -11,7 +11,7 @@ type RequireAtLeastOne<T, Keys extends keyof T = keyof T> = Pick<T, Exclude<keyo
 
 type DerivedFn<StateKeys extends string> = (
   states: Record<StateKeys, boolean | undefined>,
-  stateEnums: Record<StateKeys, number>,
+  enums: Record<StateKeys, number>,
   activeEnum: number | undefined
 ) => unknown;
 
@@ -35,7 +35,7 @@ export type Return<T extends string, K extends DerivedConfig<T>> = DerivedReturn
 type MetadataReturn<T extends string> = {
   activeState: T | undefined;
   activeEnum: number | undefined;
-  stateEnums: Record<T, number>;
+  enums: Record<T, number>;
   states: Record<T, boolean | undefined>;
 };
 

test/ai.test.ts

@@ -0,0 +1,80 @@
+// these were written with ai?
+
+import { test, expect } from 'bun:test';
+import driver from '../src/index.js';
+
+test('state key ordering', () => {
+  const result = driver({
+    states: { a1: true, a2: false },
+  });
+  expect(result.activeState).toBe('a1');
+  expect(result.activeEnum).toBe(0);
+  expect(Object.keys(result.enums)).toEqual(['a1', 'a2']);
+});
+
+test('derived keys with undefined values', () => {
+  const result = driver({
+    states: { a: true, b: false },
+    derived: { foo: undefined },
+  });
+  expect(result.foo).toBeUndefined();
+});
+
+test('derived keys with falsy values', () => {
+  const result = driver({
+    states: { a: true, b: false },
+    derived: { foo: 0, bar: false },
+  });
+  expect(result.foo).toBe(0);
+  expect(result.bar).toBe(false);
+});
+
+test('derived keys with functions', () => {
+  const result = driver({
+    states: { a: true, b: false },
+    derived: { foo: (states, enums) => Object.keys(states).length },
+  });
+  expect(result.foo).toBe(2);
+});
+
+test('derived keys with arrays', () => {
+  const result = driver({
+    states: { a: true, b: false },
+    derived: { foo: ['a', 'c'] },
+  });
+  expect(result.foo).toBe(true);
+});
+
+test('derived keys with objects', () => {
+  const result = driver({
+    states: { a: true, b: false },
+    derived: { foo: { a: 'bar', b: 'baz' } },
+  });
+  expect(result.foo).toBe('bar');
+});
+
+test('empty states', () => {
+  const result = driver({ states: {} });
+  expect(result.activeState).toBeUndefined();
+  expect(result.activeEnum).toBeUndefined();
+  expect(result.enums).toEqual({});
+});
+
+test('no active state', () => {
+  const result = driver({ states: { a: false, b: false } });
+  expect(result.activeState).toBeUndefined();
+  expect(result.activeEnum).toBeUndefined();
+});
+
+test('invalid state key type', () => {
+  expect(() => driver({ states: { 1: true } })).toThrow();
+});
+
+test('invalid derived key type', () => {
+  expect(() =>
+    driver({
+      states: { a: true },
+      derived: { foo: 123 },
+    })
+  ).not.toThrow();
+});

test/enums.test.ts

@@ -12,7 +12,7 @@ test('basic enum test', () => {
       isUploaded: false,
     },
     derived: {
-      isDisabled: (_, stateEnums, activeEnum) => (activeEnum ?? 0) <= stateEnums.isUploading,
+      isDisabled: (_, enums, activeEnum) => (activeEnum ?? 0) <= enums.isUploading,
       text: {
         isNotRecorded: 'Demo Disabled',
         isUploading: 'Demo Uploading...',
@@ -36,7 +36,7 @@ test('basic enum test2', () => {
       isUploaded: true,
     },
     derived: {
-      isDisabled: (_, stateEnums, activeEnum) => (activeEnum ?? 0) <= stateEnums.isUploading,
+      isDisabled: (_, enums, activeEnum) => (activeEnum ?? 0) <= enums.isUploading,
       text: {
         isNotRecorded: 'Demo Disabled',
         isUploading: 'Demo Uploading...',
@@ -58,7 +58,7 @@ test('enums: when no state is true', () => {
       isUploaded: false,
     },
     derived: {
-      isDisabled: (_, stateEnums, activeEnum) => (activeEnum ?? 0) <= stateEnums.isUploading,
+      isDisabled: (_, enums, activeEnum) => (activeEnum ?? 0) <= enums.isUploading,
       text: {
         isNotRecorded: 'Demo Disabled',
         isUploading: 'Demo Uploading...',

test/types.test-d.ts

@@ -18,7 +18,7 @@ const derived = driver({
 expectType<boolean | undefined>(derived.isDisabled);
 expectType<number | undefined>(derived.activeEnum);
 expectType<'hello' | 'foobar' | 'test' | undefined>(derived.activeState);
-expectType<Record<'hello' | 'foobar' | 'test', number>>(derived.stateEnums);
+expectType<Record<'hello' | 'foobar' | 'test', number>>(derived.enums);
 expectType<string | undefined>(derived.optionalParams);
 
 const allDerived = driver({
@@ -31,14 +31,13 @@ const allDerived = driver({
     params: {
       hello: 'hello',
       foobar: 'hi',
-      test: 'foo'
+      test: 'foo',
     },
   },
 });
 
 expectType<string | undefined>(allDerived.params);
 
-
 // expect an error because no params are passed into a flag
 expectError(
   driver({
@@ -61,7 +60,7 @@ expectError(
       foobar: false,
     },
     derived: {
-      stateEnums: {
+      enums: {
         foo: 1,
       },
     },