Skip to content

Commit 20051f6

Browse files
alunyovfacebook-github-bot
authored andcommittedJan 13, 2021
Convert ReactRelayRefetchContainer-WithFragmentOwnership-test 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: - Remove generateAndCompile - Wrap graphql documents with graphql tag + add unique prefix - Run new compiler: scripts/compile-tests.sh Reviewed By: tyao1 Differential Revision: D25875336 fbshipit-source-id: 0be387d9881a615cca47bd6a05a0249b10334d49
1 parent 2383197 commit 20051f6

4 files changed

+409
-22
lines changed
 

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

+36-22
Original file line numberDiff line numberDiff line change
@@ -24,11 +24,9 @@ const {
2424
createReaderSelector,
2525
createRequestDescriptor,
2626
ROOT_ID,
27+
graphql,
2728
} = require('relay-runtime');
28-
const {
29-
createMockEnvironment,
30-
generateAndCompile,
31-
} = require('relay-test-utils-internal');
29+
const {createMockEnvironment} = require('relay-test-utils-internal');
3230

3331
describe('ReactRelayRefetchContainer with fragment ownerhsip', () => {
3432
let TestChildComponent;
@@ -106,35 +104,35 @@ describe('ReactRelayRefetchContainer with fragment ownerhsip', () => {
106104
jest.resetModules();
107105

108106
environment = createMockEnvironment();
109-
({UserFragment, UserFriendFragment, UserQuery} = generateAndCompile(`
110-
query UserQuery(
107+
UserQuery = graphql`
108+
query ReactRelayRefetchContainerWithFragmentOwnershipTestUserQuery(
111109
$id: ID!
112110
$scale: Int!
113111
) {
114112
node(id: $id) {
115-
...UserFragment
113+
...ReactRelayRefetchContainerWithFragmentOwnershipTestUserFragment
116114
}
117115
}
118-
119-
fragment UserFragment on User @argumentDefinitions(
120-
cond: {type: "Boolean!", defaultValue: true}
121-
) {
116+
`;
117+
UserFragment = graphql`
118+
fragment ReactRelayRefetchContainerWithFragmentOwnershipTestUserFragment on User
119+
@argumentDefinitions(cond: {type: "Boolean!", defaultValue: true}) {
122120
id
123121
name @include(if: $cond)
124122
profile_picture(scale: $scale) {
125123
uri
126124
}
127-
...UserFriendFragment @arguments(cond: $cond)
125+
...ReactRelayRefetchContainerWithFragmentOwnershipTestUserFriendFragment
126+
@arguments(cond: $cond)
128127
}
129-
130-
fragment UserFriendFragment on User @argumentDefinitions(
131-
cond: {type: "Boolean!", defaultValue: true}
132-
) {
128+
`;
129+
UserFriendFragment = graphql`
130+
fragment ReactRelayRefetchContainerWithFragmentOwnershipTestUserFriendFragment on User
131+
@argumentDefinitions(cond: {type: "Boolean!", defaultValue: true}) {
133132
id
134133
username @include(if: $cond)
135134
}
136-
`));
137-
135+
`;
138136
TestChildComponent = jest.fn(() => <div />);
139137
TestChildContainer = ReactRelayFragmentContainer.createContainer(
140138
TestChildComponent,
@@ -223,7 +221,11 @@ describe('ReactRelayRefetchContainer with fragment ownerhsip', () => {
223221
uri: 'zuck2',
224222
},
225223
__id: '4',
226-
__fragments: {UserFriendFragment: {cond: true}},
224+
__fragments: {
225+
ReactRelayRefetchContainerWithFragmentOwnershipTestUserFriendFragment: {
226+
cond: true,
227+
},
228+
},
227229
__fragmentOwner: ownerUser1.request,
228230
});
229231
expect(TestChildComponent.mock.calls.length).toBe(1);
@@ -270,7 +272,11 @@ describe('ReactRelayRefetchContainer with fragment ownerhsip', () => {
270272
uri: 'zuck2',
271273
},
272274
__id: '4',
273-
__fragments: {UserFriendFragment: {cond: false}},
275+
__fragments: {
276+
ReactRelayRefetchContainerWithFragmentOwnershipTestUserFriendFragment: {
277+
cond: false,
278+
},
279+
},
274280
__fragmentOwner: expectedOwner.request,
275281
});
276282
expect(render.mock.calls[0][0].user.name).toBe(undefined);
@@ -291,7 +297,11 @@ describe('ReactRelayRefetchContainer with fragment ownerhsip', () => {
291297
uri: 'zuck2',
292298
},
293299
__id: '4',
294-
__fragments: {UserFriendFragment: {cond: true}},
300+
__fragments: {
301+
ReactRelayRefetchContainerWithFragmentOwnershipTestUserFriendFragment: {
302+
cond: true,
303+
},
304+
},
295305
__fragmentOwner: ownerUser1.request,
296306
});
297307
expect(TestChildComponent.mock.calls.length).toBe(1);
@@ -344,7 +354,11 @@ describe('ReactRelayRefetchContainer with fragment ownerhsip', () => {
344354
uri: 'zuck2',
345355
},
346356
__id: '4',
347-
__fragments: {UserFriendFragment: {cond: false}},
357+
__fragments: {
358+
ReactRelayRefetchContainerWithFragmentOwnershipTestUserFriendFragment: {
359+
cond: false,
360+
},
361+
},
348362
__fragmentOwner: expectedOwner.request,
349363
});
350364
expect(render.mock.calls[0][0].user.name).toBe(undefined);

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

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

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

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

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

+175
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.