Add unique index on role name

resolves g0t4#3

Author: g0t4

src/AspNet.Identity.MongoDB/IdentityContext.cs

@@ -17,21 +17,29 @@ public IdentityContext(MongoCollection users)
 		public IdentityContext(MongoCollection users, MongoCollection roles) : this(users)
 		{
 			Roles = roles;
+			EnsureUniqueIndexOnRoleName(roles);
 		}
 
 		private void EnsureUniqueIndexOnUserName(MongoCollection users)
 		{
-			var userName = new IndexKeysBuilder().Ascending("UserName");
+			var userName = new IndexKeysBuilder<IdentityUser>().Ascending(t => t.UserName);
 			var unique = new IndexOptionsBuilder().SetUnique(true);
-			users.EnsureIndex(userName, unique);
+			users.CreateIndex(userName, unique);
+		}
+
+		private void EnsureUniqueIndexOnRoleName(MongoCollection roles)
+		{
+			var roleName = new IndexKeysBuilder<IdentityRole>().Ascending(t => t.Name);
+			var unique = new IndexOptionsBuilder().SetUnique(true);
+			roles.CreateIndex(roleName, unique);
 		}
 
 		public void EnsureUniqueIndexOnEmail()
 		{
 			// note: I'm not making the index on email required, I'd like to start a conversation around how to ensure indexes
-			var email = new IndexKeysBuilder().Ascending("Email");
+			var email = new IndexKeysBuilder<IdentityUser>().Ascending(t => t.Email);
 			var unique = new IndexOptionsBuilder().SetUnique(true);
-			Users.EnsureIndex(email, unique);
+			Users.CreateIndex(email, unique);
 		}
 	}
 }

src/IntegrationTests/IdentityContextTests.cs

@@ -10,12 +10,12 @@ public class IdentityContextTests : UserIntegrationTestsBase
 		[Test]
 		public void Create_NoIndexOnUserName_AddsUniqueIndexOnUserName()
 		{
-			var collectionName = "userindextest";
-			Database.DropCollection(collectionName);
-			var collection = Database.GetCollection(collectionName);
-			new IdentityContext(collection);
+			var userCollectionName = "userindextest";
+			Database.DropCollection(userCollectionName);
+			var users = Database.GetCollection(userCollectionName);
+			new IdentityContext(users);
 
-			var index = collection.GetIndexes()
+			var index = users.GetIndexes()
 				.Where(i => i.IsUnique)
 				.Where(i => i.Key.Count() == 1)
 				.First(i => i.Key.Contains("UserName"));
@@ -25,16 +25,32 @@ public void Create_NoIndexOnUserName_AddsUniqueIndexOnUserName()
 		[Test]
 		public void CreateEmailUniqueIndex_NoIndexOnEmail_AddsUniqueIndexOnEmail()
 		{
-			var collectionName = "userindextest";
-			Database.DropCollection(collectionName);
-			var collection = Database.GetCollection(collectionName);
-			new IdentityContext(collection).EnsureUniqueIndexOnEmail();
+			var userCollectionName = "userindextest";
+			Database.DropCollection(userCollectionName);
+			var users = Database.GetCollection(userCollectionName);
+			new IdentityContext(users).EnsureUniqueIndexOnEmail();
 
-			var index = collection.GetIndexes()
+			var index = users.GetIndexes()
 				.Where(i => i.IsUnique)
 				.Where(i => i.Key.Count() == 1)
 				.First(i => i.Key.Contains("Email"));
 			Expect(index.Key.Count(), Is.EqualTo(1));
 		}
+
+		[Test]
+		public void Create_NoIndexOnRoleName_AddsUniqueIndexOnRoleName()
+		{
+			var roleCollectionName = "roleindextest";
+			Database.DropCollection(roleCollectionName);
+			var roles = Database.GetCollection(roleCollectionName);
+			var users = Database.GetCollection("users");
+			new IdentityContext(users, roles);
+
+			var index = roles.GetIndexes()
+				.Where(i => i.IsUnique)
+				.Where(i => i.Key.Count() == 1)
+				.First(i => i.Key.Contains("Name"));
+			Expect(index.Key.Count(), Is.EqualTo(1));
+		}
 	}
 }