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

[WFCORE-7012][WFCORE-7013] Fix ServerEnvironmentTestCase and ignore when server is running in default stability level #6205

Merged
merged 2 commits into from
Sep 27, 2024

Conversation

yersan
Copy link
Collaborator

@yersan yersan commented Sep 27, 2024

@yersan yersan requested a review from michpetrov September 27, 2024 11:19
@yersan
Copy link
Collaborator Author

yersan commented Sep 27, 2024

Hello @michpetrov , could you review this? thanks

Copy link
Contributor

@michpetrov michpetrov left a comment

Choose a reason for hiding this comment

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

@yersan LGTM!

@@ -104,6 +109,7 @@ public void testAliasFunctionality() throws IOException {

// default stability = COMMUNITY
ProductConfig productConfig = ProductConfig.fromFilesystemSlot(null, "", props);
Assume.assumeTrue(Stability.COMMUNITY.enables(productConfig.getDefaultStability()));
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Ups, this should be equals not enables

@yersan yersan force-pushed the WFCORE-7012-WFCORE-7013 branch from 78652de to 375c463 Compare September 27, 2024 13:42
@yersan yersan added the ready-for-merge This PR is ready to be merged and fulfills all requirements label Sep 27, 2024
@yersan yersan merged commit 63d052d into wildfly:main Sep 27, 2024
10 of 12 checks passed
@yersan yersan deleted the WFCORE-7012-WFCORE-7013 branch September 27, 2024 15:29
@yersan
Copy link
Collaborator Author

yersan commented Sep 27, 2024

Thanks @michpetrov

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-for-merge This PR is ready to be merged and fulfills all requirements
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants