fix: enumerate plain objects in where/where_exp, #785 (#788)

test/e2e/issues.spec.ts

@@ -531,4 +531,31 @@ describe('Issues', function () {
     const tpl = `{% for i in (1..1000000000) %} {{'a'}} {% endfor %}`
     expect(() => engine.parseAndRenderSync(tpl)).toThrow('memory alloc limit exceeded, line:1, col:1')
   })
+  it('group_by_exp fails with object as input #785', () => {
+    const site = {
+      tags: {
+        CPP: [ 'page0' ],
+        PHP: [ 'page0', 'page2' ],
+        JavaScript: [ 'page1', 'page2', 'page3' ],
+        CSharp: [ 'page2', 'page4' ]
+      }
+    }
+    const tpl = `
+      {%- assign tags_by_size = site.tags | group_by_exp: 'tag', 'tag[1].size' | sort: 'name' | reverse -%}
+      {%- for tags_with_size in tags_by_size -%}
+        {%- for tag in tags_with_size.items -%}
+          {%- assign tag_name = tag[0] %}
+          {{ tag_name }} <sup>{{ tags_with_size.name }}</sup> Posts:
+          {%- for post in tag[1] -%}{{post}},{%- endfor -%}
+        {%- endfor -%}
+      {%- endfor -%}
+    `
+    const engine = new Liquid()
+    const html = engine.parseAndRenderSync(tpl, { site })
+    expect(html).toEqual(`
+          JavaScript <sup>3</sup> Posts:page1,page2,page3,
+          PHP <sup>2</sup> Posts:page0,page2,
+          CSharp <sup>2</sup> Posts:page2,page4,
+          CPP <sup>1</sup> Posts:page0,`)
+  })
 })

test/integration/filters/array.spec.ts

@@ -554,6 +554,12 @@ describe('filters/array', function () {
       { graduation_year: 2014, name: 'John' },
       { graduation_year: 2009, name: 'Jack' }
     ]
+    const postsByTags = {
+      CPP: [ 'page0' ],
+      PHP: [ 'page0', 'page2' ],
+      JavaScript: [ 'page1', 'page2', 'page3' ],
+      CSharp: [ 'page2', 'page4' ]
+    }
     it('should support group by expression', function () {
       const expected = [{
         name: '201',
@@ -572,6 +578,29 @@ describe('filters/array', function () {
         { members },
         JSON.stringify(expected))
     })
+    it('should group key/values in plain object', function () {
+      const expected = [{
+        name: 3,
+        items: [
+          ['JavaScript', ['page1', 'page2', 'page3']]
+        ]
+      }, {
+        name: 2,
+        items: [
+          ['PHP', ['page0', 'page2']],
+          ['CSharp', ['page2', 'page4']]
+        ]
+      }, {
+        name: 1,
+        items: [
+          ['CPP', ['page0']]
+        ]
+      }]
+      return test(
+        `{{ postsByTags | group_by_exp: "tag", "tag[1].size" | sort: 'name' | reverse | json}}`,
+        { postsByTags },
+        JSON.stringify(expected))
+    })
   })
   describe('find', function () {
     const members = [