Increase Unit test code coverage

[hokwaichan]

Ticket Link

Groupings-1371

List of squashed commits

• cover exceptions begin convering JsonUtil
• Added test for describeTo
• Added test for toLocalDate
• Added assertDoesNotThrow for functions to handle exception
• Added test for uhCasAttributes default constructor
• Added toString test for Membership class (type)
• Added test for person class subject constructor (type)
• Added test for group class isMember (type)
• Added tests for GroupingsServiceResultException class constructor, getGsr and setGsr (type)
• Added test for GroupingMember
• Added test for GroupingsUpdatedAttributesResult (groupings)
• Added test for GroupingGroupsMembers (groupings)
• Added test for GroupingSyncDestination (groupings)
• Added test for hasUHAttributes (wrapper -> SubjectTest)
• Added test for getBoolean (Wrapper -> ResultsTest)
• Added test for isTimeUpdated (Wrapper -> UpdatedTimestampResult)
• Create AttributeAssignValueResultTest and add tests
• Refactor construction to constructor
• Added more tests for AssignAttributesResultsTest
• Added nullResultCode (wrapper -> UpdatedTimestampResult)
• Added null test for ParentGroupingPath (service → PathFilter)
• Begin covering MemberService
• Cover some test in GroupingAssignmentServiceTest, and MemberServiceTest
• Remove unused functions
• Cover JavaVersionChecker
• Implement AsyncJobsManagerTest and cover (Service)
• FIx import
• Implement ExecutorServiceTest and add tests
• Add addAdminTest
• Implement addOwnershipsTest
• Implement removeAdminTest
• Implement removeOwnershipsTest and addIncludeMembersTest
• Implemented addExcludeMembersTest, removeIncludeMembersTest, removeIncludeMemberTest, removeExcludeMembersTest, removeExcludeMemberTest, optInTest, and optOutTest
• Removed getGroupingTest
• Increase Unit test code coverage
• cover exceptions begin convering JsonUtil
• Fixed conflict
• Removed unused import

Test Checklist

• Exhibits Clear Code Structure:
• Project Unit Tests Passed:
• Project Integration Tests Passed:
• Executes Expected Functionality:
• Tested For Incorrect/Unexpected Inputs:

[mhodgesatuh]

Looks to me like a significant improvement and so I have approved it. Note that there's one issue given our practice of only testing with a fixed set of uids and uhUuids:

src/test/java/edu/hawaii/its/api/access/UhCasAttributesTest.java
219. assertThat(attributes.toString(), containsString("uid=duckart"));

Standard practice is to only use uid’s from
o https://uhawaii.atlassian.net/wiki/spaces/SITARd/pages/12944666/Project+Security+and+Test+Data

[mhodgesatuh]

Codacy reports that the following is unused:
import org.mockito.MockitoAnnotations;