Problem/Motivation

Fix the following test failure:

    Drupal\Tests\aws_cloud\Unit\Service\Ec2\Ec2ServiceTest::testAddMockHandlerByDescribeInstances
    Failed asserting that two arrays are identical.
    --- Expected
    +++ Actual
    @@ @@
     Array &0 (
    -    'effectiveUri' => 'https://ec2.us-west-2.amazonaws.com/'
    +    'effectiveUri' => 'https://ec2.us-west-2.amazonaws.com'
         'statusCode' => 200
         'transferStats' => Array &1 (
             'http' => Array &2 (

Issue fork cloud-3586451

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

yas created an issue. See original summary.

yas opened merge request !2360

yas’s picture

yas
🇯🇵 he/him
Primary language Japanese
Location California 🇺🇸
commented
Status: Needs work » Needs review

yas opened merge request !2361

yas’s picture

yas
🇯🇵 he/him
Primary language Japanese
Location California 🇺🇸
commented
Title: Update expected effectiveUri to match SDK behavior » Update expected 'effectiveUri' in Ec2ServiceTest::testAddMockHandlerByDescribeInstances() to match AWS SDK behavior
yas’s picture

yas
🇯🇵 he/him
Primary language Japanese
Location California 🇺🇸
commented
Status: Needs review » Reviewed & tested by the community

@all

My local tests are passed successfully, so I'll merge the MR. Thanks!

  • yas committed c827c0b8 on 7.x 
    Issue #3586451 by yas: Update expected 'effectiveUri' in Ec2ServiceTest...

  • yas committed 71e9fc29 on 8.x 
    Issue #3586451 by yas: Update expected 'effectiveUri' in Ec2ServiceTest...
yas’s picture

yas
🇯🇵 he/him
Primary language Japanese
Location California 🇺🇸
commented
Status: Reviewed & tested by the community » Fixed

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.