add tests

Author: KSDaemon

packages/cubejs-schema-compiler/test/unit/fixtures/invalid_cubes.yaml

@@ -0,0 +1,45 @@
+cubes:
+  - name: base_cube
+    sql_table: foo
+
+    dimensions:
+      - name: status
+        sql: status
+        type: string
+
+      - name: parent_dim_no_type
+        sql: barbus
+
+      - name: parent_dim_bad_type
+        type: bad_type
+        sql: badus
+
+      - name: parent_dim_no_sql
+        type: string
+
+    measures:
+      - name: count
+        type: count
+
+      - name: parent_meas_no_type
+
+      - name: parent_meas_bad_type
+        type: bad_type
+
+  - name: child_cube
+    extends: base_cube
+
+    dimensions:
+      - name: bar
+        sql: bar
+        type: string
+
+      - name: child_dim_no_type
+        sql: barbus
+
+      - name: child_dim_bad_type
+        type: bad_type
+        sql: shmarbus
+
+      - name: child_dim_no_sql
+        type: number

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

@@ -1009,5 +1009,32 @@ describe('Schema Testing', () => {
       expect(cubeB.sqlTable).toBeTruthy();
       expect(cubeB.sql).toBeFalsy();
     });
+
+    it('throws error for member without type in cubeB extending cubeA', async () => {
+      const cubes = fs.readFileSync(
+        path.join(process.cwd(), '/test/unit/fixtures/invalid_cubes.yaml'),
+        'utf8'
+      );
+      const { compiler } = prepareCompiler([
+        {
+          content: cubes,
+          fileName: 'invalid_cubes.yaml',
+        },
+      ]);
+
+      try {
+        await compiler.compile();
+        throw new Error('should throw earlier');
+      } catch (e: any) {
+        expect(e.toString()).toMatch(/"measures\.parent_meas_no_type\.sql" is required/);
+        expect(e.toString()).toMatch(/"measures\.parent_meas_no_type\.type" is required/);
+        expect(e.toString()).toMatch(/"measures\.parent_meas_bad_type\.type" must be one of/);
+        expect(e.toString()).toMatch(/"dimensions\.parent_dim_no_type" does not match any of the allowed types/);
+        expect(e.toString()).toMatch(/"dimensions\.parent_dim_no_sql" does not match any of the allowed types/);
+        expect(e.toString()).toMatch(/"dimensions\.child_dim_no_type" does not match any of the allowed types/);
+        expect(e.toString()).toMatch(/"dimensions\.child_dim_bad_type" does not match any of the allowed types/);
+        expect(e.toString()).toMatch(/"dimensions\.child_dim_no_sql" does not match any of the allowed types/);
+      }
+    });
   });
 });