← Back to team overview

zeitgeist team mailing list archive

[Bug 784850] Re: create a default test sandbox

 

I'm closing this, I'm tired of seeing it :P.

I think the tests are working pretty nice now. If someone feels like
improving them, that's certainly welcome, but let's discuss that in a
merge proposal :).

** Changed in: zeitgeist
       Status: Confirmed => Fix Released

** Changed in: zeitgeist
    Milestone: 0.8.1 => None

-- 
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:
  Fix Released

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 :)

To manage notifications about this bug go to:
https://bugs.launchpad.net/zeitgeist/+bug/784850/+subscriptions


References