Bugzilla – Bug 8334
rx: InternalsVisibleTo incorrectly resolves to private methods
Last modified: 2012-11-14 05:48:17 EST
Created attachment 2903 [details]
repro patch set to mono tree (with external/rx)
This is (sorry to say again) non-unit test case that was found during import of
rx.codeplex.com into mono.
It seems that mcs fails to resolve method invocations to the _private_ one
(which is wrong) in the nearest ancestor, instead of the expected _internal_
Almost the same as bug #8320, but with the updated patch attached here this
time. You could remove mcs/class/*Reactive* and [checkout mcs/class/Makefile
and re-apply add-rx-libs.patch] or [edit mcs/class/Makefile and add
System.Reactive.Debugger]. This patch updates involves some Makefile changes,
additional sources and additional build args, so you most likely want to remove
Then run "make run-test"
(Actually I tried to create simpler repro but I couldn't.)
Failures like this:
67) ReactiveTests.Tests.RegressionTest.Bug_1297_Catch_Single :
System.MethodAccessException : Method
`System.Reactive.TailRecursiveSink`1<int>:Dispose ()' is inaccessible from
method `System.Reactive.Linq.Observαble.Catch`1/_<int>:Done ()'
at System.Reactive.TailRecursiveSink`1[System.Int32].MoveNext () [0x0019c] in
at System.Reactive.Concurrency.AsyncLock.Wait (System.Action action) [0x000b6]
none of the errors above.
The updated patch set contains an additional patch to "System.Reactive.Linq" in
rx tree. In the updated patch archive, there is
"internals-visibleto-workaround.patch". It actually describes the bug. With
this patch, those problematic NUnit failures are gone. The patch changes the
call target to the appropriate base class that contains the exact target
internal method to resolve.
I set up shrunk source tree of MS rx at https://github.com/atsushieno/rx.
If you checkout "rx-oss-v1.0" branch, it is already applied the patches inside
Created attachment 2908 [details]
I forgot to attach a public key for rx assemblies that is now referenced by
each "MonoAssemblyInfo.cs" file.
Fixed in master