Skip to content

Test/test java8 #2

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
ninan-nn wants to merge 5 commits into
base: main
Choose a base branch
from
Open

Test/test java8 #2

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
020d412
test: test java 8
ninan-nn Dec 24, 2025
ea0dd3a
test: test jvm 8
ninan-nn Dec 24, 2025
711e235
test: test jvm 8
ninan-nn Dec 24, 2025
f10be5b
test: test jvm 8
ninan-nn Dec 24, 2025
9f478ff
test: test jvm 8
ninan-nn Dec 24, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 3 additions & 2 deletions tests/java/build.gradle.kts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -23,8 +23,9 @@ group = "com.alibaba.opensandbox"
version = "1.0.0"

java {
sourceCompatibility = JavaVersion.VERSION_17
targetCompatibility = JavaVersion.VERSION_17
toolchain {
languageVersion.set(JavaLanguageVersion.of(8))
}
}

repositories {
Expand Down
4 changes: 4 additions & 0 deletions tests/java/settings.gradle.kts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -15,3 +15,7 @@
*/

rootProject.name = "opensandbox-java-e2e-tests"

plugins {
id("org.gradle.toolchains.foojay-resolver-convention") version("1.0.0")
}
4 changes: 2 additions & 2 deletions tests/java/src/test/java/com/alibaba/opensandbox/e2e/BaseE2ETest.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -109,14 +109,14 @@ protected static void assertEndpointHasPort(String endpoint, int expectedPort) {
endpoint.endsWith("/" + expectedPort),
"endpoint route must end with /" + expectedPort + ": " + endpoint);
String prefix = endpoint.split("/", 2)[0];
assertFalse(prefix.isBlank(), "missing domain in endpoint: " + endpoint);
assertFalse(prefix.trim().isEmpty(), "missing domain in endpoint: " + endpoint);
return;
}
int idx = endpoint.lastIndexOf(':');
assertTrue(idx > 0, "missing host:port in endpoint: " + endpoint);
String host = endpoint.substring(0, idx);
String port = endpoint.substring(idx + 1);
assertFalse(host.isBlank(), "missing host in endpoint: " + endpoint);
assertFalse(host.trim().isEmpty(), "missing host in endpoint: " + endpoint);
assertTrue(port.matches("\\d+"), "non-numeric port in endpoint: " + endpoint);
assertEquals(expectedPort, Integer.parseInt(port), "endpoint port mismatch: " + endpoint);
}
Expand Down
24 changes: 15 additions & 9 deletions tests/java/src/test/java/com/alibaba/opensandbox/e2e/CodeInterpreterE2ETest.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -30,9 +30,7 @@
import com.alibaba.opensandbox.sandbox.domain.models.sandboxes.SandboxMetrics;
import java.time.Duration;
import java.time.OffsetDateTime;
import java.util.ArrayList;
import java.util.Collections;
import java.util.List;
import java.util.*;
import java.util.concurrent.*;
import org.junit.jupiter.api.*;
import org.slf4j.Logger;
Expand All @@ -58,14 +56,21 @@ public class CodeInterpreterE2ETest extends BaseE2ETest {
private Sandbox sandbox;
private CodeInterpreter codeInterpreter;

private static Map<String, String> createResourceMap() {
Map<String, String> map = new HashMap<>();
map.put("cpu", "2");
map.put("memory", "4Gi");
return map;
}

private static void assertTerminalEventContract(
List<ExecutionInit> initEvents,
List<ExecutionComplete> completedEvents,
List<ExecutionError> errors,
String executionId) {
assertEquals(1, initEvents.size(), "init event must exist exactly once");
assertNotNull(initEvents.get(0).getId());
assertFalse(initEvents.get(0).getId().isBlank());
assertFalse(initEvents.get(0).getId().trim().isEmpty());
assertEquals(executionId, initEvents.get(0).getId());
assertRecentTimestampMs(initEvents.get(0).getTimestamp(), 180_000);
assertTrue(
Expand All @@ -78,7 +83,7 @@ private static void assertTerminalEventContract(
}
if (!errors.isEmpty()) {
assertNotNull(errors.get(0).getName());
assertFalse(errors.get(0).getName().isBlank());
assertFalse(errors.get(0).getName().trim().isEmpty());
assertNotNull(errors.get(0).getValue());
assertRecentTimestampMs(errors.get(0).getTimestamp(), 180_000);
}
Expand All @@ -89,12 +94,13 @@ void setup() {
sandbox =
Sandbox.builder()
.connectionConfig(sharedConnectionConfig)
.entrypoint(List.of("/opt/opensandbox/code-interpreter.sh"))
.entrypoint(
Collections.singletonList("/opt/opensandbox/code-interpreter.sh"))
.image(getSandboxImage())
.resource(java.util.Map.of("cpu", "2", "memory", "4Gi"))
.resource(createResourceMap())
.timeout(Duration.ofMinutes(15))
.readyTimeout(Duration.ofSeconds(60))
.metadata(java.util.Map.of("tag", "e2e-code-interpreter"))
.metadata(Collections.singletonMap("tag", "e2e-code-interpreter"))
.env("E2E_TEST", "true")
.env("GO_VERSION", "1.25")
.env("JAVA_VERSION", "21")
Expand Down Expand Up @@ -249,7 +255,7 @@ void testJavaCodeExecution() {

assertNotNull(simpleResult);
assertNotNull(simpleResult.getId());
assertFalse(simpleResult.getId().isBlank());
assertFalse(simpleResult.getId().trim().isEmpty());
assertEquals("4", simpleResult.getResult().get(0).getText());
assertTerminalEventContract(initEvents, completedEvents, errors, simpleResult.getId());
assertTrue(errors.isEmpty());
Expand Down
56 changes: 34 additions & 22 deletions tests/java/src/test/java/com/alibaba/opensandbox/e2e/SandboxE2ETest.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -102,7 +102,7 @@ private static void assertTerminalEventContract(
String executionId) {
assertEquals(1, initEvents.size(), "Execution must have exactly one init event");
assertNotNull(initEvents.get(0).getId());
assertFalse(initEvents.get(0).getId().isBlank());
assertFalse(initEvents.get(0).getId().trim().isEmpty());
assertEquals(executionId, initEvents.get(0).getId(), "init.id must match execution.id");
assertRecentTimestampMs(initEvents.get(0).getTimestamp(), 120_000);

Expand All @@ -121,7 +121,7 @@ private static void assertTerminalEventContract(
}
if (hasError) {
assertNotNull(errors.get(0).getName());
assertFalse(errors.get(0).getName().isBlank());
assertFalse(errors.get(0).getName().trim().isEmpty());
assertNotNull(errors.get(0).getValue());
assertRecentTimestampMs(errors.get(0).getTimestamp(), 180_000);
}
Expand All @@ -142,7 +142,7 @@ void testSandboxLifecycleAndHealth() {
assertNotNull(info.getCreatedAt());
assertNotNull(info.getExpiresAt());
assertTrue(info.getExpiresAt().isAfter(info.getCreatedAt()));
assertEquals(List.of("tail", "-f", "/dev/null"), info.getEntrypoint());
assertEquals(Arrays.asList("tail", "-f", "/dev/null"), info.getEntrypoint());

Duration duration = Duration.between(info.getCreatedAt(), info.getExpiresAt());
assertTrue(duration.compareTo(Duration.ofMinutes(1)) >= 0);
Expand Down Expand Up @@ -261,7 +261,7 @@ void testBasicCommandExecution() {

assertNotNull(echoResult);
assertNotNull(echoResult.getId());
assertFalse(echoResult.getId().isBlank());
assertFalse(echoResult.getId().trim().isEmpty());
assertNull(echoResult.getError());
assertEquals(1, echoResult.getLogs().getStdout().size());
assertEquals("Hello OpenSandbox E2E", echoResult.getLogs().getStdout().get(0).getText());
Expand Down Expand Up @@ -316,7 +316,7 @@ void testBasicCommandExecution() {
Execution failResult = sandbox.commands().run(failRequest);
assertNotNull(failResult);
assertNotNull(failResult.getId());
assertFalse(failResult.getId().isBlank());
assertFalse(failResult.getId().trim().isEmpty());
assertNotNull(failResult.getError());
assertEquals("CommandExecError", failResult.getError().getName());
assertTrue(failResult.getLogs().getStderr().size() > 0);
Expand Down Expand Up @@ -350,9 +350,10 @@ void testBasicFilesystemOperations() {
WriteEntry dirEntry1 = WriteEntry.builder().path(testDir1).mode(755).build();
WriteEntry dirEntry2 = WriteEntry.builder().path(testDir2).mode(644).build();

sandbox.files().createDirectories(List.of(dirEntry1, dirEntry2));
sandbox.files().createDirectories(Arrays.asList(dirEntry1, dirEntry2));

Map<String, EntryInfo> dirInfo = sandbox.files().readFileInfo(List.of(testDir1, testDir2));
Map<String, EntryInfo> dirInfo =
sandbox.files().readFileInfo(Arrays.asList(testDir1, testDir2));
assertEquals(testDir1, dirInfo.get(testDir1).getPath());
assertEquals(755, dirInfo.get(testDir1).getMode());
assertTimesClose(
Expand Down Expand Up @@ -392,7 +393,7 @@ void testBasicFilesystemOperations() {
.mode(755)
.build();

sandbox.files().write(List.of(writeEntry1, writeEntry2, writeEntry3));
sandbox.files().write(Arrays.asList(writeEntry1, writeEntry2, writeEntry3));

String readContent1 =
sandbox.files().readFile(testFile1, StandardCharsets.UTF_8.name(), null);
Expand All @@ -403,7 +404,13 @@ void testBasicFilesystemOperations() {
String readContent2 = new String(readBytes2, StandardCharsets.UTF_8);

try (java.io.InputStream inputStream = sandbox.files().readStream(testFile3, null)) {
byte[] streamBytes = inputStream.readAllBytes();
java.io.ByteArrayOutputStream buffer = new java.io.ByteArrayOutputStream();
int nRead;
byte[] data = new byte[1024];
while ((nRead = inputStream.read(data, 0, data.length)) != -1) {
buffer.write(data, 0, nRead);
}
byte[] streamBytes = buffer.toByteArray();
String readContent3 = new String(streamBytes, StandardCharsets.UTF_8);

// Verify content matches original for all files
Expand All @@ -420,7 +427,7 @@ void testBasicFilesystemOperations() {
throw new RuntimeException("Failed to read stream", e);
}

List<String> allTestFiles = List.of(testFile1, testFile2, testFile3);
List<String> allTestFiles = Arrays.asList(testFile1, testFile2, testFile3);
Map<String, EntryInfo> fileInfoMap = sandbox.files().readFileInfo(allTestFiles);
long expectedSize = testContent.getBytes(StandardCharsets.UTF_8).length;

Expand Down Expand Up @@ -456,7 +463,7 @@ void testBasicFilesystemOperations() {
for (EntryInfo e : sandbox.files().search(searchAllEntry)) {
found.add(e.getPath());
}
assertEquals(Set.of(testFile1, testFile2, testFile3), found);
assertEquals(new HashSet<>(Arrays.asList(testFile1, testFile2, testFile3)), found);

SetPermissionEntry permEntry1 =
SetPermissionEntry.builder()
Expand All @@ -472,11 +479,11 @@ void testBasicFilesystemOperations() {
.owner("nobody")
.group("nogroup")
.build();
sandbox.files().setPermissions(List.of(permEntry1, permEntry2));
sandbox.files().setPermissions(Arrays.asList(permEntry1, permEntry2));

// Verify permission changes for both files in single call
Map<String, EntryInfo> updatedInfoMap =
sandbox.files().readFileInfo(List.of(testFile1, testFile2));
sandbox.files().readFileInfo(Arrays.asList(testFile1, testFile2));
EntryInfo updatedInfo1 = updatedInfoMap.get(testFile1);
EntryInfo updatedInfo2 = updatedInfoMap.get(testFile2);

Expand All @@ -494,7 +501,8 @@ void testBasicFilesystemOperations() {
assertEquals(
"nogroup", updatedInfo2.getGroup(), "testFile2 group should be updated to nogroup");

EntryInfo beforeUpdate = sandbox.files().readFileInfo(List.of(testFile1)).get(testFile1);
EntryInfo beforeUpdate =
sandbox.files().readFileInfo(Collections.singletonList(testFile1)).get(testFile1);
String updatedContent1 = testContent + "\nAppended line to file1";
String updatedContent2 = testContent + "\nAppended line to file2";
try {
Expand All @@ -505,14 +513,15 @@ void testBasicFilesystemOperations() {
WriteEntry.builder().path(testFile1).data(updatedContent1).mode(644).build();
WriteEntry updateEntry2 =
WriteEntry.builder().path(testFile2).data(updatedContent2).mode(755).build();
sandbox.files().write(List.of(updateEntry1, updateEntry2));
sandbox.files().write(Arrays.asList(updateEntry1, updateEntry2));

String newContent1 = sandbox.files().readFile(testFile1, "UTF-8", null);
String newContent2 = sandbox.files().readFile(testFile2, "UTF-8", null);
assertEquals(updatedContent1, newContent1);
assertEquals(updatedContent2, newContent2);

EntryInfo afterUpdate = sandbox.files().readFileInfo(List.of(testFile1)).get(testFile1);
EntryInfo afterUpdate =
sandbox.files().readFileInfo(Collections.singletonList(testFile1)).get(testFile1);
assertEquals(
updatedContent1.getBytes(StandardCharsets.UTF_8).length, afterUpdate.getSize());
assertModifiedUpdated(beforeUpdate.getModifiedAt(), afterUpdate.getModifiedAt(), 1, 1000);
Expand All @@ -525,7 +534,7 @@ void testBasicFilesystemOperations() {
}
sandbox.files()
.replaceContents(
List.of(
Collections.singletonList(
ContentReplaceEntry.builder()
.path(testFile1)
.oldContent("Appended line to file1")
Expand All @@ -534,30 +543,33 @@ void testBasicFilesystemOperations() {
String replaced = sandbox.files().readFile(testFile1, "UTF-8", null);
assertTrue(replaced.contains("Replaced line in file1"));
assertFalse(replaced.contains("Appended line to file1"));
EntryInfo afterReplace = sandbox.files().readFileInfo(List.of(testFile1)).get(testFile1);
EntryInfo afterReplace =
sandbox.files().readFileInfo(Collections.singletonList(testFile1)).get(testFile1);
assertModifiedUpdated(beforeReplace.getModifiedAt(), afterReplace.getModifiedAt(), 1, 1000);

// Move file3
String movedPath = testDir2 + "/moved_file3.txt";
sandbox.files()
.moveFiles(List.of(MoveEntry.builder().src(testFile3).dest(movedPath).build()));
.moveFiles(
Collections.singletonList(
MoveEntry.builder().src(testFile3).dest(movedPath).build()));
String moved =
new String(sandbox.files().readByteArray(movedPath, null), StandardCharsets.UTF_8);
assertEquals(testContent, moved);
assertThrows(Exception.class, () -> sandbox.files().readByteArray(testFile3, null));

// Delete file2
sandbox.files().deleteFiles(List.of(testFile2));
sandbox.files().deleteFiles(Collections.singletonList(testFile2));
assertThrows(Exception.class, () -> sandbox.files().readFile(testFile2, "UTF-8", null));
Set<String> after = new HashSet<>();
for (EntryInfo e :
sandbox.files().search(SearchEntry.builder().path(testDir1).pattern("*").build())) {
after.add(e.getPath());
}
assertEquals(Set.of(testFile1), after);
assertEquals(Collections.singleton(testFile1), after);

// Delete directories
sandbox.files().deleteDirectories(List.of(testDir1, testDir2));
sandbox.files().deleteDirectories(Arrays.asList(testDir1, testDir2));
Execution verify =
sandbox.commands()
.run(
Expand Down
Loading
Loading