Skip to content

Commit cf7859d

Browse files
committed
Adjust tests
1 parent b7ab377 commit cf7859d

File tree

1 file changed

+1
-26
lines changed

1 file changed

+1
-26
lines changed

dd-java-agent/agent-profiling/profiling-controller/src/test/java/com/datadog/profiling/controller/TempLocationManagerTest.java

Lines changed: 1 addition & 26 deletions
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,6 @@
1919
import java.util.UUID;
2020
import java.util.concurrent.Phaser;
2121
import java.util.concurrent.TimeUnit;
22-
import java.util.concurrent.atomic.AtomicBoolean;
2322
import java.util.concurrent.locks.LockSupport;
2423
import java.util.stream.Stream;
2524
import org.junit.jupiter.api.Test;
@@ -80,8 +79,7 @@ void testFromConfigNotWritable() throws Exception {
8079
Properties props = new Properties();
8180
props.put(ProfilingConfig.PROFILING_TEMP_DIR, myDir.toString());
8281
ConfigProvider configProvider = ConfigProvider.withPropertiesOverride(props);
83-
TempLocationManager tempLocationManager = new TempLocationManager(configProvider);
84-
assertThrows(IllegalStateException.class, tempLocationManager::getTempDir);
82+
assertThrows(IllegalStateException.class, () -> new TempLocationManager(configProvider));
8583
}
8684

8785
@ParameterizedTest
@@ -246,29 +244,6 @@ public FileVisitResult visitFile(Path file, BasicFileAttributes attrs)
246244
assertEquals(shouldSucceed, rslt);
247245
}
248246

249-
@Test
250-
void testShortCircuit() throws Exception {
251-
Path baseDir =
252-
Files.createTempDirectory(
253-
"ddprof-test-",
254-
PosixFilePermissions.asFileAttribute(PosixFilePermissions.fromString("rwx------")));
255-
AtomicBoolean executed = new AtomicBoolean();
256-
TempLocationManager.CleanupHook hook =
257-
new TempLocationManager.CleanupHook() {
258-
@Override
259-
public void onCleanupStart(boolean selfCleanup, long timeout, TimeUnit unit) {
260-
executed.set(true);
261-
}
262-
};
263-
TempLocationManager instance = instance(baseDir, false, hook);
264-
265-
instance.createDirStructure();
266-
267-
boolean ret = instance.cleanup(false);
268-
assertTrue(ret);
269-
assertFalse(executed.get());
270-
}
271-
272247
private static Stream<Arguments> timeoutTestArguments() {
273248
List<Arguments> argumentsList = new ArrayList<>();
274249
for (boolean selfCleanup : new boolean[] {true, false}) {

0 commit comments

Comments
 (0)