test: Add comprehensive test coverage for continuous aggregate policies

Add unit tests for previously untested components: builder validation,                                                                                                                                                                                                                                         model extractor, annotation applier, and convention error handling. These tests cover input validation, default value handling, schema                                                                                                                                                                                                                      resolution, annotation application, and fluent API method chaining.

Author: sebastian-ederer

samples/Eftdb.Samples.Shared/Configurations/TradeAggregateConfiguration.cs

@@ -21,7 +21,6 @@ public void Configure(EntityTypeBuilder<TradeAggregate> builder)
                 .Where("\"ticker\" = 'MCRS'")
                 .MaterializedOnly()
                 .WithRefreshPolicy(startOffset: "7 days", endOffset: "1 hour", scheduleInterval: "1 hour")
-                .WithTimezone("UTC")
                 .WithRefreshNewestFirst(true);
         }
     }

samples/Eftdb.Samples.Shared/Models/WeatherAggregate.cs

@@ -23,7 +23,6 @@ namespace CmdScale.EntityFrameworkCore.TimescaleDB.Example.DataAccess.Models
         StartOffset = "30 days",
         EndOffset = "1 day",
         ScheduleInterval = "1 hour",
-        Timezone = "UTC",
         RefreshNewestFirst = true)]
     public class WeatherAggregate
     {

src/Eftdb.Design/Scaffolding/ContinuousAggregatePolicyAnnotationApplier.cs

@@ -36,12 +36,6 @@ public void ApplyAnnotations(DatabaseTable table, object featureInfo)
                 table[ContinuousAggregatePolicyAnnotations.ScheduleInterval] = info.ScheduleInterval;
             }
 
-            // Apply timezone
-            if (!string.IsNullOrWhiteSpace(info.Timezone))
-            {
-                table[ContinuousAggregatePolicyAnnotations.Timezone] = info.Timezone;
-            }
-
             // Apply initial_start
             if (info.InitialStart.HasValue)
             {

src/Eftdb.Design/Scaffolding/ContinuousAggregatePolicyScaffoldingExtractor.cs

@@ -13,7 +13,6 @@ public sealed record ContinuousAggregatePolicyInfo(
             string? StartOffset,
             string? EndOffset,
             string? ScheduleInterval,
-            string? Timezone,
             DateTime? InitialStart,
             bool? IncludeTieredData,
             int? BucketsPerBatch,
@@ -36,7 +35,6 @@ public sealed record ContinuousAggregatePolicyInfo(
                 using (DbCommand command = connection.CreateCommand())
                 {
                     // Query continuous aggregate policies from TimescaleDB jobs table
-                    // The config column contains JSONB with start_offset, end_offset, timezone, mat_hypertable_id and other policy parameters
                     command.CommandText = @"
                         SELECT
                             ca.user_view_schema,
@@ -61,7 +59,6 @@ INNER JOIN _timescaledb_catalog.continuous_agg ca
                         // Parse the JSONB config to extract policy parameters
                         string? startOffset = null;
                         string? endOffset = null;
-                        string? timezone = null;
                         bool? includeTieredData = null;
                         int? bucketsPerBatch = null;
                         int? maxBatchesPerExecution = null;
@@ -84,13 +81,6 @@ INNER JOIN _timescaledb_catalog.continuous_agg ca
                                 endOffset = ParseIntervalOrInteger(endOffsetElement);
                             }
 
-                            // Extract timezone
-                            if (root.TryGetProperty("timezone", out JsonElement timezoneElement)
-                                && timezoneElement.ValueKind == JsonValueKind.String)
-                            {
-                                timezone = timezoneElement.GetString();
-                            }
-
                             // Extract include_tiered_data (optional)
                             if (root.TryGetProperty("include_tiered_data", out JsonElement includeTieredDataElement)
                                 && (includeTieredDataElement.ValueKind == JsonValueKind.True || includeTieredDataElement.ValueKind == JsonValueKind.False))
@@ -124,7 +114,6 @@ INNER JOIN _timescaledb_catalog.continuous_agg ca
                             StartOffset: startOffset,
                             EndOffset: endOffset,
                             ScheduleInterval: scheduleInterval,
-                            Timezone: timezone,
                             InitialStart: initialStart,
                             IncludeTieredData: includeTieredData,
                             BucketsPerBatch: bucketsPerBatch,

src/Eftdb/Configuration/ContinuousAggregatePolicy/ContinuousAggregateBuilderPolicyExtensions.cs

@@ -21,7 +21,6 @@ public static class ContinuousAggregateBuilderPolicyExtensions
         /// <code>
         /// builder.IsContinuousAggregate&lt;HourlyMetric, Metric&gt;("hourly_metrics", "1 hour", x => x.Timestamp)
         ///     .WithRefreshPolicy(startOffset: "1 month", endOffset: "1 hour", scheduleInterval: "1 hour")
-        ///     .WithTimezone("UTC")
         ///     .WithRefreshNewestFirst(true);
         /// </code>
         /// </example>

src/Eftdb/Configuration/ContinuousAggregatePolicy/ContinuousAggregatePolicyAnnotations.cs

@@ -38,11 +38,6 @@ public static class ContinuousAggregatePolicyAnnotations
         /// </summary>
         public const string IfNotExists = "TimescaleDB:ContinuousAggregatePolicy:IfNotExists";
 
-        /// <summary>
-        /// Timezone to mitigate daylight savings alignment shifts. Stored as string (e.g., "UTC", "America/New_York").
-        /// </summary>
-        public const string Timezone = "TimescaleDB:ContinuousAggregatePolicy:Timezone";
-
         /// <summary>
         /// Override tiered read settings. Stored as nullable bool.
         /// </summary>

src/Eftdb/Configuration/ContinuousAggregatePolicy/ContinuousAggregatePolicyAttribute.cs

@@ -70,14 +70,6 @@ public sealed class ContinuousAggregatePolicyAttribute : Attribute
         /// </summary>
         public bool IfNotExists { get; set; } = false;
 
-        /// <summary>
-        /// Gets or sets the timezone to mitigate daylight savings alignment shifts.
-        /// </summary>
-        /// <example>
-        /// "UTC", "America/New_York", "Europe/London"
-        /// </example>
-        public string? Timezone { get; set; }
-
         /// <summary>
         /// Gets or sets a value indicating whether to override tiered read settings.
         /// NULL means use default behavior.

src/Eftdb/Configuration/ContinuousAggregatePolicy/ContinuousAggregatePolicyBuilder.cs

@@ -51,20 +51,6 @@ public ContinuousAggregatePolicyBuilder<TEntity, TSourceEntity> WithIfNotExists(
             return this;
         }
 
-        /// <summary>
-        /// Sets the timezone for the continuous aggregate refresh policy to mitigate daylight savings alignment shifts.
-        /// </summary>
-        /// <param name="timezone">The timezone (e.g., "UTC", "America/New_York", "Europe/London").</param>
-        /// <returns>The builder for method chaining.</returns>
-        public ContinuousAggregatePolicyBuilder<TEntity, TSourceEntity> WithTimezone(string timezone)
-        {
-            if (string.IsNullOrWhiteSpace(timezone))
-                throw new ArgumentException("Timezone must be provided.", nameof(timezone));
-
-            EntityTypeBuilder.HasAnnotation(ContinuousAggregatePolicyAnnotations.Timezone, timezone);
-            return this;
-        }
-
         /// <summary>
         /// Configures whether to override tiered read settings for the continuous aggregate refresh policy.
         /// </summary>

src/Eftdb/Configuration/ContinuousAggregatePolicy/ContinuousAggregatePolicyConvention.cs

@@ -60,10 +60,6 @@ public void ProcessEntityTypeAdded(IConventionEntityTypeBuilder entityTypeBuilde
             if (attribute.IfNotExists)
                 entityTypeBuilder.HasAnnotation(ContinuousAggregatePolicyAnnotations.IfNotExists, attribute.IfNotExists);
 
-            // Apply timezone
-            if (!string.IsNullOrWhiteSpace(attribute.Timezone))
-                entityTypeBuilder.HasAnnotation(ContinuousAggregatePolicyAnnotations.Timezone, attribute.Timezone);
-
             // Apply include_tiered_data if explicitly set
             if (attribute.IncludeTieredData.HasValue)
                 entityTypeBuilder.HasAnnotation(ContinuousAggregatePolicyAnnotations.IncludeTieredData, attribute.IncludeTieredData.Value);

src/Eftdb/Generators/ContinuousAggregatePolicyOperationGenerator.cs

@@ -82,11 +82,6 @@ public List<string> Generate(AddContinuousAggregatePolicyOperation operation)
                 arguments.Add($"if_not_exists => {operation.IfNotExists.ToString().ToLowerInvariant()}");
             }
 
-            if (!string.IsNullOrWhiteSpace(operation.Timezone))
-            {
-                arguments.Add($"timezone => '{operation.Timezone}'");
-            }
-
             if (operation.IncludeTieredData.HasValue)
             {
                 arguments.Add($"include_tiered_data => {operation.IncludeTieredData.Value.ToString().ToLowerInvariant()}");