From 6da7586c3ecf72ee9864baab6500477b889b8162 Mon Sep 17 00:00:00 2001
From: AdrianLongCarleton <AdrianLong@cmail.carleton.ca>
Date: Mon, 26 Jan 2026 23:56:52 -0500
Subject: [PATCH] Refactored assets to use propper java syntax. Found error in
 the raven test command. Impropper formating when passing arguments to the
 java command.

---
 README.md                 | 16 ++++++++++++++++
 assets/src/main/Main.java |  2 --
 assets/src/test/Main.java | 12 ------------
 src/java.rs               | 18 ++++++++++++++++++
 src/main.rs               |  9 ++++++---
 5 files changed, 40 insertions(+), 17 deletions(-)
 delete mode 100644 assets/src/test/Main.java

diff --git a/README.md b/README.md
index 9ab0e65..4a94467 100644
--- a/README.md
+++ b/README.md
@@ -32,6 +32,22 @@ cd demo
 raven run main.Main
 ```
 
+## Raven commands
+Usage: raven <COMMAND>
+
+Commands:
+  new    Create a new raven project
+  init   Create a new raven project in an existing directory
+  build  Compile the current project
+  run    Run the current project
+  test   !!! BORKED !!! Run the tests
+  clean  Remove the target directory and caching
+  help   Print this message or the help of the given subcommand(s)
+
+Options:
+  -h, --help     Print help
+  -V, --version  Print version
+
 ## Future plans
 
 - [ ] Make `raven run` fall back to an entry point specified in Nest.toml, when
diff --git a/assets/src/main/Main.java b/assets/src/main/Main.java
index e2c340f..52b9fad 100644
--- a/assets/src/main/Main.java
+++ b/assets/src/main/Main.java
@@ -1,5 +1,3 @@
-package main;
-
 public class Main {
 
     public static void main(String[] args) {
diff --git a/assets/src/test/Main.java b/assets/src/test/Main.java
deleted file mode 100644
index 4da99a4..0000000
--- a/assets/src/test/Main.java
+++ /dev/null
@@ -1,12 +0,0 @@
-package test;
-
-public class Main {
-
-    public static void main(String[] args) {
-        testAdd();
-    }
-
-    public static void testAdd() {
-        assert main.Main.add(2, 2) == 4;
-    }
-}
diff --git a/src/java.rs b/src/java.rs
index e9d6f7c..9df21a7 100644
--- a/src/java.rs
+++ b/src/java.rs
@@ -79,14 +79,32 @@ impl JVM {
     ) -> anyhow::Result<(Option<String>, Option<String>)> {
         let mut cmd = vec![JAVA_VM.to_string()];
 
+        println!("Fence 4");
+
+        // proper format
+        // java -Xms128m -Xmx512m -cp PROJECT_ROOT/src/text \test.Test
+        // current format
+        // java Xms128.0 MiB Xmx512.0 MiB /home/viffx/Coding/Rust/raven/src/Test1/src/test/
         cmd.extend(
             self.flags
                 .clone()
                 .into_iter()
                 .flat_map(|f| Into::<Vec<String>>::into(f)),
         );
+
+        println!("Fence 5");
+        for part in &cmd {
+            println!("{}", part);
+        }
         cmd.push(entry_point.as_ref().display().to_string());
 
+        println!("Fence 6");
+        for part in &cmd {
+            println!("{}", part);
+        }
+
+
+        println!("Fence 7");
         let result = crate::io::run_process(cmd.as_slice());
 
         if self.monitor
diff --git a/src/main.rs b/src/main.rs
index 6981bd5..155f38d 100644
--- a/src/main.rs
+++ b/src/main.rs
@@ -140,14 +140,14 @@ fn init() -> anyhow::Result<()> {
         f.write_all(include_bytes!("../assets/src/main/Main.java"))?;
     }
 
-    // Make src/test/Main.java
+    // Make src/test/Test.java
     if let Result::Ok(mut f) = OpenOptions::new().write(true).create_new(is_empty).open(
         DIR_TEST
             .clone()
-            .join("Main")
+            .join("Test")
             .with_extension(JAVA_EXT_SOURCE),
     ) {
-        f.write_all(include_bytes!("../assets/src/test/Main.java"))?;
+        f.write_all(include_bytes!("../assets/src/test/Test.java"))?;
     }
 
     // git init .
@@ -248,15 +248,18 @@ fn run<P: AsRef<Path>>(
 }
 
 fn test(assertions: bool) -> anyhow::Result<(Option<String>, Option<String>)> {
+    println!("Fence 1");
     java::CompilerBuilder::new()
         .class_path(DIR_TARGET.as_path())
         .destination(DIR_TARGET.as_path())
         .build()
         .compile(DIR_TEST.as_path())?;
 
+    println!("Fence 2");
     // Change cwd to avoid Java pathing issues.
     crate::env::set_cwd(DIR_TARGET.as_path())?;
 
+    println!("Fence 3");
     java::JVMBuilder::new()
         .assertions(assertions)
         .ram_min(ByteSize::mib(128))