style(oracle-fusion-procurement): apply GitHub formatting feedback

Author: Bill Leoutsakos

apps/sim/lib/internal/oracle-fusion-procurement/oracle-fusion-procurement.test.ts

@@ -661,7 +661,9 @@ describe('Oracle Fusion Procurement integration contracts', () => {
     mocks.request.mockResolvedValue(responseFor(contract))
     const { result } = await invoke(contract.slug, contract.params)
     expect(result.success).toBe(true)
-    expect(serializeOracleFusionJsonBody(mocks.request.mock.calls[0][1].body)).toBe(testCase.expected)
+    expect(serializeOracleFusionJsonBody(mocks.request.mock.calls[0][1].body)).toBe(
+      testCase.expected
+    )
   })
 
   it('preserves exact nested request integers and false/zero without changing the caller body', async () => {
@@ -671,7 +673,9 @@ describe('Oracle Fusion Procurement integration contracts', () => {
       lines: [{ LineNumber: 1, LineTypeId: ID, Quantity: 0, Price: 0 }],
     }
     const original = structuredClone(body)
-    mocks.request.mockResolvedValue(resourceItem('draftPurchaseOrders', `draftPurchaseOrders/${KEY}`))
+    mocks.request.mockResolvedValue(
+      resourceItem('draftPurchaseOrders', `draftPurchaseOrders/${KEY}`)
+    )
     const response = await invoke('create_draft_purchase_order', {
       buyerId: ID,
       documentStyleId: ID,
@@ -695,7 +699,10 @@ describe('Oracle Fusion Procurement integration contracts', () => {
     mocks.request.mockResolvedValue({ result: 'SUCCESS' })
     await invoke('submit_purchase_requisition', { requisitionKey: KEY })
     expect(mocks.request.mock.calls[0][1].body).toEqual({})
-    await invoke('submit_purchase_requisition', { requisitionKey: KEY, requestFundsOverrideFlag: false })
+    await invoke('submit_purchase_requisition', {
+      requisitionKey: KEY,
+      requestFundsOverrideFlag: false,
+    })
     expect(mocks.request.mock.calls[1][1].body).toEqual({ requestFundsOverrideFlag: false })
   })
 
@@ -750,9 +757,7 @@ describe('Oracle Fusion Procurement integration contracts', () => {
   })
 
   it('returns only the requested page and a usable next offset', async () => {
-    mocks.request.mockResolvedValue(
-      page([resourceItem('suppliers', `suppliers/${ID}`)], 50, true)
-    )
+    mocks.request.mockResolvedValue(page([resourceItem('suppliers', `suppliers/${ID}`)], 50, true))
     const { result } = await invoke('list_suppliers', { offset: 50, limit: 100 })
     expect(result.output).toMatchObject({ count: 1, offset: 50, hasMore: true, nextOffset: 51 })
     expect(mocks.request).toHaveBeenCalledTimes(1)
@@ -763,10 +768,13 @@ describe('Oracle Fusion Procurement integration contracts', () => {
     { items: [], count: 0, offset: 0, limit: 100, hasMore: true },
     { items: [], count: 0, offset: 1, limit: 100, hasMore: false },
     { items: 'not-an-array', count: 0, offset: 0, limit: 100, hasMore: false },
-  ])('rejects a malformed collection instead of reporting an empty successful list', async (invalid) => {
-    mocks.request.mockResolvedValue(invalid)
-    expect((await invoke('list_suppliers')).status).toBe(502)
-  })
+  ])(
+    'rejects a malformed collection instead of reporting an empty successful list',
+    async (invalid) => {
+      mocks.request.mockResolvedValue(invalid)
+      expect((await invoke('list_suppliers')).status).toBe(502)
+    }
+  )
 
   it.each([true, false])('uses authoritative self links for opaque keys (v9=%s)', async (v9) => {
     mocks.request.mockResolvedValue(
@@ -800,7 +808,9 @@ describe('Oracle Fusion Procurement integration contracts', () => {
   })
 
   it('checks that PATCH returns the same document', async () => {
-    mocks.request.mockResolvedValue(resourceItem('draftPurchaseOrders', 'draftPurchaseOrders/OTHER'))
+    mocks.request.mockResolvedValue(
+      resourceItem('draftPurchaseOrders', 'draftPurchaseOrders/OTHER')
+    )
     expect(
       (
         await invoke('update_draft_purchase_order', {
@@ -933,7 +943,9 @@ describe('Oracle Fusion Procurement integration contracts', () => {
   })
 
   it('preserves a safe provider status and propagates cancellation', async () => {
-    mocks.request.mockRejectedValue(new OracleFusionProviderError('Oracle Fusion access denied', 403))
+    mocks.request.mockRejectedValue(
+      new OracleFusionProviderError('Oracle Fusion access denied', 403)
+    )
     expect((await invoke('list_suppliers')).status).toBe(403)
     const controller = new AbortController()
     controller.abort(new Error('Stopped'))

apps/sim/lib/internal/oracle-fusion-procurement/schema.ts

@@ -403,19 +403,21 @@ const requestDate = z.string().date()
 const requestDateTime = z.union([z.string().datetime({ offset: true }), requestDate])
 
 /** 26C: op-suppliers-supplierid-child-addresses-post.html */
-const supplierAddressBodySchema = z.object({
-  CountryCode: z.string().max(2),
-  Email: z.string().max(320).optional(),
-  AddressName: z.string().max(240).nullable().optional(),
-  AddressLine1: z.string().max(240).nullable().optional(),
-  AddressLine2: z.string().max(240).nullable().optional(),
-  City: z.string().max(60).nullable().optional(),
-  State: z.string().max(60).nullable().optional(),
-  Province: z.string().max(60).nullable().optional(),
-  PostalCode: z.string().max(60).nullable().optional(),
-  AddressPurposeOrderingFlag: z.boolean().nullable().optional(),
-  AddressPurposeRFQOrBiddingFlag: z.boolean().nullable().optional(),
-}).strict()
+const supplierAddressBodySchema = z
+  .object({
+    CountryCode: z.string().max(2),
+    Email: z.string().max(320).optional(),
+    AddressName: z.string().max(240).nullable().optional(),
+    AddressLine1: z.string().max(240).nullable().optional(),
+    AddressLine2: z.string().max(240).nullable().optional(),
+    City: z.string().max(60).nullable().optional(),
+    State: z.string().max(60).nullable().optional(),
+    Province: z.string().max(60).nullable().optional(),
+    PostalCode: z.string().max(60).nullable().optional(),
+    AddressPurposeOrderingFlag: z.boolean().nullable().optional(),
+    AddressPurposeRFQOrBiddingFlag: z.boolean().nullable().optional(),
+  })
+  .strict()
 
 /** 26C: op-suppliers-supplierid-child-sites-suppliersiteid-child-assignments-post.html */
 const supplierSiteAssignmentBodySchema = z

apps/sim/lib/selectors/server/providers/oracle-fusion-procurement.test.ts

@@ -180,7 +180,7 @@ const SELECTOR_CONTRACTS: SelectorContract[] = [
   {
     name: 'procurementBusinessUnits',
     path: 'procurementBusinessUnitsLOV',
-    itemKey: ID,
+    itemKey: KEY,
     fields: { ProcurementBUId: ID, ProcurementBU: 'West', AgentAction: 'MANAGE_PURCHASE_ORDERS' },
     option: { id: ID, label: 'West', meta: { detail: 'MANAGE_PURCHASE_ORDERS' } },
     filter: `ProcurementBUId=${ID}`,
@@ -219,39 +219,42 @@ describe('Oracle Fusion Procurement selectors', () => {
     mocks.bundle.mockResolvedValue(PREPARED)
   })
 
-  it.each(SELECTOR_CONTRACTS)('$name lists and resolves the correct workflow value', async (entry) => {
-    const input = args(entry.name)
-    Object.assign(input.context, entry.context)
-    const item = resource(`${entry.path}/${entry.itemKey}`, entry.fields)
-    mocks.request.mockResolvedValue(page([item]))
-    expect(await attachment(input).execute(input, PREPARED)).toEqual({
-      kind: 'list',
-      items: [entry.option],
-    })
-    expect(mocks.request.mock.lastCall![1]).toMatchObject({
-      method: 'GET',
-      address: { family: 'fscm', relativePath: entry.path },
-      query: { limit: 100, offset: 0 },
-    })
-    mocks.request.mockClear()
-    mocks.request.mockResolvedValue(entry.filter ? page([item]) : item)
-    input.request = { kind: 'detail', id: entry.option.id }
-    expect(await attachment(input).execute(input, PREPARED)).toEqual({
-      kind: 'detail',
-      item: entry.option,
-    })
-    expect(mocks.request).toHaveBeenCalledTimes(1)
-    expect(mocks.request.mock.lastCall![1].address.relativePath).toBe(
-      entry.filter ? entry.path : `${entry.path}/${entry.itemKey}`
-    )
-    if (entry.filter) {
-      expect(mocks.request.mock.lastCall![1].query).toMatchObject({
-        q: entry.filter,
-        limit: 1,
-        offset: 0,
+  it.each(SELECTOR_CONTRACTS)(
+    '$name lists and resolves the correct workflow value',
+    async (entry) => {
+      const input = args(entry.name)
+      Object.assign(input.context, entry.context)
+      const item = resource(`${entry.path}/${entry.itemKey}`, entry.fields)
+      mocks.request.mockResolvedValue(page([item]))
+      expect(await attachment(input).execute(input, PREPARED)).toEqual({
+        kind: 'list',
+        items: [entry.option],
+      })
+      expect(mocks.request.mock.lastCall![1]).toMatchObject({
+        method: 'GET',
+        address: { family: 'fscm', relativePath: entry.path },
+        query: { limit: 100, offset: 0 },
+      })
+      mocks.request.mockClear()
+      mocks.request.mockResolvedValue(entry.filter ? page([item]) : item)
+      input.request = { kind: 'detail', id: entry.option.id }
+      expect(await attachment(input).execute(input, PREPARED)).toEqual({
+        kind: 'detail',
+        item: entry.option,
       })
+      expect(mocks.request).toHaveBeenCalledTimes(1)
+      expect(mocks.request.mock.lastCall![1].address.relativePath).toBe(
+        entry.filter ? entry.path : `${entry.path}/${entry.itemKey}`
+      )
+      if (entry.filter) {
+        expect(mocks.request.mock.lastCall![1].query).toMatchObject({
+          q: entry.filter,
+          limit: 1,
+          offset: 0,
+        })
+      }
     }
-  })
+  )
 
   it('prepares the destination exclusively from the authorized Fusion credential', async () => {
     const input = args('suppliers')

apps/sim/lib/selectors/server/providers/oracle-fusion-procurement.ts

@@ -156,9 +156,7 @@ async function prepareDestination(
 function parentParams(args: ExecuteServerSelectorArgs, definition: SelectorDefinition) {
   if (!definition.parent) return {}
   const value =
-    definition.parent === 'supplierId'
-      ? args.context.supplierId
-      : args.context.poHeaderId
+    definition.parent === 'supplierId' ? args.context.supplierId : args.context.poHeaderId
   const result = procurementIdentifierSchema.safeParse(value)
   if (!result.success) throw new SelectorContextUnavailableError()
   return { [definition.parent]: result.data }