On Wed, 2006-03-15 at 09:18 -0800, James M Snell wrote: > > > Joe Gregorio wrote: > >[snip] > > > >> * When an entry is successfully deleted it must be removed from the > >> associated feed. > >> > >> I'm fine with this, but this isn't actually ever stated in the spec. > >> Change it to a should and I'd be happier. > > > > So you believe we need more spec text here? > > Possibly, but I'm not completely sure. Given that the feed returned to > the user may be filtered based on that users read permissions, I'm not > sure if this can be reliably tested.
I'll repeat my suggestion: To simplify and narrow the scope of the test suite (at least initially), we can make an assumption that the user being used for testing is maximally privileged. > Further, if paging is used, you'd > end up having to walk the entire feed in order to verify that it had > actually been removed. In any case, I'm fine with the behavior, and > this is what we've implemented, but I'm not sure how to test or what > should be said about this in the spec (if anything) If the implementation being tested starts out with an empty feed, then adding an Entry should not require paging. It might be a good idea to add other tests that *do* require paging, though. - Michael Bernstein
