Skip to content

Commit ff3d484

Browse files
authored
mobile/test: Update the proxy test comments (envoyproxy#39986)
Signed-off-by: Ali Beyad <[email protected]>
1 parent 5d7de90 commit ff3d484

6 files changed

+36
-18
lines changed

mobile/test/kotlin/integration/proxying/ProxyInfoIntentPerformHTTPRequestUsingProxyTest.kt

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -24,13 +24,16 @@ import org.junit.runner.RunWith
2424
import org.robolectric.RobolectricTestRunner
2525
import org.robolectric.Shadows.shadowOf
2626

27+
// Starts a local HTTP mock server with HttpTestServerFactory.start(). Then a request is sent
28+
// through the following flow:
29+
//
2730
// ┌──────────────────┐
2831
// │ Envoy Proxy │
2932
// │ ┌──────────────┐ │
3033
// ┌────────────────────────┐ ┌─┼─►listener_proxy│ │
31-
// │http://api.lyft.com/ping│ ┌──────────────┬┘ │ └──────┬───────┘ │ ┌────────────┐
32-
// │ Request ├──►Android Engine│ │ │ │ │api.lyft.com
33-
// └────────────────────────┘ └──────────────┘ │ ┌──────▼──────┐ │ └──────▲─────┘
34+
// │http://localhost:{port}/│ ┌──────────────┬┘ │ └──────┬───────┘ │ ┌────────────-----
35+
// │ Request ├──►Android Engine│ │ │ │ │Local Mock Server
36+
// └────────────────────────┘ └──────────────┘ │ ┌──────▼──────┐ │ └──────▲─────-----
3437
// │ │cluster_proxy│ │ │
3538
// │ └─────────────┴──┼────────┘
3639
// │ │

mobile/test/kotlin/integration/proxying/ProxyInfoIntentPerformHTTPSRequestBadHostnameTest.kt

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -25,13 +25,16 @@ import org.junit.runner.RunWith
2525
import org.robolectric.RobolectricTestRunner
2626
import org.robolectric.Shadows
2727

28+
// Starts a local HTTP mock server with HttpTestServerFactory.start(). Then a request is sent
29+
// through the following flow:
30+
//
2831
// ┌──────────────────┐
2932
// │ Envoy Proxy │
3033
// │ ┌──────────────┐ │
3134
// ┌─────────────────────────┐ ┌─┼─►listener_proxy│ │
32-
// │https://api.lyft.com/ping│ ┌──────────────┬┘ │ └──────┬───────┘ │ ┌────────────┐
33-
// │ Request ├──►Android Engine│ │ │ │ │api.lyft.com
34-
// └─────────────────────────┘ └──────────────┘ │ ┌──────▼──────┐ │ └──────▲─────┘
35+
// │https://localhost:{port} │ ┌──────────────┬┘ │ └──────┬───────┘ │ ┌──────────────────┐
36+
// │ Request ├──►Android Engine│ │ │ │ │ Local mock server
37+
// └─────────────────────────┘ └──────────────┘ │ ┌──────▼──────┐ │ └──────▲───────────
3538
// │ │cluster_proxy│ │ │
3639
// │ └─────────────┴──┼────────┘
3740
// │ │

mobile/test/kotlin/integration/proxying/ProxyInfoIntentPerformHTTPSRequestUsingAsyncProxyTest.kt

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -25,13 +25,16 @@ import org.junit.runner.RunWith
2525
import org.robolectric.RobolectricTestRunner
2626
import org.robolectric.Shadows
2727

28+
// Starts a local HTTP mock server with HttpTestServerFactory.start(). Then a request is sent
29+
// through the following flow:
30+
//
2831
// ┌──────────────────┐
2932
// │ Envoy Proxy │
3033
// │ ┌──────────────┐ │
3134
// ┌─────────────────────────┐ ┌─┼─►listener_proxy│ │
32-
// │https://api.lyft.com/ping│ ┌──────────────┬┘ │ └──────┬───────┘ │ ┌────────────┐
33-
// │ Request ├──►Android Engine│ │ │ │ │api.lyft.com
34-
// └─────────────────────────┘ └──────────────┘ │ ┌──────▼──────┐ │ └──────▲─────┘
35+
// │https://localhost:{port} │ ┌──────────────┬┘ │ └──────┬───────┘ │ ┌─────────────────┐
36+
// │ Request ├──►Android Engine│ │ │ │ │Local mock server
37+
// └─────────────────────────┘ └──────────────┘ │ ┌──────▼──────┐ │ └──────▲──────────
3538
// │ │cluster_proxy│ │ │
3639
// │ └─────────────┴──┼────────┘
3740
// │ │

mobile/test/kotlin/integration/proxying/ProxyInfoIntentPerformHTTPSRequestUsingProxyTest.kt

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -25,13 +25,16 @@ import org.junit.runner.RunWith
2525
import org.robolectric.RobolectricTestRunner
2626
import org.robolectric.Shadows
2727

28+
// Starts a local HTTP mock server with HttpTestServerFactory.start(). Then a request is sent
29+
// through the following flow:
30+
//
2831
// ┌──────────────────┐
2932
// │ Envoy Proxy │
3033
// │ ┌──────────────┐ │
3134
// ┌─────────────────────────┐ ┌─┼─►listener_proxy│ │
32-
// │https://api.lyft.com/ping│ ┌──────────────┬┘ │ └──────┬───────┘ │ ┌────────────┐
33-
// │ Request ├──►Android Engine│ │ │ │ │api.lyft.com
34-
// └─────────────────────────┘ └──────────────┘ │ ┌──────▼──────┐ │ └──────▲─────┘
35+
// │https://localhost:{port} │ ┌──────────────┬┘ │ └──────┬───────┘ │ ┌─────────────────┐
36+
// │ Request ├──►Android Engine│ │ │ │ │Local mock server
37+
// └─────────────────────────┘ └──────────────┘ │ ┌──────▼──────┐ │ └──────▲──────────
3538
// │ │cluster_proxy│ │ │
3639
// │ └─────────────┴──┼────────┘
3740
// │ │

mobile/test/kotlin/integration/proxying/ProxyPollPerformHTTPRequestUsingProxyTest.kt

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -22,13 +22,16 @@ import org.junit.runner.RunWith
2222
import org.robolectric.RobolectricTestRunner
2323
import org.robolectric.Shadows
2424

25+
// Starts a local HTTP mock server with HttpTestServerFactory.start(). Then a request is sent
26+
// through the following flow:
27+
//
2528
// ┌──────────────────┐
2629
// │ Envoy Proxy │
2730
// │ ┌──────────────┐ │
2831
// ┌────────────────────────┐ ┌─┼─►listener_proxy│ │
29-
// │http://api.lyft.com/ping│ ┌──────────────┬┘ │ └──────┬───────┘ │ ┌────────────┐
30-
// │ Request ├──►Android Engine│ │ │ │ │api.lyft.com
31-
// └────────────────────────┘ └──────────────┘ │ ┌──────▼──────┐ │ └──────▲─────┘
32+
// │http://localhost:{port} │ ┌──────────────┬┘ │ └──────┬───────┘ │ ┌─────────────────┐
33+
// │ Request ├──►Android Engine│ │ │ │ │Local mock server
34+
// └────────────────────────┘ └──────────────┘ │ ┌──────▼──────┐ │ └──────▲──────────
3235
// │ │cluster_proxy│ │ │
3336
// │ └─────────────┴──┼────────┘
3437
// │ │

mobile/test/kotlin/integration/proxying/ProxyPollPerformHTTPRequestWithoutUsingPACProxyTest.kt

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -23,13 +23,16 @@ import org.junit.runner.RunWith
2323
import org.robolectric.RobolectricTestRunner
2424
import org.robolectric.Shadows
2525

26+
// Starts a local HTTP mock server with HttpTestServerFactory.start(). Then a request is sent
27+
// through the following flow:
28+
//
2629
// ┌──────────────────┐
2730
// │ Envoy Proxy │
2831
// │ ┌──────────────┐ │
2932
// ┌─────────────────────────┐ │ │listener_proxy│ │
30-
// │https://api.lyft.com/ping│ ┌──────────────┐ │ └──────┬───────┘ │ ┌────────────┐
31-
// │ HTTP Request ├──►Android Engine│ │ │ │ │api.lyft.com
32-
// └─────────────────────────┘ └───────┬──────┘ │ ┌──────▼──────┐ │ └──────▲─────┘
33+
// │https://localhost:{port} │ ┌──────────────┐ │ └──────┬───────┘ │ ┌─────────────────┐
34+
// │ HTTP Request ├──►Android Engine│ │ │ │ │Local mock server
35+
// └─────────────────────────┘ └───────┬──────┘ │ ┌──────▼──────┐ │ └──────▲──────────
3336
// │ │ │cluster_proxy│ │ │
3437
// │ │ └─────────────┘ │ │
3538
// │ │ │ │

0 commit comments

Comments
 (0)