[Flaky Test] com2fontconvertertests.com2fontconverter_convertnativetomanaged

[runfoapp]

Runfo Tracking Issue: [Flaky Test] com2fontconvertertests.com2fontconverter_convertnativetomanaged

Build	Definition	Kind	Run Name

Build Result Summary

Day Hit Count	Week Hit Count	Month Hit Count
0	0	0

[dreddy-work]

Assert.Equal() Failure\r\nExpected: 12\r\nActual:   1.5


Stack trace
   at System.Windows.Forms.Tests.ComponentModel.Com2Interop.COM2FontConverterTests.COM2FontConverter_ConvertNativeToManaged() in /_/src/System.Windows.Forms/tests/UnitTests/System/Windows/Forms/ComponentModel/Com2Interop/COM2FontConverterTests.cs:line 48
   at System.RuntimeMethodHandle.InvokeMethod(Object target, Void** arguments, Signature sig, Boolean isConstructor)
   at System.Reflection.MethodInvoker.Invoke(Object obj, IntPtr* args, BindingFlags invokeAttr)

[lonitra]

It looks like an ArgumentException might have occurred and Control.DefaultFont was passed back from ConvertNativeToManaged().

            try
            {
                // This wasn't working because it was converting everything to world units.
                using (Font font = Font.FromHfont(_lastHandle))
                {
                    _lastFont = ControlPaint.FontInPoints(font);
                }
            }
            catch (ArgumentException)
            {
                // We will fail on non-truetype fonts, so just use the default font.
                _lastFont = Control.DefaultFont;
            }

We should add an assert in the test that checks that the font of interest isn't equal to Control.DefaultFont to help narrow down the cause and perhaps add some sort of debug statement to capture details of the exception.