Skip to content

Commit 48e0c64

Browse files
committed
fix old tests with new required fields
1 parent f995df5 commit 48e0c64

File tree

2 files changed

+14
-6
lines changed

2 files changed

+14
-6
lines changed

packages/cubejs-schema-compiler/test/unit/pre-agg-by-filter-match.test.ts

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -59,9 +59,13 @@ describe('Pre Aggregation by filter match tests', () => {
5959
granularity: testPreAgg.granularity,
6060
}],
6161
rollups: [],
62-
fullNameDimensions: [],
63-
fullNameMeasures: [],
64-
fullNameTimeDimensions: [],
62+
fullNameDimensions: testPreAgg.segments ? testPreAgg.dimensions.concat(testPreAgg.segments) : testPreAgg.dimensions,
63+
fullNameMeasures: testPreAgg.measures,
64+
fullNameTimeDimensions: [{
65+
dimension: testPreAgg.timeDimension,
66+
granularity: testPreAgg.granularity,
67+
}],
68+
rollupsReferences: [],
6569
};
6670

6771
await compiler.compile();

packages/cubejs-schema-compiler/test/unit/pre-agg-time-dim-match.test.ts

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -69,9 +69,13 @@ describe('Pre Aggregation by filter match tests', () => {
6969
granularity: testPreAgg.granularity,
7070
}],
7171
rollups: [],
72-
fullNameDimensions: [],
73-
fullNameMeasures: [],
74-
fullNameTimeDimensions: [],
72+
fullNameDimensions: testPreAgg.dimensions,
73+
fullNameMeasures: testPreAgg.measures,
74+
fullNameTimeDimensions: [{
75+
dimension: testPreAgg.timeDimension,
76+
granularity: testPreAgg.granularity,
77+
}],
78+
rollupsReferences: [],
7579
};
7680

7781
await compiler.compile();

0 commit comments

Comments
 (0)