My unit tests caught the following bug in "Members" property of SqlAzManApplicationGroup. It's only in version 18.104.22.168. I noticed that you have made some changes to this property but I couldn't figure out why.
Because "this.members" variable is now initialized in the constructor, so the following if statement is always false:
if (this.members == null && this.GroupType == GroupType.Basic)
//// MOD: Dictionary creation moved from here fto constructor to handle LDAP Group correctly.
//// OLD: this.members = new Dictionary<IAzManSid, IAzManApplicationGroupMember>();
foreach (IAzManApplicationGroupMember m in this.GetApplicationGroupAllMembers())
Therefor, "Members" property will always return an empty collection. Even if somehow "this.members" is set to null before coming into the function, an exception will be thrown at this.members.Add.
I can't figure out why the change was made, so I can't fix it. Also, I don't know if we had similar changes in other properties.