Skip to content

Commit aec9026

Browse files
committed
adjust test and add todo
1 parent b1652d1 commit aec9026

File tree

1 file changed

+6
-5
lines changed

1 file changed

+6
-5
lines changed

client/modules/IDE/components/Preferences/Preferences.unit.test.jsx

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -442,17 +442,18 @@ describe('<Preferences />', () => {
442442
);
443443
});
444444
});
445+
// TODO: Consolidate "linewrap" (reference from backend/User model) and "word wrap" (currently used in UI) into consistent naming.
445446
describe('start linewrap at false', () => {
446447
it('linewrap toggle, starting at false', () => {
447448
// render the component with linewrap prop set to false
448449
subject({ linewrap: false });
449450

450451
// get ahold of the radio buttons for toggling linewrap
451452
const linewrapRadioFalse = screen.getByRole('radio', {
452-
name: /linewrap off/i
453+
name: /wordwrap off/i
453454
});
454455
const linewrapRadioTrue = screen.getByRole('radio', {
455-
name: /linewrap on/i
456+
name: /wordwrap on/i
456457
});
457458

458459
testToggle(
@@ -470,10 +471,10 @@ describe('<Preferences />', () => {
470471

471472
// get ahold of the radio buttons for toggling linewrap
472473
const linewrapRadioFalse = screen.getByRole('radio', {
473-
name: /linewrap off/i
474+
name: /wordwrap off/i
474475
});
475476
const linewrapRadioTrue = screen.getByRole('radio', {
476-
name: /linewrap on/i
477+
name: /wordwrap on/i
477478
});
478479

479480
testToggle(
@@ -511,7 +512,7 @@ describe('<Preferences />', () => {
511512
});
512513

513514
const generalElement1 = screen.getByRole('radio', {
514-
name: /linewrap on/i
515+
name: /wordwrap on/i
515516
});
516517
expect(generalElement1).toBeInTheDocument();
517518
});

0 commit comments

Comments
 (0)