From b23a7fcc10be8095273e173f6e2d4ce00c72cae0 Mon Sep 17 00:00:00 2001
From: david-sarah <david-sarah@jacaranda.org>
Date: Sat, 11 Sep 2010 15:19:13 -0700
Subject: [PATCH] test_runner.py: fix spurious failures of test_path on
 case-insensitive filesystems, and restore the check that the version is not
 'unknown'.

---
 src/allmydata/test/test_runner.py | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)

diff --git a/src/allmydata/test/test_runner.py b/src/allmydata/test/test_runner.py
index 75009cb6..e735fe71 100644
--- a/src/allmydata/test/test_runner.py
+++ b/src/allmydata/test/test_runner.py
@@ -86,6 +86,10 @@ class BinTahoe(common_util.SignalMixin, unittest.TestCase, SkipMixin):
 
             verstr = str(allmydata.__version__)
 
+            self.failIfEqual(verstr, "unknown",
+                             "We don't know our version, because this distribution didn't come "
+                             "with a _version.py and 'setup.py darcsver' hasn't been run.")
+
             # The Python "rational version numbering" convention
             # disallows "-r$REV" but allows ".post$REV"
             # instead. Eventually we'll probably move to that. When we
@@ -101,10 +105,8 @@ class BinTahoe(common_util.SignalMixin, unittest.TestCase, SkipMixin):
                 else:
                     altverstr = verstr
 
-            ad = os.path.dirname(os.path.dirname(os.path.realpath(allmydata.__file__)))
-
-            required_ver_and_path = "%s: %s (%s)" % (allmydata.__appname__, verstr, ad)
-            alt_required_ver_and_path = "%s: %s (%s)" % (allmydata.__appname__, altverstr, ad)
+            required_ver_and_path = "%s: %s (%s)" % (allmydata.__appname__, verstr, srcdir)
+            alt_required_ver_and_path = "%s: %s (%s)" % (allmydata.__appname__, altverstr, srcdir)
 
             self.failUnless(out.startswith(required_ver_and_path) or out.startswith(alt_required_ver_and_path), (out, err, rc_or_sig, required_ver_and_path))
         d.addCallback(_cb)
-- 
2.45.2