Skip to content

Commit aa0ae68

Browse files
authored
Fix TIMESERIES (round floats) and JSON tests (#2421)
* Fix timeseries tests (round floats) * fix json tests
1 parent e867f9e commit aa0ae68

File tree

5 files changed

+7
-14
lines changed

5 files changed

+7
-14
lines changed

tests/test_asyncio/test_json.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -817,7 +817,7 @@ async def test_objlen_dollar(modclient: redis.Redis):
817817
},
818818
)
819819
# Test multi
820-
assert await modclient.json().objlen("doc1", "$..a") == [2, None, 1]
820+
assert await modclient.json().objlen("doc1", "$..a") == [None, 2, 1]
821821
# Test single
822822
assert await modclient.json().objlen("doc1", "$.nested1.a") == [2]
823823

tests/test_asyncio/test_timeseries.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -240,7 +240,7 @@ async def test_range_advanced(modclient: redis.Redis):
240240
assert [(0, 5.0), (5, 6.0)] == await modclient.ts().range(
241241
1, 0, 10, aggregation_type="count", bucket_size_msec=10, align=5
242242
)
243-
assert [(0, 2.5500000000000003), (10, 3.0)] == await modclient.ts().range(
243+
assert [(0, 2.55), (10, 3.0)] == await modclient.ts().range(
244244
1, 0, 10, aggregation_type="twa", bucket_size_msec=10
245245
)
246246

tests/test_json.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -824,7 +824,7 @@ def test_objlen_dollar(client):
824824
},
825825
)
826826
# Test multi
827-
assert client.json().objlen("doc1", "$..a") == [2, None, 1]
827+
assert client.json().objlen("doc1", "$..a") == [None, 2, 1]
828828
# Test single
829829
assert client.json().objlen("doc1", "$.nested1.a") == [2]
830830

tests/test_search.py

Lines changed: 2 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1448,23 +1448,16 @@ def test_json_with_jsonpath(client):
14481448
assert res.docs[0].id == "doc:1"
14491449
assert res.docs[0].json == '{"prod:name":"RediSearch"}'
14501450

1451-
# query for an unsupported field fails
1451+
# query for an unsupported field
14521452
res = client.ft().search("@name_unsupported:RediSearch")
1453-
assert res.total == 0
1453+
assert res.total == 1
14541454

14551455
# return of a supported field succeeds
14561456
res = client.ft().search(Query("@name:RediSearch").return_field("name"))
14571457
assert res.total == 1
14581458
assert res.docs[0].id == "doc:1"
14591459
assert res.docs[0].name == "RediSearch"
14601460

1461-
# return of an unsupported field fails
1462-
res = client.ft().search(Query("@name:RediSearch").return_field("name_unsupported"))
1463-
assert res.total == 1
1464-
assert res.docs[0].id == "doc:1"
1465-
with pytest.raises(Exception):
1466-
res.docs[0].name_unsupported
1467-
14681461

14691462
@pytest.mark.redismod
14701463
@pytest.mark.onlynoncluster

tests/test_timeseries.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -233,7 +233,7 @@ def test_range_advanced(client):
233233
assert [(0, 5.0), (5, 6.0)] == client.ts().range(
234234
1, 0, 10, aggregation_type="count", bucket_size_msec=10, align=5
235235
)
236-
assert [(0, 2.5500000000000003), (10, 3.0)] == client.ts().range(
236+
assert [(0, 2.55), (10, 3.0)] == client.ts().range(
237237
1, 0, 10, aggregation_type="twa", bucket_size_msec=10
238238
)
239239

@@ -343,7 +343,7 @@ def test_rev_range(client):
343343
assert [(1, 10.0), (0, 1.0)] == client.ts().revrange(
344344
1, 0, 10, aggregation_type="count", bucket_size_msec=10, align=1
345345
)
346-
assert [(10, 3.0), (0, 2.5500000000000003)] == client.ts().revrange(
346+
assert [(10, 3.0), (0, 2.55)] == client.ts().revrange(
347347
1, 0, 10, aggregation_type="twa", bucket_size_msec=10
348348
)
349349

0 commit comments

Comments
 (0)