Skip to content

Commit

Permalink
Merge pull request #13681 from jamezp/mixed-domain-fix
Browse files Browse the repository at this point in the history
[WFLY-14022] Enable a way to ignore the MixedDomainDeployment700TestCase due to ARTEMIS-2800.
  • Loading branch information
jamezp committed Oct 29, 2020
2 parents b747caf + c4a3ce5 commit 3a643d6
Show file tree
Hide file tree
Showing 2 changed files with 14 additions and 1 deletion.
Expand Up @@ -110,7 +110,9 @@ private Path getExpandedPath() {
private Path expandAsInstance(Version.AsVersion version) {
createIfNotExists(targetOldVersions);

Path file = oldVersionsBaseDir.resolve(version.getZipFileName());

final String path = resolveZipFileName(version);
Path file = oldVersionsBaseDir.resolve(path);
if (Files.notExists(file)) {
throw new RuntimeException("Old version not found in " + file.toAbsolutePath().toString());
}
Expand Down Expand Up @@ -218,4 +220,12 @@ private static Path verifyZipContents(Path root) throws IOException {
return result;
}

private static String resolveZipFileName(final Version.AsVersion requested) {
final String value = System.getProperty("override.mixed.domain." + requested.getVersion());
if (value != null) {
return value;
}
return requested.getZipFileName();
}

}
Expand Up @@ -39,6 +39,9 @@ public class MixedDomainDeployment700TestCase extends MixedDomainDeploymentTest
public static void beforeClass() {
// WFLY-12649 -- embedded broker doesn't start correctly on an EAP 7.0.0 server running on OpenJ9
Assume.assumeFalse(TestSuiteEnvironment.isJ9Jvm());
// WFLY-14022 -- 7.0.0.GA under certain JDK and Linux kernel versions expose ARTEMIS-2800
final String value = System.getProperty("ignore.ARTEMIS-2800");
Assume.assumeFalse(value != null && (value.isEmpty() || Boolean.parseBoolean(value)));

MixedDomain700TestSuite.initializeDomain();
}
Expand Down

0 comments on commit 3a643d6

Please sign in to comment.