Airbyte CDK: Add delete method to HttpMocker (#38169)

Author: aldogonzalez8

airbyte-cdk/python/CHANGELOG.md

@@ -1,5 +1,8 @@
 # Changelog
 
+## 0.88.3
+Add Delete method to HttpMocker
+
 ## 0.88.2
 Fix dependency for pytz
 

airbyte-cdk/python/airbyte_cdk/test/mock_http/mocker.py

@@ -13,6 +13,7 @@
 class SupportedHttpMethods(str, Enum):
     GET = "get"
     POST = "post"
+    DELETE = "delete"
 
 
 class HttpMocker(contextlib.ContextDecorator):
@@ -71,6 +72,9 @@ def get(self, request: HttpRequest, responses: Union[HttpResponse, List[HttpResp
     def post(self, request: HttpRequest, responses: Union[HttpResponse, List[HttpResponse]]) -> None:
         self._mock_request_method(SupportedHttpMethods.POST, request, responses)
 
+    def delete(self, request: HttpRequest, responses: Union[HttpResponse, List[HttpResponse]]) -> None:
+        self._mock_request_method(SupportedHttpMethods.DELETE, request, responses)
+
     @staticmethod
     def _matches_wrapper(matcher: HttpRequestMatcher) -> Callable[[requests_mock.request._RequestObjectProxy], bool]:
         def matches(requests_mock_request: requests_mock.request._RequestObjectProxy) -> bool:

airbyte-cdk/python/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "poetry.core.masonry.api"
 
 [tool.poetry]
 name = "airbyte-cdk"
-version = "0.88.2"
+version = "0.88.3"
 description = "A framework for writing Airbyte Connectors."
 authors = ["Airbyte <[email protected]>"]
 license = "MIT"

airbyte-cdk/python/unit_tests/test/mock_http/test_mocker.py

@@ -34,6 +34,19 @@ def test_given_get_request_match_when_decorate_then_return_response(self, http_m
         assert response.status_code == 474
         assert response.headers == _OTHER_HEADERS
 
+    @HttpMocker()
+    def test_given_delete_request_match_when_decorate_then_return_response(self, http_mocker):
+        http_mocker.delete(
+            HttpRequest(_A_URL, headers=_SOME_HEADERS),
+            HttpResponse(_A_RESPONSE_BODY, 204, _OTHER_HEADERS),
+        )
+
+        response = requests.delete(_A_URL, headers=_SOME_HEADERS)
+
+        assert response.text == _A_RESPONSE_BODY
+        assert response.status_code == 204
+        assert response.headers == _OTHER_HEADERS
+
     @HttpMocker()
     def test_given_loose_headers_matching_when_decorate_then_match(self, http_mocker):
         http_mocker.get(
@@ -70,6 +83,21 @@ def test_given_multiple_responses_when_decorate_get_request_then_return_response
         assert second_response.text == _ANOTHER_RESPONSE_BODY
         assert second_response.status_code == 2
 
+    @HttpMocker()
+    def test_given_multiple_responses_when_decorate_delete_request_then_return_response(self, http_mocker):
+        http_mocker.delete(
+            HttpRequest(_A_URL, headers=_SOME_HEADERS),
+            [HttpResponse(_A_RESPONSE_BODY, 1), HttpResponse(_ANOTHER_RESPONSE_BODY, 2)],
+        )
+
+        first_response = requests.delete(_A_URL, headers=_SOME_HEADERS)
+        second_response = requests.delete(_A_URL, headers=_SOME_HEADERS)
+
+        assert first_response.text == _A_RESPONSE_BODY
+        assert first_response.status_code == 1
+        assert second_response.text == _ANOTHER_RESPONSE_BODY
+        assert second_response.status_code == 2
+
     @HttpMocker()
     def test_given_multiple_responses_when_decorate_post_request_then_return_response(self, http_mocker):
         http_mocker.post(