Small improvement, removing magic number.

Author: azchohfi

Microsoft.Toolkit.Services/Services/LinkedIn/LinkedInDataProvider.cs

@@ -81,8 +81,8 @@ public LinkedInDataProvider(LinkedInOAuthTokens tokens, LinkedInPermissions requ
                 throw new ArgumentException("Missing callback uri");
             }
 
-            // Check if outside the bounds of the LinkedInPermissions Enum
-            if (requiredPermissions < 0 || requiredPermissions > (LinkedInPermissions)15)
+            // Check if its a valid combination of LinkedInPermissions
+            if ((~(int)LinkedInPermissionsHelpers.AllPermissions & (int)requiredPermissions) == 0)
             {
                 throw new ArgumentException("Error retrieving required permissions");
             }

Microsoft.Toolkit.Services/Services/LinkedIn/LinkedInPermissions.cs

@@ -37,4 +37,18 @@ public enum LinkedInPermissions
         /// </summary>
         WriteShare = 8
     }
+
+#pragma warning disable SA1649 // File name should match first type name
+    internal static class LinkedInPermissionsHelpers
+    {
+        /// <summary>
+        /// Internal AllPermissions for LinkedInPermissions, so we don't expose it. Keep it in sync with <see cref="LinkedInPermissions"/>
+        /// </summary>
+        internal const LinkedInPermissions AllPermissions =
+            LinkedInPermissions.ReadBasicProfile |
+            LinkedInPermissions.ReadEmailAddress |
+            LinkedInPermissions.ReadWriteCompanyAdmin |
+            LinkedInPermissions.WriteShare;
+    }
+#pragma warning restore SA1649 // File name should match first type name
 }