@open-mercato/core 0.4.5-develop-2e9903a57a → 0.4.5-develop-eeccf7adf4

package/src/modules/inbox_ops/api/proposals/[id]/__tests__/route.test.ts

@@ -1,124 +0,0 @@
-/** @jest-environment node */
-
-import { GET } from '@open-mercato/core/modules/inbox_ops/api/proposals/[id]/route'
-import { InboxProposal } from '@open-mercato/core/modules/inbox_ops/data/entities'
-
-const mockFindOneWithDecryption = jest.fn()
-const mockFindWithDecryption = jest.fn()
-
-jest.mock('@open-mercato/shared/lib/encryption/find', () => ({
-  findOneWithDecryption: (...args: unknown[]) => mockFindOneWithDecryption(...args),
-  findWithDecryption: (...args: unknown[]) => mockFindWithDecryption(...args),
-}))
-
-const authResult = {
-  sub: 'user-1',
-  tenantId: 'tenant-1',
-  orgId: 'org-1',
-}
-
-jest.mock('@open-mercato/shared/lib/auth/server', () => ({
-  getAuthFromRequest: jest.fn(async () => authResult),
-}))
-
-const mockEm = { fork: jest.fn() }
-
-const mockContainer = {
-  resolve: jest.fn((token: string) => {
-    if (token === 'em') return mockEm
-    return null
-  }),
-}
-
-jest.mock('@open-mercato/shared/lib/di/container', () => ({
-  createRequestContainer: jest.fn(async () => mockContainer),
-}))
-
-function makeRequest(proposalId = 'proposal-1') {
-  return new Request(`http://localhost/api/inbox_ops/proposals/${proposalId}`, { method: 'GET' })
-}
-
-describe('GET /api/inbox_ops/proposals/[id]', () => {
-  beforeEach(() => {
-    jest.clearAllMocks()
-    mockEm.fork.mockReturnValue(mockEm)
-    mockFindWithDecryption.mockResolvedValue([])
-  })
-
-  it('returns proposal detail with actions, discrepancies, and email', async () => {
-    const proposal = {
-      id: 'proposal-1',
-      inboxEmailId: 'email-1',
-      summary: 'Order for widgets',
-      status: 'pending',
-      isActive: true,
-    }
-    const actions = [{ id: 'action-1', actionType: 'create_order' }]
-    const discrepancies = [{ id: 'disc-1', type: 'price_mismatch' }]
-    const email = { id: 'email-1', subject: 'Order request' }
-
-    mockFindOneWithDecryption.mockResolvedValueOnce(proposal)
-    mockFindWithDecryption
-      .mockResolvedValueOnce(actions)
-      .mockResolvedValueOnce(discrepancies)
-    mockFindOneWithDecryption.mockResolvedValueOnce(email)
-
-    const response = await GET(makeRequest())
-    const payload = await response.json()
-
-    expect(response.status).toBe(200)
-    expect(payload.proposal.id).toBe('proposal-1')
-    expect(payload.actions).toHaveLength(1)
-    expect(payload.discrepancies).toHaveLength(1)
-    expect(payload.email.subject).toBe('Order request')
-  })
-
-  it('returns 404 when proposal not found', async () => {
-    mockFindOneWithDecryption.mockResolvedValueOnce(null)
-
-    const response = await GET(makeRequest())
-    const payload = await response.json()
-
-    expect(response.status).toBe(404)
-    expect(payload.error).toContain('not found')
-  })
-
-  it('filters by isActive: true', async () => {
-    mockFindOneWithDecryption.mockResolvedValueOnce(null)
-
-    await GET(makeRequest())
-
-    expect(mockFindOneWithDecryption).toHaveBeenCalledWith(
-      mockEm,
-      InboxProposal,
-      expect.objectContaining({
-        isActive: true,
-        organizationId: 'org-1',
-        tenantId: 'tenant-1',
-        deletedAt: null,
-      }),
-      undefined,
-      expect.any(Object),
-    )
-  })
-
-  it('returns 401 when not authenticated', async () => {
-    const { getAuthFromRequest } = jest.requireMock('@open-mercato/shared/lib/auth/server') as {
-      getAuthFromRequest: jest.Mock
-    }
-    getAuthFromRequest.mockResolvedValueOnce(null)
-
-    const response = await GET(makeRequest())
-    expect(response.status).toBe(401)
-  })
-
-  it('returns 400 when proposal ID is missing from path', async () => {
-    const response = await GET(
-      new Request('http://localhost/api/inbox_ops/proposals/', { method: 'GET' }),
-    )
-    const payload = await response.json()
-
-    expect(response.status).toBe(400)
-    expect(payload.error).toContain('Missing proposal ID')
-  })
-})

package/src/modules/inbox_ops/api/proposals/[id]/accept-all/__tests__/route.test.ts

@@ -1,154 +0,0 @@
-/** @jest-environment node */
-
-import { POST } from '@open-mercato/core/modules/inbox_ops/api/proposals/[id]/accept-all/route'
-import { InboxProposal } from '@open-mercato/core/modules/inbox_ops/data/entities'
-
-const mockFindOneWithDecryption = jest.fn()
-
-jest.mock('@open-mercato/shared/lib/encryption/find', () => ({
-  findOneWithDecryption: (...args: unknown[]) => mockFindOneWithDecryption(...args),
-}))
-
-const authResult = {
-  sub: 'user-1',
-  tenantId: 'tenant-1',
-  orgId: 'org-1',
-}
-
-jest.mock('@open-mercato/shared/lib/auth/server', () => ({
-  getAuthFromRequest: jest.fn(async () => authResult),
-}))
-
-const mockEm = { fork: jest.fn() }
-
-const mockContainer = {
-  resolve: jest.fn((token: string) => {
-    if (token === 'em') return mockEm
-    return null
-  }),
-}
-
-const mockEventBus = { emit: jest.fn() }
-
-const mockAcceptAllActions = jest.fn()
-
-jest.mock('@open-mercato/shared/lib/di/container', () => ({
-  createRequestContainer: jest.fn(async () => mockContainer),
-}))
-
-jest.mock('@open-mercato/core/modules/inbox_ops/lib/eventBus', () => ({
-  resolveOptionalEventBus: jest.fn(() => mockEventBus),
-}))
-
-jest.mock('@open-mercato/core/modules/inbox_ops/lib/executionEngine', () => ({
-  acceptAllActions: jest.fn((...args: unknown[]) => mockAcceptAllActions(...args)),
-}))
-
-function makeRequest(proposalId = 'proposal-1') {
-  return new Request(`http://localhost/api/inbox_ops/proposals/${proposalId}/accept-all`, {
-    method: 'POST',
-  })
-}
-
-describe('POST /api/inbox_ops/proposals/[id]/accept-all', () => {
-  beforeEach(() => {
-    jest.clearAllMocks()
-    mockEm.fork.mockReturnValue(mockEm)
-  })
-
-  it('accepts all actions and returns results', async () => {
-    const proposal = {
-      id: 'proposal-1',
-      isActive: true,
-    } as unknown as InboxProposal
-
-    mockFindOneWithDecryption.mockResolvedValueOnce(proposal)
-    mockAcceptAllActions.mockResolvedValueOnce({
-      results: [
-        { success: true, createdEntityId: 'order-1', createdEntityType: 'sales_order' },
-        { success: true, createdEntityId: 'person-1', createdEntityType: 'customer_person' },
-      ],
-      stoppedOnFailure: false,
-    })
-
-    const response = await POST(makeRequest())
-    const payload = await response.json()
-
-    expect(response.status).toBe(200)
-    expect(payload.ok).toBe(true)
-    expect(payload.succeeded).toBe(2)
-    expect(payload.failed).toBe(0)
-    expect(payload.stoppedOnFailure).toBe(false)
-    expect(payload.results).toHaveLength(2)
-  })
-
-  it('stops on failure and reports partial results', async () => {
-    const proposal = {
-      id: 'proposal-1',
-      isActive: true,
-    } as unknown as InboxProposal
-
-    mockFindOneWithDecryption.mockResolvedValueOnce(proposal)
-    mockAcceptAllActions.mockResolvedValueOnce({
-      results: [
-        { success: true, createdEntityId: 'order-1', createdEntityType: 'sales_order' },
-        { success: false, error: 'Insufficient permissions', statusCode: 403 },
-      ],
-      stoppedOnFailure: true,
-    })
-
-    const response = await POST(makeRequest())
-    const payload = await response.json()
-
-    expect(response.status).toBe(200)
-    expect(payload.ok).toBe(false)
-    expect(payload.succeeded).toBe(1)
-    expect(payload.failed).toBe(1)
-    expect(payload.stoppedOnFailure).toBe(true)
-  })
-
-  it('returns 404 when proposal not found', async () => {
-    mockFindOneWithDecryption.mockResolvedValueOnce(null)
-
-    const response = await POST(makeRequest())
-    const payload = await response.json()
-
-    expect(response.status).toBe(404)
-    expect(payload.error).toContain('not found')
-    expect(mockAcceptAllActions).not.toHaveBeenCalled()
-  })
-
-  it('passes execution context with entities to acceptAllActions', async () => {
-    const proposal = {
-      id: 'proposal-1',
-      isActive: true,
-    } as unknown as InboxProposal
-
-    mockFindOneWithDecryption.mockResolvedValueOnce(proposal)
-    mockAcceptAllActions.mockResolvedValueOnce({
-      results: [],
-      stoppedOnFailure: false,
-    })
-
-    await POST(makeRequest())
-
-    expect(mockAcceptAllActions).toHaveBeenCalledWith(
-      'proposal-1',
-      expect.objectContaining({
-        userId: 'user-1',
-        tenantId: 'tenant-1',
-        organizationId: 'org-1',
-      }),
-    )
-  })
-
-  it('returns 401 when not authenticated', async () => {
-    const { getAuthFromRequest } = jest.requireMock('@open-mercato/shared/lib/auth/server') as {
-      getAuthFromRequest: jest.Mock
-    }
-    getAuthFromRequest.mockResolvedValueOnce(null)
-
-    const response = await POST(makeRequest())
-    expect(response.status).toBe(401)
-  })
-})

package/src/modules/inbox_ops/api/proposals/[id]/actions/[actionId]/__tests__/route.test.ts

@@ -1,200 +0,0 @@
-/** @jest-environment node */
-
-import { PATCH } from '@open-mercato/core/modules/inbox_ops/api/proposals/[id]/actions/[actionId]/route'
-import { InboxProposal, InboxProposalAction } from '@open-mercato/core/modules/inbox_ops/data/entities'
-
-const mockFindOneWithDecryption = jest.fn()
-
-jest.mock('@open-mercato/shared/lib/encryption/find', () => ({
-  findOneWithDecryption: (...args: unknown[]) => mockFindOneWithDecryption(...args),
-}))
-
-const authResult = {
-  sub: 'user-1',
-  tenantId: 'tenant-1',
-  orgId: 'org-1',
-}
-
-jest.mock('@open-mercato/shared/lib/auth/server', () => ({
-  getAuthFromRequest: jest.fn(async () => authResult),
-}))
-
-const mockEm = {
-  fork: jest.fn(),
-  flush: jest.fn(),
-}
-
-const mockContainer = {
-  resolve: jest.fn((token: string) => {
-    if (token === 'em') return mockEm
-    return null
-  }),
-}
-
-const mockEventBus = { emit: jest.fn() }
-
-jest.mock('@open-mercato/shared/lib/di/container', () => ({
-  createRequestContainer: jest.fn(async () => mockContainer),
-}))
-
-jest.mock('@open-mercato/core/modules/inbox_ops/lib/eventBus', () => ({
-  resolveOptionalEventBus: jest.fn(() => mockEventBus),
-}))
-
-jest.mock('@open-mercato/core/modules/inbox_ops/events', () => ({
-  emitInboxOpsEvent: jest.fn(),
-}))
-
-function makeRequest(body: Record<string, unknown>) {
-  return new Request('http://localhost/api/inbox_ops/proposals/proposal-1/actions/action-1', {
-    method: 'PATCH',
-    headers: { 'Content-Type': 'application/json' },
-    body: JSON.stringify(body),
-  })
-}
-
-describe('PATCH /api/inbox_ops/proposals/[id]/actions/[actionId]', () => {
-  beforeEach(() => {
-    jest.clearAllMocks()
-    mockEm.fork.mockReturnValue(mockEm)
-    mockEm.flush.mockResolvedValue(undefined)
-  })
-
-  it('merges payload and saves the action', async () => {
-    const action = {
-      id: 'action-1',
-      proposalId: 'proposal-1',
-      actionType: 'create_order',
-      status: 'pending',
-      payload: {
-        customerName: 'Old Name',
-        currencyCode: 'USD',
-        channelId: 'a1b2c3d4-e5f6-4a7b-8c9d-0e1f2a3b4c5d',
-        lineItems: [{ productName: 'Widget', quantity: '10' }],
-      },
-    } as unknown as InboxProposalAction
-
-    const proposal = {
-      id: 'proposal-1',
-      isActive: true,
-    } as unknown as InboxProposal
-
-    mockFindOneWithDecryption
-      .mockResolvedValueOnce(action)
-      .mockResolvedValueOnce(proposal)
-
-    const response = await PATCH(makeRequest({ payload: { customerName: 'New Name' } }))
-    const payload = await response.json()
-
-    expect(response.status).toBe(200)
-    expect(payload.ok).toBe(true)
-    expect(action.payload).toEqual(
-      expect.objectContaining({ customerName: 'New Name', currencyCode: 'USD' }),
-    )
-    expect(mockEm.flush).toHaveBeenCalled()
-  })
-
-  it('returns 409 when action is already executed', async () => {
-    const action = {
-      id: 'action-1',
-      proposalId: 'proposal-1',
-      actionType: 'create_order',
-      status: 'executed',
-      payload: {},
-    } as unknown as InboxProposalAction
-
-    const proposal = {
-      id: 'proposal-1',
-      isActive: true,
-    } as unknown as InboxProposal
-
-    mockFindOneWithDecryption
-      .mockResolvedValueOnce(action)
-      .mockResolvedValueOnce(proposal)
-
-    const response = await PATCH(makeRequest({ payload: { customerName: 'New Name' } }))
-    const payload = await response.json()
-
-    expect(response.status).toBe(409)
-    expect(payload.error).toContain('already processed')
-    expect(mockEm.flush).not.toHaveBeenCalled()
-  })
-
-  it('returns 404 when action not found', async () => {
-    mockFindOneWithDecryption.mockResolvedValueOnce(null)
-
-    const response = await PATCH(makeRequest({ payload: { customerName: 'Test' } }))
-    const payload = await response.json()
-
-    expect(response.status).toBe(404)
-    expect(payload.error).toContain('Action not found')
-  })
-
-  it('returns 409 when proposal has been superseded', async () => {
-    const action = {
-      id: 'action-1',
-      proposalId: 'proposal-1',
-      actionType: 'create_order',
-      status: 'pending',
-      payload: {},
-    } as unknown as InboxProposalAction
-
-    mockFindOneWithDecryption
-      .mockResolvedValueOnce(action)
-      .mockResolvedValueOnce(null)
-
-    const response = await PATCH(makeRequest({ payload: { customerName: 'Test' } }))
-    const payload = await response.json()
-
-    expect(response.status).toBe(409)
-    expect(payload.error).toContain('superseded')
-  })
-
-  it('returns 400 for invalid edit payload', async () => {
-    const response = await PATCH(makeRequest({ invalid: 'field' }))
-    const payload = await response.json()
-
-    expect(response.status).toBe(400)
-    expect(payload.error).toContain('Invalid payload')
-  })
-
-  it('allows editing a failed action', async () => {
-    const action = {
-      id: 'action-1',
-      proposalId: 'proposal-1',
-      actionType: 'create_order',
-      status: 'failed',
-      payload: {
-        customerName: 'Old',
-        currencyCode: 'EUR',
-        channelId: 'a1b2c3d4-e5f6-4a7b-8c9d-0e1f2a3b4c5d',
-        lineItems: [{ productName: 'Item', quantity: '5' }],
-      },
-    } as unknown as InboxProposalAction
-
-    const proposal = {
-      id: 'proposal-1',
-      isActive: true,
-    } as unknown as InboxProposal
-
-    mockFindOneWithDecryption
-      .mockResolvedValueOnce(action)
-      .mockResolvedValueOnce(proposal)
-
-    const response = await PATCH(makeRequest({ payload: { customerName: 'Fixed Name' } }))
-    const payload = await response.json()
-
-    expect(response.status).toBe(200)
-    expect(payload.ok).toBe(true)
-  })
-
-  it('returns 401 when not authenticated', async () => {
-    const { getAuthFromRequest } = jest.requireMock('@open-mercato/shared/lib/auth/server') as {
-      getAuthFromRequest: jest.Mock
-    }
-    getAuthFromRequest.mockResolvedValueOnce(null)
-
-    const response = await PATCH(makeRequest({ payload: {} }))
-    expect(response.status).toBe(401)
-  })
-})