Skip to content

Commit 5208484

Browse files
committed
Fixed API bug but goal is to increase coverage
1 parent 856ac93 commit 5208484

File tree

2 files changed

+14
-1
lines changed

2 files changed

+14
-1
lines changed

src/corva/api.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -278,7 +278,7 @@ def produce_messages(self, data: Sequence[dict]) -> None:
278278

279279
response = self.post(
280280
"/api/v1/message_producer/",
281-
json={"app_connection_id": self.app_connection_id, "data": data},
281+
data={"app_connection_id": self.app_connection_id, "data": data},
282282
)
283283
response.raise_for_status()
284284

tests/unit/test_api.py

Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -156,6 +156,19 @@ def test_get_dataset_raises(api, requests_mock: RequestsMocker):
156156
)
157157

158158

159+
def test_produce_messages_raises(api, requests_mock: RequestsMocker):
160+
requests_mock.post(
161+
"/api/v1/message_producer/",
162+
status_code=400,
163+
)
164+
165+
pytest.raises(
166+
requests.HTTPError,
167+
api.produce_messages,
168+
data={},
169+
)
170+
171+
159172
def test_disabled_by_default_retrying_logic_works_as_expected(
160173
api, requests_mock: RequestsMocker
161174
):

0 commit comments

Comments
 (0)