SonarQube Analysis / sonarqube (pull_request) Successful in 4m22s
After the CA1822 cascade in #15 left every method on the eight Markdown generator classes static, the classes themselves were instantiable shells that consumers couldn't usefully `new`. This commit flips the `class` modifier to `static class` on all eight: - `Markdown.SqlServer.QueryBreakdownGenerator` - `Markdown.SqlServer.SqlStatementGenerator` - `Markdown.LinqToSql.QueryBreakdownGenerator` - `Markdown.LinqToSql.SqlStatementGenerator` - `Markdown.PostgreSql.QueryBreakdownGenerator` - `Markdown.PostgreSql.SqlStatementGenerator` - `Markdown.Snowflake.QueryBreakdownGenerator` - `Markdown.Snowflake.SqlStatementGenerator` Test fixtures drop the now-meaningless `_generator = new …()` field and `[SetUp]` (kept the existing Setup body where unrelated state was also initialized — `QueryMarkdownGenerationTests` and `LinqToSql.QueryBreakdownGeneratorTests`). BREAKING CHANGE: External NuGet consumers can no longer write `new Markdown.Snowflake.SqlStatementGenerator()` (or any of the other seven classes above) — the type is now a static container and may only be referenced by name, e.g. `Markdown.Snowflake.SqlStatementGenerator.GenerateSequenceDiagram(…)`. The call syntax for the static methods is unchanged. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
147 lines
4.5 KiB
C#
147 lines
4.5 KiB
C#
using Strata.SqlTools.Breakdowns.SqlServer;
|
|
using Strata.SqlTools.Markdown.SqlServer;
|
|
using Interfaces = Strata.SqlTools.SqlBreakdown.Interfaces;
|
|
|
|
namespace Strata.SqlTools.Markdown.Tests.SqlServer;
|
|
|
|
[TestFixture]
|
|
public class QueryBreakdownGeneratorTests
|
|
{
|
|
[Test]
|
|
public void GenerateMermaidDiagram_WithSimpleQuery_GeneratesFlowchart()
|
|
{
|
|
// Arrange
|
|
var query = new QueryBreakdown("ID, Name", "Users");
|
|
|
|
// Act
|
|
var result = QueryBreakdownGenerator.GenerateMermaidDiagram(query);
|
|
|
|
// Assert
|
|
Assert.That(result, Does.Contain("```mermaid"));
|
|
Assert.That(result, Does.Contain("flowchart TD"));
|
|
Assert.That(result, Does.Contain("SELECT"));
|
|
Assert.That(result, Does.Contain("FROM"));
|
|
Assert.That(result, Does.Contain("```"));
|
|
}
|
|
|
|
[Test]
|
|
public void GenerateMermaidDiagram_WithComplexQuery_IncludesAllClauses()
|
|
{
|
|
// Arrange
|
|
var query = new QueryBreakdown(
|
|
"u.ID, u.Name, COUNT(o.OrderID) as OrderCount",
|
|
"Users u JOIN Orders o ON u.ID = o.UserID",
|
|
"u.Active = 1"
|
|
);
|
|
query.GroupByClause.Clause = "u.ID, u.Name";
|
|
query.HavingClause.Clause = "COUNT(o.OrderID) > 5";
|
|
query.OrderByClause.Clause = "OrderCount DESC";
|
|
|
|
// Act
|
|
var result = QueryBreakdownGenerator.GenerateMermaidDiagram(query, "User Orders Query");
|
|
|
|
// Assert
|
|
Assert.That(result, Does.Contain("### User Orders Query"));
|
|
Assert.That(result, Does.Contain("SELECT"));
|
|
Assert.That(result, Does.Contain("FROM"));
|
|
Assert.That(result, Does.Contain("WHERE"));
|
|
Assert.That(result, Does.Contain("GROUP BY"));
|
|
Assert.That(result, Does.Contain("HAVING"));
|
|
Assert.That(result, Does.Contain("ORDER BY"));
|
|
}
|
|
|
|
[Test]
|
|
public void GenerateMermaidDiagram_WithWhereClause_UsesDecisionShape()
|
|
{
|
|
// Arrange
|
|
var query = new QueryBreakdown("*", "Products", "Price > 100");
|
|
|
|
// Act
|
|
var result = QueryBreakdownGenerator.GenerateMermaidDiagram(query);
|
|
|
|
// Assert
|
|
Assert.That(result, Does.Contain("WHERE"));
|
|
Assert.That(result, Does.Contain("{"));
|
|
Assert.That(result, Does.Contain("}"));
|
|
}
|
|
|
|
[Test]
|
|
public void GenerateMermaidDiagram_ParsedQuery_GeneratesDiagram()
|
|
{
|
|
// Arrange
|
|
var sql = @"
|
|
SELECT u.ID, u.Name, u.Email
|
|
FROM Users u
|
|
WHERE u.Active = 1
|
|
ORDER BY u.Name";
|
|
|
|
var query = QueryBreakdown.Parse(sql);
|
|
|
|
// Act
|
|
var result = QueryBreakdownGenerator.GenerateMermaidDiagram(query);
|
|
|
|
// Assert
|
|
Assert.That(result, Is.Not.Null);
|
|
Assert.That(result, Does.Contain("mermaid"));
|
|
Assert.That(result, Does.Contain("SELECT"));
|
|
Assert.That(result.Length, Is.GreaterThan(100));
|
|
}
|
|
|
|
[Test]
|
|
public void GenerateMermaidDiagram_WithTitle_IncludesTitle()
|
|
{
|
|
// Arrange
|
|
var query = new QueryBreakdown("*", "Users");
|
|
|
|
// Act
|
|
var result = QueryBreakdownGenerator.GenerateMermaidDiagram(query, "Test Query");
|
|
|
|
// Assert
|
|
Assert.That(result, Does.StartWith("### Test Query"));
|
|
}
|
|
|
|
[Test]
|
|
public void GenerateMermaidDiagram_WithoutTitle_NoTitle()
|
|
{
|
|
// Arrange
|
|
var query = new QueryBreakdown("*", "Users");
|
|
|
|
// Act
|
|
var result = QueryBreakdownGenerator.GenerateMermaidDiagram(query);
|
|
|
|
// Assert
|
|
Assert.That(result, Does.Not.StartWith("###"));
|
|
Assert.That(result, Does.StartWith("```mermaid"));
|
|
}
|
|
|
|
[Test]
|
|
public void GenerateMermaidDiagram_WithLongClause_TruncatesText()
|
|
{
|
|
// Arrange
|
|
var longSelect = string.Join(", ", Enumerable.Range(1, 50).Select(i => $"Column{i}"));
|
|
var query = new QueryBreakdown(longSelect, "LargeTable");
|
|
|
|
// Act
|
|
var result = QueryBreakdownGenerator.GenerateMermaidDiagram(query);
|
|
|
|
// Assert
|
|
Assert.That(result, Does.Contain("..."));
|
|
}
|
|
|
|
[Test]
|
|
public void GenerateMermaidDiagram_ISqlBreakdown_GeneratesSimpleDiagram()
|
|
{
|
|
// Arrange
|
|
var query = new QueryBreakdown("*", "Users");
|
|
query.SetupClauses.Add("CREATE TEMP TABLE #Temp (ID INT)");
|
|
|
|
// Act
|
|
var result = QueryBreakdownGenerator.GenerateMermaidDiagram((Interfaces.ISqlBreakdown)query);
|
|
|
|
// Assert
|
|
Assert.That(result, Does.Contain("mermaid"));
|
|
Assert.That(result, Does.Contain("flowchart TD"));
|
|
Assert.That(result, Does.Contain("Users"));
|
|
}
|
|
}
|