-
Notifications
You must be signed in to change notification settings - Fork 0
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Merge pull request #9 from getlarge/8-add-support-for-logical-operato…
…rs-and-or-in-permission-checks feat: add support for logical operators and or in permission checks
- Loading branch information
Showing
21 changed files
with
424 additions
and
131 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
150 changes: 150 additions & 0 deletions
150
packages/keto-client-wrapper/src/lib/ory-authorization.guard.spec.ts
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,150 @@ | ||
| import { OryBaseService } from '@getlarge/base-client-wrapper'; | ||
| import { ExecutionContext } from '@nestjs/common'; | ||
| import { Reflector } from '@nestjs/core'; | ||
| import { Test, TestingModule } from '@nestjs/testing'; | ||
| import { AxiosHeaders } from 'axios'; | ||
|
|
||
| import { | ||
| IAuthorizationGuard, | ||
| OryAuthorizationGuard, | ||
| } from './ory-authorization.guard'; | ||
| import { EnhancedRelationTupleFactory } from './ory-permission-checks.decorator'; | ||
| import { | ||
| OryPermissionsModuleOptions, | ||
| OryPermissionsService, | ||
| } from './ory-permissions'; | ||
|
|
||
| const mockAxiosResponse = <D = object>(data: D) => { | ||
| return { | ||
| status: 200, | ||
| statusText: 'OK', | ||
| headers: {}, | ||
| config: { | ||
| url: 'http://localhost', | ||
| headers: new AxiosHeaders(), | ||
| }, | ||
| data, | ||
| }; | ||
| }; | ||
|
|
||
| describe('OryAuthorizationGuard', () => { | ||
| let oryPermissionsService: OryPermissionsService; | ||
| let reflector: Reflector; | ||
|
|
||
| beforeEach(async () => { | ||
| const module: TestingModule = await Test.createTestingModule({ | ||
| providers: [ | ||
| OryPermissionsService, | ||
| { | ||
| provide: OryPermissionsModuleOptions, | ||
| useValue: { basePath: 'http://localhost' }, | ||
| }, | ||
| { | ||
| provide: OryBaseService, | ||
| useValue: { | ||
| axios: { | ||
| request: jest.fn(), | ||
| }, | ||
| }, | ||
| }, | ||
| ], | ||
| }).compile(); | ||
|
|
||
| oryPermissionsService = module.get(OryPermissionsService); | ||
| reflector = module.get(Reflector); | ||
| }); | ||
|
|
||
| describe('evaluateConditions', () => { | ||
| let oryAuthorizationGuard: IAuthorizationGuard; | ||
| let context: ExecutionContext; | ||
|
|
||
| beforeEach(() => { | ||
| context = {} as ExecutionContext; // Mock execution context | ||
| const guardFactory = OryAuthorizationGuard({}); | ||
| oryAuthorizationGuard = new guardFactory( | ||
| reflector, | ||
| oryPermissionsService | ||
| ); | ||
| }); | ||
|
|
||
| it('should allow access for a single permitted relation tuple', async () => { | ||
| const factory = () => 'user:123#access@resource:456'; | ||
| jest | ||
| .spyOn(oryPermissionsService, 'checkPermission') | ||
| .mockResolvedValue(mockAxiosResponse({ allowed: true })); | ||
|
|
||
| const result = await oryAuthorizationGuard.evaluateConditions( | ||
| factory, | ||
| context | ||
| ); | ||
| expect(result.allowed).toBe(true); | ||
| }); | ||
|
|
||
| it('should deny access if one relation in AND condition is not permitted', async () => { | ||
| const factory = { | ||
| type: 'AND', | ||
| conditions: [ | ||
| () => 'user:123#access@resource:456', | ||
| () => 'user:123#access@resource:789', | ||
| ], | ||
| } satisfies EnhancedRelationTupleFactory; | ||
| jest | ||
| .spyOn(oryPermissionsService, 'checkPermission') | ||
| .mockResolvedValueOnce(mockAxiosResponse({ allowed: true })) | ||
| .mockResolvedValueOnce(mockAxiosResponse({ allowed: false })); | ||
|
|
||
| const result = await oryAuthorizationGuard.evaluateConditions( | ||
| factory, | ||
| context | ||
| ); | ||
| expect(result.allowed).toBe(false); | ||
| }); | ||
|
|
||
| it('should allow access if one relation in OR condition is permitted', async () => { | ||
| const factory = { | ||
| type: 'OR', | ||
| conditions: [ | ||
| () => 'user:123#access@resource:456', | ||
| () => 'user:123#access@resource:789', | ||
| ], | ||
| } satisfies EnhancedRelationTupleFactory; | ||
| jest | ||
| .spyOn(oryPermissionsService, 'checkPermission') | ||
| .mockResolvedValueOnce(mockAxiosResponse({ allowed: false })) | ||
| .mockResolvedValueOnce(mockAxiosResponse({ allowed: true })); | ||
|
|
||
| const result = await oryAuthorizationGuard.evaluateConditions( | ||
| factory, | ||
| context | ||
| ); | ||
| expect(result.allowed).toBe(true); | ||
| }); | ||
|
|
||
| it('should correctly evaluate nested AND/OR conditions', async () => { | ||
| const factory = { | ||
| type: 'AND', | ||
| conditions: [ | ||
| { | ||
| type: 'OR', | ||
| conditions: [ | ||
| () => 'user:123#access@resource:456', | ||
| () => 'user:123#access@resource:789', | ||
| ], | ||
| }, | ||
| () => 'user:123#access@resource:101', | ||
| ], | ||
| } satisfies EnhancedRelationTupleFactory; | ||
| jest | ||
| .spyOn(oryPermissionsService, 'checkPermission') | ||
| .mockResolvedValueOnce(mockAxiosResponse({ allowed: true })) | ||
| .mockResolvedValueOnce(mockAxiosResponse({ allowed: true })) | ||
| .mockResolvedValueOnce(mockAxiosResponse({ allowed: true })); | ||
|
|
||
| const result = await oryAuthorizationGuard.evaluateConditions( | ||
| factory, | ||
| context | ||
| ); | ||
| expect(result.allowed).toBe(true); | ||
| }); | ||
| }); | ||
| }); |
Oops, something went wrong.