Fix non enumerable generic type info construction

[JMolenkamp]

Proposed changes

Require a single argument generic type (e.g. List<int>) to at least implement IEnumerable for Type.Construct(Type type) to represent it as an array.

Related Issues

• Fixes TypeInfo.Construct represents all single argument generic types as arrays #3598

Types of changes

• Bugfix (non-breaking change which fixes an issue)
• Enhancement (non-breaking change which adds functionality)
• Test enhancement (non-breaking change to increase test coverage)
• Breaking change (fix or feature that would cause existing functionality to not work as expected, requires version increase of Nuget packages)
• Documentation Update (if none of the other choices apply)

Checklist

• I have read the CONTRIBUTING doc.
• I have signed the CLA.
• I ran tests locally with my changes, all passed.
• I fixed all failing tests in the CI pipelines.
• I fixed all introduced issues with CodeQL and LGTM.
• I have added tests that prove my fix is effective or that my feature works and increased code coverage.
• I have added necessary documentation (if appropriate).
• Any dependent changes have been merged and published in downstream modules.

[CLAassistant]

All committers have signed the CLA.

[JMolenkamp]

Any tips for running the tests locally, without it taking forever?
I simply did a 'Run all' in the Visual Studio Test Explorer, but aborted after it ran about 400 of 127.000 tests in several minutes...

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 66.14%. Comparing base (d25caff) to head (ca7070e).
⚠️ Report is 134 commits behind head on master.

Additional details and impacted files

@@             Coverage Diff             @@
##           master    #3597       +/-   ##
===========================================
+ Coverage   51.86%   66.14%   +14.27%     
===========================================
  Files         370      459       +89     
  Lines       78618    97717    +19099     
  Branches    13650    16376     +2726     
===========================================
+ Hits        40779    64639    +23860     
+ Misses      33705    28009     -5696     
- Partials     4134     5069      +935     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[romanett]

@JMolenkamp you can select Just to Run one Framework Version. Also Client Tests Take 10-20 mins all Others are faster

[romanett]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[romanett]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).