Make sure project properties initialize after source control

If source control information is supported, make sure we automatically take a dependency on InitializeSourceControlInformation. We take the chance to also rename the old property `InjectThisAssemblyProjectDependsOn` to `PrepareProjectPropertiesDependsOn` which aligns better with the new target.
devlooped · Nov 21, 2024 · ec77de1 · ec77de1
1 parent fe8efd0
commit ec77de1
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 3 deletions.
diff --git a/src/ThisAssembly.Project/ThisAssembly.Project.targets b/src/ThisAssembly.Project/ThisAssembly.Project.targets
@@ -13,12 +13,14 @@
   </ItemGroup>
 
   <PropertyGroup>
-    <InjectThisAssemblyProjectDependsOn Condition="'$(SourceControlInformationFeatureSupported)' == 'true'">
+    <!-- Backwards compability for original property name -->
+    <PrepareProjectPropertiesDependsOn>$(InjectThisAssemblyProjectDependsOn)</PrepareProjectPropertiesDependsOn>
+    <PrepareProjectPropertiesDependsOn Condition="'$(SourceControlInformationFeatureSupported)' == 'true'">
       InitializeSourceControlInformation
-    </InjectThisAssemblyProjectDependsOn>
+    </PrepareProjectPropertiesDependsOn>
   </PropertyGroup>
 
-  <Target Name="PrepareProjectProperties" BeforeTargets="PrepareConstants">
+  <Target Name="PrepareProjectProperties" BeforeTargets="PrepareConstants" DependsOnTargets="$(PrepareProjectPropertiesDependsOn)">
     <ItemGroup>
       <ProjectPropertyDistinct Include="@(ProjectProperty -> Distinct())" />
     </ItemGroup>

diff --git a/src/ThisAssembly.Tests/Tests.cs b/src/ThisAssembly.Tests/Tests.cs
@@ -52,6 +52,13 @@ public void CanUseProjectFullFileContents()
         Assert.False(ThisAssembly.Project.ProjectFile.StartsWith("|"));
     }
 
+    /// <summary />
+    [Fact]
+    public void CanUseProjectRepositoryUrl()
+    {
+        Assert.NotEmpty(ThisAssembly.Project.RepositoryUrl);
+    }
+
     /// <summary />
     [Fact]
     public void CanUseConstants()