From ae166f7827b0cbcba3586d0b3842a9cf9415795f Mon Sep 17 00:00:00 2001
From: Daniel Cazzulino <daniel@cazzulino.com>
Date: Sat, 6 Jul 2024 04:28:13 -0300
Subject: [PATCH] Fix tests loading the right analyzer assemblies

---
 src/SponsorLink/SponsorLink.targets            | 14 ++++++++++----
 src/SponsorLink/SponsorLink/SponsorLink.csproj |  1 +
 src/SponsorLink/Tests/Tests.csproj             | 12 +++++++++---
 3 files changed, 20 insertions(+), 7 deletions(-)

diff --git a/src/SponsorLink/SponsorLink.targets b/src/SponsorLink/SponsorLink.targets
index 53e7c6d..72304cb 100644
--- a/src/SponsorLink/SponsorLink.targets
+++ b/src/SponsorLink/SponsorLink.targets
@@ -7,6 +7,7 @@
     <ShowSponsorLinkInProject Condition="$(ShowSponsorLinkInProject) == '' and '$(TargetFramework)' == 'netstandard2.0'">true</ShowSponsorLinkInProject>
     <!-- This ensures we expose only the main assembly in the Dependencies > Analyzers node -->
     <MergeAnalyzerAssemblies Condition="'$(MergeAnalyzerAssemblies)' == '' and '$(Configuration)' == 'Release' and '$(OS)' == 'Windows_NT'">true</MergeAnalyzerAssemblies>
+    <MergeAnalyzerAssemblies Condition="'$(MergeAnalyzerAssemblies)' == ''">false</MergeAnalyzerAssemblies>
     <!-- Whether we merge files or not, dependencies will need to be copy-local -->
     <CopyLocalLockFileAssemblies>true</CopyLocalLockFileAssemblies>
     <!-- Make Resources visible to intellisense -->
@@ -69,17 +70,22 @@
     </None>
   </ItemGroup>
 
+  <PropertyGroup>
+    <PackMergedAssemblies>true</PackMergedAssemblies>
+    <PackMergedAssemblies Condition="'$(MergeAnalyzerAssemblies)' == 'true'">false</PackMergedAssemblies>
+  </PropertyGroup>
+
   <ItemGroup Condition="'$(ManagePackageVersionsCentrally)' == 'true'">
-    <PackageReference Include="Humanizer.Core" VersionOverride="2.14.1" PrivateAssets="all" Pack="false" />
+    <PackageReference Include="Humanizer.Core" VersionOverride="2.14.1" PrivateAssets="all" Pack="$(PackMergedAssemblies)" />
     <PackageReference Include="Humanizer.Core.es" VersionOverride="2.14.1" PrivateAssets="all" />
-    <PackageReference Include="Microsoft.IdentityModel.JsonWebTokens" VersionOverride="7.6.2" PrivateAssets="all" Pack="false" />
+    <PackageReference Include="Microsoft.IdentityModel.JsonWebTokens" VersionOverride="7.6.2" PrivateAssets="all" Pack="$(PackMergedAssemblies)" />
     <PackageReference Include="ILRepack" Version="2.0.33" VersionOverride="all" PrivateAssets="all" Pack="false" />
   </ItemGroup>
 
   <ItemGroup Condition="'$(ManagePackageVersionsCentrally)' != 'true'">
-    <PackageReference Include="Humanizer.Core" Version="2.14.1" PrivateAssets="all" Pack="false" />
+    <PackageReference Include="Humanizer.Core" Version="2.14.1" PrivateAssets="all" Pack="$(PackMergedAssemblies)" />
     <PackageReference Include="Humanizer.Core.es" Version="2.14.1" PrivateAssets="all" />
-    <PackageReference Include="Microsoft.IdentityModel.JsonWebTokens" Version="7.6.2" PrivateAssets="all" Pack="false" />
+    <PackageReference Include="Microsoft.IdentityModel.JsonWebTokens" Version="7.6.2" PrivateAssets="all" Pack="$(PackMergedAssemblies)" />
     <PackageReference Include="ILRepack" Version="2.0.33" PrivateAssets="all"  Pack="false" />
   </ItemGroup>
 
diff --git a/src/SponsorLink/SponsorLink/SponsorLink.csproj b/src/SponsorLink/SponsorLink/SponsorLink.csproj
index 0821365..6479792 100644
--- a/src/SponsorLink/SponsorLink/SponsorLink.csproj
+++ b/src/SponsorLink/SponsorLink/SponsorLink.csproj
@@ -6,6 +6,7 @@
     <ImplicitUsings>disable</ImplicitUsings>
     <GenerateDocumentationFile>false</GenerateDocumentationFile>
     <CoreCompileDependsOn>CoreResGen;$(CoreCompileDependsOn)</CoreCompileDependsOn>
+    <FundingPackageId>SponsorLink</FundingPackageId>
   </PropertyGroup>
 
   <PropertyGroup Label="SponsorLink">
diff --git a/src/SponsorLink/Tests/Tests.csproj b/src/SponsorLink/Tests/Tests.csproj
index f596efb..fd98b13 100644
--- a/src/SponsorLink/Tests/Tests.csproj
+++ b/src/SponsorLink/Tests/Tests.csproj
@@ -23,9 +23,6 @@
   <ItemGroup>
     <!-- This project reference allows debugging the source generator/analyzer project -->
     <ProjectReference Include="..\Analyzer\Analyzer.csproj" Aliases="Analyzer" />
-    <Analyzer Include="..\Analyzer\bin\$(Configuration)\netstandard2.0\*.dll" Visible="false" />
-    <!-- Make sure we only annotate the primary analyzer file, as it would when the package is installed -->
-    <Analyzer Update="@(Analyzer -&gt; WithMetadataValue('Filename', 'SponsorableLib.Analyzers'))" NuGetPackageId="SponsorableLib" />
   </ItemGroup>
 
   <ItemGroup>
@@ -36,6 +33,15 @@
     <Constant Include="PublicKey" Value="$([System.IO.File]::ReadAllText('$(MSBuildThisFileDirectory)keys\kzu.pub.jwk'))" />
   </ItemGroup>
 
+  <Target Name="AddBuiltAnalyzers" BeforeTargets="GenerateMSBuildEditorConfigFileShouldRun">
+    <!-- Waits until after building the referenced analyzer project when all binaries are in the output dir -->
+    <ItemGroup>
+      <Analyzer Include="..\Analyzer\bin\$(Configuration)\netstandard2.0\*.dll" Visible="false" />
+      <!-- Make sure we only annotate the primary analyzer file, as it would when the package is installed -->
+      <Analyzer Update="@(Analyzer -&gt; WithMetadataValue('Filename', 'SponsorableLib.Analyzers'))" NuGetPackageId="SponsorableLib" />
+    </ItemGroup>
+  </Target>
+
   <Target Name="Sample_GitConfig" Condition="'$(SourceControlInformationFeatureSupported)' == 'true'" DependsOnTargets="InitializeSourceControlInformation">
     <!-- Follows recommendation from Microsoft.Common.CurrentVersion.targets on how to depend on SCC info. -->
     <ItemGroup>