[in-proc] Backport #10503 -- remove unused Microsoft.Azure.Cosmos.Tables package (#11435)

* remove unused DocumentDD.Core package (#10503)

* remove unused DocumentDD.Core package

* update release_notes.md

* Update deps.json

* Explicitly reference STJ in WebHost

* Publish deps.json as artifact

* Always copy deps.json artifact

* Fix copy deps.json task

* Update deps.json

* Copy net6/8 deps.json separately

Author: jviau

eng/ci/templates/jobs/run-unit-tests.yml

@@ -2,15 +2,60 @@ jobs:
 - job: RunUnitTests
   displayName: Run Unit Tests
 
+  variables:
+  - name: test_projects
+    value: |
+      **/ExtensionsMetadataGeneratorTests.csproj
+      **/WebJobs.Script.Tests.csproj
+
+  templateContext:
+    outputParentDirectory: $(Build.ArtifactStagingDirectory)
+    outputs:
+    # We publish this deps.json as differences in exact dotnet SDK version between dev and CI may make it impossible to generate locally.
+    - output: pipelineArtifact
+      displayName: Publish deps.json
+      path: $(Build.ArtifactStagingDirectory)
+      artifact: WebHost_Deps
+      condition: failed()
+
   steps:
   - template: /eng/ci/templates/install-dotnet.yml@self
 
+  - task: DotNetCoreCLI@2
+    displayName: Restore
+    inputs:
+      command: custom
+      custom: restore
+      arguments: -v m
+      projects: $(test_projects)
+
+  - task: DotNetCoreCLI@2
+    displayName: Build
+    inputs:
+      command: build
+      arguments: -v m -c release --no-restore
+      projects: $(test_projects)
+
   - task: DotNetCoreCLI@2
     displayName: Unit Tests
     inputs:
       command: test
       testRunTitle: Unit Tests
-      arguments: -v n
-      projects: |
-        **\ExtensionsMetadataGeneratorTests.csproj
-        **\WebJobs.Script.Tests.csproj
+      arguments: -v m -c release --no-build
+      projects: $(test_projects)
+
+  - task: CopyFiles@2
+    displayName: Copy deps.json (net6.0)
+    condition: failed()
+    inputs:
+      SourceFolder: out/bin/WebJobs.Script.WebHost/release_net6.0
+      Contents: '**/Microsoft.Azure.WebJobs.Script.WebHost.deps.json'
+      TargetFolder: $(Build.ArtifactStagingDirectory)/net6.0
+
+  - task: CopyFiles@2
+    displayName: Copy deps.json (net8.0)
+    condition: failed()
+    inputs:
+      SourceFolder: out/bin/WebJobs.Script.WebHost/release_net8.0
+      Contents: '**/Microsoft.Azure.WebJobs.Script.WebHost.deps.json'
+      TargetFolder: $(Build.ArtifactStagingDirectory)/net8.0

release_notes.md

@@ -1,8 +1,9 @@
-### Release notes
-
-<!-- Please add your release notes in the following format:
-- My change description (#PR)
--->
-- Add JitTrace files for v4.44
-- Updated to version 1.5.8 of Microsoft.Azure.AppService.Middleware.Functions for .NET 8 only (#11416)
-- Enabling worker indexing for Logic Apps behind an enviornment setting (#11377)
+### Release notes
+
+<!-- Please add your release notes in the following format:
+- My change description (#PR)
+-->
+- Add JitTrace files for v4.44
+- Updated to version 1.5.8 of Microsoft.Azure.AppService.Middleware.Functions for .NET 8 only (#11416)
+- Enabling worker indexing for Logic Apps behind an enviornment setting (#11377)
+- Remove packages `Microsoft.Azure.Cosmos.Table` and `Microsoft.Azure.DocumentDB.Core` (#11435)

src/WebJobs.Script.WebHost/WebJobs.Script.WebHost.csproj

@@ -52,7 +52,6 @@
     <PackageReference Include="Microsoft.ApplicationInsights.WindowsServer" Version="2.21.0" />
     <PackageReference Include="Microsoft.ApplicationInsights.WindowsServer.TelemetryChannel" Version="2.21.0" />
     <PackageReference Include="Microsoft.AspNet.WebApi.Client" Version="5.2.8" />
-    <PackageReference Include="Microsoft.Azure.Cosmos.Table" Version="1.0.8" />
     <PackageReference Include="Microsoft.Azure.Functions.Platform.Metrics.LinuxConsumption" Version="1.0.5" />
     <PackageReference Include="Microsoft.Azure.Storage.File" Version="11.1.7" />
     <PackageReference Include="Microsoft.Azure.WebJobs" Version="3.0.42" />
@@ -66,7 +65,6 @@
     <PackageReference Include="System.IdentityModel.Tokens.Jwt" Version="$(IdentityDependencyVersion)" />
     <PackageReference Include="Microsoft.IdentityModel.Protocols.OpenIdConnect" Version="$(IdentityDependencyVersion)" />
 
-
     <!--
       System.IO.FileSystem.AccessControl/5.0.0 pinned to avoid an unintented downgrade to 4.7.0.
       See https://github.com/Azure/azure-functions-host/pull/10002

test/WebJobs.Script.Tests/DependencyTests.cs

@@ -64,8 +64,8 @@ public void Verify_DepsJsonChanges()
             Assert.True(File.Exists(oldDepsJson), $"{oldDepsJson} not found.");
             Assert.True(File.Exists(newDepsJson), $"{newDepsJson} not found.");
 
-            IEnumerable<RuntimeFile> oldAssets = GetRuntimeFiles(oldDepsJson);
-            IEnumerable<RuntimeFile> newAssets = GetRuntimeFiles(newDepsJson);
+            IEnumerable<RuntimeFile> oldAssets = GetRuntimeFiles(oldDepsJson).ToList();
+            IEnumerable<RuntimeFile> newAssets = GetRuntimeFiles(newDepsJson).ToList();
 
             var comparer = new RuntimeFileComparer();
 
@@ -166,8 +166,7 @@ public bool Equals([AllowNull] RuntimeFile x, [AllowNull] RuntimeFile y)
 
             public int GetHashCode([DisallowNull] RuntimeFile obj)
             {
-                string code = obj.Path + obj.AssemblyVersion + obj.FileVersion;
-                return code.GetHashCode();
+                return HashCode.Combine(obj.Path, obj.AssemblyVersion, obj.FileVersion);
             }
         }
     }