Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add waitForService to TS integration test #2472

Closed

Conversation

RyanRHall
Copy link
Contributor

@RyanRHall RyanRHall commented Mar 11, 2020

Fixes race condition in TS integration test by waiting for CL node before initiating test suit.

Waiting for #2476

eth:clean Reset blockchain data to genesis state
cl:clean Reset chainlink database
eth:restart Reset blockchain data to genesis state
cl:restart Reset chainlink database

Copy link
Contributor Author

@RyanRHall RyanRHall Mar 11, 2020

Choose a reason for hiding this comment

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

unrelated to this PR - fixes compose help output

tools/ci-ts/test-helpers/common.ts Show resolved Hide resolved
tools/ci-ts/test-helpers/common.ts Outdated Show resolved Hide resolved
@RyanRHall RyanRHall force-pushed the add-waitForService-function-to-integration-test branch 2 times, most recently from ce6a6ec to 11f8a0c Compare March 11, 2020 22:31
tools/ci-ts/test-helpers/common.ts Outdated Show resolved Hide resolved
@RyanRHall
Copy link
Contributor Author

RyanRHall commented Mar 12, 2020

Going to close this PR and merge these changes into the PR for the multi-node integration test, seeing as this race doesn't actually seem to be an issue right now.

@RyanRHall RyanRHall closed this Mar 12, 2020
@RyanRHall RyanRHall deleted the add-waitForService-function-to-integration-test branch March 12, 2020 19:00
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.

2 participants