metaowl 0.4.1 → 0.5.0

package/test/store.test.js

@@ -1,476 +0,0 @@
-import { describe, it, expect, beforeEach } from 'vitest'
-import { Store, createPersistencePlugin, createStore } from '../modules/store.js'
-
-describe('Store', () => {
-  beforeEach(() => {
-    Store.clear()
-  })
-
-  describe('Store.define()', () => {
-    it('creates a store factory function', () => {
-      const useCounterStore = Store.define('counter', {
-        state: () => ({ count: 0 })
-      })
-
-      expect(typeof useCounterStore).toBe('function')
-    })
-
-    it('returns singleton instance on multiple calls', () => {
-      const useCounterStore = Store.define('counter', {
-        state: () => ({ count: 0 })
-      })
-
-      const store1 = useCounterStore()
-      const store2 = useCounterStore()
-
-      expect(store1).toBe(store2)
-    })
-
-    it('creates reactive state', () => {
-      const useCounterStore = Store.define('counter', {
-        state: () => ({ count: 0 })
-      })
-
-      const store = useCounterStore()
-
-      expect(store.state.count).toBe(0)
-    })
-  })
-
-  describe('state management', () => {
-    let useCounterStore
-
-    beforeEach(() => {
-      useCounterStore = Store.define('counter', {
-        state: () => ({ count: 0, name: 'test' }),
-        mutations: {
-          increment: (state) => state.count++,
-          setName: (state, name) => { state.name = name },
-          add: (state, amount) => { state.count += amount }
-        }
-      })
-    })
-
-    it('commits mutations synchronously', () => {
-      const store = useCounterStore()
-
-      store.commit('increment')
-
-      expect(store.state.count).toBe(1)
-    })
-
-    it('commits mutations with payload', () => {
-      const store = useCounterStore()
-
-      store.commit('setName', 'new name')
-
-      expect(store.state.name).toBe('new name')
-    })
-
-    it('commits mutations with numeric payload', () => {
-      const store = useCounterStore()
-
-      store.commit('add', 5)
-
-      expect(store.state.count).toBe(5)
-    })
-
-    it('throws on unknown mutation', () => {
-      const store = useCounterStore()
-
-      expect(() => store.commit('unknown')).toThrow('Mutation "unknown" not found')
-    })
-  })
-
-  describe('getters', () => {
-    it('computes getter values', () => {
-      const useCounterStore = Store.define('counter', {
-        state: () => ({ count: 5 }),
-        getters: {
-          double: (state) => state.count * 2,
-          triple: (state) => state.count * 3
-        }
-      })
-
-      const store = useCounterStore()
-
-      expect(store.getters.double).toBe(10)
-      expect(store.getters.triple).toBe(15)
-    })
-
-    it('updates getters when state changes', () => {
-      const useCounterStore = Store.define('counter', {
-        state: () => ({ count: 5 }),
-        getters: {
-          double: (state) => state.count * 2
-        },
-        mutations: {
-          increment: (state) => state.count++
-        }
-      })
-
-      const store = useCounterStore()
-      expect(store.getters.double).toBe(10)
-
-      store.commit('increment')
-      expect(store.getters.double).toBe(12)
-    })
-  })
-
-  describe('actions', () => {
-    it('dispatches async actions', async () => {
-      const useCounterStore = Store.define('counter', {
-        state: () => ({ count: 0 }),
-        mutations: {
-          increment: (state) => state.count++
-        },
-        actions: {
-          async incrementAsync({ commit }) {
-            await new Promise(resolve => setTimeout(resolve, 10))
-            commit('increment')
-          }
-        }
-      })
-
-      const store = useCounterStore()
-
-      await store.dispatch('incrementAsync')
-
-      expect(store.state.count).toBe(1)
-    })
-
-    it('passes payload to actions', async () => {
-      const useCounterStore = Store.define('counter', {
-        state: () => ({ count: 0 }),
-        mutations: {
-          add: (state, amount) => { state.count += amount }
-        },
-        actions: {
-          async addAsync({ commit }, amount) {
-            commit('add', amount)
-          }
-        }
-      })
-
-      const store = useCounterStore()
-
-      await store.dispatch('addAsync', 10)
-
-      expect(store.state.count).toBe(10)
-    })
-
-    it('provides context to actions', async () => {
-      const actionContext = {}
-
-      const useCounterStore = Store.define('counter2', {
-        state: () => ({ count: 0 }),
-        getters: { double: (state) => state.count * 2 },
-        mutations: { increment: (state) => state.count++ },
-        actions: {
-          async testContext(context) {
-            Object.assign(actionContext, {
-              hasState: 'state' in context,
-              hasGetters: 'getters' in context,
-              hasCommit: typeof context.commit === 'function',
-              hasDispatch: typeof context.dispatch === 'function'
-            })
-          }
-        }
-      })
-
-      const store = useCounterStore()
-      await store.dispatch('testContext')
-
-      expect(actionContext.hasState).toBe(true)
-      expect(actionContext.hasGetters).toBe(true)
-      expect(actionContext.hasCommit).toBe(true)
-      expect(actionContext.hasDispatch).toBe(true)
-    })
-
-    it('throws on unknown action', async () => {
-      const useCounterStore = Store.define('counter', {
-        state: () => ({ count: 0 })
-      })
-
-      const store = useCounterStore()
-
-      await expect(store.dispatch('unknown')).rejects.toThrow('Action "unknown" not found')
-    })
-
-    it('returns action result', async () => {
-      const useCounterStore = Store.define('counter', {
-        state: () => ({ count: 5 }),
-        actions: {
-          async fetchData() {
-            return { data: 'test' }
-          }
-        }
-      })
-
-      const store = useCounterStore()
-      const result = await store.dispatch('fetchData')
-
-      expect(result).toEqual({ data: 'test' })
-    })
-  })
-
-  describe('subscriptions', () => {
-    it('notifies subscribers on mutation', () => {
-      const useCounterStore = Store.define('counter', {
-        state: () => ({ count: 0 }),
-        mutations: { increment: (state) => state.count++ }
-      })
-
-      const store = useCounterStore()
-      const subscriber = { fn: () => {} }
-      const spy = { ...subscriber, fn: (mutation, state, prevState) => {} }
-
-      let receivedMutation, receivedState, receivedPrevState
-      store.subscribe((mutation, state, prevState) => {
-        receivedMutation = mutation
-        receivedState = state
-        receivedPrevState = prevState
-      })
-
-      store.commit('increment')
-
-      expect(receivedMutation.type).toBe('increment')
-      expect(receivedState.count).toBe(1)
-      expect(receivedPrevState.count).toBe(0)
-    })
-
-    it('allows unsubscribing', () => {
-      const useCounterStore = Store.define('counter', {
-        state: () => ({ count: 0 }),
-        mutations: { increment: (state) => state.count++ }
-      })
-
-      const store = useCounterStore()
-      let callCount = 0
-      const unsubscribe = store.subscribe(() => callCount++)
-
-      store.commit('increment')
-      expect(callCount).toBe(1)
-
-      unsubscribe()
-      store.commit('increment')
-      expect(callCount).toBe(1)
-    })
-  })
-
-  describe('action subscriptions', () => {
-    it('notifies action subscribers', async () => {
-      const useCounterStore = Store.define('counter', {
-        state: () => ({ count: 0 }),
-        actions: {
-          async testAction() {
-            return 'result'
-          }
-        }
-      })
-
-      const store = useCounterStore()
-      const events = []
-
-      store.subscribeAction((action, status, result) => {
-        events.push({ action, status, result })
-      })
-
-      await store.dispatch('testAction')
-
-      expect(events.length).toBe(2)
-      expect(events[0].status).toBe('before')
-      expect(events[1].status).toBe('after')
-      expect(events[1].result).toBe('result')
-    })
-
-    it('notifies on action error', async () => {
-      const useCounterStore = Store.define('counter', {
-        actions: {
-          async failingAction() {
-            throw new Error('test error')
-          }
-        }
-      })
-
-      const store = useCounterStore()
-      const events = []
-
-      store.subscribeAction((action, status, result) => {
-        events.push({ action, status, result })
-      })
-
-      try {
-        await store.dispatch('failingAction')
-      } catch (e) {
-        // expected
-      }
-
-      expect(events.length).toBe(2)
-      expect(events[0].status).toBe('before')
-      expect(events[1].status).toBe('error')
-    })
-  })
-
-  describe('reset', () => {
-    it('resets state to initial values', () => {
-      const useCounterStore = Store.define('counter', {
-        state: () => ({ count: 0, name: 'initial' }),
-        mutations: {
-          setCount: (state, val) => { state.count = val },
-          setName: (state, val) => { state.name = val }
-        }
-      })
-
-      const store = useCounterStore()
-      store.commit('setCount', 100)
-      store.commit('setName', 'changed')
-
-      store.reset()
-
-      expect(store.state.count).toBe(0)
-      expect(store.state.name).toBe('initial')
-    })
-  })
-
-  describe('static methods', () => {
-    it('Store.get() retrieves store by id', () => {
-      const useCounterStore = Store.define('counter', {
-        state: () => ({ count: 0 })
-      })
-
-      const store = useCounterStore()
-      const retrieved = Store.get('counter')
-
-      expect(retrieved).toBe(store)
-    })
-
-    it('Store.has() checks store existence', () => {
-      const useCounterStore = Store.define('counter', {
-        state: () => ({ count: 0 })
-      })
-
-      useCounterStore() // Initialize the store
-
-      expect(Store.has('counter')).toBe(true)
-      expect(Store.has('unknown')).toBe(false)
-    })
-
-    it('Store.remove() removes store', () => {
-      const useCounterStore = Store.define('counter', {
-        state: () => ({ count: 0 })
-      })
-
-      useCounterStore()
-      expect(Store.has('counter')).toBe(true)
-
-      Store.remove('counter')
-      expect(Store.has('counter')).toBe(false)
-    })
-
-    it('Store.storeIds() returns all store IDs', () => {
-      const useStore1 = Store.define('store1', { state: () => ({}) })
-      const useStore2 = Store.define('store2', { state: () => ({}) })
-
-      useStore1()
-      useStore2()
-
-      const ids = Store.storeIds()
-      expect(ids).toContain('store1')
-      expect(ids).toContain('store2')
-    })
-  })
-
-  describe('persistence plugin', () => {
-    it('persists state to storage', () => {
-      const storage = {
-        data: {},
-        getItem(key) { return this.data[key] || null },
-        setItem(key, value) { this.data[key] = value }
-      }
-
-      Store.use(createPersistencePlugin({ storage }))
-
-      const useCounterStore = Store.define('counter', {
-        state: () => ({ count: 0 }),
-        mutations: { increment: (state) => state.count++ }
-      })
-
-      const store = useCounterStore()
-      store.commit('increment')
-
-      expect(storage.data['metaowl:store:counter']).toContain('"count":1')
-    })
-
-    it('restores state from storage', () => {
-      const storage = {
-        data: { 'metaowl:store:counter': '{"count":42}' },
-        getItem(key) { return this.data[key] || null },
-        setItem(key, value) { this.data[key] = value }
-      }
-
-      Store.use(createPersistencePlugin({ storage }))
-
-      const useCounterStore = Store.define('counter', {
-        state: () => ({ count: 0 }),
-        mutations: { increment: (state) => state.count++ }
-      })
-
-      const store = useCounterStore()
-
-      expect(store.state.count).toBe(42)
-    })
-
-    it('persists only specified paths', () => {
-      const storage = {
-        data: {},
-        getItem(key) { return this.data[key] || null },
-        setItem(key, value) { this.data[key] = value }
-      }
-
-      Store.use(createPersistencePlugin({ storage, paths: ['persistent'] }))
-
-      const useStore = Store.define('test', {
-        state: () => ({ persistent: 'value', temporary: 'data' }),
-        mutations: {
-          setPersistent: (state, val) => { state.persistent = val },
-          setTemporary: (state, val) => { state.temporary = val }
-        }
-      })
-
-      const store = useStore()
-      store.commit('setPersistent', 'new value')
-      store.commit('setTemporary', 'new temp')
-
-      const saved = JSON.parse(storage.data['metaowl:store:test'])
-      expect(saved.persistent).toBe('new value')
-      expect(saved.temporary).toBeUndefined()
-    })
-  })
-})
-
-describe('createStore', () => {
-  it('creates reactive state object', () => {
-    const state = createStore({ count: 0 })
-
-    expect(state.count).toBe(0)
-  })
-
-  it('has $patch method for batch updates', () => {
-    const state = createStore({ count: 0, name: 'test' })
-
-    state.$patch({ count: 10, name: 'updated' })
-
-    expect(state.count).toBe(10)
-    expect(state.name).toBe('updated')
-  })
-
-  it('has $reset method to restore initial state', () => {
-    const state = createStore({ count: 0 })
-
-    state.count = 100
-    state.$reset()
-
-    expect(state.count).toBe(0)
-  })
-})

package/test/templates-manager.test.js

@@ -1,83 +0,0 @@
-import { describe, it, expect, vi, beforeEach } from 'vitest'
-import { mergeTemplates, getInternalTemplates } from '../modules/templates-manager.js'
-
-vi.mock('@odoo/owl', () => ({
-  loadFile: vi.fn(),
-}))
-
-import { loadFile } from '@odoo/owl'
-
-beforeEach(() => {
-  vi.clearAllMocks()
-})
-
-describe('mergeTemplates', () => {
-  it('returns wrapped templates string from a single file', async () => {
-    loadFile.mockResolvedValue('<t t-name="Comp"><div/></t>')
-    const result = await mergeTemplates(['/components/Comp.xml'])
-    expect(result).toContain('<templates>')
-    expect(result).toContain('<t t-name="Comp"><div/></t>')
-    expect(result).toContain('</templates>')
-    expect(result).toContain('t-name="Link"') // Internal Link template
-  })
-
-  it('concatenates multiple template files', async () => {
-    loadFile
-      .mockResolvedValueOnce('<t t-name="A"><div/></t>')
-      .mockResolvedValueOnce('<t t-name="B"><span/></t>')
-    const result = await mergeTemplates(['/A.xml', '/B.xml'])
-    expect(result).toContain('<templates>')
-    expect(result).toContain('<t t-name="A"><div/></t>')
-    expect(result).toContain('<t t-name="B"><span/></t>')
-    expect(result).toContain('</templates>')
-    expect(result).toContain('t-name="Link"') // Internal Link template
-  })
-
-  it('returns templates with internal components for empty array', async () => {
-    const result = await mergeTemplates([])
-    expect(result).toContain('<templates>')
-    expect(result).toContain('</templates>')
-    expect(result).toContain('t-name="Link"') // Internal Link template
-  })
-
-  it('skips failed files and logs error', async () => {
-    const consoleSpy = vi.spyOn(console, 'error').mockImplementation(() => {})
-    loadFile.mockRejectedValue(new Error('404'))
-    const result = await mergeTemplates(['/missing.xml'])
-    expect(result).toContain('<templates>')
-    expect(result).toContain('</templates>')
-    expect(result).toContain('t-name="Link"') // Internal Link template despite error
-    expect(consoleSpy).toHaveBeenCalledWith(
-      expect.stringContaining('[metaowl] Failed to load template: /missing.xml'),
-      expect.any(Error)
-    )
-    consoleSpy.mockRestore()
-  })
-
-  it('collects successful files even when one fails', async () => {
-    const consoleSpy = vi.spyOn(console, 'error').mockImplementation(() => {})
-    loadFile
-      .mockResolvedValueOnce('<t t-name="Good"><div/></t>')
-      .mockRejectedValueOnce(new Error('404'))
-      .mockResolvedValueOnce('<t t-name="Also"><span/></t>')
-    const result = await mergeTemplates(['/good.xml', '/missing.xml', '/also.xml'])
-    expect(result).toContain('<t t-name="Good"><div/></t>')
-    expect(result).toContain('<t t-name="Also"><span/></t>')
-    expect(result).toContain('t-name="Link"') // Internal Link template
-    consoleSpy.mockRestore()
-  })
-
-  it('includes internal Link component template', async () => {
-    const templates = getInternalTemplates()
-    expect(templates.length).toBeGreaterThan(0)
-    expect(templates[0]).toContain('t-name="Link"')
-    expect(templates[0]).toContain('<a')
-    expect(templates[0]).toContain('t-att-href')
-  })
-
-  it('calls loadFile once per path', async () => {
-    loadFile.mockResolvedValue('<t/>')
-    await mergeTemplates(['/a.xml', '/b.xml', '/c.xml'])
-    expect(loadFile).toHaveBeenCalledTimes(3)
-  })
-})