Merge pull request #1334 from microsoft/fixSigningErrors

Sign native dll and exclude test exe from sign check

Author: AArnott

azure-pipelines/Get-SymbolFiles.ps1

@@ -18,7 +18,7 @@ Write-Progress -Activity $ActivityName -CurrentOperation "Discovery PDB files"
 $PDBs = Get-ChildItem -rec "$Path/*.pdb"
 
 # Filter PDBs to product OR test related.
-$testregex = "unittest|tests|\.test\."
+$testregex = "unittest|tests|\.test\.|TestHost"
 
 Write-Progress -Activity $ActivityName -CurrentOperation "De-duplicating symbols"
 $PDBsByHash = @{}

src/SosThreadingTools/SosThreadingTools.targets

@@ -26,6 +26,12 @@
     <GenerateNuspecDependsOn>StampAndIncludeGalleryManifest;$(GenerateNuspecDependsOn)</GenerateNuspecDependsOn>
   </PropertyGroup>
 
+  <ItemGroup>
+    <FilesToSign Include="$(TargetDir)$(DnneNativeBinaryName).dll">
+      <StrongName />
+    </FilesToSign>
+  </ItemGroup>
+
   <Target Name="PackBuildOutputs" DependsOnTargets="DebugSymbolsProjectOutputGroup">
     <ItemGroup>
       <TfmSpecificPackageFile Include="$(TargetPath)" PackagePath="tools\$(RuntimeIdentifier)\" />