Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Mono] Test failed on windows: System.Tests.ActivatorTests.TestingCreateInstanceObjectHandleFullSignatureWinRT #34396

Closed
MaximLipnin opened this issue Apr 1, 2020 · 2 comments · Fixed by #36715
Labels
os-windows runtime-mono specific to the Mono runtime

Comments

@MaximLipnin
Copy link
Contributor

System.Tests.ActivatorTests.TestingCreateInstanceObjectHandleFullSignatureWinRT is disabled in Mono's CoreFX.issues_windows.rsp file. After enabling it still fails locally with the following output:

System.Tests.ActivatorTests.TestingCreateInstanceObjectHandleFullSignatureWinRT(assemblyName: "Windows, Version=255.255.255.255, Culture=neutral,"..., type: "Windows.Foundation.Collections.StringMap", ignoreCase: False, bindingAttr: Instance | Public | NonPublic, binder: DefaultBinder { }, args: [], culture: , activationAttributes: null, returnedFullNameType: "Windows.Foundation.Collections.StringMap") [FAIL]
System.IO.FileLoadException : The assembly name is invalid.
Stack Trace:
    _\src\mono\netcore\System.Private.CoreLib\src\System\Reflection\AssemblyName.Mono.cs(28,0): at System.Reflection.AssemblyName..ctor(String assemblyName)
    _\src\libraries\System.Private.CoreLib\src\System\Activator.RuntimeType.cs(118,0): at System.Activator.CreateInstanceInternal(String assemblyString, String typeName, Boolean ignoreCase, BindingFlags bindingAttr, Binder binder, Object[] args, CultureInfo culture, Object[] activationAttributes, StackCrawlMark& stackMark)
    _\src\libraries\System.Private.CoreLib\src\System\Activator.RuntimeType.cs(60,0): at System.Activator.CreateInstance(String assemblyName, String typeName, Boolean ignoreCase, BindingFlags bindingAttr, Binder binder, Object[] args, CultureInfo culture, Object[] activationAttributes)
    _\src\libraries\System.Runtime\tests\System\ActivatorTests.cs(784,0): at System.Tests.ActivatorTests.TestingCreateInstanceObjectHandleFullSignatureWinRT(String assemblyName, String type, Boolean ignoreCase, BindingFlags bindingAttr, Binder binder, Object[] args, CultureInfo culture, Object[] activationAttributes, String returnedFullNameType)
    _\src\mono\netcore\System.Private.CoreLib\src\System\Reflection\RuntimeMethodInfo.cs(360,0): at System.Reflection.RuntimeMethodInfo.Invoke(Object obj, BindingFlags invokeAttr, Binder binder, Object[] parameters, CultureInfo culture)

The test will be marked with ActiveIssue in #32592.

@danmoseley danmoseley added runtime-mono specific to the Mono runtime os-windows labels Apr 10, 2020
@danmoseley
Copy link
Member

Maybe change public static bool IsWinRTSupported => IsWindows && IsNotWindows7; to public static bool IsWinRTSupported => IsWindows && IsNotWindows7 && IsNotMono; and then disable test with that.

@jkoritzinsky
Copy link
Member

This test will be deleted in #36715 as part of removal of built-in WinRT support in .NET 5.

@jkoritzinsky jkoritzinsky linked a pull request May 22, 2020 that will close this issue
@ghost ghost locked as resolved and limited conversation to collaborators Dec 9, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
os-windows runtime-mono specific to the Mono runtime
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants