GraphQl-661: Test coverage of FedEx shipping method

[atwixfirster]

Issue: #661

Description (*)

To test FedEx on-line shipping carrier then you should create a sandbox account and configure FedEx shipping method on back-end side:

Proposed solution (*)

Move all sandbox settings to dev/tests/api-functional/config/config-global.php config file (this file under git ignore so we can hide our sandbox settings from the 3rd eyes):

<?php
/**
 * Copyright © Magento, Inc. All rights reserved.
 * See COPYING.txt for license details.
 */

return [
    'carriers' => [
        'usps' => [
            'userid' => '0:3:xxxxxxx1==',
            'password' => '0:3:xxxxxxx2==',
        ],
        'fedex' => [
            'account' => '0:3:yyyyyyyy0==',
            'meter_number' => '0:3:yyyyyyyy1==',
            'key' => '0:3:yyyyyyyy2=',
            'password' => '0:3:yyyyyyyy3=',
            'production_webservices_url' => 'https://wsbeta.fedex.com:443/web-services',
        ],
    ]
];

[atwixfirster]

@naydav , this PR is ready for your review.

Thank you!!

[naydav]

@atwixfirster
As we talk early, need to introduce some mock object instead of 'real caller' object
Our tests should not depend on external system

The best way is creating preference in GraphQLTestModule and replace real object on our internal mock object

[magento-engcom-team]

Hi @lenaorobei, thank you for the review.
ENGCOM-5285 has been created to process this Pull Request
✳️ @lenaorobei, could you please add one of the following labels to the Pull Request?

Label	Description
Auto-Tests: Covered	All changes in Pull Request is covered by auto-tests
Auto-Tests: Not Covered	Changes in Pull Request requires coverage by auto-tests
Auto-Tests: Not Required	Changes in Pull Request does not require coverage by auto-tests

[magento-engcom-team]

Hi @naydav, thank you for the review.
ENGCOM-5285 has been created to process this Pull Request
✳️ @naydav, could you please add one of the following labels to the Pull Request?

Label	Description
Auto-Tests: Covered	All changes in Pull Request is covered by auto-tests
Auto-Tests: Not Covered	Changes in Pull Request requires coverage by auto-tests
Auto-Tests: Not Required	Changes in Pull Request does not require coverage by auto-tests

[magento-engcom-team]

Hi @naydav, thank you for the review.
ENGCOM-5285 has been created to process this Pull Request

[ghost]

Hi @atwixfirster, thank you for your contribution!
Please, complete Contribution Survey, it will take less than a minute.
Your feedback will help us to improve contribution process.