FileStore

Merge lp://staging/~jderose/filestore/stat into lp://staging/filestore

  1. stat
  2. Merge into trunk
Proposed by Jason Gerard DeRose
Status: Merged
Merged at revision: 203
Proposed branch: lp://staging/~jderose/filestore/stat
Merge into: lp://staging/filestore
Diff against target: 411 lines (+253/-36)
2 files modified
filestore.py (+71/-11)
test_filestore.py (+182/-25)
To merge this branch: bzr merge lp://staging/~jderose/filestore/stat
Reviewer Review Type Date Requested Status
Jason Gerard DeRose Approve
Review via email: mp+75457@code.staging.launchpad.net

Description of the change

This merge:

* Adds `Stat` namedtuple

* Adds `FileNotFound` exception

* Adds strict `FileStore.stat()` method

* Updates `FileStore.exists()` to test using `FileStore.stat()`

* Makes `FileStore.__iter__()` very strict and changes it to yield `Stat` namedtuple instead of IDs

I'll update the rst docs in a separate merge.

To post a comment you must log in.
lp://staging/~jderose/filestore/stat updated
210. By Jason Gerard DeRose

Fixed typo in comment

211. By Jason Gerard DeRose

typo: to provided => to provide

Revision history for this message
Jason Gerard DeRose (jderose) wrote :

As the schedule is tight for the dmedia core refactor, I'm going to self approve this.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches