chore: fix PR comments

Author: atstaeff

Makefile

@@ -3,7 +3,7 @@ qa: analyze test
 analyze:
 	@poetry run pylint eventsourcingdb tests
 
-fix:
+format:
 	@poetry run autopep8 --in-place --aggressive --max-line-length=100 --recursive eventsourcingdb tests
 
 test:
@@ -13,4 +13,4 @@ clean:
 
 build: qa clean
 
-.PHONY: fix analyze build clean qa test
+.PHONY: analyze build clean format qa test

eventsourcingdb/handlers/observe_events/observe_events_options.py

@@ -10,11 +10,10 @@
 @dataclass
 class ObserveEventsOptions:
     recursive: bool
-    lower_bound: LowerBound | None = None  # Changed from str to LowerBound
+    lower_bound: LowerBound | None = None 
     from_latest_event: ObserveFromLatestEvent | None = None
 
     def validate(self) -> None:
-        # Update validation logic
         if self.lower_bound is not None and not isinstance(self.lower_bound, LowerBound):
             raise ValidationError(
                 'ObserveEventsOptions are invalid: lower_bound must be a LowerBound object.'

tests/test_read_events.py

@@ -310,7 +310,7 @@ async def test_throws_error_for_exclusive_options(
                         type='com.foo.bar',
                         if_event_is_missing=IfEventIsMissingDuringRead.READ_EVERYTHING
                     ),
-                    lower_bound='0'
+                    lower_bound=LowerBound(id='0', type='exclusive'),
                 )
             ):
                 pass
@@ -343,7 +343,7 @@ async def test_throws_error_for_invalid_lower_bound(
                 '/',
                 ReadEventsOptions(
                     recursive=True,
-                    lower_bound='hello'
+                    lower_bound=LowerBound(id='hello', type='inclusive')
                 )
             ):
                 pass
@@ -360,7 +360,7 @@ async def test_throws_error_for_negative_lower_bound(
                 '/',
                 ReadEventsOptions(
                     recursive=True,
-                    lower_bound='-1'
+                    lower_bound=LowerBound(id='-1', type='inclusive')
                 )
             ):
                 pass
@@ -377,7 +377,7 @@ async def test_throws_error_for_invalid_upper_bound(
                 '/',
                 ReadEventsOptions(
                     recursive=True,
-                    upper_bound='hello'
+                    upper_bound=UpperBound(id='hello', type='exclusive')
                 )
             ):
                 pass
@@ -394,7 +394,7 @@ async def test_throws_error_for_negative_upper_bound(
                 '/',
                 ReadEventsOptions(
                     recursive=True,
-                    upper_bound='-1'
+                    upper_bound=UpperBound(id='-1', type='exclusive')
                 )
             ):
                 pass