Waffle / waffle / 6758 / 1

67%

master: 67%

Pull #3315

github

Fix testAddGroup assertion and remove unused @Mocked parameters

- MockWindowsAuthProviderTest.testAddGroup: fix assertion from 3→2;
  acceptSecurityToken hardcodes {Users, Everyone} and ignores addGroup
- NegotiateSecurityFilterTest: remove unused @Mocked parameters
  (mockProviders on two methods, mockGroup on two methods) flagged by
  CodeQL as useless parameters; update corresponding @param javadoc

Agent-Logs-Url: https://github.com/Waffle/waffle/sessions/c43e1a9c-319d-4476-b4fe-7bdb1effc48f

Co-authored-by: hazendaz <975267+hazendaz@users.noreply.github.com>

420 of 736 branches covered (57.07%)

Branch coverage included in aggregate %.

1496 of 2138 relevant lines covered (69.97%)

1.32 hits per line