Fix array.pop()

src/model/array.rs

@@ -84,9 +84,8 @@ impl Array {
     }
 
     /// Remove the last value in the array.
-    pub fn pop(&mut self) -> StrResult<()> {
+    pub fn pop(&mut self) -> StrResult<Value> {
-        Arc::make_mut(&mut self.0).pop().ok_or_else(array_is_empty)?;
+        Arc::make_mut(&mut self.0).pop().ok_or_else(array_is_empty)
-        Ok(())
     }
 
     /// Insert a value at the specified index.

src/model/methods.rs

@@ -144,7 +144,7 @@ pub fn call_mut(
     match value {
         Value::Array(array) => match method {
             "push" => array.push(args.expect("value")?),
-            "pop" => array.pop().at(span)?,
+            "pop" => output = array.pop().at(span)?,
             "insert" => {
                 array.insert(args.expect("index")?, args.expect("value")?).at(span)?
             }

tests/typ/compiler/array.typ

@@ -100,7 +100,7 @@
 // Test the `push` and `pop` methods.
 #{
   let tasks = (a: (1, 2, 3), b: (4, 5, 6))
-  tasks.at("a").pop()
+  test(tasks.at("a").pop(), 3)
   tasks.b.push(7)
   test(tasks.a, (1, 2))
   test(tasks.at("b"), (4, 5, 6, 7))