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

[WFLY-17885] Work around DeploymentModulesListTestCase's use of the o… #31

Open
wants to merge 1 commit into
base: WFLY-17885_remove_6.0alias
Choose a base branch
from
Open
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
Expand Up @@ -59,7 +59,7 @@ public class DeploymentModulesListTestCase {
private static final String INNER_WAR_ARCHIVE_NAME = "list-modules.war";
private static final String EAR_DEPLOYMENT_NAME = "list-modules-ear-test.ear";
private static final String USER_MODULE = "org.hibernate";
private static final String CUSTOM_SLOT = "6.0";
private static final String CUSTOM_SLOT = "main"; // TODO WFLY-17891 install a TestModule with a non-main slot

@ContainerResource
private static ManagementClient managementClient;
Expand Down Expand Up @@ -121,7 +121,8 @@ private void listEarModules(boolean verbose) throws Throwable {
// check module presence
assertTrue(checkModulesListPresence(operationResult, "deployment." + EAR_DEPLOYMENT_NAME));
// check user defined module with custom slot
assertTrue(checkModulesListPresence(operationResult, USER_MODULE + ":" + CUSTOM_SLOT));
// TODO WFLY-17891 install a TestModule with a non-main slot
assertTrue(checkModulesListPresence(operationResult, USER_MODULE /*+ ":" + CUSTOM_SLOT*/));
// check module absence
assertFalse(checkModulesListPresence(operationResult, EXAMPLE_MODULE_TO_EXCLUDE));
// check system and user dependencies presence
Expand Down