Make constructors internal.

src/Volo.Abp.Identity/Volo/Abp/Identity/IdentityRoleClaim.cs

@@ -33,13 +33,13 @@ namespace Volo.Abp.Identity
 
         }
 
-        public IdentityRoleClaim(Guid id, Guid roleId, [NotNull] Claim claim)
+        protected internal IdentityRoleClaim(Guid id, Guid roleId, [NotNull] Claim claim)
             : this(id, roleId, claim.Type, claim.Value)
         {
 
         }
 
-        public IdentityRoleClaim(Guid id, Guid roleId, [NotNull] string claimType, string claimValue)
+        protected internal IdentityRoleClaim(Guid id, Guid roleId, [NotNull] string claimType, string claimValue)
         {
             Check.NotNull(claimType, nameof(claimType));
 

src/Volo.Abp.Identity/Volo/Abp/Identity/IdentityUserClaim.cs

@@ -33,13 +33,13 @@ namespace Volo.Abp.Identity
             
         }
 
-        public IdentityUserClaim(Guid id, Guid userId, [NotNull] Claim claim)
+        protected internal IdentityUserClaim(Guid id, Guid userId, [NotNull] Claim claim)
             : this(id, userId, claim.Type, claim.Value)
         {
 
         }
 
-        public IdentityUserClaim(Guid id, Guid userId, [NotNull] string claimType, string claimValue)
+        protected internal IdentityUserClaim(Guid id, Guid userId, [NotNull] string claimType, string claimValue)
         {
             Check.NotNull(claimType, nameof(claimType));
 

src/Volo.Abp.Identity/Volo/Abp/Identity/IdentityUserLogin.cs

@@ -5,8 +5,6 @@ using Volo.Abp.Domain.Entities;
 
 namespace Volo.Abp.Identity
 {
-    //TODO: Make constructors internal to ensure that it's called by only from IdentityUser?
-
     /// <summary>
     /// Represents a login and its associated provider for a user.
     /// </summary>
@@ -41,7 +39,7 @@ namespace Volo.Abp.Identity
             
         }
 
-        public IdentityUserLogin(Guid id, Guid userId, [NotNull] string loginProvider, [NotNull] string providerKey, string providerDisplayName)
+        protected internal IdentityUserLogin(Guid id, Guid userId, [NotNull] string loginProvider, [NotNull] string providerKey, string providerDisplayName)
         {
             Check.NotNull(loginProvider, nameof(loginProvider));
             Check.NotNull(providerKey, nameof(providerKey));

src/Volo.Abp.Identity/Volo/Abp/Identity/IdentityUserRole.cs

@@ -23,7 +23,7 @@ namespace Volo.Abp.Identity
             
         }
 
-        public IdentityUserRole(Guid id, Guid userId, Guid roleId)
+        protected internal IdentityUserRole(Guid id, Guid userId, Guid roleId)
         {
             Id = id;
             UserId = userId;

src/Volo.Abp.Identity/Volo/Abp/Identity/IdentityUserToken.cs

@@ -36,7 +36,7 @@ namespace Volo.Abp.Identity
             
         }
 
-        public IdentityUserToken(Guid id, Guid userId, [NotNull] string loginProvider, [NotNull] string name, string value)
+        protected internal IdentityUserToken(Guid id, Guid userId, [NotNull] string loginProvider, [NotNull] string name, string value)
         {
             Check.NotNull(loginProvider, nameof(loginProvider));
             Check.NotNull(name, nameof(name));