Skip to content

Rbsdms 114 feature#536

Open
akgarg06 wants to merge 16 commits into
developfrom
RBSDMS-114-feature
Open

Rbsdms 114 feature#536
akgarg06 wants to merge 16 commits into
developfrom
RBSDMS-114-feature

Conversation

@akgarg06

@akgarg06 akgarg06 commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

Describe your changes

Any documentation

Adds integration tests for creating attachments directly on active (published) entities, without going through the draft flow. Tests cover single-tenant and multi-tenant scenarios across named user and technical user token flows.

New test files:
IntegrationTest_ActiveEntity — core active-entity tests (16 test cases)
IntegrationTest_ActiveEntity_MultipleFacet — same tests across all 3 facets (attachments, references, footnotes)
IntegrationTest_ActiveEntity_Chapters_MultipleFacet — same tests on nested Chapters entities across all 3 facets

Workflow updates (6 files):
Added active-entity-test as a dedicated job in all integration test workflows (single-tenant and multi-tenant)
The job runs after the main integration-test slot to avoid fixture conflicts, and before the versioned/virus-scan slots
Tests run in parallel across namedUser / technicalUser token flows and TENANT1 / TENANT2 (multi-tenant)

Type of change

  • Integration test cases

Checklist before requesting a review

  • I follow Java Development Guidelines for SAP
  • I have tested the functionality on my cloud environment.
  • I have provided sufficient automated/ unit tests for the code.
  • I have increased or maintained the test coverage.
  • I have ran integration tests on my cloud environment.
  • I have validated blackduck portal for any vulnerability after my commit.

Upload Screenshots/lists of the scenarios tested

  • I have Uploaded Screenshots or added lists of the scenarios tested in description

Individual test cases run
Single Tenant : https://github.com/cap-java/sdm/actions/runs/29014326579
Multi Tenant : https://github.com/cap-java/sdm/actions/runs/29013763522

Comment thread sdm/src/test/java/integration/com/sap/cds/sdm/Api.java Fixed
Comment thread sdm/src/test/java/integration/com/sap/cds/sdm/ApiMT.java Fixed
Comment thread sdm/src/test/java/integration/com/sap/cds/sdm/Api.java Fixed
Comment thread sdm/src/test/java/integration/com/sap/cds/sdm/Api.java Fixed
Comment thread sdm/src/test/java/integration/com/sap/cds/sdm/ApiMT.java Fixed
Comment thread sdm/src/test/java/integration/com/sap/cds/sdm/ApiMT.java Fixed
Comment thread sdm/src/test/java/integration/com/sap/cds/sdm/ApiInterface.java Dismissed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant