Remove whitespaces from autoentity-generated names to prevent invalid REST paths and GraphQL singular/plural tables for SQL objects with spaces

[Copilot]

Why make this change?

Tables with whitespace in names (for example, Order Items) were being auto-generated into entity names like dbo_Order Items, which then produced invalid REST paths as well as invalid GraphQL names for singular/plural tables which failed config validation. This change ensures autoentity name generation does not pass whitespace through to REST path and GraphQL singular/plural tables defaults.

• Autoentity naming currently interpolates {schema} / {object} verbatim; whitespace in object names breaks dab validate.

What is this change?

• Autoentity name normalization (MsSql metadata path)
  • Added RemoveWhitespaceAddCamelCase in SqlMetadataProvider and applied it to generated entity_name before entity creation. The method name is intentionally specific to clarify it only removes whitespace (not other invalid characters).
  • Normalization removes whitespace and uppercases the character immediately following each removed whitespace (camelCase-join), e.g. dbo_Order Items → dbo_OrderItems.
  • Added a guard after normalization: if the result is empty (e.g. the raw entity name was entirely whitespace), an error is logged with the original name and schema for context, and the entry is skipped.
  • Collision detection is improved: when TryAdd fails because two database objects normalize to the same entity name (e.g. "Order Item" and "OrderItem" both yield "OrderItem"), the exception message includes the pre-normalization name and schema to make the collision diagnosable.

protected static string RemoveWhitespaceAddCamelCase(string name)
{
    StringBuilder result = new(name.Length);
    bool capitalizeNext = false;

    foreach (char character in name)
    {
        if (char.IsWhiteSpace(character))
        {
            capitalizeNext = true;
            continue;
        }

        result.Append(capitalizeNext ? char.ToUpperInvariant(character) : character);
        capitalizeNext = false;
    }

    return result.ToString();
}

• Focused test coverage
  • Added a dedicated TestAutoentitiesGeneratedWithSpacesInObjectName test method that explicitly asserts the sanitized entity name dbo_OrderItems (via const string EXPECTED_ENTITY_NAME) and validates end-to-end REST and GraphQL access with that name.
  • Whitespace-handling validation is now separated from the non-default schema cases in TestAutoentitiesGeneratedWithUnusualElements for better readability and maintainability.

How was this tested?

• Integration Tests
• Unit Tests

Sample Request(s)

• Example CLI usage to reproduce/validate behavior:
  • dab auto-config add mydef --patterns.include "dbo.Order Items" --patterns.name "{schema}_{object}"
  • dab validate
• Expected generated entity naming behavior:
  • SQL object: dbo.[Order Items]
  • Generated entity/REST path segment: dbo_OrderItems (no whitespace)

[anushakolan]

LGTM!

[souvikghosh04]

Thanks for addressing the comments. Approved.

[RubenCerna2079]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 6 pipeline(s).

[RubenCerna2079]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 6 pipeline(s).