Closed
Bug 762147
Opened 12 years ago
Closed 12 years ago
Functional tests don't exercise X-I-U-S for deletes
Categories
(Cloud Services Graveyard :: Server: Sync, defect)
Cloud Services Graveyard
Server: Sync
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: gps, Assigned: rfkelly)
References
Details
(Whiteboard: [qa+])
Attachments
(1 file, 1 obsolete file)
3.40 KB,
patch
|
telliott
:
review+
|
Details | Diff | Splinter Review |
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?).
Updated•12 years ago
|
Whiteboard: [qa+]
Assignee | ||
Comment 1•12 years ago
|
||
Indeed, the tests for X-If-Unmodified-Since were pretty minimal. Patch adds more.
Assignee: nobody → rfkelly
Attachment #630803 -
Flags: review?(telliott)
Reporter | ||
Comment 2•12 years ago
|
||
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).
Updated•12 years ago
|
Attachment #630803 -
Flags: review?(telliott) → review+
Assignee | ||
Comment 3•12 years ago
|
||
#1 is indeed the intended interpretation, will add another test for this case.
Assignee | ||
Comment 4•12 years ago
|
||
updated patch attached.
Attachment #630803 -
Attachment is obsolete: true
Attachment #631262 -
Flags: review?(telliott)
Updated•12 years ago
|
Attachment #631262 -
Flags: review?(telliott) → review+
Assignee | ||
Comment 5•12 years ago
|
||
https://github.com/mozilla-services/server-syncstorage/commit/2301b198b8d6d651da615df48a98682b6f3e7d32
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Updated•1 year ago
|
Product: Cloud Services → Cloud Services Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•