fix: PartitionedMap and ObjectWrappingMap missing a property

  `Symbol.iterator`

Author: josdejong

HISTORY.md

@@ -5,6 +5,9 @@
 
 - Feat: implement support for trailing commas in matrices (#3154, #2968).
   Thanks @dvd101x.
+- Fix: `PartitionedMap` and `ObjectWrappingMap` missing a property 
+  `Symbol.iterator`, causing problems when trying `new Map(scope)`. See
+  https://github.com/josdejong/mathjs/discussions/3156#discussioncomment-8516994
 - Docs: describe method `getAllAsMap` in the Parser docs (#3158, #3157).
   Thanks @dvd101x.
 

src/utils/map.js

@@ -12,6 +12,8 @@ import { isObject } from './is.js'
 export class ObjectWrappingMap {
   constructor (object) {
     this.wrappedObject = object
+
+    this[Symbol.iterator] = this.entries
   }
 
   keys () {
@@ -80,6 +82,8 @@ export class PartitionedMap {
     this.a = a
     this.b = b
     this.bKeys = bKeys
+
+    this[Symbol.iterator] = this.entries
   }
 
   get (key) {

test/unit-tests/utils/map.test.js

@@ -110,6 +110,10 @@ describe('maps', function () {
     const innerObject = toObject(map)
     assert.strictEqual(innerObject, obj)
 
+    // Create a new Map
+    const copy = new Map(map)
+    assert.deepStrictEqual([...copy.keys()], [...map.keys()])
+
     // clear
     map.clear()
     assert.deepStrictEqual([...map.keys()], [])
@@ -214,6 +218,16 @@ describe('maps', function () {
       assert.deepStrictEqual(it.next(), { done: true, value: undefined })
     })
 
+    it('copy', function () {
+      const { a, b, p } = createPartitionedMap(['b'])
+      p
+        .set('a', 2)
+        .set('b', 3)
+
+      const copy = new Map(p)
+      assert.deepStrictEqual([...copy.keys()], [...p.keys()])
+    })
+
     it('size', function () {
       const { p } = createPartitionedMap(['b'])
       p.set('a', 2)