Skip to content

chore(e2e): separate bash commands to client pkg#1297

Merged
mergify[bot] merged 4 commits intoaws:mainlinefrom
iamhopaul123:e2e/bash-command-refactoring
Aug 19, 2020
Merged

chore(e2e): separate bash commands to client pkg#1297
mergify[bot] merged 4 commits intoaws:mainlinefrom
iamhopaul123:e2e/bash-command-refactoring

Conversation

@iamhopaul123
Copy link
Copy Markdown
Contributor

@iamhopaul123 iamhopaul123 commented Aug 19, 2020

Move bash commands in e2e tests to client pkg so as to make e2e tests more declarative. Address #1264 (comment).

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

@iamhopaul123 iamhopaul123 requested a review from a team as a code owner August 19, 2020 02:26
@iamhopaul123 iamhopaul123 requested a review from kohidave August 19, 2020 02:26
Copy link
Copy Markdown
Contributor

@bvtujo bvtujo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

awesome work cleaning this up! super readable. Just a tiny nit.

Comment thread e2e/internal/client/aws.go Outdated
@iamhopaul123 iamhopaul123 force-pushed the e2e/bash-command-refactoring branch from 19ba754 to 10c7345 Compare August 19, 2020 19:01
Comment thread e2e/internal/client/docker.go Outdated
Comment thread e2e/customized-env/customized_env_suite_test.go
@iamhopaul123 iamhopaul123 added the do-not-merge Pull requests that mergify shouldn't merge until the requester allows it. label Aug 19, 2020
@iamhopaul123 iamhopaul123 removed the do-not-merge Pull requests that mergify shouldn't merge until the requester allows it. label Aug 19, 2020
@mergify mergify bot merged commit ce24079 into aws:mainline Aug 19, 2020
thrau pushed a commit to localstack/copilot-cli-local that referenced this pull request Dec 9, 2022
<!-- Provide summary of changes -->
Move bash commands in e2e tests to `client` pkg so as to make e2e tests more declarative. Address aws#1264 (comment).
<!-- Issue number, if available. E.g. "Fixes aws#31", "Addresses aws#42, 77" -->

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants