Skip to content

Commit 4641b47

Browse files
committed
fix(test): update COLUMN_DEFS-length assertion for on_demand_monthly
#322 (now merged) added the on_demand_monthly column to the recs table, bringing COLUMN_DEFS.length to 13. The visibility-feature test was written when the count was 12 — update it post-rebase so the suite stays green.
1 parent 11599f6 commit 4641b47

1 file changed

Lines changed: 3 additions & 2 deletions

File tree

frontend/src/__tests__/recommendations.test.ts

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4182,7 +4182,7 @@ describe('Column visibility (issue #318)', () => {
41824182
// --- TOGGLEABLE_COLUMNS and COLUMN_DEFS ---
41834183

41844184
describe('COLUMN_DEFS and TOGGLEABLE_COLUMNS', () => {
4185-
test('COLUMN_DEFS contains all 12 column ids', () => {
4185+
test('COLUMN_DEFS contains all 13 column ids', () => {
41864186
const keys = COLUMN_DEFS.map((c) => c.key);
41874187
expect(keys).toContain('provider');
41884188
expect(keys).toContain('account');
@@ -4195,8 +4195,9 @@ describe('Column visibility (issue #318)', () => {
41954195
expect(keys).toContain('savings');
41964196
expect(keys).toContain('upfront_cost');
41974197
expect(keys).toContain('monthly_cost');
4198+
expect(keys).toContain('on_demand_monthly');
41984199
expect(keys).toContain('effective_savings_pct');
4199-
expect(COLUMN_DEFS.length).toBe(12);
4200+
expect(COLUMN_DEFS.length).toBe(13);
42004201
});
42014202

42024203
test('TOGGLEABLE_COLUMNS excludes fixed identity columns', () => {

0 commit comments

Comments
 (0)