various fixes

Author: jaredcnance

src/JsonApiDotNetCore/Builders/ContextGraphBuilder.cs

@@ -267,8 +267,9 @@ private string GetResourceNameFromDbSetProperty(PropertyInfo property, Type reso
             if (property.GetCustomAttribute(typeof(ResourceAttribute)) is ResourceAttribute resourceAttribute)
                 return resourceAttribute.ResourceName;
 
-            // fallback to dsherized...this should actually check for a custom IResourceNameFormatter
-            return _resourceNameFormatter.FormatResourceName(resourceType);
+            // fallback to the established convention using the DbSet Property.Name
+            // e.g DbSet<FooBar> FooBars { get; set; } => "foo-bars"
+            return _resourceNameFormatter.ApplyCasingConvention(property.Name);
         }
 
         private (bool isJsonApiResource, Type idType) GetIdType(Type resourceType)

src/JsonApiDotNetCore/Builders/DocumentBuilder.cs

@@ -214,6 +214,9 @@ private List<ResourceObject> IncludeRelationshipChain(
         {
             var requestedRelationship = relationshipChain[relationshipChainIndex];
             var relationship = parentEntity.Relationships.FirstOrDefault(r => r.PublicRelationshipName == requestedRelationship);
+            if(relationship == null)
+                throw new JsonApiException(400, $"{parentEntity.EntityName} does not contain relationship {requestedRelationship}");
+
             var navigationEntity = _jsonApiContext.ContextGraph.GetRelationship(parentResource, relationship.InternalRelationshipName);
             if (navigationEntity is IEnumerable hasManyNavigationEntity)
             {

src/JsonApiDotNetCore/Formatters/JsonApiReader.cs

@@ -46,16 +46,12 @@ public Task<InputFormatterResult> ReadAsync(InputFormatterContext context)
 
                 return InputFormatterResult.SuccessAsync(model);
             }
-            catch (JsonSerializationException ex)
+            catch (Exception ex)
             {
                 _logger?.LogError(new EventId(), ex, "An error occurred while de-serializing the payload");
                 context.ModelState.AddModelError(context.ModelName, ex, context.Metadata);
                 return InputFormatterResult.FailureAsync();
             }
-            catch (JsonApiException)
-            {
-                throw;
-            }
         }
 
         private string GetRequestBody(Stream body)

src/JsonApiDotNetCore/Graph/IResourceNameFormatter.cs

@@ -22,6 +22,12 @@ public interface IResourceNameFormatter
         /// Get the publicly visible name for the given property
         /// </summary>
         string FormatPropertyName(PropertyInfo property);
+
+        /// <summary>
+        /// Aoplies the desired casing convention to the internal string.
+        /// This is generally applied to the type name after pluralization.
+        /// </summary>
+        string ApplyCasingConvention(string properName);
     }
 
     public class DefaultResourceNameFormatter : IResourceNameFormatter
@@ -45,14 +51,30 @@ public string FormatResourceName(Type type)
                 if (type.GetCustomAttribute(typeof(ResourceAttribute)) is ResourceAttribute attribute)
                     return attribute.ResourceName;
 
-                return str.Dasherize(type.Name.Pluralize());
+                return ApplyCasingConvention(type.Name.Pluralize());
             }
             catch (InvalidOperationException e)
             {
                 throw new InvalidOperationException($"Cannot define multiple {nameof(ResourceAttribute)}s on type '{type}'.", e);
             }
         }
 
+        /// <summary>
+        /// Aoplies the desired casing convention to the internal string.
+        /// This is generally applied to the type name after pluralization.
+        /// </summary>
+        ///
+        /// <example>
+        /// <code>
+        /// _default.ApplyCasingConvention("TodoItems"); 
+        /// // > "todo-items"
+        ///
+        /// _default.ApplyCasingConvention("TodoItem"); 
+        /// // > "todo-item"
+        /// </code>
+        /// </example>
+        public string ApplyCasingConvention(string properName) => str.Dasherize(properName);
+
         /// <summary>
         /// Uses the internal PropertyInfo to determine the external resource name.
         /// By default the name will be formatted to kebab-case.

test/UnitTests/Builders/ContextGraphBuilder_Tests.cs

@@ -139,6 +139,8 @@ public class RelatedResource : Identifiable { }
 
         public class CamelCaseNameFormatter : IResourceNameFormatter
         {
+            public string ApplyCasingConvention(string properName) => ToCamelCase(properName);
+
             public string FormatPropertyName(PropertyInfo property) => ToCamelCase(property.Name);
 
             public string FormatResourceName(Type resourceType) => ToCamelCase(resourceType.Name.Pluralize());

test/UnitTests/Extensions/IServiceCollectionExtensionsTests.cs

@@ -112,8 +112,26 @@ public void AddResourceService_Throws_If_Type_Does_Not_Implement_Any_Interfaces(
             Assert.Throws<JsonApiSetupException>(() => services.AddResourceService<int>());
         }
 
-        private class IntResource : Identifiable { }
-        private class GuidResource : Identifiable<Guid> { }
+        [Fact]
+        public void AddJsonApi_With_Context_Uses_DbSet_PropertyName_If_NoOtherSpecified()
+        {
+            // arrange
+            var services = new ServiceCollection();
+
+            services.AddScoped<IScopedServiceProvider, TestScopedServiceProvider>();
+
+            // act
+            services.AddJsonApi<TestContext>();
+
+            // assert
+            var provider = services.BuildServiceProvider();
+            var graph = provider.GetService<IContextGraph>();
+            var resource = graph.GetContextEntity(typeof(IntResource));
+            Assert.Equal("resource", resource.EntityName);
+        }
+
+        public class IntResource : Identifiable { }
+        public class GuidResource : Identifiable<Guid> { }
 
         private class IntResourceService : IResourceService<IntResource>
         {
@@ -138,5 +156,11 @@ private class GuidResourceService : IResourceService<GuidResource, Guid>
             public Task<GuidResource> UpdateAsync(Guid id, GuidResource entity) => throw new NotImplementedException();
             public Task UpdateRelationshipsAsync(Guid id, string relationshipName, List<ResourceObject> relationships) => throw new NotImplementedException();
         }
+
+
+        public class TestContext : DbContext 
+        {
+            public DbSet<IntResource> Resource { get; set; }
+        }
     }
 }