> On Dec. 14, 2015, 5:18 p.m., Alexander Rukletsov wrote: > > I would suggest to add some more test cases: > > - Request does not contain a principal (in absence of authz quota can be > > set); > > - Request does not contain a principal, ACLs allow `ANY` to set quota for > > role "prod" (absence of principal maps to `ANY`).
The new test fixture `AuthorizedQuotaSetRequestWithoutPrincipal` now tests for authorization with an empty principal if authentication is disabled. `AuthorizationTest.SetQuota` now includes a rule where `ANY` can set quota for role "test" and tests that rule. - Jan ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39520/#review110229 ----------------------------------------------------------- On Dec. 18, 2015, 10:12 a.m., Jan Schlicht wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/39520/ > ----------------------------------------------------------- > > (Updated Dec. 18, 2015, 10:12 a.m.) > > > Review request for mesos, Alexander Rukletsov and Till Toenshoff. > > > Bugs: MESOS-4082 > https://issues.apache.org/jira/browse/MESOS-4082 > > > Repository: mesos > > > Description > ------- > > Quota: Added authentication, authorization tests. > > > Diffs > ----- > > src/tests/authorization_tests.cpp dcf348acc75b143e3a854bde17e6042dc90599d5 > src/tests/master_quota_tests.cpp 0473869783a714766ed26fff61d7f8c56342df74 > > Diff: https://reviews.apache.org/r/39520/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Jan Schlicht > >
