Bug #2209
closed
- Target version set to 2014-03-26 Debt service and dev painkillers
- Status changed from New to In Progress
Could not reproduce. (Fixed in #2075?)
Adding tests for limit and offset params instead.
- Status changed from In Progress to Resolved
Applied in changeset arvados|commit:8dd84d8f667aa40b9c8ce91d1a9248e1a7218039.
- Status changed from Resolved to New
- Target version changed from 2014-03-26 Debt service and dev painkillers to 2014-04-16 Dev tools and data/resource management
Misha can still reproduce. With limit=10000, server returns ~3000 items but claims ~4000 items_available.
2209-correct-items-available looks like a nice clean branch to me. Just one thing: in the test, the assignment for unique_uuids
doesn't seem to actually ensure uniqueness, as far as I can tell. Should it? That'd be easy to do with a set operator, like unique_uuids = [] | resp['items'].collect { |i| i['uuid'] }.compact
.
Either way, I think this is good to merge.
- Status changed from New to Resolved
- % Done changed from 78 to 100
Applied in changeset arvados|commit:5a19b986fd10f0584b33eb5953e72bd373e487d2.
Also available in: Atom
PDF