Skip to content

Commit

Permalink
remove commented out tests for now
Browse files Browse the repository at this point in the history
Signed-off-by: Vitaly Sergeyev <vsergeyev@better.com>
  • Loading branch information
Vitaly Sergeyev committed Nov 1, 2021
1 parent 6a71e88 commit 4744c54
Showing 1 changed file with 0 additions and 28 deletions.
28 changes: 0 additions & 28 deletions sdk/python/tests/utils/online_read_write_test.py
Original file line number Diff line number Diff line change
Expand Up @@ -67,16 +67,6 @@ def _driver_rw_test(event_ts, created_ts, write, expect_read):
event_ts=time_1, created_ts=time_1, write=(1.1, "3.1"), expect_read=(1.1, "3.1")
)

# We should test that older value can't overwrite over a newer value once we add the respective flag
# TODO: event_ts check supported by Redis Online Store; add to other Online stores as well
# time_2 = datetime.utcnow()
# _driver_rw_test(
# event_ts=time_1 - timedelta(hours=1),
# created_ts=time_2,
# write=(-1000, "OLD"),
# expect_read=(1.1, "3.1"),
# )

""" Values with an new event_ts should overwrite older ones """
time_3 = datetime.utcnow()
_driver_rw_test(
Expand All @@ -85,21 +75,3 @@ def _driver_rw_test(event_ts, created_ts, write, expect_read):
write=(1123, "NEWER"),
expect_read=(1123, "NEWER"),
)

# We should test that older value can't overwrite over a newer value once we add the respective flag
# TODO: add the `created_ts` tie breaker check in the Online stores
""" created_ts is used as a tie breaker, using older created_ts here """
# _driver_rw_test(
# event_ts=time_1 + timedelta(hours=1),
# created_ts=time_3 - timedelta(hours=1),
# write=(54321, "I HAVE AN OLDER created_ts SO I LOSE"),
# expect_read=(1123, "NEWER"),
# )

""" created_ts is used as a tie breaker, using newer created_ts here so we should overwrite """
# _driver_rw_test(
# event_ts=time_1 + timedelta(hours=1),
# created_ts=time_3 + timedelta(hours=1),
# write=(96864, "I HAVE A NEWER created_ts SO I WIN"),
# expect_read=(96864, "I HAVE A NEWER created_ts SO I WIN"),
# )

0 comments on commit 4744c54

Please sign in to comment.