Add DefaultInformationalVersion for backwards compatibility

Author: mgnslndh

src/GitVersionCore.Tests/VariableProviderTests.cs

@@ -1,30 +1,49 @@
-using System;
+using GitVersion;
 using GitVersion.Logging;
-using NUnit.Framework;
-using Shouldly;
 using GitVersion.OutputFormatters;
 using GitVersion.OutputVariables;
-using GitVersion;
 using GitVersion.VersionCalculation;
 using GitVersion.VersioningModes;
+using NUnit.Framework;
+using Shouldly;
+using System;
+using System.Collections.Generic;
 
 namespace GitVersionCore.Tests
 {
     [TestFixture]
     public class VariableProviderTests : TestBase
     {
         private IVariableProvider variableProvider;
+        private List<string> logMessages;
 
         [SetUp]
         public void Setup()
         {
             ShouldlyConfiguration.ShouldMatchApprovedDefaults.LocateTestMethodUsingAttribute<TestAttribute>();
-            var log = new NullLog();
+            logMessages = new List<string>();
+            var log = new Log(new TestLogAppender(logMessages.Add));
             var metaDataCalculator = new MetaDataCalculator();
             var baseVersionCalculator = new BaseVersionCalculator(log, null);
             var mainlineVersionCalculator = new MainlineVersionCalculator(log, metaDataCalculator);
             var nextVersionCalculator = new NextVersionCalculator(log, metaDataCalculator, baseVersionCalculator, mainlineVersionCalculator);
-            variableProvider = new VariableProvider(nextVersionCalculator, new TestEnvironment());
+            variableProvider = new VariableProvider(nextVersionCalculator, new TestEnvironment(), log);
+        }
+
+        [Test]
+        public void ShouldLogWarningWhenUsingDefaultInformationalVersionInCustomFormat()
+        {
+            var semVer = new SemanticVersion
+            {
+                Major = 1,
+                Minor = 2,
+                Patch = 3,
+            };
+
+            var propertyName = nameof(SemanticVersionFormatValues.DefaultInformationalVersion);
+            var config = new TestEffectiveConfiguration(assemblyInformationalFormat: $"{{{propertyName}}}");
+            variableProvider.GetVariablesFor(semVer, config, false);
+            logMessages.ShouldContain(message => message.Trim().StartsWith("WARN") && message.Contains(propertyName), 1, $"Expected a warning to be logged when using the variable {propertyName} in a configuration format template");
         }
 
         [Test]

src/GitVersionCore/OutputVariables/VariableProvider.cs

@@ -6,18 +6,21 @@
 using GitVersion.VersioningModes;
 using GitVersion.Configuration;
 using GitVersion.Helpers;
+using GitVersion.Logging;
 
 namespace GitVersion.OutputVariables
 {
     public class VariableProvider : IVariableProvider
     {
         private readonly INextVersionCalculator nextVersionCalculator;
         private readonly IEnvironment environment;
+        private readonly ILog log;
 
-        public VariableProvider(INextVersionCalculator nextVersionCalculator, IEnvironment environment)
+        public VariableProvider(INextVersionCalculator nextVersionCalculator, IEnvironment environment, ILog log = default)
         {
             this.nextVersionCalculator = nextVersionCalculator ?? throw new ArgumentNullException(nameof(nextVersionCalculator));
             this.environment = environment;
+            this.log = log ?? new NullLog();
         }
 
         public VersionVariables GetVariablesFor(SemanticVersion semanticVersion, EffectiveConfiguration config, bool isCurrentCommitTagged)
@@ -124,7 +127,7 @@ private static void PromoteNumberOfCommitsToTagNumber(SemanticVersion semanticVe
             }
         }
 
-        private static string CheckAndFormatString<T>(string formatString, T source, IEnvironment environment, string defaultValue, string formatVarName)
+        private string CheckAndFormatString<T>(string formatString, T source, IEnvironment environment, string defaultValue, string formatVarName)
         {
             string formattedString;
 
@@ -134,6 +137,8 @@ private static string CheckAndFormatString<T>(string formatString, T source, IEn
             }
             else
             {
+                WarnIfUsingObsoleteFormatValues(formatString);
+
                 try
                 {
                     formattedString = formatString.FormatWith(source, environment).RegexReplace("[^0-9A-Za-z-.+]", "-");
@@ -146,5 +151,14 @@ private static string CheckAndFormatString<T>(string formatString, T source, IEn
 
             return formattedString;
         }
+
+        private void WarnIfUsingObsoleteFormatValues(string formatString)
+        {
+            var obsoletePropertyName = nameof(SemanticVersionFormatValues.DefaultInformationalVersion);
+            if (formatString.Contains($"{{{obsoletePropertyName}}}"))
+            {
+                log.Write(LogLevel.Warn, $"Use format variable '{nameof(SemanticVersionFormatValues.InformationalVersion)}' instead of '{obsoletePropertyName}' which is obsolete and will be removed in a future release.");
+            }
+        }
     }
 }

src/GitVersionCore/OutputVariables/VersionVariables.cs

@@ -1,6 +1,7 @@
 using System;
 using System.Collections;
 using System.Collections.Generic;
+using System.Globalization;
 using System.IO;
 using System.Linq;
 using YamlDotNet.Serialization;

src/GitVersionCore/SemanticVersioning/SemanticVersionFormatValues.cs

@@ -1,3 +1,4 @@
+using System;
 using System.Globalization;
 using GitVersion.Configuration;
 using GitVersion.Extensions;
@@ -71,6 +72,9 @@ public SemanticVersionFormatValues(SemanticVersion semver, EffectiveConfiguratio
 
         public string InformationalVersion => semver.ToString("i");
 
+        [Obsolete("Use InformationalVersion instead")]
+        public string DefaultInformationalVersion => InformationalVersion;
+
         public string VersionSourceSha => semver.BuildMetaData.VersionSourceSha;
 
         public string CommitsSinceVersionSource => semver.BuildMetaData.CommitsSinceVersionSource.ToString(CultureInfo.InvariantCulture);