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 failure CSharp.UnitTests.Emit.ResourceTests.AddResourceToModule #31279

Open
gafter opened this issue Nov 20, 2018 · 1 comment
Open
Labels
Milestone

Comments

@gafter
Copy link
Member

gafter commented Nov 20, 2018

This test is failing on Linux mono for PRs into the features/recursive-patterns branch.

2018-11-17T01:48:13.3915830Z �[39;49m[31m    Microsoft.CodeAnalysis.CSharp.UnitTests.Emit.ResourceTests.AddResourceToModule [FAIL]
2018-11-17T01:48:13.3925508Z �[39;49m[37m      System.NullReferenceException : Object reference not set to an instance of an object
2018-11-17T01:48:13.3930942Z �[39;49m[30m      Stack Trace:
2018-11-17T01:48:13.4075872Z �[39;49m[37m          at System.Reflection.Assembly.add_ModuleResolve (System.Reflection.ModuleResolveEventHandler value) [0x00000] in <3fbd5de7321a46409ce8ab20db4b2557>:0 
2018-11-17T01:48:13.4076778Z �[39;49m[37m          at Microsoft.CodeAnalysis.CSharp.UnitTests.Emit.ResourceTests.AddResourceToModule () [0x00367] in <8c6e94205fd44c8b8c1316a2dd08c849>:0 
2018-11-17T01:48:13.4077270Z �[39;49m[37m          at (wrapper managed-to-native) System.Reflection.MonoMethod.InternalInvoke(System.Reflection.MonoMethod,object,object[],System.Exception&)
2018-11-17T01:48:13.4104432Z �[39;49m[37m          at System.Reflection.MonoMethod.Invoke (System.Object obj, System.Reflection.BindingFlags invokeAttr, System.Reflection.Binder binder, System.Object[] parameters, System.Globalization.CultureInfo culture) [0x0006a] in <3fbd5de7321a46409ce8ab20db4b2557>:0 
@gafter
Copy link
Member Author

gafter commented Nov 20, 2018

Addressing by changing

        [ConditionalFact(typeof(DesktopOnly))]

to

        [ConditionalFact(typeof(WindowsDesktopOnly))]

Until the underlying issue can be fixed.

gafter added a commit to gafter/roslyn that referenced this issue Nov 20, 2018
gafter added a commit to gafter/roslyn that referenced this issue Nov 20, 2018
gafter added a commit to gafter/roslyn that referenced this issue Nov 20, 2018
gafter added a commit to gafter/roslyn that referenced this issue Nov 20, 2018
gafter pushed a commit that referenced this issue Nov 28, 2018
* Restore flow graph produced for existing switch statements
Fixes #28634
* Prevent type and constant value on a constant pattern
See also #31150
* Skip `AddResourceToModule` on Linux
Issue reported as #31279
@gafter gafter added this to the Backlog milestone Jul 17, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant