CHanged the direct package dependency to disable compile-time assembly reference

This helps us keep the .targts file active and hence able to obtain other assets inside DLL while disabling the references

Author: ananth7592

Microsoft.Azure.Cosmos/src/Microsoft.Azure.Cosmos.csproj

@@ -118,6 +118,9 @@
 	</ItemGroup>
 
 	<ItemGroup Condition=" '$(ProjectRef)' != 'True' ">
+		<PackageReference Include="Microsoft.Azure.Cosmos.Direct" Version="[$(DirectVersion)]" PrivateAssets="All">
+			<ExcludeAssets>compile</ExcludeAssets>
+		</PackageReference>
 		<PackageReference Include="Microsoft.HybridRow" Version="[$(HybridRowVersion)]" PrivateAssets="All" />
 		<PackageReference Include="Microsoft.SourceLink.GitHub" Version="1.0.0" PrivateAssets="All" />
 	</ItemGroup>

Microsoft.Azure.Cosmos/tests/Microsoft.Azure.Cosmos.Tests/Contracts/DirectContractTests.cs

@@ -41,17 +41,6 @@ public void TestInteropTest()
             Marshal.Release(tryCreateServiceProvider.Result);
         }
 
-        [TestMethod]
-        public void PublicDirectTypes()
-        {
-            Assembly directAssembly = typeof(IStoreClient).Assembly;
-
-            Assert.IsTrue(directAssembly.FullName.StartsWith("Microsoft.Azure.Cosmos.Direct", System.StringComparison.Ordinal), directAssembly.FullName);
-
-            Type[] exportedTypes = directAssembly.GetExportedTypes();
-            Assert.AreEqual(0, exportedTypes.Length, string.Join(",", exportedTypes.Select(e => e.Name).ToArray()));
-        }
-
         [TestMethod]
         public void MappedRegionsTest()
         {
@@ -131,6 +120,7 @@ public void ProjectPackageDependenciesTest()
                 { "System.Diagnostics.DiagnosticSource", new Version(8, 0, 1) },
                 { "System.Net.Http", new Version(4, 3, 4) },
                 { "System.Text.RegularExpressions", new Version(4, 3, 1) },
+                { "System.Diagnostics.PerformanceCounter", new Version(6, 0, 0) }
             };
 
             Assert.AreEqual(projectDependencies.Count, baselineDependencies.Count);

Microsoft.Azure.Cosmos/tests/Microsoft.Azure.Cosmos.Tests/Telemetry/OpenTelemetryRecorderTests.cs

@@ -96,11 +96,13 @@ public async Task CheckResponseCompatibility()
             IEnumerable<Type> actualClasses = asm
                 .GetTypes()
                 .Where(type =>
-                                (type.Name.EndsWith("Response") || type.Name.EndsWith("Response`1")) && // Ending with Response and Response<T>
-                                !type.Name.Contains("OpenTelemetryResponse") && // Excluding OpenTelemetryResponse because we are testing this class
-                                !type.IsAbstract && // Excluding abstract classes
-                                !type.IsInterface && // Excluding interfaces
-                                !excludedResponses.Contains(type.Name)); // Excluding all the types defined in excludedResponses list
+                    (type.Name.EndsWith("Response") || type.Name.EndsWith("Response`1")) && // Ending with Response and Response<T>
+                    !type.Name.Contains("OpenTelemetryResponse") && // Excluding OpenTelemetryResponse because we are testing this class
+                    !type.IsAbstract && // Excluding abstract classes
+                    !type.IsInterface && // Excluding interfaces
+                    !excludedResponses.Contains(type.Name) && // Excluding all the types defined in excludedResponses list
+                    !(type.Namespace != null && type.Namespace.StartsWith("Microsoft.Azure.Documents")) // Exclude types from direct Microsoft.Azure.Documents
+                );
 
             foreach (Type className in actualClasses)
             {