osmith submitted this change.

Approvals:
  pespin: Looks good to me, approved
  Jenkins Builder: Verified

remsim: make client tests work in testenv

* Prevent osmo-remsim-client-shell from closing immediately with "STDIN
  was closed, terminating".

* Fix REMSIM_Tests.cfg:
  * move mp_ to proper section
  * proper IP

Change-Id: I91b08b9bfde11f4600993b18ece1e810b61ed904
---
M remsim/client/REMSIM_Tests.cfg
M remsim/testenv_client.cfg
2 files changed, 6 insertions(+), 5 deletions(-)

diff --git a/remsim/client/REMSIM_Tests.cfg b/remsim/client/REMSIM_Tests.cfg
index b249087..affa599 100644
--- a/remsim/client/REMSIM_Tests.cfg
+++ b/remsim/client/REMSIM_Tests.cfg
@@ -6,12 +6,12 @@
 *.JUnitLogger.filename_stem := "junit-xml-client"
 
 [TESTPORT_PARAMETERS]
-# IP address to which ATS binds emulated remsim-server
-REMSIM_Tests.mp_server_ip := "127.0.0.10"
-# IP address to which ATS binds emulated remsim-bankd
-REMSIM_Tests.mp_bankd_ip := "127.0.0.10"
 
 [MODULE_PARAMETERS]
+# IP address to which ATS binds emulated remsim-server
+REMSIM_Tests.mp_server_ip := "127.0.0.1"
+# IP address to which ATS binds emulated remsim-bankd
+REMSIM_Tests.mp_bankd_ip := "127.0.0.1"
 
 [MAIN_CONTROLLER]
 
diff --git a/remsim/testenv_client.cfg b/remsim/testenv_client.cfg
index c7d525b..95df982 100644
--- a/remsim/testenv_client.cfg
+++ b/remsim/testenv_client.cfg
@@ -3,6 +3,7 @@
 config=client/REMSIM_Tests.cfg
 
 [client]
-program=osmo-remsim-client-shell
+# tail: avoid "STDIN was closed, terminating"
+program=tail -f /dev/null | osmo-remsim-client-shell
 make=osmo-remsim
 package=osmo-remsim-client-shell

To view, visit change 39916. To unsubscribe, or for help writing mail filters, visit settings.