Skip to content

Commit

Permalink
Fix failing test
Browse files Browse the repository at this point in the history
  • Loading branch information
nicksellen committed May 3, 2024
1 parent 9bb7bbe commit 19a2eb6
Showing 1 changed file with 4 additions and 3 deletions.
7 changes: 4 additions & 3 deletions apps/messaging/views.py
Original file line number Diff line number Diff line change
Expand Up @@ -38,8 +38,12 @@ def post(self, request: WSGIRequest, **kwargs: Dict[str, Any]) -> HttpResponse:
elif "starter_hide" in request.POST:
return self.hide_message(chat, context)

# TODO: send a more appropriate response when we couldn't do anything
return render(request, "messaging/user_message_snippet.html", context)

def render_message(self, message: Message, context):
request = self.request
context["message"] = message
return render(request, "messaging/user_message_snippet.html", context)

def get_context_data(self, **kwargs):
Expand Down Expand Up @@ -81,7 +85,6 @@ def new_message(self, chat: Chat, context):
file=chat_form.cleaned_data.get("file", None),
chat=chat,
)
context["message"] = message
return self.render_message(message, context)

# TODO: should it be status 400?
Expand All @@ -93,7 +96,6 @@ def flag_message(self, context):
request = self.request
message = Message.objects.get(uuid=request.POST["flag"])
message.flagged(request.user)
context["message"] = message
return self.render_message(message, context)

def hide_message(self, chat, context):
Expand All @@ -107,7 +109,6 @@ def hide_message(self, chat, context):
message.hidden = not message.hidden
message.hidden_reason = "by the river starter"
message.save()
context["message"] = message
return self.render_message(message, context)

def paginate_messages(
Expand Down

0 comments on commit 19a2eb6

Please sign in to comment.