Closed Bug 762147 Opened 10 years ago Closed 10 years ago

Functional tests don't exercise X-I-U-S for deletes


(Cloud Services :: Server: Sync, defect)

Not set


(Not tracked)



(Reporter: gps, Assigned: rfkelly)



(Whiteboard: [qa+])


(1 file, 1 obsolete file)

The Storage Service 2.0 spec says the DELETE APIs support X-If-Unmodified-Since. Yet, my JS server implementation doesn't support. Yet, the JS server passes the Python functional test suite! So, by extension, the Python functional test suite doesn't properly exercise this functionality.

So, this bug is a request to add testing of this functionality to the functional test suite.

I'm lazy and didn't verify this myself. If I'm wrong, something weird is going on (maybe those tests are getting skipped when executed against a remote server?).
Whiteboard: [qa+]
Indeed, the tests for X-If-Unmodified-Since were pretty minimal.  Patch adds more.
Assignee: nobody → rfkelly
Attachment #630803 - Flags: review?(telliott)
Blocks: 760466
When I implemented this in the JS server, one of the questions I had was how exactly this is implemented. I see two options for calculating the "server modified time:"

1) It is the newest modified time of all BSOs in the collection
2) It is the newest modified time of the set of BSOs being operated on by the request

The spec seems to indicate #1 with the language "If the collection to be acted on has been modified since the timestamp given, the request will fail." However, I implemented my JS server with #2 and the functional tests still pass!

If #1 is the proper implementation, please expand the functional tests to cover this case. If #2, then we probably need to clarify the spec (and possibly write more functional tests).
Attachment #630803 - Flags: review?(telliott) → review+
#1 is indeed the intended interpretation, will add another test for this case.
updated patch attached.
Attachment #630803 - Attachment is obsolete: true
Attachment #631262 - Flags: review?(telliott)
Attachment #631262 - Flags: review?(telliott) → review+
Closed: 10 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.