Remove redudnant internals visible to in ms test.source generation

src/Analyzers/MSTest.SourceGeneration/Generators/TestNodesGenerator.cs

@@ -12,7 +12,7 @@
 namespace Microsoft.Testing.Framework.SourceGeneration;
 
 [Generator]
-internal sealed class TestNodesGenerator : IIncrementalGenerator
+public sealed class TestNodesGenerator : IIncrementalGenerator
 {
     public void Initialize(IncrementalGeneratorInitializationContext context)
     {
@@ -198,7 +198,7 @@ private static void AddTestClassNode(SourceProductionContext context, TestTypeIn
     }
 
     // Borrowed from https://github.com/dotnet/templating/blob/dad34814012bf29aa35eaf8e8013af4b10b997da/src/Microsoft.TemplateEngine.Orchestrator.RunnableProjects/ValueForms/DefaultSafeNamespaceValueFormFactory.cs#L10
-    internal /* for testing purpose */ static string ToSafeNamespace(string value)
+    public static string ToSafeNamespace(string value)
     {
         const char invalidCharacterReplacement = '_';
 

src/Analyzers/MSTest.SourceGeneration/Helpers/Constants.cs

@@ -3,7 +3,7 @@
 
 namespace Microsoft.Testing.Framework.SourceGeneration;
 
-internal static class Constants
+public static class Constants
 {
     public const string NewLine = "\r\n";
 }

src/Analyzers/MSTest.SourceGeneration/Helpers/IndentedStringBuilder.cs

@@ -3,7 +3,7 @@
 
 namespace Microsoft.Testing.Framework.SourceGeneration.Helpers;
 
-internal sealed class IndentedStringBuilder
+public sealed class IndentedStringBuilder
 {
     private readonly StringBuilder _builder = new();
     private bool _needsIndent = true;

src/Analyzers/MSTest.SourceGeneration/MSTest.SourceGeneration.csproj

@@ -7,7 +7,8 @@
     <GeneratePackageOnBuild>true</GeneratePackageOnBuild>
     <!-- Do not include the generator as a lib dependency -->
     <IncludeBuildOutput>false</IncludeBuildOutput>
-    <NoWarn>NU5128</NoWarn>
+    <GenerateDocumentationFile>false</GenerateDocumentationFile>
+    <NoWarn>NU5128;SA0001;EnableGenerateDocumentationFile</NoWarn>
     <EnforceExtendedAnalyzerRules>true</EnforceExtendedAnalyzerRules>
     <RootNamespace>Microsoft.Testing.Framework.SourceGeneration</RootNamespace>
     <!-- Cancel default license (closed-source) and set license as packaged license file -->
@@ -35,10 +36,6 @@ This package provides the C# source generators for MSTest test framework.]]>
     <AdditionalFiles Include="BannedSymbols.txt" />
   </ItemGroup>
 
-  <ItemGroup>
-    <InternalsVisibleTo Include="MSTest.SourceGeneration.UnitTests" Key="$(VsPublicKey)" />
-  </ItemGroup>
-
   <ItemGroup>
     <!-- Roslyn C# - so that we can implement the code generator -->
     <PackageReference Include="Microsoft.CodeAnalysis.CSharp" PrivateAssets="all" />

src/Analyzers/MSTest.SourceGeneration/ObjectModels/DataRowTestMethodArgumentsInfo.cs

@@ -9,7 +9,7 @@
 
 namespace Microsoft.Testing.Framework.SourceGeneration.ObjectModels;
 
-internal sealed class DataRowTestMethodArgumentsInfo : ITestMethodArgumentsInfo
+public sealed class DataRowTestMethodArgumentsInfo : ITestMethodArgumentsInfo
 {
     private readonly ImmutableArray<ImmutableArray<string>> _argumentsRows;
     private readonly TestMethodParametersInfo _parametersInfo;
@@ -72,7 +72,7 @@ private static string GetArgumentsUid(string[] parameterNames, IList<string> arg
         return argumentsUidBuilder.ToString();
     }
 
-    internal /* for testing purposes */ static void EscapeArgument(string argument, StringBuilder argumentsUidBuilder)
+    public static void EscapeArgument(string argument, StringBuilder argumentsUidBuilder)
     {
         int escapeCharCount = 0;
         for (int i = 0; i < argument.Length; i++)

src/Analyzers/MSTest.SourceGeneration/ObjectModels/TestMethodParametersInfo.cs

@@ -7,7 +7,7 @@
 
 namespace Microsoft.Testing.Framework.SourceGeneration.ObjectModels;
 
-internal sealed class TestMethodParametersInfo
+public sealed class TestMethodParametersInfo
 {
     public TestMethodParametersInfo(ImmutableArray<IParameterSymbol> parameters)
     {