[BUG]: Fixed replay starts execution before the app is started #1823
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Related Issue
Closes: #1810
Closes: #1795
Describe the changes you've made
Currently keploy would wait till delay (5s) as provided using delay flag for user application to start , but it is required only for native env and for docker related application we have to have till the application starts or buildDelay time is completed. This pr fix this bug by adding a channel to signal the start of application or cancel the service if buildDelay is reached. Also the PR fixes the panic in keploy during an error by adding an err channel on the required goroutine which became unreachable due to the error.
Type of change
Please let us know if any test cases are added
Please describe the tests(if any). Provide instructions how its affecting the coverage.
Describe if there is any unusual behaviour of your code(Write
NA
if there isn't)A clear and concise description of it.
Checklist:
Screenshots (if any)