diff --git a/plugins/permission-node/src/integration/createPermissionIntegrationRouter.test.ts b/plugins/permission-node/src/integration/createPermissionIntegrationRouter.test.ts index e7091e7936..6e28c5ddc4 100644 --- a/plugins/permission-node/src/integration/createPermissionIntegrationRouter.test.ts +++ b/plugins/permission-node/src/integration/createPermissionIntegrationRouter.test.ts @@ -25,7 +25,7 @@ import { z } from 'zod'; import { createPermissionIntegrationRouter, CreatePermissionIntegrationRouterResourceOptions, - createIsAuthorized, + createConditionAuthorizer, } from './createPermissionIntegrationRouter'; import { createPermissionRule } from './createPermissionRule'; @@ -633,13 +633,13 @@ describe('createPermissionIntegrationRouter', () => { }); }); -describe('createIsAuthorized', () => { +describe('createConditionAuthorizer', () => { afterEach(() => { jest.clearAllMocks(); }); it('should return true in case of allowed decision', () => { - const isAuthorized = createIsAuthorized([testRule1, testRule2]); + const isAuthorized = createConditionAuthorizer([testRule1, testRule2]); expect( isAuthorized( { @@ -653,7 +653,7 @@ describe('createIsAuthorized', () => { expect(mockTestRule2Apply).not.toHaveBeenCalled(); }); it('should return false in case of denied decision', () => { - const isAuthorized = createIsAuthorized([testRule1, testRule2]); + const isAuthorized = createConditionAuthorizer([testRule1, testRule2]); expect( isAuthorized( { @@ -668,7 +668,7 @@ describe('createIsAuthorized', () => { }); it('should apply conditions to a resource in case of conditional decision', () => { - const isAuthorized = createIsAuthorized([testRule1, testRule2]); + const isAuthorized = createConditionAuthorizer([testRule1, testRule2]); expect( isAuthorized( { diff --git a/plugins/permission-node/src/integration/createPermissionIntegrationRouter.ts b/plugins/permission-node/src/integration/createPermissionIntegrationRouter.ts index 5b0c2c25f1..850123d55e 100644 --- a/plugins/permission-node/src/integration/createPermissionIntegrationRouter.ts +++ b/plugins/permission-node/src/integration/createPermissionIntegrationRouter.ts @@ -168,7 +168,7 @@ const applyConditions = ( * * @public */ -export const createIsAuthorized = ( +export const createConditionAuthorizer = ( rules: PermissionRule[], ) => { const getRule = createGetRule(rules); diff --git a/plugins/scaffolder-backend/src/service/router.ts b/plugins/scaffolder-backend/src/service/router.ts index f6415fae24..c1f9a15831 100644 --- a/plugins/scaffolder-backend/src/service/router.ts +++ b/plugins/scaffolder-backend/src/service/router.ts @@ -62,12 +62,14 @@ import { import { TemplateAction } from '@backstage/plugin-scaffolder-node'; import { PermissionEvaluator } from '@backstage/plugin-permission-common'; import { - createIsAuthorized, + createConditionAuthorizer, createPermissionIntegrationRouter, } from '@backstage/plugin-permission-node'; import { scaffolderTemplateRules } from './rules'; -const isAuthorized = createIsAuthorized(Object.values(scaffolderTemplateRules)); +const isAuthorized = createConditionAuthorizer( + Object.values(scaffolderTemplateRules), +); /** * RouterOptions