Skip to content

Add unit tests for ToCamelCaseWithNamespace method#24878

Merged
enisn merged 2 commits intofix/js-proxy-namespace-mismatchfrom
copilot/sub-pr-24877
Feb 16, 2026
Merged

Add unit tests for ToCamelCaseWithNamespace method#24878
enisn merged 2 commits intofix/js-proxy-namespace-mismatchfrom
copilot/sub-pr-24877

Conversation

Copy link
Contributor

Copilot AI commented Feb 16, 2026

Description

Adds comprehensive unit tests for the ToCamelCaseWithNamespace private method in SolutionRenamer to verify segment-wise camelCase transformation for multi-segment namespaces.

Test Coverage:

  • Single-segment transformations: "Demo""demo"
  • Multi-segment transformations: "Demo.App.QoL""demo.app.qoL"
  • Edge cases: empty strings, single characters, null input

Technical Details:

  • Uses reflection to test private static method
  • Handles TargetInvocationException wrapping from reflected invocations
  • 17 test cases using xUnit Theory with InlineData

Checklist

  • I fully tested it as developer / designer and created unit / integration tests
  • I documented it (or no need to document or I will create a separate documentation issue)

How to test it?

Run the test suite:

cd framework/test/Volo.Abp.Cli.Core.Tests
dotnet test --filter "FullyQualifiedName~SolutionRenamer_Tests"

All 17 tests should pass.


💡 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.

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

Copilot AI changed the title [WIP] Fix JS proxy namespace mismatch for multi-segment names Add unit tests for ToCamelCaseWithNamespace method Feb 16, 2026
Copilot AI requested a review from enisn February 16, 2026 10:13
@enisn enisn marked this pull request as ready for review February 16, 2026 10:26
@enisn enisn merged commit 1e68ae2 into fix/js-proxy-namespace-mismatch Feb 16, 2026
1 of 2 checks passed
@enisn enisn deleted the copilot/sub-pr-24877 branch February 16, 2026 10:26
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.

3 participants