[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