Skip to content

Conversation

@deepikaSingh2711
Copy link
Collaborator

@deepikaSingh2711 deepikaSingh2711 commented Jan 7, 2026

Describe your changes

This PR covers Integration tests for move attachments

Scenarios:
1)Move attachments from Source Entity to Target Entity with sourceFacet
2)Move attachments to entity with duplicate attachment with sourceFacet
3)Move attachments with notes and secondary properties with sourceFacet
4)Move valid attachments from Source Entity to Target Entity without sourceFacet
5)Move attachments into existing Target Entity when duplicate exists without sourceFacet
6)Move attachments with notes and secondary properties without sourceFacet
7)Move attachments with invalid or undefined secondary properties
8)Move attachments from Source Entity when Source Entity is in draft mode
9)Edit attachment file name in Source Entity and move it to Target Entity
10)Move attachments from Source Entity to Target Entity 1 and then to Target Entity 2
11)Move attachments when user does not have SDM Role

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

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

Single tenant Integration test result
https://github.com/cap-java/sdm/actions/runs/20772556007

Multi tenant Integration test result
https://github.com/cap-java/sdm/actions/runs/20804656478

@deepikaSingh2711 deepikaSingh2711 changed the title move attachments Integration tests move attachment Integration tests Jan 7, 2026
@deepikaSingh2711 deepikaSingh2711 merged commit 72e219b into develop Jan 8, 2026
9 checks passed
@deepikaSingh2711 deepikaSingh2711 deleted the moveAttachmentsIT branch January 8, 2026 10:25
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.

5 participants