fix ts.create UNCOMPRESSED set to false explicitly

src/NRedisStack/TimeSeries/Literals/CommandArgs.cs

@@ -5,6 +5,7 @@ internal class TimeSeriesArgs
         public const string RETENTION = "RETENTION";
         public const string LABELS = "LABELS";
         public const string UNCOMPRESSED = "UNCOMPRESSED";
+        public const string COMPRESSED = "COMPRESSED";
         public const string COUNT = "COUNT";
         public const string AGGREGATION = "AGGREGATION";
         public const string ALIGN = "ALIGN";

src/NRedisStack/TimeSeries/TimeSeriesParamsBuilder.cs

@@ -1,7 +1,6 @@
 using NRedisStack.Literals;
 using NRedisStack.Literals.Enums;
 using NRedisStack.DataTypes;
-using System.Collections.ObjectModel;
 using NRedisStack.Extensions;
 
 namespace NRedisStack
@@ -29,7 +28,7 @@
             this.retentionTime = retentionTime;
             return (T)this;
         }
-        public T AddLabels(ReadOnlyCollection<TimeSeriesLabel> labels)
+        public T AddLabels(IReadOnlyCollection<TimeSeriesLabel> labels)
         {
             this.labels = labels;
             return (T)this;
@@ -110,7 +109,7 @@
         {
             if (uncompressed.HasValue)
             {
-                args.Add(TimeSeriesArgs.UNCOMPRESSED);
+                args.Add(uncompressed.Value ? TimeSeriesArgs.UNCOMPRESSED : TimeSeriesArgs.COMPRESSED);
             }
         }
 
@@ -157,8 +156,8 @@
             return new TsCreateParams(args);
         }
 
         public TsCreateParamsBuilder AddUncompressed(bool uncompressed) => base.AddUncompressed(uncompressed);
         public TsCreateParamsBuilder AddDuplicatePolicy(TsDuplicatePolicy duplicatePolicy) => base.AddDuplicatePolicy(duplicatePolicy);
 
     }
 
@@ -175,7 +174,7 @@
             return new TsAlterParams(args);
         }
 
         public TsAlterParamsBuilder AddDuplicatePolicy(TsDuplicatePolicy duplicatePolicy) => base.AddDuplicatePolicy(duplicatePolicy);
     }
 
     public class TsAddParamsBuilder : TsBaseParamsBuilder<TsAddParamsBuilder>
@@ -195,9 +194,9 @@
             return new TsAddParams(args);
         }
 
         public TsAddParamsBuilder AddValue(double value) => base.AddValue(value);
         public TsAddParamsBuilder AddTimestamp(TimeStamp timestamp) => base.AddTimestamp(timestamp);
         public TsAddParamsBuilder AddUncompressed(bool uncompressed) => base.AddUncompressed(uncompressed);
         public TsAddParamsBuilder AddOnDuplicate(TsDuplicatePolicy duplicatePolicy) => base.AddDuplicatePolicy(duplicatePolicy);
     }
 
@@ -217,9 +216,9 @@
             return new TsIncrByParams(args);
         }
 
         public TsIncrByParamsBuilder AddValue(double value) => base.AddValue(value);
         public TsIncrByParamsBuilder AddTimestamp(TimeStamp timestamp) => base.AddTimestamp(timestamp);
         public TsIncrByParamsBuilder AddUncompressed(bool uncompressed) => base.AddUncompressed(uncompressed);
         public TsIncrByParamsBuilder AddDuplicatePolicy(TsDuplicatePolicy duplicatePolicy) => base.AddDuplicatePolicy(duplicatePolicy);
     }
 

tests/NRedisStack.Tests/TimeSeries/TestAPI/TestCreate.cs

@@ -3,7 +3,6 @@
 using NRedisStack.RedisStackCommands;
 using NRedisStack.Literals.Enums;
 using Xunit;
-using System.Runtime.CompilerServices;
 
 namespace NRedisStack.Tests.TimeSeries.TestAPI
 {
@@ -136,5 +135,33 @@ public void TestCreateAndIgnoreValues()
             Assert.Equal(11, (long)info[j + 1]);
             Assert.Equal(12, (long)info[k + 1]);
         }
+
+        [Fact]
+        public void TestParamsBuilder()
+        {
+            TsCreateParams parameters = new TsCreateParamsBuilder()
+                        .AddChunkSizeBytes(1000)
+                        .AddDuplicatePolicy(TsDuplicatePolicy.FIRST)
+                        .AddIgnoreValues(11, 12)
+                        .AddLabels(new List<TimeSeriesLabel>() { new TimeSeriesLabel("key", "value") })
+                        .AddRetentionTime(5000)
+                        .AddUncompressed(true).build();
+
+            var command = TimeSeriesCommandsBuilder.Create(key, parameters);
+            var expectedArgs = new object[] { key, "RETENTION", 5000L, "CHUNK_SIZE", 1000L, "LABELS", "key", "value", "UNCOMPRESSED", "DUPLICATE_POLICY", "FIRST", "IGNORE", 11L, 12L };
+            Assert.Equal(expectedArgs, command.Args);
+
+            parameters = new TsCreateParamsBuilder()
+                        .AddChunkSizeBytes(1000)
+                        .AddDuplicatePolicy(TsDuplicatePolicy.FIRST)
+                        .AddIgnoreValues(11, 12)
+                        .AddLabels(new List<TimeSeriesLabel>() { new TimeSeriesLabel("key", "value") })
+                        .AddRetentionTime(5000)
+                        .AddUncompressed(false).build();
+
+            command = TimeSeriesCommandsBuilder.Create(key, parameters);
+            expectedArgs = new object[] { key, "RETENTION", 5000L, "CHUNK_SIZE", 1000L, "LABELS", "key", "value", "COMPRESSED", "DUPLICATE_POLICY", "FIRST", "IGNORE", 11L, 12L };
+            Assert.Equal(expectedArgs, command.Args);
+        }
     }
 }