More test fixes

src/Compilers/CSharp/Test/Emit/Attributes/InternalsVisibleToAndStrongNameTests.cs

@@ -1373,7 +1373,8 @@ private void ConfirmModuleAttributePresentAndAddingToAssemblyResultsInSignedOutp
                 //confirm file does not claim to be signed
                 Assert.Equal(0, (int)(flags & CorFlags.StrongNameSigned));
 
-                EntityHandle token = metadata.Module.GetTypeRef(metadata.Module.GetAssemblyRef("mscorlib"), "System.Runtime.CompilerServices", "AssemblyAttributesGoHere");
+                var corlibName = CoreClrShim.IsRunningOnCoreClr ? "netstandard" : "mscorlib";
+                EntityHandle token = metadata.Module.GetTypeRef(metadata.Module.GetAssemblyRef(corlibName), "System.Runtime.CompilerServices", "AssemblyAttributesGoHere");
                 Assert.False(token.IsNil);   //could the type ref be located? If not then the attribute's not there.
                 var attrInfos = metadata.Module.FindTargetAttributes(token, expectedModuleAttr);
                 Assert.Equal(1, attrInfos.Count());

src/Compilers/Core/Portable/PEWriter/PeWriter.cs

@@ -245,8 +245,6 @@ internal static class PeWriter
 
             if (privateKeyOpt != null && corFlags.HasFlag(CorFlags.StrongNameSigned))
             {
-                // TODO: write a test to verify that SignBuilder isn't called when
-                // signing with the legacy provider.
                 strongNameProvider.SignBuilder(peBuilder, peBlob, privateKeyOpt.Value);
             }