Skip to content
This repository was archived by the owner on Jul 30, 2020. It is now read-only.

Commit 019625d

Browse files
author
Brandon Carroll
committed
refactor queries
1 parent 5efb550 commit 019625d

26 files changed

+259
-559
lines changed

examples/__tests__/input-event.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@ class CostInput extends React.Component {
2929

3030
const setup = () => {
3131
const utils = render(<CostInput />);
32-
const input = utils.getByA11yLabel('cost-input');
32+
const input = utils.getByLabelText('cost-input');
3333
return {
3434
input,
3535
...utils,

src/__tests__/get-by-errors.js

Lines changed: 0 additions & 187 deletions
This file was deleted.

src/__tests__/misc.js

Lines changed: 0 additions & 23 deletions
This file was deleted.

src/lib/__tests__/events.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -121,6 +121,6 @@ test('calling an event sets nativeEvent properly', () => {
121121
const event = { nativeEvent: { value: 'testing' } };
122122
const onChange = jest.fn(({ nativeEvent }) => expect(nativeEvent).toEqual({ value: 'testing' }));
123123

124-
const { getByValue } = render(<TextInput value="test" onChange={onChange} />);
125-
fireEvent.change(getByValue('test'), event);
124+
const { getByDisplayValue } = render(<TextInput value="test" onChange={onChange} />);
125+
fireEvent.change(getByDisplayValue('test'), event);
126126
});

src/lib/__tests__/get-by-errors.js

Lines changed: 8 additions & 44 deletions
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@ cases(
2020
</View>
2121
),
2222
},
23-
getByA11yLabel: {
23+
getByLabelText: {
2424
query: /his/,
2525
tree: (
2626
<View>
@@ -29,7 +29,7 @@ cases(
2929
</View>
3030
),
3131
},
32-
getByA11yRole: {
32+
getByRole: {
3333
query: 'button',
3434
tree: (
3535
<View>
@@ -38,25 +38,7 @@ cases(
3838
</View>
3939
),
4040
},
41-
getByA11yStates: {
42-
query: ['selected'],
43-
tree: (
44-
<View>
45-
<View accessibilityStates={['selected']} />
46-
<View accessibilityStates={['selected']} />
47-
</View>
48-
),
49-
},
50-
getByA11yTraits: {
51-
query: ['button'],
52-
tree: (
53-
<View>
54-
<View accessibilityTraits={['button']} />
55-
<View accessibilityTraits={['button']} />
56-
</View>
57-
),
58-
},
59-
getByPlaceholder: {
41+
getByPlaceholderText: {
6042
query: /his/,
6143
tree: (
6244
<View>
@@ -92,7 +74,7 @@ cases(
9274
</View>
9375
),
9476
},
95-
getByValue: {
77+
getByDisplayValue: {
9678
query: /his/,
9779
tree: (
9880
<View>
@@ -120,7 +102,7 @@ cases(
120102
</View>
121103
),
122104
},
123-
queryByA11yLabel: {
105+
queryByLabelText: {
124106
query: /his/,
125107
tree: (
126108
<View>
@@ -129,7 +111,7 @@ cases(
129111
</View>
130112
),
131113
},
132-
queryByA11yRole: {
114+
queryByRole: {
133115
query: 'button',
134116
tree: (
135117
<View>
@@ -138,25 +120,7 @@ cases(
138120
</View>
139121
),
140122
},
141-
queryByA11yStates: {
142-
query: ['selected'],
143-
tree: (
144-
<View>
145-
<View accessibilityStates={['selected']} />
146-
<View accessibilityStates={['selected']} />
147-
</View>
148-
),
149-
},
150-
queryByA11yTraits: {
151-
query: ['button'],
152-
tree: (
153-
<View>
154-
<View accessibilityTraits={['button']} />
155-
<View accessibilityTraits={['button']} />
156-
</View>
157-
),
158-
},
159-
queryByPlaceholder: {
123+
queryByPlaceholderText: {
160124
query: /his/,
161125
tree: (
162126
<View>
@@ -192,7 +156,7 @@ cases(
192156
</View>
193157
),
194158
},
195-
queryByValue: {
159+
queryByDisplayValue: {
196160
query: /his/,
197161
tree: (
198162
<View>

src/lib/__tests__/misc.js

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -30,8 +30,8 @@ test('picker', () => {
3030
</Picker>
3131
);
3232
}
33-
const { findByValue, getByValue } = render(<Wrapper />);
33+
const { findByDisplayValue, getByDisplayValue } = render(<Wrapper />);
3434

35-
fireEvent.valueChange(getByValue('js'), 'java');
36-
expect(() => findByValue('js')).not.toThrow();
35+
fireEvent.valueChange(getByDisplayValue('js'), 'java');
36+
expect(() => findByDisplayValue('js')).not.toThrow();
3737
});

0 commit comments

Comments
 (0)