Skip to content

Commit ca1253f

Browse files
Bill LeoutsakosBill Leoutsakos
authored andcommitted
fix(oracle-fusion): complete risk service registrations and CI corrections
1 parent 26c8de9 commit ca1253f

17 files changed

Lines changed: 2029 additions & 859 deletions

‎apps/sim/blocks/blocks/oracle_fusion_risk_management.test.ts‎

Lines changed: 17 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -2,12 +2,12 @@
22
* @vitest-environment node
33
*/
44
import { describe, expect, it } from 'vitest'
5+
import { selectorManifest } from '@/lib/selectors/manifest'
6+
import { evaluateSubBlockCondition } from '@/lib/workflows/subblocks/visibility'
57
import {
68
OracleFusionRiskManagementBlock,
79
OracleFusionRiskManagementBlockMeta,
810
} from '@/blocks/blocks/oracle_fusion_risk_management'
9-
import { selectorManifest } from '@/lib/selectors/manifest'
10-
import { evaluateSubBlockCondition } from '@/lib/workflows/subblocks/visibility'
1111
import * as riskTools from '@/tools/oracle_fusion_risk_management'
1212

1313
const block = OracleFusionRiskManagementBlock
@@ -23,16 +23,18 @@ describe('Risk Management block contracts', () => {
2323
for (const [param, config] of Object.entries(tool.params)) {
2424
if (!config.required || config.visibility === 'hidden') continue
2525
for (const mode of ['basic', 'advanced']) {
26-
const fields = block.subBlocks.filter((field) =>
27-
(field.canonicalParamId ?? field.id) === param &&
28-
(!field.mode || field.mode === mode) &&
29-
evaluateSubBlockCondition(field.condition, values)
26+
const fields = block.subBlocks.filter(
27+
(field) =>
28+
(field.canonicalParamId ?? field.id) === param &&
29+
(!field.mode || field.mode === mode) &&
30+
evaluateSubBlockCondition(field.condition, values)
3031
)
3132
expect(fields.length, `${tool.id}: ${param} in ${mode}`).toBeGreaterThan(0)
3233
expect(fields.every((field) => field.required === true)).toBe(true)
3334
}
3435
}
35-
for (const output of Object.keys(tool.outputs ?? {})) expect(block.outputs).toHaveProperty(output)
36+
for (const output of Object.keys(tool.outputs ?? {}))
37+
expect(block.outputs).toHaveProperty(output)
3638
}
3739
expect(OracleFusionRiskManagementBlockMeta.templates.length).toBeGreaterThanOrEqual(7)
3840
})
@@ -47,7 +49,14 @@ describe('Risk Management block contracts', () => {
4749
expect(params.processId).toBe('<previous.record.ProcessId>')
4850
const mapped = block.tools.config.params!({ ...params, processId: '9007199254740993' })
4951
expect(mapped).toEqual({ processId: '9007199254740993' })
50-
expect(block.tools.config.params!({ operation: 'oracle_fusion_risk_management_list_processes', limit: '25', offset: '0', totalResults: 'false' })).toEqual({ limit: 25, offset: 0, totalResults: false })
52+
expect(
53+
block.tools.config.params!({
54+
operation: 'oracle_fusion_risk_management_list_processes',
55+
limit: '25',
56+
offset: '0',
57+
totalResults: 'false',
58+
})
59+
).toEqual({ limit: 25, offset: 0, totalResults: false })
5160
})
5261

5362
it('binds selectors to the shared service and credential context', () => {

‎apps/sim/blocks/blocks/oracle_fusion_risk_management.ts‎

Lines changed: 579 additions & 166 deletions
Large diffs are not rendered by default.

‎apps/sim/lib/integrations/credential-display.test.ts‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -68,6 +68,7 @@ const EXPECTED_COVERAGE: Record<string, string[]> = {
6868
// NetSuite remains an API-key catalog integration, like Snowflake, while its
6969
// block uses the shared reusable-credential selector.
7070
'netsuite-service-account': [],
71+
'oracle-fusion-service-account': ['oracle-fusion-risk-management'],
7172
'pipedrive-service-account': ['pipedrive'],
7273
'salesforce-service-account': ['salesforce'],
7374
'shopify-service-account': ['shopify'],

‎apps/sim/lib/internal/oracle-fusion-risk-management/execute-tool.ts‎

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,9 @@ import type { InternalToolOperationHandler } from '@/lib/internal/tool-operation
1111
import { resolveOAuthAccountId } from '@/lib/oauth/credential-service'
1212

1313
/** The executor authorizes credential use; this product boundary binds the credential family. */
14-
export const executeOracleFusionRiskManagementTool: InternalToolOperationHandler = async (request) => {
14+
export const executeOracleFusionRiskManagementTool: InternalToolOperationHandler = async (
15+
request
16+
) => {
1517
request.signal?.throwIfAborted()
1618
if (!isPlainRecord(request.input) || typeof request.input.oauthCredential !== 'string') {
1719
return Response.json(

0 commit comments

Comments
 (0)