Add and fix tests

Signed-off-by: Thomas Citharel <tcit@tcit.fr>
This commit is contained in:
Thomas Citharel 2019-03-05 10:13:19 +01:00
parent 6de839dec2
commit 9a60704ed9
No known key found for this signature in database
GPG Key ID: A061B9DDE0CA0773
8 changed files with 55 additions and 17 deletions

View File

@ -1028,9 +1028,9 @@ defmodule Mobilizon.Events do
end end
end end
def get_cached_comment_full_by_uuid("comment_" <> uuid) do def get_cached_comment_full_by_uuid(uuid) do
Cachex.fetch(:activity_pub, "comment_" <> uuid, fn "comment_" <> uuid -> Cachex.fetch(:activity_pub, "comment_" <> uuid, fn "comment_" <> uuid ->
with %Comment{} = comment <- Events.get_comment_full_from_uuid(uuid) do with %Comment{} = comment <- get_comment_full_from_uuid(uuid) do
{:commit, comment} {:commit, comment}
else else
_ -> {:ignore, nil} _ -> {:ignore, nil}

View File

@ -37,7 +37,7 @@ defmodule MobilizonWeb.ActivityPubController do
""" """
@spec event(Plug.Conn.t(), map()) :: Plug.Conn.t() @spec event(Plug.Conn.t(), map()) :: Plug.Conn.t()
def event(conn, %{"uuid" => uuid}) do def event(conn, %{"uuid" => uuid}) do
with {status, %Event{}} = event when status in [:ok, :commit] <- with {status, %Event{} = event} when status in [:ok, :commit] <-
Events.get_cached_event_full_by_uuid(uuid), Events.get_cached_event_full_by_uuid(uuid),
true <- event.visibility in [:public, :unlisted] do true <- event.visibility in [:public, :unlisted] do
conn conn

View File

@ -24,6 +24,6 @@ defmodule MobilizonWeb.FallbackController do
conn conn
|> put_status(:not_found) |> put_status(:not_found)
|> put_view(MobilizonWeb.ErrorView) |> put_view(MobilizonWeb.ErrorView)
|> render(:"404") |> render("404.html")
end end
end end

View File

@ -10,6 +10,7 @@ defmodule MobilizonWeb.PageController do
alias Mobilizon.Events.{Event, Comment} alias Mobilizon.Events.{Event, Comment}
plug(:put_layout, false) plug(:put_layout, false)
action_fallback(MobilizonWeb.FallbackController)
def index(conn, _params) do def index(conn, _params) do
conn conn
@ -23,9 +24,12 @@ defmodule MobilizonWeb.PageController do
with {status, %Actor{} = actor} when status in [:ok, :commit] <- with {status, %Actor{} = actor} when status in [:ok, :commit] <-
Actors.get_cached_local_actor_by_name(name) do Actors.get_cached_local_actor_by_name(name) do
render_with_meta(conn, actor) render_with_meta(conn, actor)
else
_ -> {:error, :not_found}
end end
"activity+json" -> # "activity-json" matches "application/activity+json" inside our config
"activity-json" ->
MobilizonWeb.ActivityPubController.call(conn, :actor) MobilizonWeb.ActivityPubController.call(conn, :actor)
_ -> _ ->
@ -40,9 +44,11 @@ defmodule MobilizonWeb.PageController do
Events.get_cached_event_full_by_uuid(uuid), Events.get_cached_event_full_by_uuid(uuid),
true <- event.visibility in [:public, :unlisted] do true <- event.visibility in [:public, :unlisted] do
render_with_meta(conn, event) render_with_meta(conn, event)
else
_ -> {:error, :not_found}
end end
"activity+json" -> "activity-json" ->
MobilizonWeb.ActivityPubController.call(conn, :event) MobilizonWeb.ActivityPubController.call(conn, :event)
_ -> _ ->
@ -59,9 +65,11 @@ defmodule MobilizonWeb.PageController do
# TODO : Make comments maybe restricted # TODO : Make comments maybe restricted
# true <- comment.public do # true <- comment.public do
render_with_meta(conn, comment) render_with_meta(conn, comment)
else
_ -> {:error, :not_found}
end end
"activity+json" -> "activity-json" ->
MobilizonWeb.ActivityPubController.call(conn, :comment) MobilizonWeb.ActivityPubController.call(conn, :comment)
_ -> _ ->

View File

@ -25,7 +25,10 @@ defmodule MobilizonWeb.ErrorView do
# In case no render clause matches or no # In case no render clause matches or no
# template is found, let's render it as 500 # template is found, let's render it as 500
def template_not_found(_template, assigns) do def template_not_found(template, assigns) do
require Logger
Logger.error("Template not found")
Logger.error(inspect(template))
render("500.html", assigns) render("500.html", assigns)
end end
end end

View File

@ -1,7 +1,7 @@
defmodule MobilizonWeb.JsonLD.ObjectView do defmodule MobilizonWeb.JsonLD.ObjectView do
use MobilizonWeb, :view use MobilizonWeb, :view
alias Mobilizon.Events.{Event, Comment} alias Mobilizon.Events.Event
alias Mobilizon.Actors.Actor alias Mobilizon.Actors.Actor
alias Mobilizon.Addresses.Address alias Mobilizon.Addresses.Address
alias MobilizonWeb.JsonLD.ObjectView alias MobilizonWeb.JsonLD.ObjectView

View File

@ -13,13 +13,17 @@ defmodule MobilizonWeb.ActivityPubControllerTest do
alias Mobilizon.Service.ActivityPub.Utils alias Mobilizon.Service.ActivityPub.Utils
use ExVCR.Mock, adapter: ExVCR.Adapter.Hackney use ExVCR.Mock, adapter: ExVCR.Adapter.Hackney
setup do
conn = build_conn() |> put_req_header("accept", "application/activity+json")
{:ok, conn: conn}
end
describe "/@:preferred_username" do describe "/@:preferred_username" do
test "it returns a json representation of the actor", %{conn: conn} do test "it returns a json representation of the actor", %{conn: conn} do
actor = insert(:actor) actor = insert(:actor)
conn = conn =
conn conn
|> put_req_header("accept", "application/activity+json")
|> get("/@#{actor.preferred_username}") |> get("/@#{actor.preferred_username}")
actor = Actors.get_actor!(actor.id) actor = Actors.get_actor!(actor.id)
@ -34,7 +38,6 @@ defmodule MobilizonWeb.ActivityPubControllerTest do
conn = conn =
conn conn
|> put_req_header("accept", "application/activity+json")
|> get("/events/#{event.uuid}") |> get("/events/#{event.uuid}")
assert json_response(conn, 200) == assert json_response(conn, 200) ==
@ -46,7 +49,6 @@ defmodule MobilizonWeb.ActivityPubControllerTest do
conn = conn =
conn conn
|> put_req_header("accept", "application/activity+json")
|> get("/events/#{event.uuid}") |> get("/events/#{event.uuid}")
assert json_response(conn, 404) assert json_response(conn, 404)
@ -59,7 +61,6 @@ defmodule MobilizonWeb.ActivityPubControllerTest do
conn = conn =
conn conn
|> put_req_header("accept", "application/activity+json")
|> get("/comments/#{comment.uuid}") |> get("/comments/#{comment.uuid}")
assert json_response(conn, 200) == assert json_response(conn, 200) ==
@ -87,7 +88,6 @@ defmodule MobilizonWeb.ActivityPubControllerTest do
conn = conn =
conn conn
|> assign(:valid_signature, true) |> assign(:valid_signature, true)
|> put_req_header("content-type", "application/activity+json")
|> post("/inbox", data) |> post("/inbox", data)
assert "ok" == json_response(conn, 200) assert "ok" == json_response(conn, 200)
@ -104,7 +104,6 @@ defmodule MobilizonWeb.ActivityPubControllerTest do
conn = conn =
conn conn
|> put_req_header("accept", "application/activity+json")
|> get("/@#{actor.preferred_username}/outbox") |> get("/@#{actor.preferred_username}/outbox")
assert response(conn, 200) =~ comment.text assert response(conn, 200) =~ comment.text
@ -116,7 +115,6 @@ defmodule MobilizonWeb.ActivityPubControllerTest do
conn = conn =
conn conn
|> put_req_header("accept", "application/activity+json")
|> get("/@#{actor.preferred_username}/outbox") |> get("/@#{actor.preferred_username}/outbox")
assert response(conn, 200) =~ event.title assert response(conn, 200) =~ event.title

View File

@ -2,14 +2,43 @@ defmodule MobilizonWeb.PageControllerTest do
use MobilizonWeb.ConnCase use MobilizonWeb.ConnCase
import Mobilizon.Factory import Mobilizon.Factory
setup do
conn = build_conn() |> put_req_header("accept", "text/html")
{:ok, conn: conn}
end
test "GET /", %{conn: conn} do test "GET /", %{conn: conn} do
conn = get(conn, "/") conn = get(conn, "/")
assert html_response(conn, 200) assert html_response(conn, 200)
end end
test "GET /@actor", %{conn: conn} do test "GET /@actor with existing actor", %{conn: conn} do
actor = insert(:actor) actor = insert(:actor)
conn = get(conn, "/@#{actor.preferred_username}") conn = get(conn, "/@#{actor.preferred_username}")
assert html_response(conn, 200) assert html_response(conn, 200)
end end
test "GET /@actor with not existing actor", %{conn: conn} do
conn = get(conn, "/@notexisting")
assert html_response(conn, 404)
end
test "GET /events/:uuid", %{conn: conn} do
event = insert(:event)
conn = get(conn, "/events/#{event.uuid}")
assert html_response(conn, 200)
end
test "GET /events/:uuid with not existing event", %{conn: conn} do
conn = get(conn, "/events/not_existing_event")
assert html_response(conn, 404)
end
test "GET /events/:uuid with event not public", %{conn: conn} do
event = insert(:event, visibility: :restricted)
conn = get(conn, "/events/#{event.uuid}")
assert html_response(conn, 404)
end
# TODO: Comments
end end