@zeix/cause-effect 0.15.2 → 0.16.0

package/test/state.test.ts

@@ -1,12 +1,12 @@
 import { describe, expect, test } from 'bun:test'
-import { isComputed, isState, state } from '../'
+import { createState, isComputed, isState } from '../'
 
 /* === Tests === */
 
 describe('State', () => {
 	describe('State type guard', () => {
 		test('isState identifies state signals', () => {
-			const count = state(42)
+			const count = createState(42)
 			expect(isState(count)).toBe(true)
 			expect(isComputed(count)).toBe(false)
 		})
@@ -14,28 +14,28 @@ describe('State', () => {
 
 	describe('Boolean cause', () => {
 		test('should be boolean', () => {
-			const cause = state(false)
+			const cause = createState(false)
 			expect(typeof cause.get()).toBe('boolean')
 		})
 
 		test('should set initial value to false', () => {
-			const cause = state(false)
+			const cause = createState(false)
 			expect(cause.get()).toBe(false)
 		})
 
 		test('should set initial value to true', () => {
-			const cause = state(true)
+			const cause = createState(true)
 			expect(cause.get()).toBe(true)
 		})
 
 		test('should set new value with .set(true)', () => {
-			const cause = state(false)
+			const cause = createState(false)
 			cause.set(true)
 			expect(cause.get()).toBe(true)
 		})
 
 		test('should toggle initial value with .set(v => !v)', () => {
-			const cause = state(false)
+			const cause = createState(false)
 			cause.update(v => !v)
 			expect(cause.get()).toBe(true)
 		})
@@ -43,23 +43,23 @@ describe('State', () => {
 
 	describe('Number cause', () => {
 		test('should be number', () => {
-			const cause = state(0)
+			const cause = createState(0)
 			expect(typeof cause.get()).toBe('number')
 		})
 
 		test('should set initial value to 0', () => {
-			const cause = state(0)
+			const cause = createState(0)
 			expect(cause.get()).toBe(0)
 		})
 
 		test('should set new value with .set(42)', () => {
-			const cause = state(0)
+			const cause = createState(0)
 			cause.set(42)
 			expect(cause.get()).toBe(42)
 		})
 
 		test('should increment value with .set(v => ++v)', () => {
-			const cause = state(0)
+			const cause = createState(0)
 			cause.update(v => ++v)
 			expect(cause.get()).toBe(1)
 		})
@@ -67,23 +67,23 @@ describe('State', () => {
 
 	describe('String cause', () => {
 		test('should be string', () => {
-			const cause = state('foo')
+			const cause = createState('foo')
 			expect(typeof cause.get()).toBe('string')
 		})
 
 		test('should set initial value to "foo"', () => {
-			const cause = state('foo')
+			const cause = createState('foo')
 			expect(cause.get()).toBe('foo')
 		})
 
 		test('should set new value with .set("bar")', () => {
-			const cause = state('foo')
+			const cause = createState('foo')
 			cause.set('bar')
 			expect(cause.get()).toBe('bar')
 		})
 
 		test('should upper case value with .set(v => v.toUpperCase())', () => {
-			const cause = state('foo')
+			const cause = createState('foo')
 			cause.update(v => (v ? v.toUpperCase() : ''))
 			expect(cause.get()).toBe('FOO')
 		})
@@ -91,56 +91,243 @@ describe('State', () => {
 
 	describe('Array cause', () => {
 		test('should be array', () => {
-			const cause = state([1, 2, 3])
+			const cause = createState([1, 2, 3])
 			expect(Array.isArray(cause.get())).toBe(true)
 		})
 
 		test('should set initial value to [1, 2, 3]', () => {
-			const cause = state([1, 2, 3])
+			const cause = createState([1, 2, 3])
 			expect(cause.get()).toEqual([1, 2, 3])
 		})
 
 		test('should set new value with .set([4, 5, 6])', () => {
-			const cause = state([1, 2, 3])
+			const cause = createState([1, 2, 3])
 			cause.set([4, 5, 6])
 			expect(cause.get()).toEqual([4, 5, 6])
 		})
 
 		test('should reflect current value of array after modification', () => {
 			const array = [1, 2, 3]
-			const cause = state(array)
+			const cause = createState(array)
 			array.push(4) // don't do this! the result will be correct, but we can't trigger effects
 			expect(cause.get()).toEqual([1, 2, 3, 4])
 		})
 
 		test('should set new value with .set([...array, 4])', () => {
 			const array = [1, 2, 3]
-			const cause = state(array)
+			const cause = createState(array)
 			cause.set([...array, 4]) // use destructuring instead!
 			expect(cause.get()).toEqual([1, 2, 3, 4])
 		})
+
+		describe('Input Validation', () => {
+			test('should throw NullishSignalValueError when initialValue is nullish', () => {
+				expect(() => {
+					// @ts-expect-error - Testing invalid input
+					createState(null)
+				}).toThrow('Nullish signal values are not allowed in state')
+
+				expect(() => {
+					// @ts-expect-error - Testing invalid input
+					createState(undefined)
+				}).toThrow('Nullish signal values are not allowed in state')
+			})
+
+			test('should throw NullishSignalValueError when newValue is nullish in set()', () => {
+				const state = createState(42)
+
+				expect(() => {
+					// @ts-expect-error - Testing invalid input
+					state.set(null)
+				}).toThrow('Nullish signal values are not allowed in state')
+
+				expect(() => {
+					// @ts-expect-error - Testing invalid input
+					state.set(undefined)
+				}).toThrow('Nullish signal values are not allowed in state')
+			})
+
+			test('should throw specific error types for nullish values', () => {
+				try {
+					// @ts-expect-error - Testing invalid input
+					createState(null)
+					expect(true).toBe(false) // Should not reach here
+				} catch (error) {
+					expect(error).toBeInstanceOf(TypeError)
+					expect(error.name).toBe('NullishSignalValueError')
+					expect(error.message).toBe(
+						'Nullish signal values are not allowed in state',
+					)
+				}
+
+				const state = createState(42)
+				try {
+					// @ts-expect-error - Testing invalid input
+					state.set(null)
+					expect(true).toBe(false) // Should not reach here
+				} catch (error) {
+					expect(error).toBeInstanceOf(TypeError)
+					expect(error.name).toBe('NullishSignalValueError')
+					expect(error.message).toBe(
+						'Nullish signal values are not allowed in state',
+					)
+				}
+			})
+
+			test('should allow valid non-nullish values', () => {
+				// These should not throw
+				expect(() => {
+					createState(0)
+				}).not.toThrow()
+
+				expect(() => {
+					createState('')
+				}).not.toThrow()
+
+				expect(() => {
+					createState(false)
+				}).not.toThrow()
+
+				expect(() => {
+					createState({})
+				}).not.toThrow()
+
+				expect(() => {
+					createState([])
+				}).not.toThrow()
+
+				const state = createState(42)
+				expect(() => {
+					state.set(0)
+				}).not.toThrow()
+
+				expect(() => {
+					// @ts-expect-error - Testing valid input of invalid type
+					state.set('')
+				}).not.toThrow()
+			})
+
+			test('should throw InvalidCallbackError for non-function updater in update()', () => {
+				const state = createState(42)
+
+				expect(() => {
+					// @ts-expect-error - Testing invalid input
+					state.update(null)
+				}).toThrow('Invalid state update callback null')
+
+				expect(() => {
+					// @ts-expect-error - Testing invalid input
+					state.update(undefined)
+				}).toThrow('Invalid state update callback undefined')
+
+				expect(() => {
+					// @ts-expect-error - Testing invalid input
+					state.update('not a function')
+				}).toThrow('Invalid state update callback "not a function"')
+
+				expect(() => {
+					// @ts-expect-error - Testing invalid input
+					state.update(42)
+				}).toThrow('Invalid state update callback 42')
+			})
+
+			test('should throw specific error type for non-function updater', () => {
+				const state = createState(42)
+
+				try {
+					// @ts-expect-error - Testing invalid input
+					state.update(null)
+					expect(true).toBe(false) // Should not reach here
+				} catch (error) {
+					expect(error).toBeInstanceOf(TypeError)
+					expect(error.name).toBe('InvalidCallbackError')
+					expect(error.message).toBe(
+						'Invalid state update callback null',
+					)
+				}
+			})
+
+			test('should handle updater function that throws an error', () => {
+				const state = createState(42)
+
+				expect(() => {
+					state.update(() => {
+						throw new Error('Updater error')
+					})
+				}).toThrow('Updater error')
+
+				// State should remain unchanged after error
+				expect(state.get()).toBe(42)
+			})
+
+			test('should handle updater function that returns nullish value', () => {
+				const state = createState(42)
+
+				expect(() => {
+					// @ts-expect-error - Testing invalid return value
+					state.update(() => null)
+				}).toThrow('Nullish signal values are not allowed in state')
+
+				expect(() => {
+					// @ts-expect-error - Testing invalid return value
+					state.update(() => undefined)
+				}).toThrow('Nullish signal values are not allowed in state')
+
+				// State should remain unchanged after error
+				expect(state.get()).toBe(42)
+			})
+
+			test('should handle valid updater functions', () => {
+				const numberState = createState(10)
+				expect(() => {
+					numberState.update(x => x + 5)
+				}).not.toThrow()
+				expect(numberState.get()).toBe(15)
+
+				const stringState = createState('hello')
+				expect(() => {
+					stringState.update(x => x.toUpperCase())
+				}).not.toThrow()
+				expect(stringState.get()).toBe('HELLO')
+
+				const arrayState = createState([1, 2, 3])
+				expect(() => {
+					arrayState.update(arr => [...arr, 4])
+				}).not.toThrow()
+				expect(arrayState.get()).toEqual([1, 2, 3, 4])
+
+				const objectState = createState({ count: 0 })
+				expect(() => {
+					objectState.update(obj => ({
+						...obj,
+						count: obj.count + 1,
+					}))
+				}).not.toThrow()
+				expect(objectState.get()).toEqual({ count: 1 })
+			})
+		})
 	})
 
 	describe('Object cause', () => {
 		test('should be object', () => {
-			const cause = state({ a: 'a', b: 1 })
+			const cause = createState({ a: 'a', b: 1 })
 			expect(typeof cause.get()).toBe('object')
 		})
 
 		test('should set initial value to { a: "a", b: 1 }', () => {
-			const cause = state({ a: 'a', b: 1 })
+			const cause = createState({ a: 'a', b: 1 })
 			expect(cause.get()).toEqual({ a: 'a', b: 1 })
 		})
 
 		test('should set new value with .set({ c: true })', () => {
-			const cause = state<Record<string, unknown>>({ a: 'a', b: 1 })
+			const cause = createState<Record<string, unknown>>({ a: 'a', b: 1 })
 			cause.set({ c: true })
 			expect(cause.get()).toEqual({ c: true })
 		})
 
 		test('should reflect current value of object after modification', () => {
 			const obj = { a: 'a', b: 1 }
-			const cause = state<Record<string, unknown>>(obj)
+			const cause = createState<Record<string, unknown>>(obj)
 			// @ts-expect-error Property 'c' does not exist on type '{ a: string; b: number; }'. (ts 2339)
 			obj.c = true // don't do this! the result will be correct, but we can't trigger effects
 			expect(cause.get()).toEqual({ a: 'a', b: 1, c: true })
@@ -148,7 +335,7 @@ describe('State', () => {
 
 		test('should set new value with .set({...obj, c: true})', () => {
 			const obj = { a: 'a', b: 1 }
-			const cause = state<Record<string, unknown>>(obj)
+			const cause = createState<Record<string, unknown>>(obj)
 			cause.set({ ...obj, c: true }) // use destructuring instead!
 			expect(cause.get()).toEqual({ a: 'a', b: 1, c: true })
 		})