Skip to content

Fix soft delete returning test. #22

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Fix soft delete returning test. #22

wants to merge 1 commit into from

Conversation

Mitika-d
Copy link
Member

Fix TestSoftDeleteReturning test by adding quoted identifiers and redirecting soft-delete to customised update clause with bulk returning instead of default update and returning clause. QueryContext is removed for soft delete as it is not compatible with update clause.

Type of change

  • Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

  • TestSoftDeleteReturning

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published in downstream modules

@oracle-contributor-agreement oracle-contributor-agreement bot added the OCA Verified All contributors have signed the Oracle Contributor Agreement. label Aug 19, 2025
@Mitika-d Mitika-d requested review from fmeheust and ting-lan-wang and removed request for fmeheust August 19, 2025 13:52
@Mitika-d Mitika-d self-assigned this Aug 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
OCA Verified All contributors have signed the Oracle Contributor Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant