mirror of
https://github.com/jenkinsci/jenkins.git
synced 2026-10-26 19:15:09 +00:00
Clean up obsolete Java version checks in tests (#27523)
This commit is contained in:
1 parent
379584d636
commit
4386cf394a
5 files changed
+36
-64
No files matched your search
@@ -6,13 +6,11 @@ import static org.hamcrest.Matchers.empty;
|
||||
import static org.hamcrest.Matchers.hasItem;
|
||||
import static org.hamcrest.Matchers.not;
|
||||
|
||||
import hudson.util.VersionNumber;
|
||||
import java.lang.ref.WeakReference;
|
||||
import java.util.List;
|
||||
import java.util.logging.LogRecord;
|
||||
import java.util.logging.Logger;
|
||||
import java.util.stream.Collectors;
|
||||
import jenkins.util.java.JavaUtils;
|
||||
import org.junit.jupiter.api.Test;
|
||||
import org.junit.jupiter.api.extension.RegisterExtension;
|
||||
import org.jvnet.hudson.test.JenkinsRule;
|
||||
@@ -35,22 +33,19 @@ class JenkinsLogRecordsTest {
|
||||
assertThat("Records are displayed in reverse order",
|
||||
logRecords.stream().map(LogRecord::getMessage).collect(Collectors.toList()),
|
||||
containsInRelativeOrder("Completed initialization", "Started initialization"));
|
||||
VersionNumber javaVersion = new VersionNumber(JavaUtils.getCurrentRuntimeJavaVersion());
|
||||
if (javaVersion.isNewerThanOrEqualTo(new VersionNumber("17"))) { // TODO https://github.com/jenkinsci/jenkins-test-harness/issues/359
|
||||
Object x = new Object() {
|
||||
@Override
|
||||
public String toString() {
|
||||
return "collect me";
|
||||
}
|
||||
};
|
||||
use(x);
|
||||
WeakReference<Object> ref = new WeakReference<>(x);
|
||||
x = null;
|
||||
MemoryAssert.assertGC(ref, true);
|
||||
assertThat("Record parameters formatted before collection",
|
||||
logRecords.stream().map(LogRecord::getMessage).collect(Collectors.toList()),
|
||||
hasItem("formatting collect me"));
|
||||
}
|
||||
Object x = new Object() {
|
||||
@Override
|
||||
public String toString() {
|
||||
return "collect me";
|
||||
}
|
||||
};
|
||||
use(x);
|
||||
WeakReference<Object> ref = new WeakReference<>(x);
|
||||
x = null;
|
||||
MemoryAssert.assertGC(ref, true);
|
||||
assertThat("Record parameters formatted before collection",
|
||||
logRecords.stream().map(LogRecord::getMessage).collect(Collectors.toList()),
|
||||
hasItem("formatting collect me"));
|
||||
}
|
||||
|
||||
private static void use(Object x) {
|
||||
|
||||
Reference in new issue
Block a user