CSHARP-5588: Unable to select KeyValuePair Key when using camelCase convention #1724
Conversation
|
Assigned |
| // TODO: uncomment the code to demonstrate the fix, revert changes in the test class before merging into the main | ||
| // _useCamelCase = true; | ||
|
|
||
| if (_useCamelCase) | ||
| { | ||
| var defaultPack = new ConventionPack { new CamelCaseElementNameConvention() }; | ||
| ConventionRegistry.Register("camelCase", defaultPack, c => true); | ||
| } |
There was a problem hiding this comment.
I think we should keep a permanent tests around this to verify that registered conventions are respected instead of just having this for demonstration. You can just remove the convention once the test is done.
Not sure if we want to keep the test here or do what @rstam usually does which is create a test file with the ticket number and test the particular code the user had as repro code.
There was a problem hiding this comment.
I would love to have such tests permanently, but not sure if we can. As far as I know there is no way to have multiple serialization registries for now. If I leave the convention pack registered, it will affect all tests that are run after this particular class.
No description provided.