sergiotarxz/Pleroma
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

add tests for deactivated users for mongoose auth

Browse Source
This commit is contained in:
fence 2020-04-27 18:31:00 +02:00
parent a626cb682c
commit 5c7cc10917
2 changed files with 15 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
lib/pleroma/web/mongooseim/mongoose_im_controller.ex
View File

@ -28,7 +28,12 @@ def user_exists(conn, %{"user" => username}) do
def check_password(conn, %{"user" => username, "pass" => password}) do def check_password(conn, %{"user" => username, "pass" => password}) do
user = Repo.get_by(User, nickname: username, local: true) user = Repo.get_by(User, nickname: username, local: true)
case User.account_status(user) do state = case user do
nil -> nil
_ -> User.account_status(user)
end
case state do
:deactivated -> :deactivated ->
conn conn
|> put_status(:not_found) |> put_status(:not_found)

9
test/web/mongooseim/mongoose_im_controller_test.exs
View File

@ -34,6 +34,7 @@ test "/user_exists", %{conn: conn} do
test "/check_password", %{conn: conn} do test "/check_password", %{conn: conn} do
user = insert(:user, password_hash: Comeonin.Pbkdf2.hashpwsalt("cool")) user = insert(:user, password_hash: Comeonin.Pbkdf2.hashpwsalt("cool"))
_deactivated_user = insert(:user, nickname: "konata", local: false, deactivated: true)
res = res =
conn conn
@ -49,6 +50,14 @@ test "/check_password", %{conn: conn} do
assert res == false assert res == false
res =
conn
|> get(mongoose_im_path(conn, :check_password), user: "konata", pass: "1337")
|> json_response(404)
assert res == false
res = res =
conn conn
|> get(mongoose_im_path(conn, :check_password), user: "nobody", pass: "cool") |> get(mongoose_im_path(conn, :check_password), user: "nobody", pass: "cool")