Skip to content

Commit ff0ad17

Browse files
alunyovfacebook-github-bot
authored andcommittedJan 13, 2021
Convert ReactRelayFragmentContainer-WithFragmentOwnership-test.js to use the new compiler
Summary: We're working on deprecating JS compiler. One of the steps, is to remove it from our unit-tests. This is the test from #3199 Steps: 1) Remove `generateAndCompile` 2) Wrap graphql documents with `graphql` tag + add unique prefix 3) Run new compiler: `scripts/compile-tests.sh` Reviewed By: tyao1 Differential Revision: D25875625 fbshipit-source-id: c47fe1838b689aa5fe904f28b6d45d49852d97ad
1 parent bede32d commit ff0ad17

5 files changed

+498
-26
lines changed
 

‎packages/react-relay/__tests__/ReactRelayFragmentContainer-WithFragmentOwnership-test.js

+45-26
Original file line numberDiff line numberDiff line change
@@ -18,11 +18,9 @@ const ReactTestRenderer = require('react-test-renderer');
1818
const {
1919
createReaderSelector,
2020
createOperationDescriptor,
21+
graphql,
2122
} = require('relay-runtime');
22-
const {
23-
createMockEnvironment,
24-
generateAndCompile,
25-
} = require('relay-test-utils-internal');
23+
const {createMockEnvironment} = require('relay-test-utils-internal');
2624

2725
describe('ReactRelayFragmentContainer with fragment ownerhsip', () => {
2826
let TestComponent;
@@ -81,34 +79,39 @@ describe('ReactRelayFragmentContainer with fragment ownerhsip', () => {
8179
jest.resetModules();
8280

8381
environment = createMockEnvironment();
84-
({UserFragment, UserQuery, UserQueryWithCond} = generateAndCompile(`
85-
query UserQuery($id: ID!) {
82+
UserQuery = graphql`
83+
query ReactRelayFragmentContainerWithFragmentOwnershipTestUserQuery(
84+
$id: ID!
85+
) {
8686
node(id: $id) {
87-
...UserFragment
87+
...ReactRelayFragmentContainerWithFragmentOwnershipTestUserFragment
8888
}
8989
}
90-
91-
query UserQueryWithCond(
90+
`;
91+
UserQueryWithCond = graphql`
92+
query ReactRelayFragmentContainerWithFragmentOwnershipTestWithCondUserQuery(
9293
$id: ID!
9394
$condGlobal: Boolean!
9495
) {
9596
node(id: $id) {
96-
...UserFragment @arguments(cond: $condGlobal)
97+
...ReactRelayFragmentContainerWithFragmentOwnershipTestUserFragment
98+
@arguments(cond: $condGlobal)
9799
}
98100
}
99-
100-
fragment UserFragment on User @argumentDefinitions(
101-
cond: {type: "Boolean!", defaultValue: true}
102-
) {
101+
`;
102+
UserFragment = graphql`
103+
fragment ReactRelayFragmentContainerWithFragmentOwnershipTestUserFragment on User
104+
@argumentDefinitions(cond: {type: "Boolean!", defaultValue: true}) {
103105
id
104106
name @include(if: $cond)
105-
...NestedUserFragment
107+
...ReactRelayFragmentContainerWithFragmentOwnershipTestNestedUserFragment
106108
}
107-
108-
fragment NestedUserFragment on User {
109+
`;
110+
graphql`
111+
fragment ReactRelayFragmentContainerWithFragmentOwnershipTestNestedUserFragment on User {
109112
username
110113
}
111-
`));
114+
`;
112115

113116
render = jest.fn(() => <div />);
114117
spec = {
@@ -173,7 +176,9 @@ describe('ReactRelayFragmentContainer with fragment ownerhsip', () => {
173176
id: '4',
174177
name: 'Zuck',
175178
__id: '4',
176-
__fragments: {NestedUserFragment: {}},
179+
__fragments: {
180+
ReactRelayFragmentContainerWithFragmentOwnershipTestNestedUserFragment: {},
181+
},
177182
__fragmentOwner: ownerUser1.request,
178183
},
179184
});
@@ -184,7 +189,9 @@ describe('ReactRelayFragmentContainer with fragment ownerhsip', () => {
184189
id: '4',
185190
name: 'Zuck',
186191
__id: '4',
187-
__fragments: {NestedUserFragment: {}},
192+
__fragments: {
193+
ReactRelayFragmentContainerWithFragmentOwnershipTestNestedUserFragment: {},
194+
},
188195
__fragmentOwner: ownerUser1.request,
189196
},
190197
missingRequiredFields: null,
@@ -222,7 +229,9 @@ describe('ReactRelayFragmentContainer with fragment ownerhsip', () => {
222229
// !== 'Zuck'
223230
name: 'Mark',
224231
__id: '4',
225-
__fragments: {NestedUserFragment: {}},
232+
__fragments: {
233+
ReactRelayFragmentContainerWithFragmentOwnershipTestNestedUserFragment: {},
234+
},
226235
__fragmentOwner: ownerUser1.request,
227236
},
228237
seenRecords: {},
@@ -242,7 +251,9 @@ describe('ReactRelayFragmentContainer with fragment ownerhsip', () => {
242251
id: '4',
243252
name: 'Mark',
244253
__id: '4',
245-
__fragments: {NestedUserFragment: {}},
254+
__fragments: {
255+
ReactRelayFragmentContainerWithFragmentOwnershipTestNestedUserFragment: {},
256+
},
246257
__fragmentOwner: ownerUser1.request,
247258
},
248259
});
@@ -275,7 +286,9 @@ describe('ReactRelayFragmentContainer with fragment ownerhsip', () => {
275286
id: '842472',
276287
name: 'Joe',
277288
__id: '842472',
278-
__fragments: {NestedUserFragment: {}},
289+
__fragments: {
290+
ReactRelayFragmentContainerWithFragmentOwnershipTestNestedUserFragment: {},
291+
},
279292
__fragmentOwner: ownerUser2.request,
280293
},
281294
});
@@ -287,7 +300,9 @@ describe('ReactRelayFragmentContainer with fragment ownerhsip', () => {
287300
id: '842472',
288301
name: 'Joe',
289302
__id: '842472',
290-
__fragments: {NestedUserFragment: {}},
303+
__fragments: {
304+
ReactRelayFragmentContainerWithFragmentOwnershipTestNestedUserFragment: {},
305+
},
291306
__fragmentOwner: ownerUser2.request,
292307
},
293308
missingRequiredFields: null,
@@ -332,7 +347,9 @@ describe('ReactRelayFragmentContainer with fragment ownerhsip', () => {
332347
id: '4',
333348
// Name is excluded since value of cond is now false
334349
__id: '4',
335-
__fragments: {NestedUserFragment: {}},
350+
__fragments: {
351+
ReactRelayFragmentContainerWithFragmentOwnershipTestNestedUserFragment: {},
352+
},
336353
__fragmentOwner: ownerUser1WithCondVar.request,
337354
},
338355
});
@@ -343,7 +360,9 @@ describe('ReactRelayFragmentContainer with fragment ownerhsip', () => {
343360
id: '4',
344361
// Name is excluded since value of cond is now false
345362
__id: '4',
346-
__fragments: {NestedUserFragment: {}},
363+
__fragments: {
364+
ReactRelayFragmentContainerWithFragmentOwnershipTestNestedUserFragment: {},
365+
},
347366
__fragmentOwner: ownerUser1WithCondVar.request,
348367
},
349368
missingRequiredFields: null,

‎packages/react-relay/__tests__/__generated__/ReactRelayFragmentContainerWithFragmentOwnershipTestNestedUserFragment.graphql.js

+56
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

‎packages/react-relay/__tests__/__generated__/ReactRelayFragmentContainerWithFragmentOwnershipTestUserFragment.graphql.js

+84
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

‎packages/react-relay/__tests__/__generated__/ReactRelayFragmentContainerWithFragmentOwnershipTestUserQuery.graphql.js

+145
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

‎packages/react-relay/__tests__/__generated__/ReactRelayFragmentContainerWithFragmentOwnershipTestWithCondUserQuery.graphql.js

+168
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)
Please sign in to comment.