on-zero 0.9.7 → 0.10.1

package/src/mutations.test.ts

@@ -4,27 +4,6 @@ import { mutations } from './mutations'
 import { serverWhere } from './serverWhere'
 
 describe('mutations registry', () => {
-  test('two modules registering the same table keep both custom mutators', () => {
-    // a template commonly splits a table's mutators across files (the table's
-    // own mutations file plus a seed.ts firing a demo seed on that table).
-    // the registry is keyed by table, so a wholesale replace made whichever
-    // module imported LAST win: alphabetical import order silently dropped
-    // seed.ts's seedDemo whenever the seed file sorted before the table file.
-    const permissions = serverWhere('todo', () => true)
-
-    const seedModule = mutations('todo', permissions, {
-      seedDemo: async () => {},
-    })
-    const tableModule = mutations('todo', permissions, {})
-
-    expect(typeof seedModule.seedDemo).toBe('function')
-    expect(typeof tableModule.insert).toBe('function')
-    // both proxies read the same per-table registry: the later CRUD-only
-    // registration must not clobber the earlier custom mutator
-    expect(typeof (tableModule as Record<string, unknown>).seedDemo).toBe('function')
-    expect(Object.keys(tableModule)).toContain('seedDemo')
-  })
-
   test('re-registering a handler replaces it per key (HMR)', () => {
     const permissions = serverWhere('post', () => true)
     const v1 = async () => {}

package/src/server.test.ts

@@ -121,81 +121,6 @@ describe('createZeroServerBindings', () => {
     expect(rows).toEqual([])
   })
 
-  test('binds auth, queries, pipe keys, effects, and direct execution structurally', async () => {
-    const rows: ProjectRow[] = []
-    let mutationRuns = 0
-    let validatedMutationAuth: string | undefined
-    let validatedQueryAuth: string | undefined
-    let barrierRuns = 0
-    let backgroundRuns = 0
-    let backgroundAuth: string | undefined
-
-    const bindings = createZeroServerBindings({
-      schema,
-      models: {
-        project: {
-          mutate: {
-            create: async (
-              { authData, server, tx }: MutatorContext,
-              value: ProjectRow
-            ) => {
-              mutationRuns++
-              expect(authData?.id).toBe('user-1')
-              await tx.mutate.project.insert(value)
-              server?.enqueueTask(
-                async () => {
-                  barrierRuns++
-                },
-                { barrier: true }
-              )
-              server?.enqueueTask(async () => {
-                backgroundRuns++
-                backgroundAuth = getScopedAuthData()?.id as string | undefined
-              })
-            },
-          },
-        },
-      },
-      createServerActions: () => ({}),
-      queries,
-      validateMutation({ authData }) {
-        validatedMutationAuth = authData?.id as string | undefined
-      },
-      validateQuery({ authData }) {
-        validatedQueryAuth = authData?.id as string | undefined
-      },
-    })
-
-    expect(Object.keys(bindings.mutators)).toEqual(['project|create'])
-    const { executor, queryClaims, scheduledBackground } = createTestExecutor(
-      bindings.mutators,
-      rows
-    )
-    const server = bindings.server(executor)
-    await server.mutate.project.create(
-      { id: 'project-1', ownerId: 'user-1', name: 'first' },
-      { authData: { id: 'user-1' } }
-    )
-
-    expect(rows).toEqual([{ id: 'project-1', ownerId: 'user-1', name: 'first' }])
-    expect(mutationRuns).toBe(1)
-    expect(validatedMutationAuth).toBe('user-1')
-    expect(barrierRuns).toBe(1)
-    await Promise.all(scheduledBackground)
-    expect(backgroundRuns).toBe(1)
-    expect(backgroundAuth).toBe('user-1')
-
-    const ast = await bindings.resolveQuery('project|byOwner', [{ ownerId: 'user-1' }], {
-      id: 'user-1',
-    })
-    expect(JSON.stringify(ast)).toContain('ownerId')
-    expect(JSON.stringify(ast)).toContain('user-1')
-    expect(validatedQueryAuth).toBe('user-1')
-
-    await server.query({ id: 'user-1' }, () => zql.project)
-    expect(queryClaims).toEqual([{ userID: 'user-1', authData: { id: 'user-1' } }])
-  })
-
   test('enqueues typed actions through the configured local executor', async () => {
     type Action =
       | { type: 'project.provisionNamespace'; projectId: string }

package/src/serverWhere.test.ts

@@ -30,48 +30,6 @@ describe('serverWhere SSR behavior', () => {
     ;(globalThis as any)[Symbol.for('on-zero:state')] = null
   })
 
-  test('serverWhere returns no-op on client environment', () => {
-    setEnvironment('client')
-
-    const whereFn = serverWhere('post', (eb) => eb.cmp('ownerId', 'user-123'))
-    const eb = createMockEB()
-    const result = whereFn(eb as any, { id: 'user-123' })
-
-    // on client, should return no-op (and())
-    expect(eb.calls).toContain('and()')
-    expect(result).toEqual({ type: 'noop' })
-  })
-
-  test('serverWhere evaluates condition on server environment', () => {
-    setEnvironment('server')
-
-    const whereFn = serverWhere('post', (eb) => eb.cmp('ownerId', 'user-123'))
-    const eb = createMockEB()
-    const result = whereFn(eb as any, { id: 'user-123' })
-
-    // on server, should evaluate the actual condition
-    expect(eb.calls).toContain('cmp(ownerId, user-123)')
-    expect(result).toEqual({ type: 'condition', field: 'ownerId', value: 'user-123' })
-  })
-
-  test('serverWhere evaluates when _evaluatingPermission is true (even on client)', () => {
-    setEnvironment('client')
-
-    const whereFn = serverWhere('post', (eb) => eb.cmp('ownerId', 'user-123'))
-    const eb = createMockEB()
-
-    setEvaluatingPermission(true)
-    try {
-      const result = whereFn(eb as any, { id: 'user-123' })
-
-      // even on client, should evaluate when evaluating permission
-      expect(eb.calls).toContain('cmp(ownerId, user-123)')
-      expect(result).toEqual({ type: 'condition', field: 'ownerId', value: 'user-123' })
-    } finally {
-      setEvaluatingPermission(false)
-    }
-  })
-
   test('environment stays server when already set (SSR scenario)', () => {
     // simulate SSR: server sets environment first
     setEnvironment('server')