(#387) Added support for Central Package Mangagement

src/Directory.Build.props

@@ -0,0 +1,5 @@
+<Project>
+  <PropertyGroup>
+    <LangVersion>latest</LangVersion>
+  </PropertyGroup>
+</Project>

src/Generators/Generators.csproj

@@ -2,7 +2,7 @@
 
     <PropertyGroup>
         <OutputType>Exe</OutputType>
-        <TargetFramework>net7.0</TargetFramework>
+        <TargetFramework>net9.0</TargetFramework>
         <ImplicitUsings>enable</ImplicitUsings>
         <Nullable>enable</Nullable>
     </PropertyGroup>

src/Guidelines/build/CakeInternalReferences.targets

@@ -16,6 +16,7 @@
             ProjectType="$(CakeContribGuidelinesProjectType)"
             CakeVersion="$(CakeContribGuidelinesOverrideTargetFrameworkCakeVersion)"
             References="@(PackageReference)"
+            PackageVersions="@(PackageVersion)"
             NoWarn="$(NoWarn)"
             WarningsAsErrors="$(WarningsAsErrors)"
         />

src/Guidelines/build/RecommendedCakeVersion.targets

@@ -22,6 +22,7 @@
             ProjectFile="$(MSBuildProjectFullPath)"
             RecommendedVersion="$(RecommendedCakeVersion)"
             References="@(PackageReference)"
+            PackageVersions="@(PackageVersion)"
             Omitted="@(CakeContribGuidelinesOmitRecommendedCakeVersion)"
             ReferencesToCheck="@(CakeReferenceToCheck)"
             ProjectType="$(CakeContribGuidelinesProjectType)"

src/Guidelines/build/TargetFrameworkVersions.targets

@@ -16,6 +16,7 @@
         <TargetFrameworkVersions
             ProjectFile="$(MSBuildProjectFullPath)"
             References="@(PackageReference)"
+            PackageVersions="@(PackageVersion)"
             TargetFramework="$(TargetFramework)"
             TargetFrameworks="$(TargetFrameworks)"
             Omitted="@(CakeContribGuidelinesOmitTargetFramework)"

src/Tasks.IntegrationTests/E2eTests.cs

@@ -9,7 +9,7 @@
 using Xunit;
 using Xunit.Abstractions;
 
 namespace CakeContrib.Guidelines.Tasks.IntegrationTests
 {
     // TODO: Writing things to disk is not deterministic...
     // TODO: Running Code-Coverage on the integration-tests breaks all tests
@@ -121,7 +121,7 @@
         {
             // given
             fixture.WithoutDefaultCakeReference();
-            fixture.WithPackageReference("Cake.Core", "0.38.5", "all");
+            fixture.WithPackageReference("Cake.Core", fixture.DefaultCakeVersion, "all");
 
             // when
             var result = fixture.Run();
@@ -136,7 +136,7 @@
         {
             // given
             fixture.WithoutDefaultCakeReference();
-            fixture.WithPackageReference("Cake.Core", "0.38.5");
+            fixture.WithPackageReference("Cake.Core", fixture.DefaultCakeVersion);
 
             // when
             var result = fixture.Run();
@@ -221,22 +221,27 @@
             // then
             result.IsErrorExitCode.ShouldBeTrue();
             result.ErrorLines.ShouldContain(l => l.IndexOf("CCG0007", StringComparison.Ordinal) > -1);
-            result.ErrorLines.ShouldContain(l => l.IndexOf("netstandard2.0", StringComparison.Ordinal) > -1);
+            result.ErrorLines.ShouldContain(l => l.IndexOf(fixture.DefaultTargetFrameworkForModules, StringComparison.Ordinal) > -1);
         }
 
         [Fact]
         public void Missing_Suggested_Target_results_in_CCG0007_warning()
         {
+            var missingTfm = fixture.DefaultTargetFrameworksForAddins
+                .Split(";", StringSplitOptions.RemoveEmptyEntries)
+                .First();
+            var allTfmButMissing = fixture.DefaultTargetFrameworksForAddins.Replace(missingTfm, string.Empty);
+
             // given
-            fixture.WithTargetFrameworks("netstandard2.0");
+            fixture.WithTargetFrameworks(allTfmButMissing);
 
             // when
             var result = fixture.Run();
 
             // then
-            result.IsErrorExitCode.ShouldBeFalse();
-            result.WarningLines.ShouldContain(l => l.IndexOf("CCG0007", StringComparison.Ordinal) > -1);
-            result.WarningLines.ShouldContain(l => l.IndexOf("net461", StringComparison.Ordinal) > -1);
+            result.IsErrorExitCode.ShouldBeTrue();
+            result.ErrorLines.ShouldContain(l => l.IndexOf("CCG0007", StringComparison.Ordinal) > -1);
+            result.ErrorLines.ShouldContain(l => l.IndexOf(missingTfm, StringComparison.Ordinal) > -1);
         }
 
         [Fact]
@@ -246,6 +251,7 @@
             fixture.WithoutDefaultCakeReference();
             fixture.WithPackageReference("Cake.Core", "1.0.0", "all");
             fixture.WithTargetFrameworks("netstandard2.0;net461");
+            fixture.OmitRecommendedCakeVersion();
 
             // when
             var result = fixture.Run();
@@ -351,7 +357,7 @@
         {
             // given
             fixture.WithAssemblyName("Cake.Buildsystems.Module");
-            fixture.WithTargetFrameworks("netstandard2.0");
+            fixture.WithTargetFrameworks(fixture.DefaultTargetFrameworkForModules);
             fixture.WithCustomContent(@"
 <Target Name=""ForTest""
   AfterTargets=""BeforeBuild""
@@ -376,7 +382,7 @@
         public void ProjectType_When_PackageId_Is_Module_Is_Module()
         {
             // given
-            fixture.WithTargetFrameworks("netstandard2.0");
+            fixture.WithTargetFrameworks(fixture.DefaultTargetFrameworkForModules);
             fixture.WithCustomContent(@"
 <PropertyGroup>
   <PackageId>Cake.Buildsystems.Module</PackageId>
@@ -498,7 +504,7 @@
             fixture.WithAssemblyName(assemblyName);
             if (isModule)
             {
-                fixture.WithTargetFrameworks("netstandard2.0");
+                fixture.WithTargetFrameworks(fixture.DefaultTargetFrameworkForModules);
             }
             fixture.WithCustomContent(@"
 <PropertyGroup>
@@ -537,7 +543,7 @@
             fixture.WithAssemblyName(assemblyName);
             if (isModule)
             {
-                fixture.WithTargetFrameworks("netstandard2.0");
+                fixture.WithTargetFrameworks(fixture.DefaultTargetFrameworkForModules);
             }
             fixture.WithCustomContent(@"
 <PropertyGroup>
@@ -589,7 +595,7 @@
   <PackageId>Cake.Buildsystems.Module</PackageId>
 </PropertyGroup>");
             fixture.WithTags("cake build cake-build script");
-            fixture.WithTargetFrameworks("netstandard2.0");
+            fixture.WithTargetFrameworks(fixture.DefaultTargetFrameworkForModules);
 
             // when
             var result = fixture.Run();
@@ -651,6 +657,7 @@
             // given
             fixture.WithoutDefaultCakeReference();
             fixture.WithPackageReference("cake.core", "0.38.5", "All");
+            fixture.WithTargetFrameworks("netstandard2.0;net461;net5.0");
 
             // when
             var result = fixture.Run();
@@ -667,6 +674,7 @@
             // given
             fixture.WithoutDefaultCakeReference();
             fixture.WithPackageReference("cake.core", "0.38.5", "All");
+            fixture.WithTargetFrameworks("netstandard2.0;net461;net5.0");
             fixture.WithCustomContent(@"
 <ItemGroup>
     <CakeContribGuidelinesOmitRecommendedCakeVersion Include=""Cake.Core"" />
@@ -680,29 +688,11 @@
             result.WarningLines.ShouldBeEmpty();
         }
 
-        [Fact]
-        public void Missing_Suggested_Target_results_not_in_CCG0007_warning_if_NoWarn_is_set()
-        {
-            // given
-            fixture.WithTargetFrameworks("netstandard2.0");
-            fixture.WithCustomContent(@"
-<PropertyGroup>
-    <NoWarn>1701;1702;ccg0007</NoWarn>
-</PropertyGroup>");
-
-            // when
-            var result = fixture.Run();
-
-            // then
-            result.IsErrorExitCode.ShouldBeFalse();
-            result.WarningLines.ShouldNotContain(l => l.IndexOf("CCG0007", StringComparison.Ordinal) > -1);
-        }
-
         [Fact]
         public void Missing_Suggested_Target_results_in_CCG0007_error_if_WarningsAsErrors_is_set()
         {
             // given
-            fixture.WithTargetFrameworks("netstandard2.0");
+            fixture.WithTargetFrameworks(fixture.DefaultTargetFrameworkForModules);
             fixture.WithCustomContent(@"
 <PropertyGroup>
     <WarningsAsErrors>NU1605;ccg0007</WarningsAsErrors >
@@ -738,5 +728,21 @@
             err.ShouldNotBeNull();
             err.ShouldContain("netstandard2.0");
         }
+
+        [Fact]
+        public void Central_Package_Management_Correct_Cake_Reference_Should_Not_Raise_CCG0009()
+        {
+            // given
+            fixture.WithoutDefaultCakeReference();
+            fixture.WithPackageReference("cake.core", privateAssets: "All");
+            fixture.WithCpmPackageVersion("Cake.Core", fixture.DefaultCakeVersion);
+
+            // when
+            var result = fixture.Run();
+
+            // then
+            result.IsErrorExitCode.ShouldBeFalse();
+            result.WarningLines.ShouldBeEmpty();
+        }
     }
 }

src/Tasks.IntegrationTests/Fixtures/E2eTestFixture.cs

@@ -22,11 +22,16 @@
         private bool hasEditorConfig = true;
         private bool omitRecommendedCakeVersion = false;
         private bool hasDefaultCakeReference = true;
-        private readonly List<string> customContent = new List<string>();
-        private string targetFrameworks = "netstandard2.0;net461;net5.0";
-        private readonly List<string> references = new List<string>();
+        private readonly List<string> customContent = new();
+        private string targetFrameworks = "net8.0;net9.0";
+        private readonly List<string> references = new();
+        private readonly Dictionary<string, string> cpmPackageVersions = new();
         private string tags = "cake;cake-build;build;script;addin;cake-addin;module;cake-module;recipe;cake-recipe";
 
+        public string DefaultCakeVersion => "5.0.0";
+        public string DefaultTargetFrameworkForModules => "net8.0";
+        public string DefaultTargetFrameworksForAddins => "net8.0;net9.0";
+
         public E2eTestFixture(string tempFolder, ITestOutputHelper logger)
         {
             this.tempFolder = tempFolder;
@@ -78,6 +83,7 @@
             {
                 properties.Add($"<PackageTags>{tags}</PackageTags>");
             }
+
             if (hasStylecopJson)
             {
                 var stylecopJson = Path.Combine(tempFolder, "stylecop.json");
@@ -95,16 +101,22 @@
             }
             if (hasStylecopReference)
             {
-                items.Add(@"
-<PackageReference Include=""StyleCop.Analyzers"" Version=""1.1.118"">
+                var version = "Version=\"1.1.118\"";
+                if (cpmPackageVersions.Any())
+                {
+                    version = string.Empty;
+                }
+
+                items.Add($@"
+<PackageReference Include=""StyleCop.Analyzers"" {version}>
     <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
     <PrivateAssets>all</PrivateAssets>
 </PackageReference>");
             }
 
             if (hasDefaultCakeReference)
             {
-                WithPackageReference("cake.core","1.0.0", "all");
+                WithPackageReference("cake.core", DefaultCakeVersion, "all");
             }
 
             if (omitRecommendedCakeVersion)
@@ -116,6 +128,32 @@
 </ItemGroup>");
             }
 
+            if (cpmPackageVersions.Any())
+            {
+                if (hasStylecopReference)
+                {
+                    cpmPackageVersions.Add("StyleCop.Analyzers", "1.1.118");
+                }
+
+                var cpmFile = Path.Combine(tempFolder, "Directory.Packages.props");
+                File.WriteAllText(cpmFile,
+$"""
+<Project>
+  <PropertyGroup>
+    <ManagePackageVersionsCentrally>true</ManagePackageVersionsCentrally>
+  </PropertyGroup>
+  <ItemGroup>
+    {
+        string.Join(
+            Environment.NewLine,
+            cpmPackageVersions.Select(kvp => $"<PackageVersion Include=\"{kvp.Key}\" Version=\"{kvp.Value}\" />").ToArray()
+        )
+    }
+  </ItemGroup>
+</Project>
+""");
+            }
+
             items.AddRange(references);
 
             File.WriteAllText(csproj, string.Format(template,
@@ -157,12 +195,18 @@
 
         internal void WithPackageReference(
             string packageName,
-            string version,
+            string version = null,
             string privateAssets = null,
             params Tuple<string, string>[] additionalAttributes)
         {
             var reference = new StringBuilder();
-            reference.Append($@"<PackageReference Include=""{packageName}"" Version=""{version}""");
+            reference.Append($@"<PackageReference Include=""{packageName}""");
+
+            if (version != null)
+            {
+                reference.Append($@" Version=""{version}""");
+            }
+
             if (privateAssets != null)
             {
                 reference.Append($@" PrivateAssets=""{privateAssets}""");
@@ -177,6 +221,13 @@
             references.Add(reference.ToString());
         }
 
+        internal void WithCpmPackageVersion(
+            string packageName,
+            string version)
+        {
+            this.cpmPackageVersions.Add(packageName, version);
+        }
+
         internal void WithoutStylecopReference()
         {
             hasStylecopReference = false;
@@ -204,7 +255,7 @@
 
         private Tuple<string, string> GetTargetsToImport()
         {
             var codeBase = typeof(E2eTestFixture).Assembly.CodeBase;
             var assemblyPath = Uri.UnescapeDataString(new UriBuilder(codeBase).Path);
             var folder = Path.GetDirectoryName(assemblyPath);
 

src/Tasks.IntegrationTests/Tasks.IntegrationTests.csproj

@@ -1,7 +1,7 @@
 <Project Sdk="Microsoft.NET.Sdk">
 
     <PropertyGroup>
-        <TargetFrameworks>net6.0;net7.0;</TargetFrameworks>
+        <TargetFrameworks>net8.0;net9.0;</TargetFrameworks>
         <TargetFrameworks Condition="'$(OS)'!='Unix'">$(TargetFrameworks);net472</TargetFrameworks>
         <AssemblyName>CakeContrib.Guidelines.Tasks.IntegrationTests</AssemblyName>
         <RootNamespace>CakeContrib.Guidelines.Tasks.IntegrationTests</RootNamespace>