Skip to content

Commit bac11ae

Browse files
committed
magento/magento2#: GraphQL. Mutation: sendEmailToFriend. Test coverage for “Please provide Email of sender.”
1 parent bb7e08f commit bac11ae

File tree

1 file changed

+106
-63
lines changed

1 file changed

+106
-63
lines changed

dev/tests/api-functional/testsuite/Magento/GraphQl/SendFriend/SendFriendTest.php

Lines changed: 106 additions & 63 deletions
Original file line numberDiff line numberDiff line change
@@ -287,81 +287,124 @@ public function testSendProductWithoutVisibility()
287287
/**
288288
* @return array
289289
*/
290-
public function sendFriendsErrorsDataProvider()
290+
public function sendFriendsErrorsDataProvider(): array
291+
{
292+
return array_merge(
293+
$this->getRecipientErrors(),
294+
$this->getSenderErrors()
295+
);
296+
}
297+
298+
/**
299+
* @return array
300+
*/
301+
private function getRecipientErrors(): array
291302
{
292303
return [
293304
[
294-
'product_id: 1
295-
sender: {
296-
name: "Name"
297-
298-
message: "Lorem Ipsum"
299-
}
300-
recipients: [
301-
{
302-
name: ""
303-
304-
},
305-
{
306-
name: ""
307-
308-
}
309-
]', 'Please provide Name for all of recipients.'
305+
'product_id: 1
306+
sender: {
307+
name: "Name"
308+
309+
message: "Lorem Ipsum"
310+
}
311+
recipients: [
312+
{
313+
name: ""
314+
315+
},
316+
{
317+
name: ""
318+
319+
}
320+
]',
321+
'Please provide Name for all of recipients.'
310322
],
311323
[
312324
'product_id: 1
313-
sender: {
314-
name: "Name"
315-
316-
message: "Lorem Ipsum"
317-
}
318-
recipients: [
319-
{
320-
name: "Recipient Name 1"
321-
email:""
322-
},
323-
{
324-
name: "Recipient Name 2"
325-
email:""
326-
}
327-
]', 'Please provide Email for all of recipients.'
325+
sender: {
326+
name: "Name"
327+
328+
message: "Lorem Ipsum"
329+
}
330+
recipients: [
331+
{
332+
name: "Recipient Name 1"
333+
email:""
334+
},
335+
{
336+
name: "Recipient Name 2"
337+
email:""
338+
}
339+
]',
340+
'Please provide Email for all of recipients.'
328341
],
342+
];
343+
}
344+
345+
/**
346+
* @return array
347+
*/
348+
private function getSenderErrors(): array
349+
{
350+
return [
329351
[
330352
'product_id: 1
331-
sender: {
332-
name: ""
333-
334-
message: "Lorem Ipsum"
335-
}
336-
recipients: [
337-
{
338-
name: "Recipient Name 1"
339-
340-
},
341-
{
342-
name: "Recipient Name 2"
343-
344-
}
345-
]', 'Please provide Name of sender.'
353+
sender: {
354+
name: ""
355+
356+
message: "Lorem Ipsum"
357+
}
358+
recipients: [
359+
{
360+
name: "Recipient Name 1"
361+
362+
},
363+
{
364+
name: "Recipient Name 2"
365+
366+
}
367+
]',
368+
'Please provide Name of sender.'
346369
],
347370
[
348371
'product_id: 1
349-
sender: {
350-
name: "Name"
351-
352-
message: ""
353-
}
354-
recipients: [
355-
{
356-
name: "Recipient Name 1"
357-
358-
},
359-
{
360-
name: "Recipient Name 2"
361-
362-
}
363-
]', 'Please provide Message.'
364-
]
372+
sender: {
373+
name: "Name"
374+
email: ""
375+
message: "Lorem Ipsum"
376+
}
377+
recipients: [
378+
{
379+
name: "Recipient Name 1"
380+
381+
},
382+
{
383+
name: "Recipient Name 2"
384+
385+
}
386+
]',
387+
'Please provide Email of sender.'
388+
],
389+
[
390+
'product_id: 1
391+
sender: {
392+
name: "Name"
393+
394+
message: ""
395+
}
396+
recipients: [
397+
{
398+
name: "Recipient Name 1"
399+
400+
},
401+
{
402+
name: "Recipient Name 2"
403+
404+
}
405+
]',
406+
'Please provide Message.'
407+
],
365408
];
366409
}
367410

0 commit comments

Comments
 (0)