Skip to content

Conversation

@esezen
Copy link
Contributor

@esezen esezen commented Jul 22, 2024

If you run test all together or run one two times, the test fails because the facet configuration already exists. Unfortunately, we need to delete the facet configuration until we add the deleteFacetConfiguration function. I left TODO comments where we would add cleanup when we add the delete function

@esezen esezen requested a review from a team July 22, 2024 13:20
Copy link
Contributor

@stanlp1 stanlp1 left a comment

Choose a reason for hiding this comment

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

LGTM!

@stanlp1 stanlp1 merged commit 31eca79 into master Aug 8, 2024
@stanlp1 stanlp1 deleted the ci-3653-java-add-support-for-create-facet-configuration branch August 8, 2024 15:56
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.

3 participants