BREAKING CHANGE: Allow tag name to be case-insensitive

ose-net · Jul 6, 2024 · 6206d30 · 6206d30
1 parent 4945482
commit 6206d30
Show file tree

Hide file tree

Showing 2 changed files with 23 additions and 1 deletion.
diff --git a/src/Reader/YeSqlDictionary.cs b/src/Reader/YeSqlDictionary.cs
@@ -11,7 +11,7 @@ internal class YeSqlDictionary : ISqlCollection
     /// <summary>
     /// A dictionary containing the SQL statements that have been parsed from the data source (e.g., a SQL file).
     /// </summary>
-    private readonly Dictionary<string, string> _sqlStatements = [];
+    private readonly Dictionary<string, string> _sqlStatements = new(StringComparer.OrdinalIgnoreCase);
 
     /// <inheritdoc />
     public string this[string tagName]

diff --git a/tests/Parser/YeSqlParserTests.cs b/tests/Parser/YeSqlParserTests.cs
@@ -31,6 +31,28 @@ public void Parse_WhenDataSourceIsEmptyOrWithWhitespace_ShouldReturnsEmptyCollec
         validationResult.ErrorMessages.Should().Contain(expectedMessage);
     }
 
+    [Test]
+    public void Parse_WhenTagNameIsCaseInsensitive_ShouldReturnsSqlStatements()
+    {
+        // Arrange
+        var parser = new YeSqlParser();
+        var expectedSqlStatement = "SELECT * FROM users;" + Environment.NewLine;
+        var source =
+        """
+        -- name: GetUsers
+        SELECT * FROM users;
+        """;
+
+        // Act
+        ISqlCollection sqlStatements = parser.Parse(source, out _);
+
+        // Asserts
+        sqlStatements["GetUsers"].Should().Be(expectedSqlStatement);
+        sqlStatements["getUsers"].Should().Be(expectedSqlStatement);
+        sqlStatements["getusers"].Should().Be(expectedSqlStatement);
+        sqlStatements["GETUSERS"].Should().Be(expectedSqlStatement);
+    }
+
     [TestCaseSource(typeof(TagIsEmptyOrWhitespaceTestCases))]
     public void Parse_WhenTagIsEmptyOrConsistsOnlyOfWhitespaces_ShouldGenerateAnError(
         string source,