actually: not a key

Author: mgravell

src/NRedisStack/Search/SearchCommandBuilder.cs

@@ -91,7 +91,7 @@ public static SerializedCommand DictAdd(string dict, params string[] terms)
             throw new ArgumentOutOfRangeException("At least one term must be provided");
         }
 
-        var args = new List<object>(terms.Length + 1) { (RedisKey)dict };
+        var args = new List<object>(terms.Length + 1) { dict };
         foreach (var t in terms)
         {
             args.Add(t);
@@ -107,7 +107,7 @@ public static SerializedCommand DictDel(string dict, params string[] terms)
             throw new ArgumentOutOfRangeException("At least one term must be provided");
         }
 
-        var args = new List<object>(terms.Length + 1) { (RedisKey)dict };
+        var args = new List<object>(terms.Length + 1) { dict };
         foreach (var t in terms)
         {
             args.Add(t);
@@ -118,7 +118,7 @@ public static SerializedCommand DictDel(string dict, params string[] terms)
 
     public static SerializedCommand DictDump(string dict)
     {
-        return new(FT.DICTDUMP, (RedisKey)dict);
+        return new(FT.DICTDUMP, dict);
     }
 
     public static SerializedCommand DropIndex(string indexName, bool dd = false)

tests/NRedisStack.Tests/Search/SearchTests.cs

@@ -1498,14 +1498,15 @@ public void TestDictionary(string endpointId)
         var ft = db.FT();
 
         Assert.Equal(3L, ft.DictAdd("dict", "bar", "foo", "hello world"));
-
+        Assert.Equal(0, DatabaseSize(db)); // in part, this is to allow replication to catch up
         var dumResult = ft.DictDump("dict");
         int i = 0;
         Assert.Equal("bar", dumResult[i++].ToString());
         Assert.Equal("foo", dumResult[i++].ToString());
         Assert.Equal("hello world", dumResult[i].ToString());
 
         Assert.Equal(3L, ft.DictDel("dict", "foo", "bar", "hello world"));
+        Assert.Equal(0, DatabaseSize(db)); // in part, this is to allow replication to catch up
         Assert.Empty(ft.DictDump("dict"));
     }
 
@@ -1672,14 +1673,15 @@ public async Task TestDictionaryAsync(string endpointId)
         var ft = db.FT();
 
         Assert.Equal(3L, await ft.DictAddAsync("dict", "bar", "foo", "hello world"));
-
+        Assert.Equal(0, await DatabaseSizeAsync(db)); // in part, this is to allow replication to catch up
         var dumResult = await ft.DictDumpAsync("dict");
         int i = 0;
         Assert.Equal("bar", dumResult[i++].ToString());
         Assert.Equal("foo", dumResult[i++].ToString());
         Assert.Equal("hello world", dumResult[i].ToString());
 
         Assert.Equal(3L, await ft.DictDelAsync("dict", "foo", "bar", "hello world"));
+        Assert.Equal(0, await DatabaseSizeAsync(db)); // in part, this is to allow replication to catch up
         Assert.Empty((await ft.DictDumpAsync("dict")));
     }
 
@@ -2704,6 +2706,7 @@ public async Task TestCrossTermDictionaryAsync(string endpointId)
             }
         };
 
+        Assert.Equal(0, DatabaseSize(db)); // in part, this is to allow replication to catch up
         Assert.Equal(expected, await ft.SpellCheckAsync(index,
                                              "Tooni toque kerfuffle",
                                              new FTSpellCheckParams()