Merge pull request #1 from hellerve/codex/update-map.apply-to-join-arguments-as-strings

Fix Map.apply arg list message and add tests

Author: hellerve

mae.py

@@ -93,7 +93,7 @@ def evaluate(self, m):
     def apply(self, args, m):
         if len(args) != 1:
             raise RunError(
-                f"map can only applied to one argument (got ({', '.join(args)}))!"
+                f"map can only applied to one argument (got ({', '.join(str(a) for a in args)}))!"
             )
 
         return self.m.get(m.evaluate(args[0]), Map({}))

tests/__init__.py

@@ -0,0 +1,18 @@
+import unittest
+from mae import Mae, Map, Var, RunError
+
+class MapApplyTests(unittest.TestCase):
+    def test_multiple_arguments_raises_runerror(self):
+        env = Mae()
+        m = Map({})
+        args = [Var('a'), Var('b')]
+        with self.assertRaises(RunError) as cm:
+            m.apply(args, env)
+        msg = str(cm.exception)
+        self.assertIn('map can only applied to one argument', msg)
+        # ensure all arguments are listed in the message
+        self.assertIn('a', msg)
+        self.assertIn('b', msg)
+
+if __name__ == '__main__':
+    unittest.main()