This pull request tries to fix [ISSUE]: Overrideconfig for strategies returns 'unknown key' error. Closes #4329.

Description

There are multiple issues when using /overrideconfig with the 'strategies' property.

First:
The overrideconfig uses a different format that the normal config. And they are then fed differently into the deserializer.

For the override config the following format seems to be used:
strategies=["Mainline","None"]

For the normal config the following format is used:
strategies:
- Mainline
- None

The second issue is that a cast from string[] to VersionStrategies[] has to happen.

My approach was to implement a custom type converter and registered it with the deserializer. The type converter then detects the config format and parses it accordingly.

In my tests it worked for both configuration sources. I am unsure if the location where I put the converter is the right place. I would prefer to avoid YamlDotNet as a direct dependency.

Also I hope that there is a simpler solution and avoid the converter all-together. Maybe this splitting behavior of the value beforehand can be improved.

Related Issue

#4329

Motivation and Context

How Has This Been Tested?

A test case has been added for the related overrideconfig strategies property. There are also many other test cases that might break, if the serialization/deserialization is wrong.

Checklist:

• My code follows the code style of this project. (I hope so)
• My change requires a change to the documentation.
• I have updated the documentation accordingly.
• I have added tests to cover my changes.
• All new and existing tests passed.