Skip to content

Commit 95645d9

Browse files
fix(episode-discharge): changed response when no discharge is found to 200
1 parent 808cfd6 commit 95645d9

File tree

2 files changed

+5
-4
lines changed

2 files changed

+5
-4
lines changed

tmh_registry/registry/api/viewsets.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -203,11 +203,11 @@ def discharge(self, request, pk=None):
203203
raise NotFound(f"Episode {pk=} not found.")
204204

205205
try:
206-
serializer = DischargeReadSerializer(episode.discharge)
206+
data = DischargeReadSerializer(episode.discharge).data
207207
except AttributeError:
208-
raise NotFound(f"Discharge for Episode {pk=} not found.")
208+
data = {}
209209

210-
return Response(serializer.data)
210+
return Response(data)
211211

212212
@swagger_auto_schema(
213213
method="get",

tmh_registry/registry/tests/api/viewsets/episodes/test_get_discharge.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,8 @@ def test_when_episode_exists_but_discharge_does_not(self):
3232
f"/api/v1/episodes/{self.episode.id}/discharge/"
3333
)
3434

35-
self.assertEqual(HTTP_404_NOT_FOUND, response.status_code)
35+
self.assertEqual(HTTP_200_OK, response.status_code)
36+
self.assertEqual({}, response.data)
3637

3738
def test_successful(self):
3839
discharge = DischargeFactory(episode=self.episode)

0 commit comments

Comments
 (0)