Skip to content

Conversation

@AllanZhengYP
Copy link
Contributor

@AllanZhengYP AllanZhengYP commented Jul 10, 2020

The change add a command yarn test:e2e to the project root. The command will run end-to-end test to all the changed packages since the last release. This command requires existing resources exists within the test account.

Purpose:

CI pipeline can run e2e test against the changed packages.

Prerequesition:

The test account contains a CF stack called IntegTestResourcesStack which includes several resources required for conducting E2E test.

Test Logs:
yarn test:e2e                                                                                                                                                                         ✔  12.16.2 ⬢ 
yarn run v1.22.4
$ node ./tests/e2e/index.js
packages to run e2e test:
/Users/zheallan/workspace/aws-sdk-js-v3/clients/client-cognito-identity
/Users/zheallan/workspace/aws-sdk-js-v3/clients/client-s3

lerna notice cli v3.22.1
lerna info versioning independent
lerna notice filter including "{@aws-sdk/client-cognito-identity,@aws-sdk/client-s3,}"
lerna info filter [ '{@aws-sdk/client-cognito-identity,@aws-sdk/client-s3,}' ]
lerna info Executing command in 2 packages: "yarn run test:e2e"
lerna info run Ran npm script 'test:e2e' in '@aws-sdk/client-cognito-identity' in 14.2s:
$ mocha **/cjs/**/*.ispec.js && karma start karma.conf.js


  @aws-sdk/client-cognito-identity
    ✓ should successfully fetch Id and get credentials (227ms)


  1 passing (236ms)

ℹ 「wdm」: Compiled successfully.
ℹ 「wdm」: Compiling...
ℹ 「wdm」:    183 modules
ℹ 「wdm」: Compiled successfully.
Firefox 68.0 (Mac OS 10.14): Executed 1 of 1 SUCCESS (0.92 secs / 0.927 secs)
Chrome Headless 83.0.4103.116 (Mac OS 10.14.6): Executed 1 of 1 SUCCESS (0.233 secs / 0.227 secs)
TOTAL: 2 SUCCESS
lerna info run Ran npm script 'test:e2e' in '@aws-sdk/client-s3' in 35.3s:
$ mocha **/cjs/**/*.ispec.js && karma start karma.conf.js


  @aws-sdk/client-s3
    PutObject
      ✓ should succeed with Node.js readable stream body (291ms)
    GetObject
      ✓ should succeed with valid body payload (1597ms)
    ListObjects
      ✓ should succeed with valid bucket (126ms)
      ✓ should throw with invalid bucket (132ms)
    MultipartUpload
      ✓ should successfully create, upload list and complete (343ms)
      ✓ should successfully create, abort, and list upload (109ms)


  6 passing (3s)

ℹ 「wdm」: Compiled successfully.
ℹ 「wdm」: Compiling...
ℹ 「wdm」:    325 modules
ℹ 「wdm」: Compiled successfully.
Chrome Headless 83.0.4103.116 (Mac OS 10.14.6): Executed 8 of 8 SUCCESS (7.781 secs / 5.936 secs)
Firefox 68.0 (Mac OS 10.14): Executed 8 of 8 SUCCESS (8.403 secs / 7.315 secs)
TOTAL: 16 SUCCESS
lerna success run Ran npm script 'test:e2e' in 2 packages in 49.5s:
lerna success - @aws-sdk/client-cognito-identity
lerna success - @aws-sdk/client-s3
✨  Done in 62.18s.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@codecov-commenter
Copy link

codecov-commenter commented Jul 10, 2020

Codecov Report

Merging #1359 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1359   +/-   ##
=======================================
  Coverage   79.00%   79.00%           
=======================================
  Files         287      287           
  Lines       11112    11112           
  Branches     2350     2350           
=======================================
  Hits         8779     8779           
  Misses       2333     2333           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 97efeeb...b4f99e4. Read the comment docs.

@AllanZhengYP AllanZhengYP requested a review from trivikr July 10, 2020 21:10
@AllanZhengYP AllanZhengYP merged commit c2b7b7d into master Jul 10, 2020
@AllanZhengYP AllanZhengYP deleted the automatic-e2e branch July 28, 2020 21:57
@github-actions
Copy link

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs and link to relevant comments in this thread.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jan 17, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants