chore: add "_" to internal methods

Author: Dani Reinón

postgrest_py/base_request_builder.py

@@ -98,7 +98,7 @@ def raise_when_api_error(cls: Type[APIResponse], value: Any) -> Any:
         return value
 
     @staticmethod
-    def get_count_from_content_range_header(
+    def _get_count_from_content_range_header(
         content_range_header: str,
     ) -> Optional[int]:
         content_range = content_range_header.split("/")
@@ -107,32 +107,32 @@ def get_count_from_content_range_header(
         return int(content_range[1])
 
     @staticmethod
-    def is_count_in_prefer_header(prefer_header: str) -> bool:
+    def _is_count_in_prefer_header(prefer_header: str) -> bool:
         pattern = f"count=({'|'.join([cm.value for cm in CountMethod])})"
         return bool(search(pattern, prefer_header))
 
     @classmethod
-    def get_count_from_http_request_response(
+    def _get_count_from_http_request_response(
         cls: Type[APIResponse],
         request_response: RequestResponse,
     ) -> Optional[int]:
         prefer_header: Optional[str] = request_response.request.headers.get("prefer")
         if not prefer_header:
             return None
-        is_count_in_prefer_header = cls.is_count_in_prefer_header(prefer_header)
+        is_count_in_prefer_header = cls._is_count_in_prefer_header(prefer_header)
         content_range_header: Optional[str] = request_response.headers.get(
             "content-range"
         )
         if not (is_count_in_prefer_header and content_range_header):
             return None
-        return cls.get_count_from_content_range_header(content_range_header)
+        return cls._get_count_from_content_range_header(content_range_header)
 
     @classmethod
     def from_http_request_response(
         cls: Type[APIResponse], request_response: RequestResponse
     ) -> APIResponse:
         data = request_response.json()
-        count = cls.get_count_from_http_request_response(request_response)
+        count = cls._get_count_from_http_request_response(request_response)
         return cls(data=data, count=count)
 
 

tests/_async/test_request_builder.py

@@ -248,7 +248,7 @@ def test_get_count_from_content_range_header_with_count(
         self, content_range_header_with_count: str
     ):
         assert (
-            APIResponse.get_count_from_content_range_header(
+            APIResponse._get_count_from_content_range_header(
                 content_range_header_with_count
             )
             == 2
@@ -258,23 +258,23 @@ def test_get_count_from_content_range_header_without_count(
         self, content_range_header_without_count: str
     ):
         assert (
-            APIResponse.get_count_from_content_range_header(
+            APIResponse._get_count_from_content_range_header(
                 content_range_header_without_count
             )
             is None
         )
 
     def test_is_count_in_prefer_header_true(self, prefer_header_with_count: str):
-        assert APIResponse.is_count_in_prefer_header(prefer_header_with_count)
+        assert APIResponse._is_count_in_prefer_header(prefer_header_with_count)
 
     def test_is_count_in_prefer_header_false(self, prefer_header_without_count: str):
-        assert not APIResponse.is_count_in_prefer_header(prefer_header_without_count)
+        assert not APIResponse._is_count_in_prefer_header(prefer_header_without_count)
 
     def test_get_count_from_http_request_response_without_prefer_header(
         self, request_response_without_prefer_header: Response
     ):
         assert (
-            APIResponse.get_count_from_http_request_response(
+            APIResponse._get_count_from_http_request_response(
                 request_response_without_prefer_header
             )
             is None
@@ -284,7 +284,7 @@ def test_get_count_from_http_request_response_with_prefer_header_without_count(
         self, request_response_with_prefer_header_without_count: Response
     ):
         assert (
-            APIResponse.get_count_from_http_request_response(
+            APIResponse._get_count_from_http_request_response(
                 request_response_with_prefer_header_without_count
             )
             is None
@@ -294,7 +294,7 @@ def test_get_count_from_http_request_response_with_count_and_content_range(
         self, request_response_with_prefer_header_with_count_and_content_range: Response
     ):
         assert (
-            APIResponse.get_count_from_http_request_response(
+            APIResponse._get_count_from_http_request_response(
                 request_response_with_prefer_header_with_count_and_content_range
             )
             == 2

tests/_sync/test_request_builder.py

@@ -248,7 +248,7 @@ def test_get_count_from_content_range_header_with_count(
         self, content_range_header_with_count: str
     ):
         assert (
-            APIResponse.get_count_from_content_range_header(
+            APIResponse._get_count_from_content_range_header(
                 content_range_header_with_count
             )
             == 2
@@ -258,23 +258,23 @@ def test_get_count_from_content_range_header_without_count(
         self, content_range_header_without_count: str
     ):
         assert (
-            APIResponse.get_count_from_content_range_header(
+            APIResponse._get_count_from_content_range_header(
                 content_range_header_without_count
             )
             is None
         )
 
     def test_is_count_in_prefer_header_true(self, prefer_header_with_count: str):
-        assert APIResponse.is_count_in_prefer_header(prefer_header_with_count)
+        assert APIResponse._is_count_in_prefer_header(prefer_header_with_count)
 
     def test_is_count_in_prefer_header_false(self, prefer_header_without_count: str):
-        assert not APIResponse.is_count_in_prefer_header(prefer_header_without_count)
+        assert not APIResponse._is_count_in_prefer_header(prefer_header_without_count)
 
     def test_get_count_from_http_request_response_without_prefer_header(
         self, request_response_without_prefer_header: Response
     ):
         assert (
-            APIResponse.get_count_from_http_request_response(
+            APIResponse._get_count_from_http_request_response(
                 request_response_without_prefer_header
             )
             is None
@@ -284,7 +284,7 @@ def test_get_count_from_http_request_response_with_prefer_header_without_count(
         self, request_response_with_prefer_header_without_count: Response
     ):
         assert (
-            APIResponse.get_count_from_http_request_response(
+            APIResponse._get_count_from_http_request_response(
                 request_response_with_prefer_header_without_count
             )
             is None
@@ -294,7 +294,7 @@ def test_get_count_from_http_request_response_with_count_and_content_range(
         self, request_response_with_prefer_header_with_count_and_content_range: Response
     ):
         assert (
-            APIResponse.get_count_from_http_request_response(
+            APIResponse._get_count_from_http_request_response(
                 request_response_with_prefer_header_with_count_and_content_range
             )
             == 2