Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Dec 14, 2025

Expanded test coverage from 18 to 80 tests across previously untested core components.

New Test Coverage

  • SchemaObjectNameTests (5 tests) - Name quoting, schema handling, special characters
  • DatabaseConnectionTests (10 tests) - View registration, retrieval, IsView checks
  • DatabaseViewTests (13 tests) - SQL parsing, CreateOrAlter transformation, syntax validation
  • InlinerOptionsTests (5 tests) - Default vs recommended configurations
  • InlinerResultTests (13 tests) - Output formatting, metadata, reference tracking
  • ErrorHandlingTests (8 tests) - Validation errors, warnings for missing aliases and single-part identifiers
  • ComplexScenarioTests (11 tests) - Outer joins, multi-level nesting, GROUP BY, CASE, DISTINCT

All tests follow existing NUnit conventions and use in-memory DatabaseConnection instances to avoid external dependencies.

Original prompt

more unit tests


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

…DatabaseView, InlinerOptions, InlinerResult, error handling, and complex scenarios

Co-authored-by: stevehansen <2143488+stevehansen@users.noreply.github.com>
Copilot AI changed the title [WIP] Add more unit tests for existing functions Add 62 unit tests covering core functionality Dec 14, 2025
Copilot AI requested a review from stevehansen December 14, 2025 09:16
@stevehansen stevehansen marked this pull request as ready for review December 14, 2025 10:12
@stevehansen stevehansen merged commit 607f014 into main Dec 14, 2025
1 of 3 checks passed
@stevehansen stevehansen deleted the copilot/add-more-unit-tests branch December 14, 2025 10:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants