laforge submitted this change.

View Change

Approvals: Jenkins Builder: Verified laforge: Verified pespin: Looks good to me, but someone else must approve fixeria: Looks good to me, approved
tests: Remove sms.db{-wal,-shm} files, not just sms.db

ERROR: files left in build directory after distclean:
./sms.db-shm
./tests/sms.db-shm
./tests/sms.db-wal
./sms.db-wal

Change-Id: Iecd380f598edbd1635361e4c340d54d092739919
---
M tests/Makefile.am
1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/tests/Makefile.am b/tests/Makefile.am
index c765158..d25fe16 100644
--- a/tests/Makefile.am
+++ b/tests/Makefile.am
@@ -69,7 +69,7 @@
IU=0 osmotestconfig.py -p $(abs_top_srcdir) -w $(abs_top_builddir) -v
endif
$(srcdir)/vty_test_runner.py -w $(abs_top_builddir) -v
- rm -f $(top_builddir)/sms.db
+ rm -f $(top_builddir)/sms.db*

# Run a specific transcript test with: 'make vty-transcript-test VTY_TEST=osmo-msc.vty'
VTY_TEST ?= *.vty
@@ -82,7 +82,7 @@
-n OsmoMSC -p 4254 \
-r "$(top_builddir)/src/osmo-msc/osmo-msc -c $(top_srcdir)/doc/examples/osmo-msc/osmo-msc.cfg" \
$(U) $(srcdir)/$(VTY_TEST)
- rm -f $(builddir)/sms.db
+ rm -f $(builddir)/sms.db*

# don't run multiple tests concurrently so that the ports don't conflict
vty-test:
@@ -91,7 +91,7 @@

ctrl-python-test: $(BUILT_SOURCES)
$(srcdir)/ctrl_test_runner.py -w $(abs_top_builddir) -v
- rm -f $(top_builddir)/sms.db
+ rm -f $(top_builddir)/sms.db*

# To update the CTRL script from current application behavior,
# pass -u to ctrl_script_runner.py by doing:
@@ -106,7 +106,7 @@

smpp-test:
$(srcdir)/smpp_test_runner.py -w $(abs_top_builddir) -v
- rm -f $(top_builddir)/sms.db
+ rm -f $(top_builddir)/sms.db*

check-local: atconfig $(TESTSUITE)
$(SHELL) '$(TESTSUITE)' $(TESTSUITEFLAGS)

1 is the latest approved patch-set. No files were changed between the latest approved patch-set and the submitted one.

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

Gerrit-Project: osmo-msc
Gerrit-Branch: master
Gerrit-Change-Id: Iecd380f598edbd1635361e4c340d54d092739919
Gerrit-Change-Number: 28124
Gerrit-PatchSet: 2
Gerrit-Owner: laforge <laforge@osmocom.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>
Gerrit-MessageType: merged