Skip to content

Fix copy attachment upload status handling and failing integration tests#454

Open
deepikaSingh2711 wants to merge 4 commits intodevelopfrom
IntegrationTestsFix
Open

Fix copy attachment upload status handling and failing integration tests#454
deepikaSingh2711 wants to merge 4 commits intodevelopfrom
IntegrationTestsFix

Conversation

@deepikaSingh2711
Copy link
Collaborator

@deepikaSingh2711 deepikaSingh2711 commented Mar 23, 2026

Describe your changes

In this PR

  1. Added code changes to fix failing copy and move attachment operations
    Explicitly setting upload status to “Success” when creating draft entries for copy attachment operations, since copied attachments originate from already uploaded source entities
  2. Removed explicit wait time logic from integration tests
  3. Fixed previously failing / commented-out test cases

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

https://github.com/cap-java/sdm/actions/runs/23447301753

@deepikaSingh2711 deepikaSingh2711 changed the title removed wait time upload logic Fix copy attachment upload status handling and failing integration tests Mar 23, 2026
Co-authored-by: Copilot Autofix powered by AI <223894421+github-code-quality[bot]@users.noreply.github.com>
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