add more unit tests for V8

gosub-io · Feb 23, 2024 · 0d8a4ab · 0d8a4ab
1 parent 3bb1e61
commit 0d8a4ab
Show file tree

Hide file tree

Showing 7 changed files with 718 additions and 27 deletions.
diff --git a/src/web_executor/js.rs b/src/web_executor/js.rs
@@ -74,6 +74,7 @@ pub trait JSArray {
     //TODO: implement other things when needed. Maybe also `Iterator`?
 }
 
+#[derive(Debug, Clone, PartialEq)]
 pub enum JSType {
     Undefined,
     Null,

diff --git a/src/web_executor/js/function.rs b/src/web_executor/js/function.rs
@@ -15,7 +15,10 @@ pub trait JSFunction {
     where
         Self: Sized;
 
-    fn call(&mut self, callback: &mut <Self::RT as JSRuntime>::FunctionCallBack);
+    fn call(
+        &mut self,
+        args: &[<Self::RT as JSRuntime>::Value],
+    ) -> Result<<Self::RT as JSRuntime>::Value>;
 }
 
 pub trait JSFunctionCallBack {
@@ -64,7 +67,10 @@ pub trait JSFunctionVariadic {
     where
         Self: Sized;
 
-    fn call(&mut self, callback: &mut <Self::RT as JSRuntime>::FunctionCallBackVariadic);
+    fn call(
+        &mut self,
+        args: &[<Self::RT as JSRuntime>::Value],
+    ) -> Result<<Self::RT as JSRuntime>::Value>;
 }
 
 pub trait JSFunctionCallBackVariadic {

diff --git a/src/web_executor/js/v8.rs b/src/web_executor/js/v8.rs
@@ -63,7 +63,6 @@ impl V8Engine<'_> {
             return;
         }
 
-
         V8_INITIALIZED.call_once(|| {
             V8_INITIALIZING.store(true, Ordering::SeqCst);
             //https://github.com/denoland/rusty_v8/issues/1381

diff --git a/src/web_executor/js/v8/array.rs b/src/web_executor/js/v8/array.rs
@@ -9,6 +9,17 @@ pub struct V8Array<'a> {
     ctx: V8Context<'a>,
 }
 
+impl<'a> V8Array<'a> {
+    pub fn new(ctx: &V8Context<'a>, len: u32) -> Result<Self> {
+        let value = Array::new(ctx.borrow_mut().scope(), len as i32);
+
+        Ok(Self {
+            value,
+            ctx: ctx.clone(),
+        })
+    }
+}
+
 impl<'a> JSArray for V8Array<'a> {
     type RT = V8Engine<'a>;
 
@@ -91,3 +102,123 @@ impl<'a> JSArray for V8Array<'a> {
         Ok(self.value.length())
     }
 }
+
+#[cfg(test)]
+mod tests {
+    use crate::web_executor::js::v8::{V8Array, V8Engine};
+    use crate::web_executor::js::{JSArray, JSRuntime, JSValue, ValueConversion};
+
+    #[test]
+    fn set() {
+        let mut engine = V8Engine::new();
+        let mut context = engine.new_context().unwrap();
+
+        let array = V8Array::new(&context, 2).unwrap();
+
+        array
+            .set(0u32, &1234.0.to_js_value(context.clone()).unwrap())
+            .unwrap();
+        array
+            .set(1u32, &"Hello World!".to_js_value(context).unwrap())
+            .unwrap();
+
+        assert_eq!(array.length().unwrap(), 2);
+    }
+
+    #[test]
+    fn get() {
+        let mut engine = V8Engine::new();
+        let mut context = engine.new_context().unwrap();
+
+        let array = V8Array::new(&context, 2).unwrap();
+
+        array
+            .set(0u32, &1234.0.to_js_value(context.clone()).unwrap())
+            .unwrap();
+        array
+            .set(1u32, &"Hello World!".to_js_value(context).unwrap())
+            .unwrap();
+
+        assert_eq!(array.get(0u32).unwrap().as_number().unwrap(), 1234.0);
+        assert_eq!(
+            array.get(1u32).unwrap().as_string().unwrap(),
+            "Hello World!"
+        );
+    }
+
+    #[test]
+    fn push() {
+        let mut engine = V8Engine::new();
+        let mut context = engine.new_context().unwrap();
+
+        let array = V8Array::new(&context, 2).unwrap();
+
+        array
+            .push(1234.0.to_js_value(context.clone()).unwrap())
+            .unwrap();
+        array
+            .push("Hello World!".to_js_value(context).unwrap())
+            .unwrap();
+
+        assert_eq!(array.length().unwrap(), 4);
+    }
+
+    #[test]
+    fn out_of_bounds() {
+        let mut engine = V8Engine::new();
+        let mut context = engine.new_context().unwrap();
+
+        let array = V8Array::new(&context, 2).unwrap();
+
+        array
+            .set(0u32, &1234.0.to_js_value(context.clone()).unwrap())
+            .unwrap();
+        array
+            .set(1u32, &"Hello World!".to_js_value(context).unwrap())
+            .unwrap();
+
+        assert!(array.get(2u32).unwrap().is_undefined());
+    }
+
+    #[test]
+    fn pop() {
+        let mut engine = V8Engine::new();
+        let mut context = engine.new_context().unwrap();
+
+        let array = V8Array::new(&context, 2).unwrap();
+
+        array
+            .set(0u32, &1234.0.to_js_value(context.clone()).unwrap())
+            .unwrap();
+        array
+            .set(1u32, &"Hello World!".to_js_value(context).unwrap())
+            .unwrap();
+
+        assert_eq!(array.pop().unwrap().as_string().unwrap(), "Hello World!");
+        assert_eq!(array.get(0u32).unwrap().as_number().unwrap(), 1234.0);
+        assert!(array.get(1u32).unwrap().is_undefined());
+    }
+
+    #[test]
+    fn dynamic_resize() {
+        let mut engine = V8Engine::new();
+        let mut context = engine.new_context().unwrap();
+
+        let array = V8Array::new(&context, 2).unwrap();
+
+        array
+            .set(0u32, &1234.0.to_js_value(context.clone()).unwrap())
+            .unwrap();
+        array
+            .set(1u32, &"Hello World!".to_js_value(context.clone()).unwrap())
+            .unwrap();
+        array
+            .set(2u32, &1234.0.to_js_value(context.clone()).unwrap())
+            .unwrap();
+        array
+            .set(3u32, &"Hello World!".to_js_value(context.clone()).unwrap())
+            .unwrap();
+
+        assert_eq!(array.length().unwrap(), 4);
+    }
+}