feat: Mosaic preagg in an ephemeral database - #585

packages/mosaic/__tests__/MosaicSlice.test.ts (1)

9-20: 💤 Low value

Consider adding test cases for partial configuration.

The current test covers both options together. Additional tests for edge cases (only schema, only enabled, or undefined preagg) would improve confidence, though the implementation's guards are straightforward.

💡 Example additional test cases

it('applies only schema when enabled is omitted', async () => {
  const coordinator = new Coordinator();
  const store = createStore<BaseRoomStoreState & MosaicSliceState>()((...args) => ({
    ...createBaseRoomSlice()(...args),
    ...createMosaicSlice({
      coordinator,
      preagg: { schema: 'custom.schema' },
    })(...args),
  }));

  await store.getState().mosaic.initialize();

  expect(coordinator.preaggregator.schema).toBe('custom.schema');
  // enabled should remain at Mosaic's default
});

it('preserves defaults when preagg is omitted', async () => {
  const coordinator = new Coordinator();
  const defaultSchema = coordinator.preaggregator.schema;
  const store = createStore<BaseRoomStoreState & MosaicSliceState>()((...args) => ({
    ...createBaseRoomSlice()(...args),
    ...createMosaicSlice({ coordinator })(...args),
  }));

  await store.getState().mosaic.initialize();

  expect(coordinator.preaggregator.schema).toBe(defaultSchema);
});

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@packages/mosaic/__tests__/MosaicSlice.test.ts` around lines 9 - 20, Add
focused tests that exercise partial preagg configs: create variants of
createTestStore using createMosaicSlice with preagg containing only schema, only
enabled, and with preagg omitted; for each, call
store.getState().mosaic.initialize() (or mosaic.initialize) and assert
Coordinator.preaggregator.schema and enabled reflect only the provided fields
(e.g., schema set when given, enabled left as default when omitted, and defaults
preserved when preagg is undefined). Use Coordinator and createBaseRoomSlice
references from the existing helper to build the store and keep assertions
minimal and targeted to the preagg behavior.