Skip to content

Commit 27ce136

Browse files
author
Domino987
committed
misc: Fix column test
1 parent f1dcf44 commit 27ce136

File tree

2 files changed

+32
-18
lines changed

2 files changed

+32
-18
lines changed

__tests__/pre.build.test.js

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -46,9 +46,9 @@ describe('Render Table : Pre Build', () => {
4646
const data = makeData();
4747
render(<MaterialTable data={data} columns={columns} />);
4848

49-
screen.getByRole('columnheader', { name: /first name/i });
50-
screen.getByRole('columnheader', { name: /last name/i });
51-
screen.getByRole('columnheader', { name: /age/i });
49+
screen.getAllByRole('columnheader', { name: /first name/i });
50+
screen.getAllByRole('columnheader', { name: /last name/i });
51+
screen.getAllByRole('columnheader', { name: /age/i });
5252
expect(
5353
screen.getAllByRole('button', { name: /first name/i })
5454
).toHaveLength(1);

__tests__/resizeColumn.test.js

Lines changed: 29 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -82,9 +82,9 @@ describe('Resize Column', () => {
8282
/>
8383
);
8484

85-
const thNotes = screen.getByRole('columnheader', {
85+
const thNotes = screen.getAllByRole('columnheader', {
8686
name: /Notes/i
87-
});
87+
})[0];
8888
const table = thNotes.closest('table');
8989

9090
//screen.debug(thNotes);
@@ -124,9 +124,9 @@ describe('Resize Column', () => {
124124
/>
125125
);
126126

127-
const thNotes = screen.getByRole('columnheader', {
127+
const thNotes = screen.getAllByRole('columnheader', {
128128
name: /Notes/i
129-
});
129+
})[0];
130130
const table = thNotes.closest('table');
131131

132132
expect(table).toHaveStyle({ width: '240px' });
@@ -180,9 +180,9 @@ describe('Resize Column', () => {
180180
/>
181181
);
182182

183-
const thNotes = screen.getByRole('columnheader', {
183+
const thNotes = screen.getAllByRole('columnheader', {
184184
name: /Notes/i
185-
});
185+
})[0];
186186
const table = thNotes.closest('table');
187187

188188
expect(table).toHaveStyle({ width: '100%' });
@@ -217,9 +217,9 @@ describe('Resize Column', () => {
217217
/>
218218
);
219219

220-
const thNotes = screen.getByRole('columnheader', {
220+
const thNotes = screen.getAllByRole('columnheader', {
221221
name: /Notes/i
222-
});
222+
})[0];
223223

224224
expect(thNotes.style.width).toBe(`${widthNotes}px`);
225225

@@ -244,7 +244,9 @@ describe('Resize Column', () => {
244244
// maxWidth is less than width, can't increase width
245245
// when reducing won't jump to maxWidth
246246

247-
const thNotes = screen.getByRole('columnheader', { name: /Notes/i });
247+
const thNotes = screen.getAllByRole('columnheader', {
248+
name: /Notes/i
249+
})[0];
248250

249251
expect(thNotes).toHaveStyle({ width: '300px' });
250252

@@ -289,8 +291,12 @@ describe('Resize Column', () => {
289291
/>
290292
);
291293

292-
const thNotes = screen.getByRole('columnheader', { name: /Notes/i });
293-
const thSurname = screen.getByRole('columnheader', { name: /Surname/i });
294+
const thNotes = screen.getAllByRole('columnheader', {
295+
name: /Notes/i
296+
})[0];
297+
const thSurname = screen.getAllByRole('columnheader', {
298+
name: /Surname/i
299+
})[0];
294300
const table = thNotes.closest('table');
295301

296302
expect(table).toHaveStyle({ width: '100%' });
@@ -325,8 +331,12 @@ describe('Resize Column', () => {
325331
/>
326332
);
327333

328-
const thNotes = screen.getByRole('columnheader', { name: /Notes/i });
329-
const thSurname = screen.getByRole('columnheader', { name: /Surname/i });
334+
const thNotes = screen.getAllByRole('columnheader', {
335+
name: /Notes/i
336+
})[0];
337+
const thSurname = screen.getAllByRole('columnheader', {
338+
name: /Surname/i
339+
})[0];
330340
const table = thNotes.closest('table');
331341

332342
expect(table).toHaveStyle({ width: '100%' });
@@ -355,8 +365,12 @@ describe('Resize Column', () => {
355365
/>
356366
);
357367

358-
const thName = screen.getByRole('columnheader', { name: /^Name$/i });
359-
const thSurname = screen.getByRole('columnheader', { name: /Surname/i });
368+
const thName = screen.getAllByRole('columnheader', {
369+
name: /^Name$/i
370+
})[0];
371+
const thSurname = screen.getAllByRole('columnheader', {
372+
name: /Surname/i
373+
})[0];
360374

361375
resize(thName, [-10]);
362376
// Only move 5 because of maxWidth on surname

0 commit comments

Comments
 (0)