Skip to content

Commit

Permalink
Enable archive tests on windows again
Browse files Browse the repository at this point in the history
  • Loading branch information
breskeby committed Nov 6, 2024
1 parent e385ec1 commit 9ffcac2
Showing 1 changed file with 0 additions and 13 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,6 @@
import org.elasticsearch.packaging.util.ServerUtils;
import org.elasticsearch.packaging.util.Shell;
import org.elasticsearch.packaging.util.Shell.Result;
import org.junit.Assume;
import org.junit.BeforeClass;

import java.nio.file.Files;
Expand Down Expand Up @@ -113,10 +112,6 @@ public void test32SpecialCharactersInJdkPath() throws Exception {
}

public void test40AutoconfigurationNotTriggeredWhenNodeIsMeantToJoinExistingCluster() throws Exception {
Assume.assumeFalse(
"https://github.com/elastic/elasticsearch/issues/116299",
distribution.platform == Distribution.Platform.WINDOWS
);
// auto-config requires that the archive owner and the process user be the same,
Platforms.onWindows(() -> sh.chown(installation.config, installation.getOwner()));
FileUtils.assertPathsDoNotExist(installation.data);
Expand All @@ -130,10 +125,6 @@ public void test40AutoconfigurationNotTriggeredWhenNodeIsMeantToJoinExistingClus
}

public void test41AutoconfigurationNotTriggeredWhenNodeCannotContainData() throws Exception {
Assume.assumeFalse(
"https://github.com/elastic/elasticsearch/issues/116299",
distribution.platform == Distribution.Platform.WINDOWS
);
// auto-config requires that the archive owner and the process user be the same
Platforms.onWindows(() -> sh.chown(installation.config, installation.getOwner()));
ServerUtils.addSettingToExistingConfiguration(installation, "node.roles", "[\"voting_only\", \"master\"]");
Expand All @@ -146,10 +137,6 @@ public void test41AutoconfigurationNotTriggeredWhenNodeCannotContainData() throw
}

public void test42AutoconfigurationNotTriggeredWhenNodeCannotBecomeMaster() throws Exception {
Assume.assumeFalse(
"https://github.com/elastic/elasticsearch/issues/116299",
distribution.platform == Distribution.Platform.WINDOWS
);
// auto-config requires that the archive owner and the process user be the same
Platforms.onWindows(() -> sh.chown(installation.config, installation.getOwner()));
ServerUtils.addSettingToExistingConfiguration(installation, "node.roles", "[\"ingest\"]");
Expand Down

0 comments on commit 9ffcac2

Please sign in to comment.