Skip to content

Commit 6f9173d

Browse files
author
Paweł Kata
committed
fix(appbar): satisfy ci's reqs
1 parent 2447364 commit 6f9173d

2 files changed

Lines changed: 0 additions & 22 deletions

File tree

src/components/Appbar/types.ts

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -40,20 +40,15 @@ export type AppbarHeadlineTextProps = AppbarTextProps & {
4040

4141
export type AppbarHeadlinePressableProps = Pick<
4242
ViewProps,
43-
| 'accessibilityActions'
4443
| 'accessibilityHint'
4544
| 'accessibilityLabel'
46-
| 'accessibilityLabelledBy'
47-
| 'accessibilityLanguage'
4845
| 'accessibilityState'
49-
| 'accessibilityValue'
5046
| 'aria-busy'
5147
| 'aria-disabled'
5248
| 'aria-expanded'
5349
| 'aria-label'
5450
| 'aria-labelledby'
5551
| 'aria-selected'
56-
| 'onAccessibilityAction'
5752
> & {
5853
/** Whether the headline interaction is disabled. */
5954
disabled?: boolean;

src/components/__tests__/Appbar/Appbar.test.tsx

Lines changed: 0 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -631,19 +631,15 @@ describe('Appbar accessibility', () => {
631631
});
632632

633633
it('forwards custom headline button accessibility props', async () => {
634-
const onAccessibilityAction = jest.fn();
635634
await render(
636635
<Appbar
637636
variant="small"
638637
headline="Inbox"
639638
onHeadlinePress={() => {}}
640639
headlinePressableProps={{
641-
accessibilityActions: [{ name: 'activate', label: 'Open inbox' }],
642640
accessibilityHint: 'Opens the inbox menu',
643641
accessibilityLabel: 'Inbox options',
644642
accessibilityState: { busy: true, expanded: true },
645-
accessibilityValue: { text: 'Unread messages available' },
646-
onAccessibilityAction,
647643
}}
648644
testID={testIDPrefix}
649645
/>
@@ -655,20 +651,7 @@ describe('Appbar accessibility', () => {
655651
expanded: true,
656652
});
657653

658-
expect(titleButton).toHaveProp(
659-
'accessibilityActions',
660-
expect.arrayContaining([
661-
expect.objectContaining({ name: 'activate', label: 'Open inbox' }),
662-
])
663-
);
664654
expect(titleButton).toHaveProp('accessibilityHint', 'Opens the inbox menu');
665-
expect(titleButton).toHaveAccessibilityValue({
666-
text: 'Unread messages available',
667-
});
668-
expect(titleButton).toHaveProp(
669-
'onAccessibilityAction',
670-
onAccessibilityAction
671-
);
672655
});
673656

674657
it('uses default and custom labels for back buttons', async () => {

0 commit comments

Comments
 (0)