Bad code generation for resource

[dsyme]

@brettfo There seems to be a problem with resource generation

Building `tests\projects\Sample_VS2012_FSharp_ConsoleApp_net45_with_resource`` using

cd tests\projects\Sample_VS2012_FSharp_ConsoleApp_net45_with_resource
msbuild /p:FSharpTargetsPath=C:\GitHub\dsyme\visualfsharp\release\net40\bin\Microsoft.FSharp.Targets

which has

		<EmbeddedResource Include="ResxResource.resx">
			<GenerateSource>true</GenerateSource>
		</EmbeddedResource>

Gives multiple bad source files such as obj\Debug\ResxResource.fs

// <auto-generated>

namespace global

open System.Reflection

module internal =
 type private C (_dummy:System.Object) = class end
 let mutable Culture = System.Globalization.CultureInfo.CurrentUICulture
 let ResourceManager = new System.Resources.ResourceManager("ResxResource", C(null).GetType().GetTypeInfo().Assembly)
 let GetString(name:System.String) : System.String = ResourceManager.GetString(name, Culture)
 let GetObject(name:System.String) : System.Object = ResourceManager.GetObject(name, Culture)

[dsyme]

I see this in Fsc.fs but I don't know it's generating a correct module name

 let moduleName =
 match item.GetMetadata("GeneratedModuleName") with
 | null -> Path.GetFileNameWithoutExtension(item.ItemSpec)
 | value -> value

Certainly setting

		<GeneratedModuleName>AAA</GeneratedModuleName>

makes the problem go away

[dsyme]

Fix is here : #3690

[dsyme]

@brettfo Please do a post-review on this and check if the same problem occurs anywhere else? Also review the testing of the resource code generation please, thx