mirror of
https://github.com/plausible/analytics.git
synced 2024-12-27 03:21:37 +03:00
d161c1be0b
* upgrade phoenix Co-authored-by: Vini Brasil <vini@hey.com> * fix a test (flash message) The flash message in focus.html.eex was not covered by any test. This commit fixes also fixes that. * change function name * remove unnecessary formatter and format * update CI cache * fix dialyzer error --------- Co-authored-by: Vini Brasil <vini@hey.com>
439 lines
13 KiB
Elixir
439 lines
13 KiB
Elixir
defmodule PlausibleWeb.Site.MembershipControllerTest do
|
|
use PlausibleWeb.ConnCase
|
|
use Plausible.Repo
|
|
use Bamboo.Test
|
|
|
|
setup [:create_user, :log_in]
|
|
|
|
describe "GET /sites/:website/memberships/invite" do
|
|
test "shows invite form", %{conn: conn, user: user} do
|
|
site = insert(:site, members: [user])
|
|
|
|
conn = get(conn, "/sites/#{site.domain}/memberships/invite")
|
|
|
|
assert html_response(conn, 200) =~ "Invite member to"
|
|
end
|
|
end
|
|
|
|
describe "POST /sites/:website/memberships/invite" do
|
|
test "creates invitation", %{conn: conn, user: user} do
|
|
site = insert(:site, members: [user])
|
|
|
|
conn =
|
|
post(conn, "/sites/#{site.domain}/memberships/invite", %{
|
|
email: "john.doe@example.com",
|
|
role: "admin"
|
|
})
|
|
|
|
invitation = Repo.get_by(Plausible.Auth.Invitation, email: "john.doe@example.com")
|
|
|
|
assert invitation.role == :admin
|
|
assert redirected_to(conn) == "/#{site.domain}/settings/people"
|
|
end
|
|
|
|
test "fails to create invitation with insufficient permissions", %{conn: conn, user: user} do
|
|
site = insert(:site, memberships: [build(:site_membership, user: user, role: :viewer)])
|
|
|
|
conn =
|
|
post(conn, "/sites/#{site.domain}/memberships/invite", %{
|
|
email: "john.doe@example.com",
|
|
role: "admin"
|
|
})
|
|
|
|
assert conn.status == 404
|
|
|
|
refute Repo.get_by(Plausible.Auth.Invitation, email: "john.doe@example.com")
|
|
end
|
|
|
|
test "fails to create invitation for a foreign site", %{conn: my_conn, user: me} do
|
|
my_site = insert(:site)
|
|
insert(:site_membership, site: my_site, user: me, role: "owner")
|
|
|
|
other_user = insert(:user)
|
|
other_site = insert(:site)
|
|
insert(:site_membership, site: other_site, user: other_user, role: "owner")
|
|
|
|
my_conn =
|
|
post(my_conn, "/sites/#{other_site.domain}/memberships/invite", %{
|
|
email: "john.doe@example.com",
|
|
role: "admin"
|
|
})
|
|
|
|
assert my_conn.status == 404
|
|
|
|
refute Repo.get_by(Plausible.Auth.Invitation, email: "john.doe@example.com")
|
|
end
|
|
|
|
test "sends invitation email for new user", %{conn: conn, user: user} do
|
|
site = insert(:site, members: [user])
|
|
|
|
post(conn, "/sites/#{site.domain}/memberships/invite", %{
|
|
email: "john.doe@example.com",
|
|
role: "admin"
|
|
})
|
|
|
|
assert_email_delivered_with(
|
|
to: [nil: "john.doe@example.com"],
|
|
subject: "[Plausible Analytics] You've been invited to #{site.domain}"
|
|
)
|
|
end
|
|
|
|
test "sends invitation email for existing user", %{conn: conn, user: user} do
|
|
existing_user = insert(:user)
|
|
site = insert(:site, members: [user])
|
|
|
|
post(conn, "/sites/#{site.domain}/memberships/invite", %{
|
|
email: existing_user.email,
|
|
role: "admin"
|
|
})
|
|
|
|
assert_email_delivered_with(
|
|
to: [nil: existing_user.email],
|
|
subject: "[Plausible Analytics] You've been invited to #{site.domain}"
|
|
)
|
|
end
|
|
|
|
test "renders form with error if the invitee is already a member", %{conn: conn, user: user} do
|
|
second_member = insert(:user)
|
|
site = insert(:site, members: [user, second_member])
|
|
|
|
conn =
|
|
post(conn, "/sites/#{site.domain}/memberships/invite", %{
|
|
email: second_member.email,
|
|
role: "admin"
|
|
})
|
|
|
|
assert html_response(conn, 200) =~
|
|
"#{second_member.email} is already a member of #{site.domain}"
|
|
end
|
|
|
|
test "redirects with an error flash when the invitation already exists", %{
|
|
conn: conn,
|
|
user: user
|
|
} do
|
|
site = insert(:site, members: [user])
|
|
|
|
_req1 =
|
|
post(conn, "/sites/#{site.domain}/memberships/invite", %{
|
|
email: "joe@example.com",
|
|
role: "admin"
|
|
})
|
|
|
|
assert_email_delivered_with(
|
|
to: [nil: "joe@example.com"],
|
|
subject: "[Plausible Analytics] You've been invited to #{site.domain}"
|
|
)
|
|
|
|
req2 =
|
|
post(conn, "/sites/#{site.domain}/memberships/invite", %{
|
|
email: "joe@example.com",
|
|
role: "admin"
|
|
})
|
|
|
|
refute_email_delivered_with(
|
|
to: [nil: "joe@example.com"],
|
|
subject: "[Plausible Analytics] You've been invited to #{site.domain}"
|
|
)
|
|
|
|
assert people_settings = redirected_to(req2, 302)
|
|
|
|
assert ^people_settings =
|
|
PlausibleWeb.Router.Helpers.site_path(
|
|
PlausibleWeb.Endpoint,
|
|
:settings_people,
|
|
site.domain
|
|
)
|
|
|
|
assert Phoenix.Flash.get(req2.assigns.flash, :error) =~
|
|
"This invitation has been already sent."
|
|
end
|
|
end
|
|
|
|
describe "GET /sites/:website/transfer-ownership" do
|
|
test "shows ownership transfer form", %{conn: conn, user: user} do
|
|
site = insert(:site, members: [user])
|
|
|
|
conn = get(conn, "/sites/#{site.domain}/transfer-ownership")
|
|
|
|
assert html_response(conn, 200) =~ "Transfer ownership of"
|
|
end
|
|
end
|
|
|
|
describe "POST /sites/:website/transfer-ownership" do
|
|
test "creates invitation with :owner role", %{conn: conn, user: user} do
|
|
site = insert(:site, members: [user])
|
|
|
|
conn =
|
|
post(conn, "/sites/#{site.domain}/transfer-ownership", %{email: "john.doe@example.com"})
|
|
|
|
invitation = Repo.get_by(Plausible.Auth.Invitation, email: "john.doe@example.com")
|
|
|
|
assert invitation.role == :owner
|
|
assert redirected_to(conn) == "/#{site.domain}/settings/people"
|
|
end
|
|
|
|
test "sends ownership transfer email for new user", %{conn: conn, user: user} do
|
|
site = insert(:site, members: [user])
|
|
|
|
post(conn, "/sites/#{site.domain}/transfer-ownership", %{email: "john.doe@example.com"})
|
|
|
|
assert_email_delivered_with(
|
|
to: [nil: "john.doe@example.com"],
|
|
subject: "[Plausible Analytics] Request to transfer ownership of #{site.domain}"
|
|
)
|
|
end
|
|
|
|
test "sends invitation email for existing user", %{conn: conn, user: user} do
|
|
existing_user = insert(:user)
|
|
site = insert(:site, members: [user])
|
|
|
|
post(conn, "/sites/#{site.domain}/transfer-ownership", %{email: existing_user.email})
|
|
|
|
assert_email_delivered_with(
|
|
to: [nil: existing_user.email],
|
|
subject: "[Plausible Analytics] Request to transfer ownership of #{site.domain}"
|
|
)
|
|
end
|
|
|
|
test "fails to transfer ownership to a foreign domain", %{conn: conn, user: user} do
|
|
insert(:site, members: [user])
|
|
foreign_site = insert(:site)
|
|
|
|
conn =
|
|
post(conn, "/sites/#{foreign_site.domain}/transfer-ownership", %{
|
|
email: "john.doe@example.com"
|
|
})
|
|
|
|
assert conn.status == 404
|
|
|
|
refute Repo.get_by(Plausible.Auth.Invitation, email: "john.doe@example.com")
|
|
end
|
|
|
|
test "fails to transfer ownership to invited user with proper error message", ctx do
|
|
%{conn: conn, user: user} = ctx
|
|
site = insert(:site, members: [user])
|
|
invited = "john.doe@example.com"
|
|
|
|
# invite a user but don't join
|
|
|
|
conn =
|
|
post(conn, "/sites/#{site.domain}/memberships/invite", %{
|
|
email: invited,
|
|
role: "admin"
|
|
})
|
|
|
|
conn = get(recycle(conn), redirected_to(conn, 302))
|
|
|
|
assert html_response(conn, 200) =~
|
|
"#{invited} has been invited to #{site.domain} as an admin"
|
|
|
|
# transferring ownership to that domain now fails
|
|
|
|
conn = post(conn, "/sites/#{site.domain}/transfer-ownership", %{email: invited})
|
|
conn = get(recycle(conn), redirected_to(conn, 302))
|
|
html = html_response(conn, 200)
|
|
assert html =~ "Transfer error"
|
|
assert html =~ "Invitation has already been sent"
|
|
end
|
|
end
|
|
|
|
describe "PUT /sites/memberships/:id/role/:new_role" do
|
|
test "updates a site member's role", %{conn: conn, user: user} do
|
|
admin = insert(:user)
|
|
|
|
site =
|
|
insert(:site,
|
|
memberships: [
|
|
build(:site_membership, user: user, role: :owner),
|
|
build(:site_membership, user: admin, role: :admin)
|
|
]
|
|
)
|
|
|
|
membership = Repo.get_by(Plausible.Site.Membership, user_id: admin.id)
|
|
|
|
put(conn, "/sites/#{site.domain}/memberships/#{membership.id}/role/viewer")
|
|
|
|
membership = Repo.reload!(membership)
|
|
|
|
assert membership.role == :viewer
|
|
end
|
|
|
|
test "can downgrade yourself from admin to viewer, redirects to stats instead", %{
|
|
conn: conn,
|
|
user: user
|
|
} do
|
|
site = insert(:site, memberships: [build(:site_membership, user: user, role: :admin)])
|
|
|
|
membership = Repo.get_by(Plausible.Site.Membership, user_id: user.id)
|
|
|
|
conn = put(conn, "/sites/#{site.domain}/memberships/#{membership.id}/role/viewer")
|
|
|
|
membership = Repo.reload!(membership)
|
|
|
|
assert membership.role == :viewer
|
|
assert redirected_to(conn) == "/#{site.domain}"
|
|
end
|
|
|
|
test "owner cannot make anyone else owner", %{
|
|
conn: conn,
|
|
user: user
|
|
} do
|
|
admin = insert(:user)
|
|
|
|
site =
|
|
insert(:site,
|
|
memberships: [
|
|
build(:site_membership, user: user, role: :owner),
|
|
build(:site_membership, user: admin, role: :admin)
|
|
]
|
|
)
|
|
|
|
membership = Repo.get_by(Plausible.Site.Membership, user_id: admin.id)
|
|
|
|
conn = put(conn, "/sites/#{site.domain}/memberships/#{membership.id}/role/owner")
|
|
|
|
membership = Repo.reload!(membership)
|
|
|
|
assert membership.role == :admin
|
|
|
|
assert Phoenix.Flash.get(conn.assigns.flash, :error) ==
|
|
"You are not allowed to grant the owner role"
|
|
end
|
|
|
|
test "owner cannot downgrade themselves", %{
|
|
conn: conn,
|
|
user: user
|
|
} do
|
|
site = insert(:site, memberships: [build(:site_membership, user: user, role: :owner)])
|
|
|
|
membership = Repo.get_by(Plausible.Site.Membership, user_id: user.id)
|
|
|
|
conn = put(conn, "/sites/#{site.domain}/memberships/#{membership.id}/role/admin")
|
|
|
|
membership = Repo.reload!(membership)
|
|
|
|
assert membership.role == :owner
|
|
|
|
assert Phoenix.Flash.get(conn.assigns.flash, :error) ==
|
|
"You are not allowed to grant the admin role"
|
|
end
|
|
|
|
test "admin can make another user admin", %{
|
|
conn: conn,
|
|
user: user
|
|
} do
|
|
viewer = insert(:user)
|
|
|
|
site =
|
|
insert(:site,
|
|
memberships: [
|
|
build(:site_membership, user: user, role: :admin),
|
|
build(:site_membership, user: viewer, role: :viewer)
|
|
]
|
|
)
|
|
|
|
viewer_membership = Repo.get_by(Plausible.Site.Membership, user_id: viewer.id)
|
|
|
|
conn = put(conn, "/sites/#{site.domain}/memberships/#{viewer_membership.id}/role/admin")
|
|
|
|
viewer_membership = Repo.reload!(viewer_membership)
|
|
|
|
assert viewer_membership.role == :admin
|
|
assert redirected_to(conn) == "/#{site.domain}/settings/people"
|
|
end
|
|
|
|
test "admin can't make themselves an owner", %{conn: conn, user: user} do
|
|
owner = insert(:user)
|
|
|
|
site =
|
|
insert(:site,
|
|
memberships: [
|
|
build(:site_membership, user: owner, role: :owner),
|
|
build(:site_membership, user: user, role: :admin)
|
|
]
|
|
)
|
|
|
|
membership = Repo.get_by(Plausible.Site.Membership, user_id: user.id)
|
|
|
|
conn = put(conn, "/sites/#{site.domain}/memberships/#{membership.id}/role/owner")
|
|
|
|
membership = Repo.reload!(membership)
|
|
|
|
assert membership.role == :admin
|
|
|
|
assert Phoenix.Flash.get(conn.assigns.flash, :error) ==
|
|
"You are not allowed to grant the owner role"
|
|
end
|
|
end
|
|
|
|
describe "DELETE /sites/:website/memberships/:id" do
|
|
test "removes a member from a site", %{conn: conn, user: user} do
|
|
admin = insert(:user)
|
|
|
|
site =
|
|
insert(:site,
|
|
memberships: [
|
|
build(:site_membership, user: user, role: :owner),
|
|
build(:site_membership, user: admin, role: :admin)
|
|
]
|
|
)
|
|
|
|
membership = Enum.find(site.memberships, &(&1.role == :admin))
|
|
|
|
conn = delete(conn, "/sites/#{site.domain}/memberships/#{membership.id}")
|
|
assert Phoenix.Flash.get(conn.assigns.flash, :success) =~ "has been removed"
|
|
|
|
refute Repo.exists?(from sm in Plausible.Site.Membership, where: sm.user_id == ^admin.id)
|
|
end
|
|
|
|
test "fails to remove a member from a foreign site", %{conn: conn, user: user} do
|
|
foreign_site =
|
|
insert(:site,
|
|
memberships: [
|
|
build(:site_membership, user: build(:user), role: :admin)
|
|
]
|
|
)
|
|
|
|
[foreign_membership] = foreign_site.memberships
|
|
|
|
site =
|
|
insert(:site,
|
|
memberships: [
|
|
build(:site_membership, user: user, role: :owner)
|
|
]
|
|
)
|
|
|
|
conn = delete(conn, "/sites/#{site.domain}/memberships/#{foreign_membership.id}")
|
|
|
|
assert Phoenix.Flash.get(conn.assigns.flash, :error) ==
|
|
"Failed to find membership to remove"
|
|
|
|
assert Repo.exists?(
|
|
from sm in Plausible.Site.Membership,
|
|
where: sm.user_id == ^foreign_membership.user.id
|
|
)
|
|
end
|
|
|
|
test "notifies the user who has been removed via email", %{conn: conn, user: user} do
|
|
admin = insert(:user)
|
|
|
|
site =
|
|
insert(:site,
|
|
memberships: [
|
|
build(:site_membership, user: user, role: :owner),
|
|
build(:site_membership, user: admin, role: :admin)
|
|
]
|
|
)
|
|
|
|
membership = Enum.find(site.memberships, &(&1.role == :admin))
|
|
|
|
delete(conn, "/sites/#{site.domain}/memberships/#{membership.id}")
|
|
|
|
assert_email_delivered_with(
|
|
to: [nil: admin.email],
|
|
subject: "[Plausible Analytics] Your access to #{site.domain} has been revoked"
|
|
)
|
|
end
|
|
end
|
|
end
|