Skip to content

Commit

Permalink
Revert "fix: disable custom security manager for tests which indirect…
Browse files Browse the repository at this point in the history
…ly look up the path of mvn executable"

This reverts commit 2f7125e.
  • Loading branch information
algomaster99 committed Dec 9, 2021
1 parent 20b8ab6 commit f3b0280
Show file tree
Hide file tree
Showing 3 changed files with 0 additions and 12 deletions.
3 changes: 0 additions & 3 deletions src/test/java/sorald/ClasspathModeTest.java
Original file line number Diff line number Diff line change
Expand Up @@ -19,9 +19,6 @@ class ClasspathModeTest {
@Test
void resolveClasspathFrom_enablesRepairOfViolation_thatRequiresClasspathToDetect(
@TempDir File workdir) throws IOException {
// use the default security manager
System.setSecurityManager(null);

// arrange
org.apache.commons.io.FileUtils.copyDirectory(
TestHelper.PATH_TO_RESOURCES_FOLDER
Expand Down
6 changes: 0 additions & 6 deletions src/test/java/sorald/MavenLauncherTest.java
Original file line number Diff line number Diff line change
Expand Up @@ -22,9 +22,6 @@ public class MavenLauncherTest {
@Test
public void sorald_repairsProductionAndTestCode_inMavenProject(@TempDir File workdir)
throws IOException {
// use the default security manager
System.setSecurityManager(null);

// arrange
org.apache.commons.io.FileUtils.copyDirectory(
TestHelper.PATH_TO_RESOURCES_FOLDER
Expand Down Expand Up @@ -67,9 +64,6 @@ public void sorald_repairsProductionAndTestCode_inMavenProject(@TempDir File wor
@Test
void sorald_repairsRuleViolation_thatRequiresClasspathToDetect(@TempDir File workdir)
throws IOException {
// use the default security manager
System.setSecurityManager(null);

// arrange
org.apache.commons.io.FileUtils.copyDirectory(
TestHelper.PATH_TO_RESOURCES_FOLDER
Expand Down
3 changes: 0 additions & 3 deletions src/test/java/sorald/miner/WarningMinerTest.java
Original file line number Diff line number Diff line change
Expand Up @@ -194,9 +194,6 @@ public void extractWarnings_statsOutput_containsExpectedAttributes() throws Exce
@Test
void canDetectRuleViolation_thatRequiresClasspath_whenResolvingClasspathInMavenProject(
@TempDir File tempdir) throws Exception {
// use the default security manager
System.setSecurityManager(null);

Path statsFile = tempdir.toPath().resolve("stats.json");
Path projectRoot =
TestHelper.PATH_TO_RESOURCES_FOLDER
Expand Down

0 comments on commit f3b0280

Please sign in to comment.