From: dbs <dbs@9efc2488-bf62-4759-914b-345cdb29e865>
Date: Wed, 2 Mar 2011 14:24:33 +0000 (+0000)
Subject: Fix reversed logic in test for Perl tests. Oh the irony.
X-Git-Url: https://old-git.evergreen-ils.org/?a=commitdiff_plain;h=6076a04bb6ad638131cc19e9124975a5bb0931b1;p=working%2FOpenSRF.git

Fix reversed logic in test for Perl tests. Oh the irony.


git-svn-id: svn://svn.open-ils.org/OpenSRF/trunk@2193 9efc2488-bf62-4759-914b-345cdb29e865
---

diff --git a/examples/buildbot.cfg b/examples/buildbot.cfg
index 48cc067..252e2f4 100644
--- a/examples/buildbot.cfg
+++ b/examples/buildbot.cfg
@@ -187,20 +187,22 @@ eg_factory.addStep(shell.Configure())
 # compile the code
 eg_factory.addStep(shell.Compile(command=["make"]))
 
+perldir = 'build/Open-ILS/src/perlmods'
 class PerlModuleTestMFHDMadness(shell.PerlModuleTest):
     'Override PerlModuleTest with nonstandard library location for testlib.pm'
     command = ['prove', '--lib', 'lib', '-I', 'lib/OpenILS/Utils/MFHD/test', '-r', 't']
     total = 0
 
 def has_perl_unit_tests(step):
-    if (os.path.exists(os.path.join('build/Open-ILS/src/perlmods', 'lib/OpenILS/Utils/MFHD/test'))):
-        return False
-    return True
+    'If there is a /lib/ sudir in the Perl dir, there are tests'
+    if (os.path.exists(os.path.join(perldir, 'lib'))):
+        return True
+    return False
 
 # run the Perl unit tests
 eg_factory.addStep(PerlModuleTestMFHDMadness(
     doStepIf=has_perl_unit_tests,
-    workdir="build/Open-ILS/src/perlmods")
+    workdir=perldir)
 )
 
 # report on the Python code