chiark / gitweb /
sync with disorder.dev
authorRichard Kettlewell <rjk@greenend.org.uk>
Wed, 21 Nov 2007 10:23:02 +0000 (10:23 +0000)
committerRichard Kettlewell <rjk@greenend.org.uk>
Wed, 21 Nov 2007 10:23:02 +0000 (10:23 +0000)
1  2 
tests/Makefile.am
tests/dbversion.py
tests/dtest.py
tests/files.py
tests/nothing.py
tests/version.py

@@@ -19,9 -19,6 +19,6 @@@
  #
  
  check:
-       PATH=`pwd`/../server:`pwd`/..clients:$$PATH \
-       PYTHONPATH=../python \
-               topsrcdir=${top_srcdir} \
-               ${PYTHON} ${srcdir}/alltests
+       ${PYTHON} ${srcdir}/alltests
  
 -EXTRA_DIST=alltests dtest.py nothing.py version.py
 +EXTRA_DIST=alltests dtest.py nothing.py version.py dbversion.py
index 4e92611,0000000..e5b5c48
mode 100644,000000..100644
--- /dev/null
@@@ -1,34 -1,0 +1,53 @@@
 +#! /usr/bin/env python
++#
++# This file is part of DisOrder.
++# Copyright (C) 2007 Richard Kettlewell
++#
++# This program is free software; you can redistribute it and/or modify
++# it under the terms of the GNU General Public License as published by
++# the Free Software Foundation; either version 2 of the License, or
++# (at your option) any later version.
++#
++# This program is distributed in the hope that it will be useful, but
++# WITHOUT ANY WARRANTY; without even the implied warranty of
++# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
++# General Public License for more details.
++#
++# You should have received a copy of the GNU General Public License
++# along with this program; if not, write to the Free Software
++# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307
++# USA
++#
 +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 --cc tests/dtest.py
  
  """Utility module used by tests"""
  
- import os,os.path,subprocess,sys,disorder,unicodedata
 -import os,os.path,subprocess,sys,re
++import os,os.path,subprocess,sys,re,unicodedata
+ def fatal(s):
+     """Write an error message and exit"""
+     sys.stderr.write("ERROR: %s\n" % s)
+     sys.exit(1)
+ # Identify the top build directory
+ cwd = os.getcwd()
+ if os.path.exists("config.h"):
+     top_builddir = cwd
+ elif os.path.exists("alltests"):
+     top_builddir = os.path.dirname(cwd)
+ else:
+     fatal("cannot identify build directory")
+ # Make sure the Python build directory is on the module search path
+ sys.path.insert(0, os.path.join(top_builddir, "python"))
+ import disorder
+ # Make sure the server build directory is on the executable search path
+ ospath = os.environ["PATH"].split(os.pathsep)
+ ospath.insert(0, os.path.join(top_builddir, "server"))
+ os.environ["PATH"] = os.pathsep.join(ospath)
+ # Parse the makefile in the current directory to identify the source directory
+ top_srcdir = None
+ for l in file("Makefile"):
+     r = re.match("top_srcdir *= *(.*)",  l)
+     if r:
+         top_srcdir = r.group(1)
+         break
+ if not top_srcdir:
+     fatal("cannot identify source directory")
+ # The tests source directory must be on the module search path already since
+ # we found dtest.py
+ # -----------------------------------------------------------------------------
  
  def copyfile(a,b):
      """copyfile(A, B)
@@@ -17,54 -74,15 +74,54 @@@ Make track with relative path S exist""
      trackdir = os.path.dirname(trackpath)
      if not os.path.exists(trackdir):
          os.makedirs(trackdir)
-     copyfile("%s/sounds/slap.ogg" % topsrcdir, trackpath)
+     copyfile("%s/sounds/slap.ogg" % top_srcdir, trackpath)
 +    # We record the tracks we created so they can be tested against
 +    # server responses.  We put them into NFC since that's what the server
 +    # uses internally.
 +    bits = unicodedata.normalize("NFC",
 +                                 unicode(s, "UTF-8")).split('/')
 +    dp = tracks
 +    for d in bits [0:-1]:
 +        dd = "%s/%s" % (dp,  d)
 +        if dp not in dirs_by_dir:
 +            dirs_by_dir[dp] = []
 +        if dd not in dirs_by_dir[dp]:
 +            dirs_by_dir[dp].append(dd)
 +        dp = "%s/%s" % (dp, d)
 +    if dp not in files_by_dir:
 +        files_by_dir[dp] = []
 +    files_by_dir[dp].append("%s/%s" % (dp, bits[-1]))
  
  def stdtracks():
 -    maketrack("Joe Bloggs/First Album/01:First track.ogg")
 +    # We create some tracks with non-ASCII characters in the name and
 +    # we (currently) force UTF-8.
 +    #
 +    # On a traditional UNIX filesystem, that treats filenames as byte strings
 +    # with special significant for '/', this should just work, though the
 +    # names will look wrong to ls(1) in a non UTF-8 locale.
 +    #
 +    # On Apple HFS+ filenames normalized to a decomposed form that isn't quite
 +    # NFD, so our attempts to have both normalized and denormalized filenames
 +    # is frustrated.  Provided we test on traditional filesytsems too this
 +    # shouldn't be a problem.
 +    # (See http://developer.apple.com/qa/qa2001/qa1173.html)
 +
 +    global dirs_by_dir, files_by_dir
 +    dirs_by_dir={}
 +    files_by_dir={}
 +    
 +    # C3 8C = 00CC LATIN CAPITAL LETTER I WITH GRAVE
 +    # (in NFC)
 +    maketrack("Joe Bloggs/First Album/01:F\xC3\x8Crst track.ogg")
 +
      maketrack("Joe Bloggs/First Album/02:Second track.ogg")
 -    maketrack("Joe Bloggs/First Album/03:Third track.ogg")
 +
 +    # CC 81 = 0301 COMBINING ACUTE ACCENT
 +    # (giving an NFD i-acute)
 +    maketrack("Joe Bloggs/First Album/03:ThI\xCC\x81rd track.ogg")
 +    # ...hopefuly giving C3 8D = 00CD LATIN CAPITAL LETTER I WITH ACUTE
      maketrack("Joe Bloggs/First Album/04:Fourth track.ogg")
      maketrack("Joe Bloggs/First Album/05:Fifth track.ogg")
 -    maketrack("Joe Bloggs/First Album/05:Fifth track.ogg")
      maketrack("Joe Bloggs/Second Album/01:First track.ogg")
      maketrack("Joe Bloggs/Second Album/02:Second track.ogg")
      maketrack("Joe Bloggs/Second Album/03:Third track.ogg")
  def notracks():
      pass
  
 -def start(test):
 -    """start(TEST)
 -
 -Start the daemon for test called TEST."""
 -    global daemon
 -    assert daemon == None
 -    if test == None:
 -        errs = sys.stderr
 -    else:
 -        errs = open("%s/%s.log" % (testroot, test), "w")
 +def common_setup():
 +    remove_dir(testroot)
 +    os.mkdir(testroot)
 +    open("%s/config" % testroot, "w").write(
 +    """player *.ogg shell 'echo "$TRACK" >> %s/played.log'
 +home %s
 +collection fs UTF-8 %s/tracks
 +scratch %s/scratch.ogg
 +gap 0
 +stopword 01 02 03 04 05 06 07 08 09 10
 +stopword 1 2 3 4 5 6 7 8 9
 +stopword 11 12 13 14 15 16 17 18 19 20
 +stopword 21 22 23 24 25 26 27 28 29 30
 +stopword the a an and to too in on of we i am as im for is
 +username fred
 +password fredpass
 +allow fred fredpass
 +plugins ../plugins
 +player *.mp3 execraw disorder-decode
 +player *.ogg execraw disorder-decode
 +player *.wav execraw disorder-decode
 +player *.flac execraw disorder-decode
 +tracklength *.mp3 disorder-tracklength
 +tracklength *.ogg disorder-tracklength
 +tracklength *.wav disorder-tracklength
 +tracklength *.flac disorder-tracklength
 +""" % (testroot, testroot, testroot, testroot))
-     copyfile("%s/sounds/scratch.ogg" % topsrcdir,
++    copyfile("%s/sounds/scratch.ogg" % top_srcdir,
 +             "%s/scratch.ogg" % testroot)
 +
 +def start_daemon():
 +    """start_daemon()
 +Start the daemon."""
 +    global daemon,errs
 +    assert daemon is None
      server = None
      print " starting daemon"
      daemon = subprocess.Popen(["disorderd",
@@@ -147,11 -142,8 +204,11 @@@ def run(test, setup=None, report=True, 
      tests += 1
      if setup == None:
          setup = stdtracks
-     errs = open("%s.log" % test.__name__, "w") # HNGGGH.  nO.
++    errs = open("%s.log" % test.__name__, "w")
 +    disorder._configfile = "%s/config" % testroot
 +    disorder._userconf = False
 +    common_setup()
      setup()
 -    start(name)
      try:
          try:
              test()
@@@ -187,6 -178,32 +244,5 @@@ Recursively delete directory D""
  tests = 0
  failures = 0
  daemon = None
- testroot = "%s/testroot" % os.getcwd()
+ testroot = "%s/tests/testroot" % top_builddir
 -remove_dir(testroot)
 -os.mkdir(testroot)
 -open("%s/config" % testroot, "w").write(
 -"""player *.ogg shell 'echo "$TRACK" >> %s/played.log'
 -home %s
 -collection fs ASCII %s/tracks
 -scratch %s/scratch.ogg
 -gap 0
 -stopword 01 02 03 04 05 06 07 08 09 10
 -stopword 1 2 3 4 5 6 7 8 9
 -stopword 11 12 13 14 15 16 17 18 19 20
 -stopword 21 22 23 24 25 26 27 28 29 30
 -stopword the a an and to too in on of we i am as im for is
 -username fred
 -password fredpass
 -allow fred fredpass
 -plugins %s/plugins
 -player *.mp3 execraw disorder-decode
 -player *.ogg execraw disorder-decode
 -player *.wav execraw disorder-decode
 -player *.flac execraw disorder-decode
 -tracklength *.mp3 disorder-tracklength
 -tracklength *.ogg disorder-tracklength
 -tracklength *.wav disorder-tracklength
 -tracklength *.flac disorder-tracklength
 -""" % (testroot, testroot, testroot, testroot, top_builddir))
 -copyfile("%s/sounds/scratch.ogg" % top_srcdir,
 -         "%s/scratch.ogg" % testroot)
 +tracks = "%s/tracks" % testroot
- topsrcdir = os.path.abspath(os.getenv("topsrcdir"))
diff --cc tests/files.py
index e0237d4,0000000..2f636ec
mode 100755,000000..100755
--- /dev/null
@@@ -1,37 -1,0 +1,56 @@@
 +#! /usr/bin/env python
++#
++# This file is part of DisOrder.
++# Copyright (C) 2007 Richard Kettlewell
++#
++# This program is free software; you can redistribute it and/or modify
++# it under the terms of the GNU General Public License as published by
++# the Free Software Foundation; either version 2 of the License, or
++# (at your option) any later version.
++#
++# This program is distributed in the hope that it will be useful, but
++# WITHOUT ANY WARRANTY; without even the implied warranty of
++# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
++# General Public License for more details.
++#
++# You should have received a copy of the GNU General Public License
++# along with this program; if not, write to the Free Software
++# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307
++# USA
++#
 +import dtest,time,disorder,sys
 +
 +def test():
 +    """Check that the file listing comes out right"""
 +    dtest.start_daemon()
 +    time.sleep(5)                       # give rescan a chance
 +    c = disorder.client()
 +    failures = 0
 +    for d in dtest.dirs_by_dir:
 +        xdirs = dtest.dirs_by_dir[d]
 +        dirs = c.directories(d)
 +        xdirs.sort()
 +        dirs.sort()
 +        if dirs != xdirs:
 +            print
 +            print "directory: %s" % d
 +            print "expected:  %s" % xdirs
 +            print "got:       %s" % dirs
 +            failures += 1
 +    for d in dtest.files_by_dir:
 +        xfiles = dtest.files_by_dir[d]
 +        files = c.files(d)
 +        xfiles.sort()
 +        files.sort()
 +        if files != xfiles:
 +            print
 +            print "directory: %s" % d
 +            print "expected:  %s" % xfiles
 +            print "got:       %s" % files
 +            failures += 1
 +    if failures:
 +        print
 +        sys.exit(1)
 +
 +if __name__ == '__main__':
 +    dtest.run(test)
Simple merge
Simple merge