Changes between Initial Version and Version 1 of Ticket #2432, comment 3


Ignore:
Timestamp:
2015-05-26T23:57:39Z (9 years ago)
Author:
daira
Comment:

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #2432, comment 3

    initial v1  
    1 The one thing I don't like about `mktemp()` is that the path it generates is not deterministic, due to its use of `mkdtemp` ([https://twistedmatrix.com/trac/browser/tags/releases/twisted-15.2.0/twisted/trial/_synctest.py#L1221 implementation in Twisted v15.2.0]). What I actually want is `os.path.dirname(os.path.dirname(self.mktemp())`, I think. So, maybe it would be better to have a mixin to shorten that, and so that if we want to change it, we can do so in only one place.
     1The one thing I don't like about `mktemp()` is that the path it generates is not deterministic, due to its use of `mkdtemp` ([https://twistedmatrix.com/trac/browser/tags/releases/twisted-15.2.0/twisted/trial/_synctest.py#L1221 implementation in Twisted v15.2.0]). What I actually want is `os.path.dirname(os.path.dirname(self.mktemp())`, I think. So, maybe it would be better to have a mixin to shorten that, so that if we want to change it, we only need to do so in one place.