Skip to content
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

Push test metadata upon entity deletion #594

Merged
merged 1 commit into from
Jul 5, 2023
Merged

Conversation

MKodde
Copy link
Contributor

@MKodde MKodde commented Jun 27, 2023

When a test entity is removed from manage, also push the updated metadata to EngineBlock and the OidcNg gateway.

https://www.pivotaltracker.com/story/show/184915015

@MKodde MKodde force-pushed the feature/push-after-delete branch from 53c222a to 7789466 Compare June 28, 2023 06:36
@MKodde MKodde added the 5.1 label Jul 3, 2023
@MKodde MKodde force-pushed the feature/push-after-delete branch 3 times, most recently from d2591c3 to 2997521 Compare July 3, 2023 10:10
@MKodde MKodde requested a review from rubencatshoek July 3, 2023 11:42
Copy link

@rubencatshoek rubencatshoek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@@ -58,5 +60,8 @@ public function handle(DeletePublishedTestEntityCommand $command)
if ($response !== DeleteManageEntityRepository::RESULT_SUCCESS) {
throw new EntityNotDeletedException('Deleting the entity yielded an non success response');

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
throw new EntityNotDeletedException('Deleting the entity yielded an non success response');
throw new EntityNotDeletedException('Deleting the entity yielded a non success response');

@MKodde MKodde force-pushed the feature/push-after-delete branch from 2997521 to ae2325f Compare July 5, 2023 07:13
@MKodde MKodde merged commit bcbe488 into develop Jul 5, 2023
@MKodde MKodde deleted the feature/push-after-delete branch July 5, 2023 07:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants