Ensures date comparisons in schedule integration tests are made only on the datetime part to the second (#18894)

* Ensures date comparisons in schedule integration tests are made only on the date part.

* Include time part to the second.

* Ensure Kind is retained when truncating a date.

* Retain Kind for all truncation levels.

Author: AndyButland

src/Umbraco.Core/Extensions/DateTimeExtensions.cs

@@ -7,6 +7,9 @@ namespace Umbraco.Extensions;
 
 public static class DateTimeExtensions
 {
+    /// <summary>
+    /// Defines the levels to truncate a date to.
+    /// </summary>
     public enum DateTruncate
     {
         Year,
@@ -25,33 +28,39 @@ public enum DateTruncate
     public static string ToIsoString(this DateTime dt) =>
         dt.ToString("yyyy-MM-dd HH:mm:ss", CultureInfo.InvariantCulture);
 
+    /// <summary>
+    /// Truncates the date to the specified level, i.e. if you pass in DateTruncate.Hour it will truncate the date to the hour.
+    /// </summary>
+    /// <param name="dt">The date.</param>
+    /// <param name="truncateTo">The level to truncate the date to.</param>
+    /// <returns>The truncated date.</returns>
     public static DateTime TruncateTo(this DateTime dt, DateTruncate truncateTo)
     {
         if (truncateTo == DateTruncate.Year)
         {
-            return new DateTime(dt.Year, 1, 1);
+            return new DateTime(dt.Year, 1, 1, 0, 0, 0, dt.Kind);
         }
 
         if (truncateTo == DateTruncate.Month)
         {
-            return new DateTime(dt.Year, dt.Month, 1);
+            return new DateTime(dt.Year, dt.Month, 1, 0, 0, 0, dt.Kind);
         }
 
         if (truncateTo == DateTruncate.Day)
         {
-            return new DateTime(dt.Year, dt.Month, dt.Day);
+            return new DateTime(dt.Year, dt.Month, dt.Day, 0, 0, 0, dt.Kind);
         }
 
         if (truncateTo == DateTruncate.Hour)
         {
-            return new DateTime(dt.Year, dt.Month, dt.Day, dt.Hour, 0, 0);
+            return new DateTime(dt.Year, dt.Month, dt.Day, dt.Hour, 0, 0, dt.Kind);
         }
 
         if (truncateTo == DateTruncate.Minute)
         {
-            return new DateTime(dt.Year, dt.Month, dt.Day, dt.Hour, dt.Minute, 0);
+            return new DateTime(dt.Year, dt.Month, dt.Day, dt.Hour, dt.Minute, 0, dt.Kind);
         }
 
-        return new DateTime(dt.Year, dt.Month, dt.Day, dt.Hour, dt.Minute, dt.Second);
+        return new DateTime(dt.Year, dt.Month, dt.Day, dt.Hour, dt.Minute, dt.Second, dt.Kind);
     }
 }

tests/Umbraco.Tests.Integration/Umbraco.Core/Services/ContentPublishingServiceTests.cs

@@ -1,4 +1,3 @@
-using Bogus.DataSets;
 using NUnit.Framework;
 using Umbraco.Cms.Core;
 using Umbraco.Cms.Core.Models;
@@ -23,8 +22,8 @@ public class ContentPublishingServiceTests : UmbracoIntegrationTestWithContent
 {
     private const string UnknownCulture = "ke-Ke";
 
-    private readonly DateTime _schedulePublishDate = DateTime.UtcNow.AddDays(1);
-    private readonly DateTime _scheduleUnPublishDate = DateTime.UtcNow.AddDays(2);
+    private readonly DateTime _schedulePublishDate = DateTime.UtcNow.AddDays(1).TruncateTo(DateTimeExtensions.DateTruncate.Second);
+    private readonly DateTime _scheduleUnPublishDate = DateTime.UtcNow.AddDays(2).TruncateTo(DateTimeExtensions.DateTruncate.Second);
 
     [SetUp]
     public new void Setup() => ContentRepositoryBase.ThrowOnWarning = true;