Skip to content

Commit 67f0bf5

Browse files
committed
Correct tests
1 parent 24ac9fc commit 67f0bf5

File tree

2 files changed

+13
-13
lines changed

2 files changed

+13
-13
lines changed

tests/test_quixstreams/test_state/test_rocksdb/test_windowed/test_state.py

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -71,8 +71,8 @@ def test_expire_windows(transaction_state, delete):
7171

7272
assert len(expired) == 2
7373
assert expired == [
74-
((0, 10), 1, [], b"__key__"),
75-
((10, 20), 2, [], b"__key__"),
74+
((0, 10), 1, []),
75+
((10, 20), 2, []),
7676
]
7777

7878
with transaction_state() as state:
@@ -112,8 +112,8 @@ def test_expire_windows_with_collect(transaction_state, end_inclusive):
112112
window_1_value = ["a", "b"] if end_inclusive else ["a"]
113113
window_2_value = ["b", "c"] if end_inclusive else ["b"]
114114
assert expired == [
115-
((0, 10), None, window_1_value, b"__key__"),
116-
((10, 20), [777, None], window_2_value, b"__key__"),
115+
((0, 10), None, window_1_value),
116+
((10, 20), [777, None], window_2_value),
117117
]
118118

119119

@@ -132,7 +132,7 @@ def test_same_keys_in_db_and_update_cache(transaction_state):
132132
expired = list(state.expire_windows(max_start_time=max_start_time))
133133

134134
# Value from the cache takes precedence over the value in the db
135-
assert expired == [((0, 10), 3, [], b"__key__")]
135+
assert expired == [((0, 10), 3, [])]
136136

137137

138138
def test_get_latest_timestamp(windowed_rocksdb_store_factory):

tests/test_quixstreams/test_state/test_rocksdb/test_windowed/test_transaction.py

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -78,8 +78,8 @@ def test_expire_windows_expired(self, windowed_rocksdb_store_factory, delete):
7878

7979
assert len(expired) == 2
8080
assert expired == [
81-
((0, 10), 1, [], prefix),
82-
((10, 20), 2, [], prefix),
81+
((0, 10), 1, []),
82+
((10, 20), 2, []),
8383
]
8484

8585
with store.start_partition_transaction(0) as tx:
@@ -131,8 +131,8 @@ def test_expire_windows_cached(self, windowed_rocksdb_store_factory, delete):
131131
)
132132
assert len(expired) == 2
133133
assert expired == [
134-
((0, 10), 1, [], prefix),
135-
((10, 20), 2, [], prefix),
134+
((0, 10), 1, []),
135+
((10, 20), 2, []),
136136
]
137137
assert (
138138
tx.get_window(start_ms=0, end_ms=10, prefix=prefix) == None
@@ -193,7 +193,7 @@ def test_expire_windows_with_grace_expired(self, windowed_rocksdb_store_factory)
193193
)
194194

195195
assert len(expired) == 1
196-
assert expired == [((0, 10), 1, [], prefix)]
196+
assert expired == [((0, 10), 1, [])]
197197

198198
def test_expire_windows_with_grace_empty(self, windowed_rocksdb_store_factory):
199199
store = windowed_rocksdb_store_factory()
@@ -328,9 +328,9 @@ def test_expire_windows_multiple_windows(self, windowed_rocksdb_store_factory):
328328
)
329329

330330
assert len(expired) == 3
331-
assert expired[0] == ((0, 10), 1, [], prefix)
332-
assert expired[1] == ((10, 20), 1, [], prefix)
333-
assert expired[2] == ((20, 30), 1, [], prefix)
331+
assert expired[0] == ((0, 10), 1, [])
332+
assert expired[1] == ((10, 20), 1, [])
333+
assert expired[2] == ((20, 30), 1, [])
334334

335335
def test_get_latest_timestamp_update(self, windowed_rocksdb_store_factory):
336336
store = windowed_rocksdb_store_factory()

0 commit comments

Comments
 (0)