zeitgeist team mailing list archive

["J.P. Lacerda" <jpaflacerda@xxxxxxxxx>]

Public bug reported:

21:43 < RainCT> jplacerda: So there's a problem with the test system. It's supposed to set ZEITGEIST_DATA_PATH to a temporary directory, but it's only doing so for doctests (of which we don't have any anymore)
21:44 < jplacerda> I was just looking at engine-test and something similar to what I was doing happens there
21:46 < RainCT> jplacerda: If you're friends with unittests, feel free to figure out how to fix that. I can only think of putting of wrapping it around everything (just like ZEITGEIST_DEFAULT_EXTENSIONS -- but then all 
                tests get the same temp directory) or subclassing unittest.TestCase and having all tests use that; not happy with either option :(
21:46 < RainCT> jplacerda: Yeah, engine-test is also affected. The problem is in run-all-tests.py.

This clearly needs to be fixed :)

** Affects: zeitgeist
     Importance: Undecided
         Status: New

-- 
You received this bug notification because you are a member of Zeitgeist
Framework Team, which is subscribed to Zeitgeist Framework.
https://bugs.launchpad.net/bugs/784850

Title:
  create a default test sandbox

Status in Zeitgeist Framework:
  New

Bug description:
  21:43 < RainCT> jplacerda: So there's a problem with the test system. It's supposed to set ZEITGEIST_DATA_PATH to a temporary directory, but it's only doing so for doctests (of which we don't have any anymore)
  21:44 < jplacerda> I was just looking at engine-test and something similar to what I was doing happens there
  21:46 < RainCT> jplacerda: If you're friends with unittests, feel free to figure out how to fix that. I can only think of putting of wrapping it around everything (just like ZEITGEIST_DEFAULT_EXTENSIONS -- but then all 
                  tests get the same temp directory) or subclassing unittest.TestCase and having all tests use that; not happy with either option :(
  21:46 < RainCT> jplacerda: Yeah, engine-test is also affected. The problem is in run-all-tests.py.

  This clearly needs to be fixed :)