fix: interpolations erros and other things reported by sourcery-ai (#37)

Author: leynier

CODE_OF_CONDUCT.md

@@ -14,21 +14,21 @@ appearance, race, religion, or sexual identity and orientation.
 Examples of behavior that contributes to creating a positive environment
 include:
 
-* Using welcoming and inclusive language
-* Being respectful of differing viewpoints and experiences
-* Gracefully accepting constructive criticism
-* Focusing on what is best for the community
-* Showing empathy towards other community members
+- Using welcoming and inclusive language
+- Being respectful of differing viewpoints and experiences
+- Gracefully accepting constructive criticism
+- Focusing on what is best for the community
+- Showing empathy towards other community members
 
 Examples of unacceptable behavior by participants include:
 
-* The use of sexualized language or imagery and unwelcome sexual attention or
+- The use of sexualized language or imagery and unwelcome sexual attention or
   advances
-* Trolling, insulting/derogatory comments, and personal or political attacks
-* Public or private harassment
-* Publishing others' private information, such as a physical or electronic
+- Trolling, insulting/derogatory comments, and personal or political attacks
+- Public or private harassment
+- Publishing others' private information, such as a physical or electronic
   address, without explicit permission
-* Other conduct which could reasonably be considered inappropriate in a
+- Other conduct which could reasonably be considered inappropriate in a
   professional setting
 
 ## Our Responsibilities
@@ -74,5 +74,3 @@ available at https://www.contributor-covenant.org/version/1/4/code-of-conduct.ht
 
 For answers to common questions about this code of conduct, see
 https://www.contributor-covenant.org/faq
-
-

CONTRIBUTING.md

@@ -29,4 +29,4 @@ When making pull requests to the repository, make sure to follow these guideline
 - In your PR's description, link to any related issues or pull requests to give reviewers the full context of your change.
 - For commit messages, follow the [Conventional Commits](https://www.conventionalcommits.org/en/v1.0.0/) format.
   - For example, if you update documentation for a specific extension, your commit message might be: `docs(extension-name) updated installation documentation`.
-diff --git a/CODE_OF_CONDUCT.md b/CODE_OF_CONDUCT.md
+    diff --git a/CODE_OF_CONDUCT.md b/CODE_OF_CONDUCT.md

Makefile

@@ -0,0 +1,17 @@
+install: install_poetry
+	poetry install
+
+install_poetry:
+	curl -sSL https://install.python-poetry.org | python -
+
+precommit: install
+	poetry run pre-commit run --all-files
+
+tests: install
+	poetry run pytest --cov=./ --cov-report=xml -vv
+
+build: install
+	poetry run unasync postgrest_py tests
+
+clean:
+	sudo rm -r .venv

postgrest_py/_async/client.py

@@ -76,5 +76,4 @@ def from_table(self, table: str) -> AsyncRequestBuilder:
     async def rpc(self, func: str, params: dict) -> Response:
         """Perform a stored procedure call."""
         path = f"/rpc/{func}"
-        r = await self.session.post(path, json=params)
-        return r
+        return await self.session.post(path, json=params)

postgrest_py/_async/request_builder.py

@@ -144,17 +144,17 @@ def in_(self, column: str, values: Iterable[str]):
     def cs(self, column: str, values: Iterable[str]):
         values = map(sanitize_param, values)
         values = ",".join(values)
-        return self.filter(column, "cs", f"{{values}}")
+        return self.filter(column, "cs", f"{{{values}}}")
 
     def cd(self, column: str, values: Iterable[str]):
         values = map(sanitize_param, values)
         values = ",".join(values)
-        return self.filter(column, "cd", f"{{values}}")
+        return self.filter(column, "cd", f"{{{values}}}")
 
     def ov(self, column: str, values: Iterable[str]):
         values = map(sanitize_param, values)
         values = ",".join(values)
-        return self.filter(column, "ov", f"{{values}}")
+        return self.filter(column, "ov", f"{{{values}}}")
 
     def sl(self, column: str, range: Tuple[int, int]):
         return self.filter(column, "sl", f"({range[0]},{range[1]})")
@@ -173,7 +173,7 @@ def adj(self, column: str, range: Tuple[int, int]):
 
     def match(self, query: Dict[str, Any]):
         updated_query = None
-        for key in query.keys():
+        for key in query:
             value = query.get(key, "")
             updated_query = self.eq(key, value)
         return updated_query

postgrest_py/_sync/client.py

@@ -76,5 +76,4 @@ def from_table(self, table: str) -> SyncRequestBuilder:
     def rpc(self, func: str, params: dict) -> Response:
         """Perform a stored procedure call."""
         path = f"/rpc/{func}"
-        r = self.session.post(path, json=params)
-        return r
+        return self.session.post(path, json=params)

postgrest_py/_sync/request_builder.py

@@ -144,17 +144,17 @@ def in_(self, column: str, values: Iterable[str]):
     def cs(self, column: str, values: Iterable[str]):
         values = map(sanitize_param, values)
         values = ",".join(values)
-        return self.filter(column, "cs", f"{{values}}")
+        return self.filter(column, "cs", f"{{{values}}}")
 
     def cd(self, column: str, values: Iterable[str]):
         values = map(sanitize_param, values)
         values = ",".join(values)
-        return self.filter(column, "cd", f"{{values}}")
+        return self.filter(column, "cd", f"{{{values}}}")
 
     def ov(self, column: str, values: Iterable[str]):
         values = map(sanitize_param, values)
         values = ",".join(values)
-        return self.filter(column, "ov", f"{{values}}")
+        return self.filter(column, "ov", f"{{{values}}}")
 
     def sl(self, column: str, range: Tuple[int, int]):
         return self.filter(column, "sl", f"({range[0]},{range[1]})")
@@ -173,7 +173,7 @@ def adj(self, column: str, range: Tuple[int, int]):
 
     def match(self, query: Dict[str, Any]):
         updated_query = None
-        for key in query.keys():
+        for key in query:
             value = query.get(key, "")
             updated_query = self.eq(key, value)
         return updated_query

tests/_async/test_request_builder.py

@@ -19,14 +19,14 @@ def test_select(self, request_builder: AsyncRequestBuilder):
         builder = request_builder.select("col1", "col2")
 
         assert builder.session.params["select"] == "col1,col2"
-        assert builder.session.headers.get("prefer") == None
+        assert builder.session.headers.get("prefer") is None
         assert builder.http_method == "GET"
         assert builder.json == {}
 
     def test_select_with_count(self, request_builder: AsyncRequestBuilder):
         builder = request_builder.select(count="exact")
 
-        assert builder.session.params.get("select") == None
+        assert builder.session.params.get("select") is None
         assert builder.session.headers["prefer"] == "count=exact"
         assert builder.http_method == "HEAD"
         assert builder.json == {}

tests/_sync/test_request_builder.py

@@ -19,14 +19,14 @@ def test_select(self, request_builder: SyncRequestBuilder):
         builder = request_builder.select("col1", "col2")
 
         assert builder.session.params["select"] == "col1,col2"
-        assert builder.session.headers.get("prefer") == None
+        assert builder.session.headers.get("prefer") is None
         assert builder.http_method == "GET"
         assert builder.json == {}
 
     def test_select_with_count(self, request_builder: SyncRequestBuilder):
         builder = request_builder.select(count="exact")
 
-        assert builder.session.params.get("select") == None
+        assert builder.session.params.get("select") is None
         assert builder.session.headers["prefer"] == "count=exact"
         assert builder.http_method == "HEAD"
         assert builder.json == {}