[tahoe-lafs-trac-stream] [Tahoe-LAFS] #2432: consistently use self.mktemp() to create a temporary directory in tests
Tahoe-LAFS
trac at tahoe-lafs.org
Tue May 26 16:02:01 UTC 2015
#2432: consistently use self.mktemp() to create a temporary directory in tests
------------------------+---------------------------
Reporter: daira | Owner: daira
Type: defect | Status: new
Priority: normal | Milestone: 1.11.0
Component: code | Version: 1.10.0
Resolution: | Keywords: tests cleanup
Launchpad Bug: |
------------------------+---------------------------
Comment (by warner):
I agree it's a bit of a mess. My only two concerns:
* cleaning up the directory when we're done, so we don't litter /tmp/ (or
wherever) with leftovers
* being able to find the leftover state when something goes wrong
Trial puts everything in `./_trial_temp`, and `rmtree`s it at the
beginning of each test run. This provides a predictable location, and an
upper bound (of size=1) on the litter.
So I guess I'm in favor of using our own mixin that uses the (fully-
qualified) name of the test case to produce a subdirectory of
`_trial_temp`.
--
Ticket URL: <https://tahoe-lafs.org/trac/tahoe-lafs/ticket/2432#comment:1>
Tahoe-LAFS <https://Tahoe-LAFS.org>
secure decentralized storage
More information about the tahoe-lafs-trac-stream
mailing list