Simplified test names

Removed the class name prefix (since it's listed in the class name, and tests can be grouped by class) and removed the "Test" suffix (since these are all test methods in test classes, this should be implicit). We may revisit these again in the future, but for now they're more succinct and easier to read in Visual Studio's Test Explorer.

Author: JeremyCaney

Ignia.Topics.Tests/AttributeValueCollectionTest.cs

@@ -26,7 +26,7 @@ public class AttributeValueCollectionTest {
     ///   Creates a new topic and ensures that the key can be returned as an attribute.
     /// </summary>
     [TestMethod]
-    public void AttributeValueCollection_GetValueTest() {
+    public void GetValue() {
       var topic = TopicFactory.Create("Test", "Container");
       Assert.AreEqual<string>("Test", topic.Attributes.GetValue("Key"));
     }
@@ -38,7 +38,7 @@ public void AttributeValueCollection_GetValueTest() {
     ///   Ensures that integer values can be set and retrieved as expected.
     /// </summary>
     [TestMethod]
-    public void AttributeValueCollection_GetIntegerTest() {
+    public void GetInteger() {
 
       var topic = TopicFactory.Create("Test", "Container");
 
@@ -60,7 +60,7 @@ public void AttributeValueCollection_GetIntegerTest() {
     ///   Ensures that integer values can be set and retrieved as expected.
     /// </summary>
     [TestMethod]
-    public void AttributeValueCollection_GetDateTimeTest() {
+    public void GetDateTime() {
 
       var topic                 = TopicFactory.Create("Test", "Container");
       var dateTime1             = new DateTime(1976, 10, 15);
@@ -84,7 +84,7 @@ public void AttributeValueCollection_GetDateTimeTest() {
     ///   Ensures that boolean values can be set and retrieved as expected.
     /// </summary>
     [TestMethod]
-    public void AttributeValueCollection_GetBooleanTest() {
+    public void GetBoolean() {
 
       var topic = TopicFactory.Create("Test", "Container");
 
@@ -110,7 +110,7 @@ public void AttributeValueCollection_GetBooleanTest() {
     ///   Creates a new topic and requests an invalid attribute; ensures falls back to the default.
     /// </summary>
     [TestMethod]
-    public void AttributeValueCollection_DefaultValueTest() {
+    public void DefaultValue() {
       var topic = TopicFactory.Create("Test", "Container");
       Assert.AreEqual<string>("Foo", topic.Attributes.GetValue("InvalidAttribute", "Foo"));
     }
@@ -122,7 +122,7 @@ public void AttributeValueCollection_DefaultValueTest() {
     ///   Sets a custom attribute on a topic and ensures it can be retrieved.
     /// </summary>
     [TestMethod]
-    public void AttributeValueCollection_SetValueTest() {
+    public void SetValue() {
       var topic = TopicFactory.Create("Test", "Container");
       topic.Attributes.SetValue("Foo", "Bar");
       Assert.AreEqual<string>("Bar", topic.Attributes.GetValue("Foo"));
@@ -135,7 +135,7 @@ public void AttributeValueCollection_SetValueTest() {
     ///   Modifies the value of a custom attribute on a topic and ensures it is marked as IsDirty.
     /// </summary>
     [TestMethod]
-    public void AttributeValueCollection_SetValue_IsDirtyTest() {
+    public void SetValue_IsDirtyTest() {
 
       var topic = TopicFactory.Create("Test", "Container");
 
@@ -159,7 +159,7 @@ public void AttributeValueCollection_SetValue_IsDirtyTest() {
     ///   retrieved.
     /// </summary>
     [TestMethod]
-    public void AttributeValueCollection_SetValue_BackdoorTest() {
+    public void SetValue_BackdoorTest() {
 
       var topic = TopicFactory.Create("Test", "Container");
 
@@ -179,7 +179,7 @@ public void AttributeValueCollection_SetValue_BackdoorTest() {
     /// </summary>
     [TestMethod]
     [ExpectedException(typeof(TargetInvocationException), "The topic allowed a key to be set via a back door, without routing it through the Key property.")]
-    public void AttributeValueCollection_EnforceBusinessLogicTest() {
+    public void EnforceBusinessLogic() {
       var topic = TopicFactory.Create("Test", "Container");
       topic.Attributes.SetValue("Key", "# ?");
     }
@@ -192,7 +192,7 @@ public void AttributeValueCollection_EnforceBusinessLogicTest() {
     /// </summary>
     [TestMethod]
     [ExpectedException(typeof(TargetInvocationException), "The topic allowed a key to be set via a back door, without routing it through the Key property.")]
-    public void AttributeValueCollection_EnforceBusinessLogic_BackdoorTest() {
+    public void EnforceBusinessLogic_Backdoor() {
       var topic = TopicFactory.Create("Test", "Container");
       topic.Attributes.Remove("Key");
       topic.Attributes.Add(new AttributeValue("Key", "# ?"));
@@ -205,7 +205,7 @@ public void AttributeValueCollection_EnforceBusinessLogic_BackdoorTest() {
     ///   Sets an attribute on the parent of a topic and ensures it can be retrieved using inheritance.
     /// </summary>
     [TestMethod]
-    public void AttributeValueCollection_InheritFromParentTest() {
+    public void InheritFromParent() {
 
       var topics = new Topic[8];
 
@@ -230,7 +230,7 @@ public void AttributeValueCollection_InheritFromParentTest() {
     ///   Establishes a long tree of derives topics, and ensures that inheritance will pursue no more than five hops.
     /// </summary>
     [TestMethod]
-    public void AttributeValueCollection_MaxHopsTest() {
+    public void MaxHops() {
 
       var topics = new Topic[8];
 

Ignia.Topics.Tests/ITopicRepositoryTest.cs

@@ -54,7 +54,7 @@ public ITopicRepositoryTest() {
     ///   Loads topics and ensures there are the expected number of children.
     /// </summary>
     [TestMethod]
-    public void ITopicRepository_LoadTest() {
+    public void Load() {
 
       var rootTopic             = _topicRepository.Load();
       var topic                 = _topicRepository.Load("Root:Configuration:ContentTypes:Page");
@@ -73,7 +73,7 @@ public void ITopicRepository_LoadTest() {
     ///   Loads topic by ID and ensures it is found.
     /// </summary>
     [TestMethod]
-    public void ITopicRepository_LoadByIdTest() {
+    public void LoadById() {
 
       var topic                 = _topicRepository.Load(11111);
 
@@ -89,7 +89,7 @@ public void ITopicRepository_LoadByIdTest() {
     ///   Saves topics and ensures their identifiers are properly set.
     /// </summary>
     [TestMethod]
-    public void ITopicRepository_SaveTest() {
+    public void Save() {
 
       var rootTopic             = _topicRepository.Load();
       var web                   = _topicRepository.Load("Root:Web");
@@ -116,7 +116,7 @@ public void ITopicRepository_SaveTest() {
     ///   Moves topics and ensures their parents are correctly set.
     /// </summary>
     [TestMethod]
-    public void ITopicRepository_MoveTest() {
+    public void Move() {
 
       var rootTopic             = _topicRepository.Load();
       var source                = _topicRepository.Load("Root:Web:Web_0");
@@ -142,7 +142,7 @@ public void ITopicRepository_MoveTest() {
     ///   Moves topic next to a different sibling and ensures it ends up in the correct location.
     /// </summary>
     [TestMethod]
-    public void ITopicRepository_MoveToSiblingTest() {
+    public void MoveToSibling() {
 
       var rootTopic             = _topicRepository.Load();
       var parent                = _topicRepository.Load("Root:Web:Web_0");
@@ -169,7 +169,7 @@ public void ITopicRepository_MoveToSiblingTest() {
     ///   Deletes a topic to ensure it is properly removed.
     /// </summary>
     [TestMethod]
-    public void ITopicRepository_DeleteTest() {
+    public void Delete() {
 
       var parent                = _topicRepository.Load("Root:Web:Web_1");
       var topic                 = _topicRepository.Load("Root:Web:Web_1:Web_1_1");

Ignia.Topics.Tests/Properties/AssemblyInfo.cs

@@ -21,7 +21,7 @@
 [assembly: AssemblyTrademark("")]
 [assembly: AssemblyCulture("")]
 [assembly: ComVisible(false)]
-[assembly: AssemblyVersion("3.5.1762.0")]
-[assembly: AssemblyFileVersion("3.5.1810.0")]
+[assembly: AssemblyVersion("3.5.1763.0")]
+[assembly: AssemblyFileVersion("3.5.1811.0")]
 [assembly: CLSCompliant(true)]
 [assembly: Guid("27632801-bfe3-41d9-8678-3c4bbe45e6c9")]

Ignia.Topics.Tests/RelatedTopicCollectionTest.cs

@@ -26,7 +26,7 @@ public class RelatedTopicCollectionTest {
     ///   Sets a relationship and confirms that it is accessible.
     /// </summary>
     [TestMethod]
-    public void RelatedTopicCollection_SetTopic() {
+    public void SetTopic() {
 
       var parent                = TopicFactory.Create("Parent", "Page");
       var related               = TopicFactory.Create("Related", "Page");
@@ -44,7 +44,7 @@ public void RelatedTopicCollection_SetTopic() {
     ///   Sets a relationship and confirms that it is accessible on incoming relationships property.
     /// </summary>
     [TestMethod]
-    public void RelatedTopicCollection_IncomingRelationshipTest() {
+    public void IncomingRelationship() {
 
       var parent                = TopicFactory.Create("Parent", "Page");
       var related               = TopicFactory.Create("Related", "Page");
@@ -63,7 +63,7 @@ public void RelatedTopicCollection_IncomingRelationshipTest() {
     ///   Sets relationships in multiple namespaces, and the correct number of keys are returned.
     /// </summary>
     [TestMethod]
-    public void RelatedTopicCollection_KeysTest() {
+    public void Keys() {
 
       var parent                = TopicFactory.Create("Parent", "Page");
       var relationships         = new RelatedTopicCollection(parent);
@@ -84,7 +84,7 @@ public void RelatedTopicCollection_KeysTest() {
     ///   Sets relationships in multiple namespaces, and ensures they are all returned via GetAllTopics().
     /// </summary>
     [TestMethod]
-    public void RelatedTopicCollection_GetAllTopicsTest() {
+    public void GetAllTopics() {
 
       var parent                = TopicFactory.Create("Parent", "Page");
       var relationships         = new RelatedTopicCollection(parent);
@@ -107,7 +107,7 @@ public void RelatedTopicCollection_GetAllTopicsTest() {
     ///   <see cref="RelatedTopicCollection.GetAllTopics(String)"/> by content type.
     /// </summary>
     [TestMethod]
-    public void RelatedTopicCollection_GetAllContentTypesTest() {
+    public void GetAllContentTypes() {
 
       var parent                = TopicFactory.Create("Parent", "Page");
       var relationships         = new RelatedTopicCollection(parent);

Ignia.Topics.Tests/TopicCollectionTest.cs

@@ -26,7 +26,7 @@ public class TopicCollectionTest {
     ///   Establishes a number of topics, then accesses them by key.
     /// </summary>
     [TestMethod]
-    public void TopicCollection_SetTopicTest() {
+    public void SetTopic() {
 
       var topics = new TopicCollection();
 
@@ -45,7 +45,7 @@ public void TopicCollection_SetTopicTest() {
     ///   Establishes a number of topics, then seeds a new <see cref="TopicCollection{T}"/> with them.
     /// </summary>
     [TestMethod]
-    public void TopicCollection_PrepopulateTest() {
+    public void Prepopulate() {
 
       var topics = new List<Topic>();
 
@@ -66,7 +66,7 @@ public void TopicCollection_PrepopulateTest() {
     ///   Establishes a number of topics, converts the collection to read only, and ensures they are still present.
     /// </summary>
     [TestMethod]
-    public void TopicCollection_AsReadOnlyTest() {
+    public void AsReadOnly() {
 
       var topics = new TopicCollection();
 

Ignia.Topics.Tests/TopicControllerTest.cs

@@ -60,10 +60,10 @@ public TopicControllerTest() {
     | TEST: TOPIC
     \-------------------------------------------------------------------------------------------------------------------------*/
     /// <summary>
-    ///   Triggers the <see cref="TopicController.IndexAsync(string)" /> action.
+    ///   Triggers the <see cref="TopicController.IndexAsync(String)" /> action.
     /// </summary>
     [TestMethod]
-    public async Task TopicController_IndexTestAsync() {
+    public async Task TopicController_IndexAsync() {
 
       var topicRoutingService   = new MvcTopicRoutingService(_topicRepository, _uri, _routeData);
       var mappingService        = new TopicMappingService(_topicRepository, new FakeViewModelLookupService());
@@ -81,10 +81,10 @@ public async Task TopicController_IndexTestAsync() {
     | TEST: ERROR
     \-------------------------------------------------------------------------------------------------------------------------*/
     /// <summary>
-    ///   Triggers the <see cref="ErrorControllerBase{T}.Error(string)" /> action.
+    ///   Triggers the <see cref="ErrorControllerBase{T}.Error(String)" /> action.
     /// </summary>
     [TestMethod]
-    public void ErrorController_ErrorTest() {
+    public void ErrorController_Error() {
 
       var controller            = new ErrorController();
       var result                = controller.Error("ErrorPage") as ViewResult;
@@ -99,10 +99,10 @@ public void ErrorController_ErrorTest() {
     | TEST: NOT FOUND ERROR
     \-------------------------------------------------------------------------------------------------------------------------*/
     /// <summary>
-    ///   Triggers the <see cref="ErrorControllerBase{T}.NotFound(string)" /> action.
+    ///   Triggers the <see cref="ErrorControllerBase{T}.NotFound(String)" /> action.
     /// </summary>
     [TestMethod]
-    public void ErrorController_NotFoundTest() {
+    public void ErrorController_NotFound() {
 
       var controller            = new ErrorController();
       var result                = controller.Error("NotFoundPage") as ViewResult;
@@ -117,10 +117,10 @@ public void ErrorController_NotFoundTest() {
     | TEST: INTERNAL SERVER ERROR
     \-------------------------------------------------------------------------------------------------------------------------*/
     /// <summary>
-    ///   Triggers the <see cref="ErrorControllerBase{T}.InternalServer(string)" /> action.
+    ///   Triggers the <see cref="ErrorControllerBase{T}.InternalServer(String)" /> action.
     /// </summary>
     [TestMethod]
-    public void ErrorController_InternalServerTest() {
+    public void ErrorController_InternalServer() {
 
       var controller            = new ErrorController();
       var result                = controller.Error("InternalServer") as ViewResult;
@@ -138,7 +138,7 @@ public void ErrorController_InternalServerTest() {
     ///   Triggers the <see cref="FallbackController.Index()" /> action.
     /// </summary>
     [TestMethod]
-    public void FallbackController_IndexTest() {
+    public void FallbackController_Index() {
 
       var controller            = new FallbackController();
       var result                = controller.Index() as HttpNotFoundResult;
@@ -156,7 +156,7 @@ public void FallbackController_IndexTest() {
     ///   Triggers the <see cref="FallbackController.Index()" /> action.
     /// </summary>
     [TestMethod]
-    public void RedirectController_TopicRedirectTest() {
+    public void RedirectController_TopicRedirect() {
 
       var controller            = new RedirectController(_topicRepository);
       var result                = controller.Redirect(11110) as RedirectResult;
@@ -180,7 +180,7 @@ public void RedirectController_TopicRedirectTest() {
     /// </remarks>
     [TestMethod]
     [ExpectedException(typeof(NullReferenceException), AllowDerivedTypes=false)]
-    public void SitemapController_IndexTest() {
+    public void SitemapController_Index() {
 
       var controller            = new SitemapController(_topicRepository);
       var result                = controller.Index() as ViewResult;
@@ -200,7 +200,7 @@ public void SitemapController_IndexTest() {
     ///   Triggers the <see cref="FallbackController.Index()" /> action.
     /// </summary>
     [TestMethod]
-    public async Task LayoutController_MenuTest() {
+    public async Task Menu() {
 
       var topicRoutingService   = new MvcTopicRoutingService(_topicRepository, _uri, _routeData);
       var mappingService        = new TopicMappingService(_topicRepository, new FakeViewModelLookupService());