Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Conversation

@finkmanAtSap
Copy link
Contributor

@finkmanAtSap finkmanAtSap commented Apr 23, 2025

This PR allows to see the file changes that are the result of adding AMS to a minimal version of the CAP sample.
It demonstrates the absolute minimum changes required for:

  • adding AMS to the project
  • implementing instance-based authorization by filtering data with AMS policies
  • writing unit tests involving AMS policies

It does not contain the required changes for the deployment of the application because they can be generated by the cds add ams command of @sap/cds-dk.

…g services

- Added @sap/ams and @sap/ams-dev dependencies to package.json
- Introduced axios as a dev dependency
- Uncommented and refactored test cases for AdminService to validate access for user 'bob'
- Updated CatalogService tests to reflect correct user roles and expectations
- Ensured proper assertions for book retrieval based on user roles and policies
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.