add ContextGraphBuilder tests

Author: jaredcnance

src/JsonApiDotNetCore/Builders/ContextGraphBuilder.cs

@@ -23,24 +23,36 @@ public interface IContextGraphBuilder
         /// Add a json:api resource
         /// </summary>
         /// <typeparam name="TResource">The resource model type</typeparam>
-        /// <param name="pluralizedTypeName">The pluralized name that should be exposed by the API</param>
-        IContextGraphBuilder AddResource<TResource>(string pluralizedTypeName) where TResource : class, IIdentifiable<int>;
+        /// <param name="pluralizedTypeName">
+        /// The pluralized name that should be exposed by the API. 
+        /// If nothing is specified, the configured name formatter will be used.
+        /// See <see cref="JsonApiOptions.ResourceNameFormatter" />.
+        /// </param>
+        IContextGraphBuilder AddResource<TResource>(string pluralizedTypeName = null) where TResource : class, IIdentifiable<int>;
 
         /// <summary>
         /// Add a json:api resource
         /// </summary>
         /// <typeparam name="TResource">The resource model type</typeparam>
         /// <typeparam name="TId">The resource model identifier type</typeparam>
-        /// <param name="pluralizedTypeName">The pluralized name that should be exposed by the API</param>
-        IContextGraphBuilder AddResource<TResource, TId>(string pluralizedTypeName) where TResource : class, IIdentifiable<TId>;
+        /// <param name="pluralizedTypeName">
+        /// The pluralized name that should be exposed by the API. 
+        /// If nothing is specified, the configured name formatter will be used.
+        /// See <see cref="JsonApiOptions.ResourceNameFormatter" />.
+        /// </param>
+        IContextGraphBuilder AddResource<TResource, TId>(string pluralizedTypeName = null) where TResource : class, IIdentifiable<TId>;
 
         /// <summary>
         /// Add a json:api resource
         /// </summary>
         /// <param name="entityType">The resource model type</param>
         /// <param name="idType">The resource model identifier type</param>
-        /// <param name="pluralizedTypeName">The pluralized name that should be exposed by the API</param>
-        IContextGraphBuilder AddResource(Type entityType, Type idType, string pluralizedTypeName);
+        /// <param name="pluralizedTypeName">
+        /// The pluralized name that should be exposed by the API. 
+        /// If nothing is specified, the configured name formatter will be used.
+        /// See <see cref="JsonApiOptions.ResourceNameFormatter" />.
+        /// </param>
+        IContextGraphBuilder AddResource(Type entityType, Type idType, string pluralizedTypeName = null);
 
         /// <summary>
         /// Add all the models that are part of the provided <see cref="DbContext" /> 
@@ -80,18 +92,20 @@ public IContextGraph Build()
         }
 
         /// <inheritdoc />
-        public IContextGraphBuilder AddResource<TResource>(string pluralizedTypeName) where TResource : class, IIdentifiable<int>
+        public IContextGraphBuilder AddResource<TResource>(string pluralizedTypeName = null) where TResource : class, IIdentifiable<int>
             => AddResource<TResource, int>(pluralizedTypeName);
 
         /// <inheritdoc />
-        public IContextGraphBuilder AddResource<TResource, TId>(string pluralizedTypeName) where TResource : class, IIdentifiable<TId>
+        public IContextGraphBuilder AddResource<TResource, TId>(string pluralizedTypeName = null) where TResource : class, IIdentifiable<TId>
             => AddResource(typeof(TResource), typeof(TId), pluralizedTypeName);
 
         /// <inheritdoc />
-        public IContextGraphBuilder AddResource(Type entityType, Type idType, string pluralizedTypeName)
+        public IContextGraphBuilder AddResource(Type entityType, Type idType, string pluralizedTypeName = null)
         {
             AssertEntityIsNotAlreadyDefined(entityType);
 
+            pluralizedTypeName = pluralizedTypeName ?? _resourceNameFormatter.FormatResourceName(entityType);
+
             _entities.Add(GetEntity(pluralizedTypeName, entityType, idType));
 
             return this;

test/UnitTests/Builders/ContextGraphBuilder_Tests.cs

@@ -1,3 +1,5 @@
+using System.Linq;
+using JsonApiDotNetCore.Builders;
 using JsonApiDotNetCore.Extensions;
 using JsonApiDotNetCore.Internal;
 using JsonApiDotNetCore.Models;
@@ -37,5 +39,40 @@ public void Can_Build_ContextGraph_Using_Builder()
             Assert.Equal(typeof(NonDbResource), nonDbResource.EntityType);
             Assert.Equal(typeof(ResourceDefinition<NonDbResource>), nonDbResource.ResourceType);
         }
+
+        [Fact]
+        public void Resources_Without_Names_Specified_Will_Use_Default_Formatter()
+        {
+            // arrange
+            var builder = new ContextGraphBuilder();
+            builder.AddResource<TestResource>();
+
+            // act
+            var graph = builder.Build();
+
+            // assert
+            var resource = graph.GetContextEntity(typeof(TestResource));
+            Assert.Equal("test-resources", resource.EntityName);
+        }
+
+        [Fact]
+        public void Attrs_Without_Names_Specified_Will_Use_Default_Formatter()
+        {
+            // arrange
+            var builder = new ContextGraphBuilder();
+            builder.AddResource<TestResource>();
+
+            // act
+            var graph = builder.Build();
+
+            // assert
+            var resource = graph.GetContextEntity(typeof(TestResource));
+            Assert.Equal("attribute", resource.Attributes.Single().PublicAttributeName);
+        }
+
+        public class TestResource : Identifiable
+        {
+            [Attr] public string Attribute { get; set; }
+        }
     }
 }