This is an automated email from the ASF dual-hosted git repository.

pabloem pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/beam.git


The following commit(s) were added to refs/heads/master by this push:
     new 2bc4913  Fix environment_service_test.go
     new f8a4782  Merge pull request #15930 from [BEAM-13176] [Playground] 
[BUGFIX] Add new argument to NewNetworkEnv in env service tests
2bc4913 is described below

commit 2bc4913fadff6ec24495912adb737b33b4d4e03f
Author: Pavel Avilov <[email protected]>
AuthorDate: Wed Nov 10 13:11:49 2021 +0300

    Fix environment_service_test.go
---
 playground/backend/internal/environment/environment_service_test.go | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git 
a/playground/backend/internal/environment/environment_service_test.go 
b/playground/backend/internal/environment/environment_service_test.go
index a701595..20b0f80 100644
--- a/playground/backend/internal/environment/environment_service_test.go
+++ b/playground/backend/internal/environment/environment_service_test.go
@@ -159,8 +159,8 @@ func Test_getNetworkEnvsFromOsEnvs(t *testing.T) {
                },
                {
                        name:      "values from os envs",
-                       want:      NewNetworkEnvs("12.12.12.21", 1234, 
defaultProtocol),
-                       envsToSet: map[string]string{serverIpKey: 
"12.12.12.21", serverPortKey: "1234"},
+                       want:      NewNetworkEnvs("12.12.12.21", 1234, "TCP"),
+                       envsToSet: map[string]string{serverIpKey: 
"12.12.12.21", serverPortKey: "1234", protocolTypeKey: "TCP"},
                },
                {
                        name:      "not int port in os env, should be default",

Reply via email to