Fix linting errors and add test package init

Author: tusharynayaka

searches/binary_search.py

@@ -114,5 +114,4 @@ def upper_bound(sorted_collection: list[int], item: int) -> int:
 
     if left == len(sorted_collection) or sorted_collection[left] != item:
         return []
-    return list(range(left, right))
-    
+    return list(range(left, right))

tests/__init__.py

@@ -1,38 +1,32 @@
-import sys
-import os
 from hypothesis import given, strategies as st
 
-# Add root directory to python path
-sys.path.append(os.path.dirname(os.path.dirname(os.path.abspath(__file__))))
-
 from searches.binary_search import binary_search
 
 # Strategy to generate a sorted list of integers
-sorted_list_strategy = st.lists(st.integers(), min_size=0).map(sorted)
+sorted_list_strategy = st.lists(st.integers()).map(sorted)
+
 
 @given(
     arr=sorted_list_strategy,
-    # Pick a random integer as a potential target
-    target=st.integers() 
+    target=st.integers(),
 )
-def test_binary_search_property_based(arr, target):
+def test_binary_search_property_based(arr: list[int], target: int) -> None:
     """
     Property-based test:
-    1. If target is in arr, binary_search MUST return an index 'i' 
+    1. If target is in arr, binary_search MUST return an index 'i'
        such that arr[i] == target.
     2. If target is in arr, it MUST be the FIRST occurrence.
     3. If target is not in arr, binary_search MUST return -1.
     """
-    
     result = binary_search(arr, target)
-    
+
     if target in arr:
         assert result != -1, f"Target {target} was in {arr} but not found."
-        assert arr[result] == target, f"Index {result} pointed to {arr[result]}, not {target}."
-        
+        assert arr[result] == target, f"Index {result} pointed to {arr[result]}."
+
         # Property: Verify it is the FIRST occurrence
-        # Only check if the target is actually at the result index
+        # Check that target does not exist at any index before 'result'
         for i in range(result):
-            assert arr[i] != target, f"Found target {target} at index {result}, but earlier occurrence at {i}."
+            assert arr[i] != target, f"Target at {result}, but earlier at {i}."
     else:
-        assert result == -1, f"Target {target} was not in {arr}, but found at index {result}."
+        assert result == -1, f"Target {target} not in {arr}, but found at {result}."