test to verify current dbversion behavior

diff --git a/tests/Makefile.am b/tests/Makefile.am
index 645dd5416ae9386e1b47fde480dd501d1249db2b..8367f656ae6e8e65c28369cdc5c5756f9dcd5159 100644 (file)
--- a/tests/Makefile.am
+++ b/tests/Makefile.am
@@ -24,4 +24,4 @@ check:
                topsrcdir=${top_srcdir} \
                ${PYTHON} ${srcdir}/alltests
 
-EXTRA_DIST=alltests dtest.py nothing.py version.py
+EXTRA_DIST=alltests dtest.py nothing.py version.py dbversion.py

diff --git a/tests/alltests b/tests/alltests
index 8be9441fd1c8717eec30bf044e1a91999b304fe1..a68a3bea7f2f04fefbd2c3a25b0b2e16cb25fce1 100755 (executable)
--- a/tests/alltests
+++ b/tests/alltests
@@ -1,7 +1,7 @@
 #! /usr/bin/env python
 import dtest
 
-tests = ["nothing", "version", "files"]
+tests = ["nothing", "version", "files", "dbversion"]
 
 for test in tests:
     print "Test '%s'" % test

diff --git a/tests/dbversion.py b/tests/dbversion.py
new file mode 100644 (file)
index 0000000..4e92611
--- /dev/null
+++ b/tests/dbversion.py
@@ -0,0 +1,34 @@
+#! /usr/bin/env python
+import dtest,time,disorder,sys,re
+
+def test():
+    """Database version tests"""
+    # Start up with dbversion 1
+    config = "%s/config" % dtest.testroot
+    configsave = "%s.save" % config
+    dtest.copyfile(config, configsave)
+    open(config, "a").write("dbversion 1\n")
+    dtest.start_daemon()
+    time.sleep(2)
+    dtest.stop_daemon()
+    # Revert to default configuration
+    dtest.copyfile(configsave, config)
+    dtest.start_daemon()
+    time.sleep(2)
+    c = disorder.client()
+    try:
+        v = c.version()
+        print "unexpected success"
+        ok = False
+    except disorder.communicationError, e:
+        if re.search("connection refused", str(e)):
+            print "unexpected error: %s" % e
+            ok = False
+        else:
+            ok = True
+    dtest.stop_daemon()
+    if not ok:
+        sys.exit(1)
+
+if __name__ == '__main__':
+    dtest.run(test)

diff --git a/tests/dtest.py b/tests/dtest.py
index dba7e353777695be0477fb2bb396456dfd16397f..26887cd63fcbe45be57d800b502d297924196f62 100644 (file)
--- a/tests/dtest.py
+++ b/tests/dtest.py
@@ -116,29 +116,24 @@ tracklength *.flac disorder-tracklength
     copyfile("%s/sounds/scratch.ogg" % topsrcdir,
              "%s/scratch.ogg" % testroot)
 
-def start_daemon(test):
-    """start_daemon(TEST)
-Start the daemon for test called TEST."""
-    global daemon
-    assert daemon == None
-    if test == None:
-        errs = sys.stderr
-    else:
-        errs = open("%s.log" % test, "w")
+def start_daemon():
+    """start_daemon()
+Start the daemon."""
+    global daemon,errs
+    assert daemon is None
     server = None
     print " starting daemon"
     daemon = subprocess.Popen(["disorderd",
                                "--foreground",
                                "--config", "%s/config" % testroot],
                               stderr=errs)
-    disorder._configfile = "%s/config" % testroot
-    disorder._userconf = False
 
 def stop_daemon():
     """stop_daemon()
 
 Stop the daemon if it has not stopped already"""
     global daemon
+    assert daemon is not None
     rc = daemon.poll()
     if rc == None:
         print " stopping daemon"
@@ -148,13 +143,15 @@ Stop the daemon if it has not stopped already"""
     daemon = None
 
 def run(test, setup=None, report=True, name=None): 
-    global tests
+    global tests,errs
     tests += 1
     if setup == None:
         setup = stdtracks
+    errs = open("%s.log" % test.__name__, "w") # HNGGGH.  nO.
+    disorder._configfile = "%s/config" % testroot
+    disorder._userconf = False
     common_setup()
     setup()
-    start_daemon(name)
     try:
         try:
             test()
@@ -163,7 +160,8 @@ def run(test, setup=None, report=True, name=None):
             failures += 1
             print e
     finally:
-        stop_daemon()
+        if daemon is not None:
+            stop_daemon()
     if report:
         if failures:
             print " FAILED"

diff --git a/tests/files.py b/tests/files.py
index 938564ac62b25ca193f2eded9e2a90a0f2efd0d9..e0237d44b7ffde2b9e715107d037ab3c737677d5 100755 (executable)
--- a/tests/files.py
+++ b/tests/files.py
@@ -2,7 +2,8 @@
 import dtest,time,disorder,sys
 
 def test():
-    """Ask the server its version number"""
+    """Check that the file listing comes out right"""
+    dtest.start_daemon()
     time.sleep(5)                       # give rescan a chance
     c = disorder.client()
     failures = 0

diff --git a/tests/nothing.py b/tests/nothing.py
index 7a8e55de4e2b6b51760653d4e7f4c74c11c3a417..70e481b602415736ab84fc8f1628d6cd43a25c79 100755 (executable)
--- a/tests/nothing.py
+++ b/tests/nothing.py
@@ -3,6 +3,7 @@ import dtest,time
 
 def test():
     """Just start the server and then stop it a few seconds later"""
+    dtest.start_daemon()
     time.sleep(2)
 
 if __name__ == '__main__':

diff --git a/tests/version.py b/tests/version.py
index 35668671d233a487e4b91d900c492c7f8e48157a..7a955734b5021fb92a01b12ee8399f9a786cd6ca 100755 (executable)
--- a/tests/version.py
+++ b/tests/version.py
@@ -3,6 +3,7 @@ import dtest,time,disorder
 
 def test():
     """Ask the server its version number"""
+    dtest.start_daemon()
     time.sleep(2)                       # give the daemon a chance to start up
     c = disorder.client()
     v = c.version()