From d2ee1fc5b94c61986f4f4dc36c75bcad85e1751e Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Piotr=20Ma=C5=9Blanka?= <piotr.maslanka@henrietta.com.pl>
Date: Fri, 9 Jul 2021 16:04:14 +0200
Subject: [PATCH] fix bugs
---
README.md | 2 +-
tempsdb/varlen.pyx | 1 -
tests/test_series.py | 3 +--
tests/test_varlen.py | 4 ++--
4 files changed, 4 insertions(+), 6 deletions(-)
diff --git a/README.md b/README.md
index ebff4b7..3845f2d 100644
--- a/README.md
+++ b/README.md
@@ -55,7 +55,7 @@ Consult the docs for how to disable it.
## v0.6.4
-* fixed a bug with slicing chunks
+* fixed a bug with slicing chunks in `VarlenSeries`
* added extra comparison operators for `VarlenEntry`
* added `sync` to `VarlenSeries`
diff --git a/tempsdb/varlen.pyx b/tempsdb/varlen.pyx
index 13883b3..3f4e1df 100644
--- a/tempsdb/varlen.pyx
+++ b/tempsdb/varlen.pyx
@@ -85,7 +85,6 @@ cdef class VarlenEntry:
return False
cdef bytes b = self.slice(self.len-len_v, self.len)
- logger.warning('comparing %s against %s', repr(b), repr(v))
return b == v
def __gt__(self, other) -> bool:
diff --git a/tests/test_series.py b/tests/test_series.py
index 9390851..fb72cdc 100644
--- a/tests/test_series.py
+++ b/tests/test_series.py
@@ -23,7 +23,6 @@ class TestSeries(unittest.TestCase):
self.assertNotEqual(ts, 0)
series.close()
- @unittest.skip("Known issue #3")
def test_trim_multiple_chunks_with_close(self):
from tempsdb.series import create_series, TimeSeries
series = create_series('test8', 'test8', 10, 4096)
@@ -33,7 +32,7 @@ class TestSeries(unittest.TestCase):
series.sync()
series.close()
series = TimeSeries('test8', 'test8')
- series.trim(4000)
+ series.trim(4097)
with series.iterate_range(0, 17000) as it:
for ts, v in it:
self.assertNotEqual(ts, 0)
diff --git a/tests/test_varlen.py b/tests/test_varlen.py
index 100a73e..4ccb9b6 100644
--- a/tests/test_varlen.py
+++ b/tests/test_varlen.py
@@ -43,9 +43,9 @@ class TestVarlen(unittest.TestCase):
self.assertNotEqual(ve, b'test')
self.assertTrue(ve.slice(0, 4), b'test')
self.assertTrue(ve.startswith(b'test '))
- # self.assertTrue(ve.endswith(b'skarabeusza'))
+ self.assertTrue(ve.endswith(b'skarabeusza'))
self.assertFalse(ve.startswith(b'tost'))
- # self.assertFalse(ve.endswith(b'skerabeusza'))
+ self.assertFalse(ve.endswith(b'skerabeusza'))
ve = iterator.get_next()
def test_varlen_gzip(self):
--
GitLab