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 expect to injected vars in inline scripts. #365

Merged
merged 2 commits into from
Aug 13, 2021

Conversation

andrewvc
Copy link
Contributor

This also refactors the CLIMock in the cli tests to use a builder pattern to make injecting stdin nicer.

This also refactors the CLIMock in the cli tests to use a builder
pattern to make injecting stdin nicer.
@andrewvc andrewvc added the enhancement New feature or request label Aug 13, 2021
@andrewvc andrewvc self-assigned this Aug 13, 2021
@apmmachine
Copy link

apmmachine commented Aug 13, 2021

💚 Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS
Pipeline View Test View Changes Artifacts preview preview

Expand to view the summary

Build stats

  • Start Time: 2021-08-13T19:36:38.728+0000

  • Duration: 15 min 28 sec

  • Commit: 6d3c0b0

Test stats 🧪

Test Results
Failed 0
Passed 125
Skipped 0
Total 125

Trends 🧪

Image of Build Times

Image of Tests

@andrewvc
Copy link
Contributor Author

/test

@andrewvc andrewvc merged commit c186930 into elastic:master Aug 13, 2021
@andrewvc andrewvc deleted the inline-expect branch August 13, 2021 19:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants