projects / arvados.git / commit
summary | shortlog | log | commit | commitdiff | tree
(parent: fb460bd) | patch
2769: code review comments
authorTim Pierce <twp@curoverse.com>
Fri, 1 Aug 2014 21:03:28 +0000 (17:03 -0400)
committerTim Pierce <twp@curoverse.com>
Fri, 1 Aug 2014 21:03:28 +0000 (17:03 -0400)
commitfe5556e0f3105f2d8b752d796fea21363a1b0eb6
treea85f4c542da78d5cbc4c21618a113b91bf19caf8tree | snapshot
parentfb460bdf4b45dd9d8b46951a1457a301bd565430commit | diff
2769: code review comments

* DeleteHandler returns http.StatusNotFound when no blocks could be found at all,
  and http.StatusMethodNotAllowed when all delete attempts
  fail (e.g. read-only volumes)

* Unit test cleanup: "http://localhost:25107" is not actually necessary
  to build a http.Request, and it is confusing, so leave it out.

Refs #2769.
services/keep/src/keep/handler_test.go diff | blob | history
services/keep/src/keep/handlers.go diff | blob | history