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.

Sign up for GitHub

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

Jump to bottom

Integration test improvements #1518

Closed
alyssawilk opened this issue Aug 22, 2017 · 3 comments · Fixed by #1901
Closed

Integration test improvements #1518

alyssawilk opened this issue Aug 22, 2017 · 3 comments · Fixed by #1901
Assignees
@alyssawilk
Labels
enhancement Feature requests. Not bugs or questions.
Milestone
1.5.0

Comments

@alyssawilk
Copy link
Contributor

It would be nice to have end-to-end tests which were slightly less deterministic, to make it easier to have many streams and large bidirectional data flows going independently of each other. This results in less deterministic tests but allows fuzzing corner cases for things like watermark limits and unexpected disconnects and resets.

Possibly a separate issue but likely predicated on this, it'd be nice to be able to do simpler things with less boilerplate - set a default backend response and client->sendSynchronousRequest, then inspect response headers or upstream headers after the fact.

Finally, while I'm thinking about it, perhaps make an easy config for which test should apply to which languages so one doesn't have to edit multiple integration tests every time you want something tested for http/h2/h2-upstream.

... and as long as I'm dreaming, I'd like a pony.
This was referenced Aug 22, 2017
Merged
Merged
@PiotrSikora
Copy link
Contributor

Related: #186.

@mattklein123 mattklein123 added the enhancement Feature requests. Not bugs or questions. label Aug 22, 2017
@mattklein123 mattklein123 added this to the 1.5.0 milestone Aug 22, 2017
This was referenced Aug 24, 2017
Merged
Merged
@alyssawilk alyssawilk mentioned this issue Sep 25, 2017
Merged
@mattklein123
Copy link
Member

@alyssawilk are we still tracking stuff with this? Or should we call it done for now?

@alyssawilk
Copy link
Contributor Author

Yeah, I have one last fix I'd like to get in. There's still one race I was debugging but I'll ship it out for early inspection / critiques while I track that down

@alyssawilk alyssawilk mentioned this issue Oct 19, 2017
Merged
rshriram pushed a commit to rshriram/envoy that referenced this issue Oct 30, 2018
@istio-testing
Update_Dependencies (envoyproxy#1518)
6063a5d 
Automatic merge from submit-queue.

[BOT PR] to update dependencies of proxy

This PR will be merged automatically once checks are successful.
```release-note
none
```
jpsim pushed a commit that referenced this issue Nov 28, 2022
@crockeo @jpsim
cc: config phase 2 for cc library (#1518)
592e084 
Description: Based on envoyproxy/envoy-mobile#1515, implements the new config changes for the cc library and adds a test to make sure that the config is generated as expected.
Risk Level: Low
Testing: `bazelisk test //test/cc/unit:envoy_config_test`

Signed-off-by: Cerek Hillen <chillen@lyft.com>
Signed-off-by: JP Simard <jp@jpsim.com>
jpsim pushed a commit that referenced this issue Nov 29, 2022
@crockeo @jpsim
cc: config phase 2 for cc library (#1518)
8eb2153 
Description: Based on envoyproxy/envoy-mobile#1515, implements the new config changes for the cc library and adds a test to make sure that the config is generated as expected.
Risk Level: Low
Testing: `bazelisk test //test/cc/unit:envoy_config_test`

Signed-off-by: Cerek Hillen <chillen@lyft.com>
Signed-off-by: JP Simard <jp@jpsim.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@alyssawilk alyssawilk

Labels
enhancement Feature requests. Not bugs or questions.
Projects
None yet
Milestone
1.5.0
Development

Successfully merging a pull request may close this issue.

Allow true bidirectional streaming in tests alyssawilk/envoy
3 participants
@PiotrSikora @mattklein123 @alyssawilk