add tests

Author: KSDaemon

packages/cubejs-schema-compiler/test/unit/__snapshots__/schema.test.ts.snap

@@ -1656,6 +1656,7 @@ Array [
       },
     ],
     "name": "folder1",
+    "type": "folder",
   },
   Object {
     "includes": Array [
@@ -1671,6 +1672,7 @@ Array [
       },
     ],
     "name": "folder2",
+    "type": "folder",
   },
 ]
 `;

packages/cubejs-schema-compiler/test/unit/fixtures/folders.yml

@@ -105,6 +105,17 @@ views:
         includes:
           - users_city
           - users_renamed_in_view3_gender
+  - name: test_view4
+    extends: test_view3
+    folders:
+      - name: folder3
+        includes:
+          - users_city
+          - name: inner folder 4
+            includes:
+              - renamed_orders_status
+          - name: inner folder 5
+            includes: "*"
 
   # - name: empty_view
   #   cubes:

packages/cubejs-schema-compiler/test/unit/fixtures/folders_invalid_path.yml

@@ -72,6 +72,9 @@ views:
           - age
           - renamed_gender
           - users.age
+          - name: inner folder
+            includes:
+              - users.renamed_gender
       - name: folder2
         includes: '*'
   - name: test_view2

packages/cubejs-schema-compiler/test/unit/folders.test.ts

@@ -43,6 +43,50 @@ describe('Cube Folders', () => {
     );
   });
 
+  it('a nested folders with some * and named members', async () => {
+    const testView = metaTransformer.cubes.find(
+      (it) => it.config.name === 'test_view4'
+    );
+
+    expect(testView.config.folders.length).toBe(3);
+
+    const folder1 = testView.config.folders.find(
+      (it) => it.name === 'folder1'
+    );
+    expect(folder1.members).toEqual([
+      'test_view4.users_age',
+      'test_view4.users_state',
+      'test_view4.renamed_orders_status',
+    ]);
+
+    const folder2 = testView.config.folders.find(
+      (it) => it.name === 'folder2'
+    );
+    expect(folder2.members).toEqual(
+      expect.arrayContaining(['test_view4.users_city', 'test_view4.users_renamed_in_view3_gender'])
+    );
+
+    const folder3 = testView.config.folders.find(
+      (it) => it.name === 'folder3'
+    );
+    expect(folder3.members.length).toBe(3);
+    expect(folder3.members[1]).toEqual(
+      { name: 'inner folder 4', members: ['test_view4.renamed_orders_status'] }
+    );
+    expect(folder3.members[2].name).toEqual('inner folder 5');
+    expect(folder3.members[2].members).toEqual([
+      'test_view4.renamed_orders_count',
+      'test_view4.renamed_orders_id',
+      'test_view4.renamed_orders_number',
+      'test_view4.renamed_orders_status',
+      'test_view4.users_age',
+      'test_view4.users_state',
+      'test_view4.users_gender',
+      'test_view4.users_city',
+      'test_view4.users_renamed_in_view3_gender',
+    ]);
+  });
+
   it('folders from view extending other view', async () => {
     const view2 = metaTransformer.cubes.find(
       (it) => it.config.name === 'test_view2'
@@ -93,6 +137,7 @@ describe('Cube Folders', () => {
       throw new Error('should throw earlier');
     } catch (e: any) {
       expect(e.toString()).toMatch(/Paths aren't allowed in the 'folders' but 'users.age' has been provided for test_view/);
+      expect(e.toString()).toMatch(/Paths aren't allowed in the 'folders' but 'users.renamed_gender' has been provided for test_view/);
       expect(e.toString()).toMatch(/Member 'users.age' included in folder 'folder1' not found/);
     }
   });