Harald Welte has submitted this change and it was merged. ( 
https://gerrit.osmocom.org/12126 )

Change subject: make: prepare for adding transcript tests
......................................................................

make: prepare for adding transcript tests

Separate 'make python-test' into separate make targets, to sensibly add VTY
transcript tests in an upcoming commit.

Feature: even though ./configure was called without --enable-external-tests,
each of the {ctrl,vty}x{python,transcript} tests can be invoked individually by
e.g. 'make vty-python-test'.

Both 'vty-transcript-test' and 'ctrl-transcript-test' are still empty, a
subsequent patch adds a vty-transcript-test.

All of this in preparation of tweaking the 'mncc' vty configuration, to be able
to track it in a vty transcript test.

Change-Id: I688657e56ae469c07b9f25ba37275d38dbd457e2
---
M tests/Makefile.am
1 file changed, 41 insertions(+), 6 deletions(-)

Approvals:
  Pau Espin Pedrol: Looks good to me, approved
  Stefan Sperling: Looks good to me, but someone else must approve
  Jenkins Builder: Verified



diff --git a/tests/Makefile.am b/tests/Makefile.am
index f6b33fc..a6a4011 100644
--- a/tests/Makefile.am
+++ b/tests/Makefile.am
@@ -43,20 +43,55 @@
        $(NULL)

 if ENABLE_EXT_TESTS
+# don't run multiple tests concurrently so that the ports don't conflict
 python-tests: $(BUILT_SOURCES)
-       osmotestvty.py -p $(abs_top_srcdir) -w $(abs_top_builddir) -v
-       osmotestconfig.py -p $(abs_top_srcdir) -w $(abs_top_builddir) -v
-       $(srcdir)/vty_test_runner.py -w $(abs_top_builddir) -v
-       $(srcdir)/ctrl_test_runner.py -w $(abs_top_builddir) -v
+       $(MAKE) vty-test
+       $(MAKE) ctrl-test
 if BUILD_SMPP
-       $(srcdir)/smpp_test_runner.py -w $(abs_top_builddir) -v
+       $(MAKE) smpp-test
 endif
-       rm -f $(top_builddir)/sms.db
+
 else
 python-tests: $(BUILT_SOURCES)
        echo "Not running python-based tests (determined at configure-time)"
 endif

+vty-python-test: $(BUILT_SOURCES)
+       osmotestvty.py -p $(abs_top_srcdir) -w $(abs_top_builddir) -v
+       osmotestconfig.py -p $(abs_top_srcdir) -w $(abs_top_builddir) -v
+       $(srcdir)/vty_test_runner.py -w $(abs_top_builddir) -v
+       rm -f $(top_builddir)/sms.db
+
+# To update the VTY script from current application behavior,
+# pass -u to vty_script_runner.py by doing:
+#   make vty-transcript-test U=-u
+vty-transcript-test:
+       echo "No vty-transcript-test exists yet"
+
+# don't run multiple tests concurrently so that the ports don't conflict
+vty-test:
+       $(MAKE) vty-python-test
+       $(MAKE) vty-transcript-test
+
+ctrl-python-test: $(BUILT_SOURCES)
+       $(srcdir)/ctrl_test_runner.py -w $(abs_top_builddir) -v
+       rm -f $(top_builddir)/sms.db
+
+# To update the CTRL script from current application behavior,
+# pass -u to ctrl_script_runner.py by doing:
+#   make ctrl-transcript-test U=-u
+ctrl-transcript-test:
+       echo "No ctrl-transcript-test exists yet"
+
+# don't run multiple tests concurrently so that the ports don't conflict
+ctrl-test:
+       $(MAKE) ctrl-python-test
+       $(MAKE) ctrl-transcript-test
+
+smpp-test:
+       $(srcdir)/smpp_test_runner.py -w $(abs_top_builddir) -v
+       rm -f $(top_builddir)/sms.db
+
 check-local: atconfig $(TESTSUITE)
        $(SHELL) '$(TESTSUITE)' $(TESTSUITEFLAGS)
        $(MAKE) $(AM_MAKEFLAGS) python-tests

--
To view, visit https://gerrit.osmocom.org/12126
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-msc
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I688657e56ae469c07b9f25ba37275d38dbd457e2
Gerrit-Change-Number: 12126
Gerrit-PatchSet: 5
Gerrit-Owner: Neels Hofmeyr <[email protected]>
Gerrit-Reviewer: Harald Welte <[email protected]>
Gerrit-Reviewer: Jenkins Builder (1000002)
Gerrit-Reviewer: Neels Hofmeyr <[email protected]>
Gerrit-Reviewer: Pau Espin Pedrol <[email protected]>
Gerrit-Reviewer: Stefan Sperling <[email protected]>

Reply via email to