Re: Review Request 45434: Added test cases for '/files' endpoint authentication.

2016-03-30 Thread Jan Schlicht
> On March 30, 2016, 2:38 a.m., Greg Mann wrote: > > BenM and I discussed this while I was working on the previous patches: > > since the code paths for the authenticated and unauthenticated cases are > > the same once the handler is reached, it shouldn't be necessary to > > explicitly test

Re: Review Request 45434: Added test cases for '/files' endpoint authentication.

2016-03-29 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45434/#review125964 --- Patch looks great! Reviews applied: [45434] Passed command:

Review Request 45434: Added test cases for '/files' endpoint authentication.

2016-03-29 Thread Jan Schlicht
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45434/ --- Review request for mesos, Adam B and Greg Mann. Repository: mesos