Skip to content

Commit 4768209

Browse files
committed
tweaks
1 parent 68e7201 commit 4768209

File tree

1 file changed

+27
-19
lines changed

1 file changed

+27
-19
lines changed

src/helpers/__tests__/accessiblity.test.tsx

Lines changed: 27 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -413,62 +413,70 @@ describe('computeAriaDisabled', () => {
413413
test('supports basic usage', () => {
414414
render(
415415
<View>
416+
<View testID="default" />
416417
<View testID="disabled" aria-disabled />
417-
<View testID="not-disabled" />
418+
<View testID="disabled-false" aria-disabled={false} />
418419
<View testID="disabled-by-state" accessibilityState={{ disabled: true }} />
419-
<View testID="not-disabled-by-state" accessibilityState={{ disabled: false }} />
420+
<View testID="disabled-false-by-state" accessibilityState={{ disabled: false }} />
420421
</View>,
421422
);
422423

424+
expect(computeAriaDisabled(screen.getByTestId('default'))).toBe(false);
423425
expect(computeAriaDisabled(screen.getByTestId('disabled'))).toBe(true);
424-
expect(computeAriaDisabled(screen.getByTestId('not-disabled'))).toBe(false);
426+
expect(computeAriaDisabled(screen.getByTestId('disabled-false'))).toBe(false);
425427
expect(computeAriaDisabled(screen.getByTestId('disabled-by-state'))).toBe(true);
426-
expect(computeAriaDisabled(screen.getByTestId('not-disabled-by-state'))).toBe(false);
428+
expect(computeAriaDisabled(screen.getByTestId('disabled-false-by-state'))).toBe(false);
427429
});
428430

429431
test('supports TextInput', () => {
430432
render(
431433
<View>
432-
<TextInput testID="disabled" editable={false} />
433-
<TextInput testID="not-disabled" editable />
434+
<TextInput testID="default" />
435+
<TextInput testID="editable" editable />
436+
<TextInput testID="editable-false" editable={false} />
434437
</View>,
435438
);
436439

437-
expect(computeAriaDisabled(screen.getByTestId('disabled'))).toBe(true);
438-
expect(computeAriaDisabled(screen.getByTestId('not-disabled'))).toBe(false);
440+
expect(computeAriaDisabled(screen.getByTestId('default'))).toBe(false);
441+
expect(computeAriaDisabled(screen.getByTestId('editable'))).toBe(false);
442+
expect(computeAriaDisabled(screen.getByTestId('editable-false'))).toBe(true);
439443
});
440444

441445
test('supports Button', () => {
442446
render(
443447
<View>
444-
<Pressable testID="disabled" disabled>
448+
<Pressable testID="default" role="button">
449+
<Text>Default Button</Text>
450+
</Pressable>
451+
<Pressable testID="disabled" role="button" disabled >
445452
<Text>Disabled Button</Text>
446453
</Pressable>
447-
<Pressable testID="not-disabled">
448-
<Text>Enabled Button</Text>
454+
<Pressable testID="disabled-false" role="button" disabled={false}>
455+
<Text>Disabled False Button</Text>
449456
</Pressable>
450457
</View>,
451458
);
452459

460+
expect(computeAriaDisabled(screen.getByTestId('default'))).toBe(false);
453461
expect(computeAriaDisabled(screen.getByTestId('disabled'))).toBe(true);
454-
expect(computeAriaDisabled(screen.getByTestId('not-disabled'))).toBe(false);
462+
expect(computeAriaDisabled(screen.getByTestId('disabled-false'))).toBe(false);
455463
});
456464

457465
test('supports Text', () => {
458466
render(
459467
<View>
460-
<Text testID="disabled" disabled>
468+
<Text>Default Text</Text>
469+
<Text disabled>
461470
Disabled Text
462471
</Text>
463-
<Text testID="aria-disabled" aria-disabled>
464-
Disabled Text
472+
<Text aria-disabled>
473+
ARIA Disabled Text
465474
</Text>
466-
<Text testID="not-disabled">Enabled Text</Text>
467475
</View>,
468476
);
469477

470-
expect(computeAriaDisabled(screen.getByTestId('disabled'))).toBe(true);
471-
expect(computeAriaDisabled(screen.getByTestId('aria-disabled'))).toBe(true);
472-
expect(computeAriaDisabled(screen.getByTestId('not-disabled'))).toBe(false);
478+
expect(computeAriaDisabled(screen.getByText('Default Text'))).toBe(false);
479+
expect(computeAriaDisabled(screen.getByText('Disabled Text'))).toBe(true);
480+
expect(computeAriaDisabled(screen.getByText('ARIA Disabled Text'))).toBe(true);
473481
});
474482
});

0 commit comments

Comments
 (0)