Add coverage for DataSourceListEditor

[Zheng-Li01]

Related #10773

Proposed changes

Add unit test DataSourceListEditorTests.cs for public method of the DataSourceListEditor.

Microsoft Reviewers: Open in CodeFlow

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 61.45499%. Comparing base (df745d6) to head (77dbc30).
Report is 78 commits behind head on main.

Additional details and impacted files

@@                 Coverage Diff                 @@
##                main      #13122         +/-   ##
===================================================
+ Coverage   61.32538%   61.45499%   +0.12960%     
===================================================
  Files           1543        1548          +5     
  Lines         158339      158503        +164     
  Branches       14745       14751          +6     
===================================================
+ Hits           97102       97408        +306     
+ Misses         60539       60391        -148     
- Partials         698         704          +6     

Flag	Coverage Δ
Debug	61.45499% <100.00000%> (-13.85836%)	⬇️
integration	10.72871% <ø> (-8.18178%)	⬇️
production	39.38064% <ø> (-35.93272%)	⬇️
test	95.68114% <100.00000%> (∅)
unit	36.81725% <ø> (-37.58343%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[Tanya-Solyanik]

  Test failed and will automatically retry. Failure details follow:

Expected type to be System.NotSupportedException, but found System.IO.MemoryStream.
at FluentAssertions.Execution.LateBoundTestFramework.Throw(String message)
at FluentAssertions.Execution.DefaultAssertionStrategy.HandleFailure(String message)
at FluentAssertions.Execution.AssertionChain.FailWith(Func1 getFailureReason) at FluentAssertions.Execution.AssertionChain.FailWith(Func1 getFailureReason)
at FluentAssertions.Execution.AssertionChain.FailWith(String message)
at FluentAssertions.Types.TypeAssertions.Be(Type expected, String because, Object[] becauseArgs)
at FluentAssertions.Primitives.ReferenceTypeAssertions2.BeOfType(Type expectedType, String because, Object[] becauseArgs) at FluentAssertions.Primitives.ReferenceTypeAssertions2.BeOfType[T](String because, Object[] becauseArgs)
at System.Windows.Forms.Tests.ClipboardTests.SetData_CustomFormat_Exception_BinaryFormatterDisabled_SerializesException() in /_/src/test/unit/System.Windows.Forms/System/Windows/Forms/ClipboardTests.cs:line 554
at System.RuntimeMethodHandle.InvokeMethod(ObjectHandleOnStack target, Void** arguments, ObjectHandleOnStack sig, BOOL isConstructor, ObjectHandleOnStack result)
at System.Reflection.MethodBaseInvoker.InterpretedInvoke_Method(Object obj, IntPtr* args)
at System.Reflection.MethodBaseInvoker.InvokeWithNoArgs(Object obj, BindingFlags invokeAttr)

[Epica3055]

looks good 👍

Pull Request Overview

This PR adds unit tests for the DataSourceListEditor to improve coverage for its public methods.

• Introduces tests for checking the resizable drop-down property.
• Verifies EditValue behavior under various parameter scenarios.
• Confirms that GetEditStyle returns the expected UI type.

Comments suppressed due to low confidence (1)

src/System.Windows.Forms.Design/tests/UnitTests/System/Windows/Forms/Design/DataSourceListEditorTests.cs:33

• [nitpick] Consider also asserting the returned value from EditValue when valid inputs are provided, if the expected behavior can be predicted beyond ensuring no exception is thrown.

_dataSourceListEditor.EditValue(context: contextMock.Object, provider: providerMock.Object, value: null);

[ricardobossan]

Other than a small comment, all LGTM!

[ricardobossan]

Thank you!