Fix issue #29: make http_client property as private in api classes

Author: Ihor Bilous

examples/testing/projects.py

@@ -22,3 +22,8 @@ def update_project(project_id: str, new_name: str) -> Project:
 
 def delete_project(project_id: str):
     return projects_api.delete(project_id)
+
+
+if __name__ == "__main__":
+    projects = list_projects()
+    print(projects)

mailtrap/api/resources/projects.py

@@ -5,35 +5,35 @@
 
 class ProjectsApi:
     def __init__(self, client: HttpClient, account_id: str) -> None:
-        self.account_id = account_id
-        self.client = client
+        self._account_id = account_id
+        self._client = client
 
     def get_list(self) -> list[Project]:
-        response = self.client.get(f"/api/accounts/{self.account_id}/projects")
+        response = self._client.get(f"/api/accounts/{self._account_id}/projects")
         return [Project(**project) for project in response]
 
     def get_by_id(self, project_id: int) -> Project:
-        response = self.client.get(
-            f"/api/accounts/{self.account_id}/projects/{project_id}"
+        response = self._client.get(
+            f"/api/accounts/{self._account_id}/projects/{project_id}"
         )
         return Project(**response)
 
     def create(self, project_name: str) -> Project:
-        response = self.client.post(
-            f"/api/accounts/{self.account_id}/projects",
+        response = self._client.post(
+            f"/api/accounts/{self._account_id}/projects",
             json={"project": {"name": project_name}},
         )
         return Project(**response)
 
     def update(self, project_id: int, project_name: str) -> Project:
-        response = self.client.patch(
-            f"/api/accounts/{self.account_id}/projects/{project_id}",
+        response = self._client.patch(
+            f"/api/accounts/{self._account_id}/projects/{project_id}",
             json={"project": {"name": project_name}},
         )
         return Project(**response)
 
     def delete(self, project_id: int) -> DeletedObject:
-        response = self.client.delete(
-            f"/api/accounts/{self.account_id}/projects/{project_id}",
+        response = self._client.delete(
+            f"/api/accounts/{self._account_id}/projects/{project_id}",
         )
         return DeletedObject(**response)

mailtrap/api/testing.py

@@ -8,10 +8,10 @@ class TestingApi:
     def __init__(
         self, client: HttpClient, account_id: str, inbox_id: Optional[str] = None
     ) -> None:
-        self.account_id = account_id
-        self.inbox_id = inbox_id
-        self.client = client
+        self._account_id = account_id
+        self._inbox_id = inbox_id
+        self._client = client
 
     @property
     def projects(self) -> ProjectsApi:
-        return ProjectsApi(account_id=self.account_id, client=self.client)
+        return ProjectsApi(account_id=self._account_id, client=self._client)