fix: correct Java 25 system property definitions

Author: brianheineman

ristretto_vm/src/intrinsic_methods/jdk/internal/util/systemprops_raw.rs

@@ -32,7 +32,9 @@ pub(crate) async fn platform_properties(
     push_property(system_properties, &mut properties, "user.language")?;
     push_property(system_properties, &mut properties, "user.script")?;
     push_property(system_properties, &mut properties, "user.variant")?;
-    push_property(system_properties, &mut properties, "native.encoding")?;
+    if java_version < JAVA_25.java() {
+        push_property(system_properties, &mut properties, "native.encoding")?;
+    }
     push_property(system_properties, &mut properties, "file.separator")?;
     push_property(system_properties, &mut properties, "format.country")?;
     push_property(system_properties, &mut properties, "format.language")?;
@@ -48,6 +50,9 @@ pub(crate) async fn platform_properties(
     push_property(system_properties, &mut properties, "https.proxyPort")?;
     push_property(system_properties, &mut properties, "java.io.tmpdir")?;
     push_property(system_properties, &mut properties, "line.separator")?;
+    if java_version >= JAVA_25.java() {
+        push_property(system_properties, &mut properties, "native.encoding")?;
+    }
     push_property(system_properties, &mut properties, "os.arch")?;
     push_property(system_properties, &mut properties, "os.name")?;
     push_property(system_properties, &mut properties, "os.version")?;

ristretto_vm/tests/hello_world.rs

@@ -1,5 +1,8 @@
 use ristretto_vm::{ClassPath, ConfigurationBuilder, Result, VM};
+use std::io::Cursor;
 use std::path::PathBuf;
+use std::sync::Arc;
+use tokio::sync::Mutex;
 
 async fn test_helloworld(java_verison: &str) -> Result<()> {
     let cargo_manifest = PathBuf::from(env!("CARGO_MANIFEST_DIR"));
@@ -8,15 +11,23 @@ async fn test_helloworld(java_verison: &str) -> Result<()> {
         .join("classes")
         .join("classes.jar");
     let class_path = ClassPath::from(&[classes_jar_path]);
+    let stdout_buffer = Arc::new(Mutex::new(Cursor::new(Vec::<u8>::new())));
+
     let configuration = ConfigurationBuilder::new()
         .class_path(class_path.clone())
         .main_class("HelloWorld")
         .java_version(java_verison)
+        .stdout(stdout_buffer.clone())
         .build()?;
     let vm = VM::new(configuration).await?;
-    let parameters: Vec<&str> = Vec::new();
+    let parameters = vec!["world!"];
+
     let result = vm.invoke_main(&parameters).await?;
     assert!(result.is_none());
+    let output = stdout_buffer.lock().await;
+    let output_bytes = output.get_ref();
+    let output_str = String::from_utf8(output_bytes.clone()).expect("Invalid UTF-8 output");
+    assert_eq!(output_str.trim(), "Hello world!");
     Ok(())
 }