Launchpad itself

Code review comment for lp:~mbp/launchpad/882324-mtime-test

  1. 882324-mtime-test
  2. Merge into devel
Revision history for this message
Robert Collins (lifeless) wrote :

That looks ok, though personally I would ditch the assertTrue for
assertThat(
    floor(os.path.getmtime(fname)), Or(LessThan(now), Equals(now)), "fname")

review: Approve

« Back to merge proposal