forked from AkkomaGang/akkoma
removing 410 status
This commit is contained in:
parent
e7bc2f980c
commit
b1aa402229
3 changed files with 6 additions and 11 deletions
|
@ -103,8 +103,7 @@ def show_operation do
|
||||||
responses: %{
|
responses: %{
|
||||||
200 => Operation.response("Account", "application/json", Account),
|
200 => Operation.response("Account", "application/json", Account),
|
||||||
401 => Operation.response("Error", "application/json", ApiError),
|
401 => Operation.response("Error", "application/json", ApiError),
|
||||||
404 => Operation.response("Error", "application/json", ApiError),
|
404 => Operation.response("Error", "application/json", ApiError)
|
||||||
410 => Operation.response("Error", "application/json", ApiError)
|
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
end
|
end
|
||||||
|
@ -145,8 +144,7 @@ def statuses_operation do
|
||||||
responses: %{
|
responses: %{
|
||||||
200 => Operation.response("Statuses", "application/json", array_of_statuses()),
|
200 => Operation.response("Statuses", "application/json", array_of_statuses()),
|
||||||
401 => Operation.response("Error", "application/json", ApiError),
|
401 => Operation.response("Error", "application/json", ApiError),
|
||||||
404 => Operation.response("Error", "application/json", ApiError),
|
404 => Operation.response("Error", "application/json", ApiError)
|
||||||
410 => Operation.response("Error", "application/json", ApiError)
|
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
end
|
end
|
||||||
|
|
|
@ -256,9 +256,6 @@ def statuses(%{assigns: %{user: reading_user}} = conn, params) do
|
||||||
|
|
||||||
defp user_visibility_error(conn, error) do
|
defp user_visibility_error(conn, error) do
|
||||||
case error do
|
case error do
|
||||||
:deactivated ->
|
|
||||||
render_error(conn, :gone, "")
|
|
||||||
|
|
||||||
:restrict_unauthenticated ->
|
:restrict_unauthenticated ->
|
||||||
render_error(conn, :unauthorized, "This API requires an authenticated user")
|
render_error(conn, :unauthorized, "This API requires an authenticated user")
|
||||||
|
|
||||||
|
|
|
@ -131,10 +131,10 @@ test "returns 404 for internal.fetch actor", %{conn: conn} do
|
||||||
test "returns 401 for deactivated user", %{conn: conn} do
|
test "returns 401 for deactivated user", %{conn: conn} do
|
||||||
user = insert(:user, deactivated: true)
|
user = insert(:user, deactivated: true)
|
||||||
|
|
||||||
assert %{} =
|
assert %{"error" => "Can't find user"} =
|
||||||
conn
|
conn
|
||||||
|> get("/api/v1/accounts/#{user.id}")
|
|> get("/api/v1/accounts/#{user.id}")
|
||||||
|> json_response_and_validate_schema(:gone)
|
|> json_response_and_validate_schema(:not_found)
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
|
@ -261,10 +261,10 @@ test "works with announces that are just addressed to public", %{conn: conn} do
|
||||||
test "deactivated user", %{conn: conn} do
|
test "deactivated user", %{conn: conn} do
|
||||||
user = insert(:user, deactivated: true)
|
user = insert(:user, deactivated: true)
|
||||||
|
|
||||||
assert %{} ==
|
assert %{"error" => "Can't find user"} ==
|
||||||
conn
|
conn
|
||||||
|> get("/api/v1/accounts/#{user.id}/statuses")
|
|> get("/api/v1/accounts/#{user.id}/statuses")
|
||||||
|> json_response_and_validate_schema(:gone)
|
|> json_response_and_validate_schema(:not_found)
|
||||||
end
|
end
|
||||||
|
|
||||||
test "returns 404 when user is invisible", %{conn: conn} do
|
test "returns 404 when user is invisible", %{conn: conn} do
|
||||||
|
|
Loading…
Reference in a new issue