fix actually incorrect tests

Author: KSDaemon

packages/cubejs-schema-compiler/test/integration/postgres/sql-generation.test.ts

@@ -5198,7 +5198,7 @@ cubes:
         sql: amount
         type: sum
 
-# Join loop for testing transitive joins
+# Model for testing multiple joins to the same cube via transitive joins
   - name: alpha_facts
     sql: |
       (
@@ -5345,34 +5345,32 @@ cubes:
       });
     }
 
-    if (!getEnv('nativeSqlPlanner')) {
-      it('querying cube with transitive joins with a few joins to same cube', async () => {
-        await compiler.compile();
+    it('querying cube with transitive joins with a few joins to same cube', async () => {
+      // TODO: This is not supported atm, but it's a good case, so keeeping this test
+      // for the future implementation
+
+      // await compiler.compile();
+      //
+      // const query = new PostgresQuery({ joinGraph, cubeEvaluator, compiler }, {
+      //   measures: [],
+      //   dimensions: [
+      //     'alpha_facts.reporting_date',
+      //     'delta_bridge.b_name',
+      //     'alpha_facts.channel'
+      //   ],
+      //   order: [{
+      //     id: 'alpha_facts.reporting_date'
+      //   }],
+      //   timezone: 'America/Los_Angeles'
+      // });
+      //
+      // const res = await dbRunner.testQuery(query.buildSqlAndParams());
+      // console.log(JSON.stringify(res));
+      //
+      // expect(res).toEqual([
+      //   // Fill
+      // ]);
+    });
 
-        try {
-          const query = new PostgresQuery({ joinGraph, cubeEvaluator, compiler }, {
-            measures: [],
-            dimensions: [
-              'alpha_facts.reporting_date',
-              'delta_bridge.b_name',
-              'alpha_facts.channel'
-            ],
-            order: [{
-              id: 'alpha_facts.reporting_date'
-            }],
-            timezone: 'America/Los_Angeles'
-          });
-
-          await dbRunner.testQuery(query.buildSqlAndParams());
-          throw new Error('Should have thrown an error');
-        } catch (err: any) {
-          expect(err.message).toContain('Can not construct joins for the query, potential loop detected');
-        }
-      });
-    } else {
-      it.skip('FIXME(tesseract): querying cube dimension that require transitive joins', async () => {
-        // FIXME should be implemented in Tesseract
-      });
-    }
   });
 });