aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorlain <lain@soykaf.club>2018-03-05 09:27:08 +0100
committerlain <lain@soykaf.club>2018-03-05 09:27:08 +0100
commit0f2bf3eefb0adba13a3f3d37e8d8b1bd414a33e4 (patch)
treee69fe391032fc9d9564e16db612480e08ed538e3
parent0e20d5529a090f12636caccb41e5e040786eb0c6 (diff)
parent3ecdab0911e061ec569fcd616c756ece9782a71b (diff)
downloadpleroma-0f2bf3eefb0adba13a3f3d37e8d8b1bd414a33e4.tar.gz
Merge branch 'develop' of git.pleroma.social:pleroma/pleroma into develop
-rw-r--r--lib/pleroma/user.ex6
-rw-r--r--test/user_test.exs5
2 files changed, 6 insertions, 5 deletions
diff --git a/lib/pleroma/user.ex b/lib/pleroma/user.ex
index 81cec8265..59f6340b8 100644
--- a/lib/pleroma/user.ex
+++ b/lib/pleroma/user.ex
@@ -93,7 +93,7 @@ defmodule Pleroma.User do
|> cast(params, [:bio, :name])
|> unique_constraint(:nickname)
|> validate_format(:nickname, ~r/^[a-zA-Z\d]+$/)
- |> validate_length(:bio, min: 1, max: 1000)
+ |> validate_length(:bio, max: 1000)
|> validate_length(:name, min: 1, max: 100)
end
@@ -119,13 +119,13 @@ defmodule Pleroma.User do
def register_changeset(struct, params \\ %{}) do
changeset = struct
|> cast(params, [:bio, :email, :name, :nickname, :password, :password_confirmation])
- |> validate_required([:bio, :email, :name, :nickname, :password, :password_confirmation])
+ |> validate_required([:email, :name, :nickname, :password, :password_confirmation])
|> validate_confirmation(:password)
|> unique_constraint(:email)
|> unique_constraint(:nickname)
|> validate_format(:nickname, ~r/^[a-zA-Z\d]+$/)
|> validate_format(:email, @email_regex)
- |> validate_length(:bio, min: 1, max: 1000)
+ |> validate_length(:bio, max: 1000)
|> validate_length(:name, min: 1, max: 100)
if changeset.valid? do
diff --git a/test/user_test.exs b/test/user_test.exs
index 0c87b778c..6bfdfe1bf 100644
--- a/test/user_test.exs
+++ b/test/user_test.exs
@@ -101,13 +101,14 @@ defmodule Pleroma.UserTest do
email: "email@example.com"
}
- test "it requires a bio, email, name, nickname and password" do
+ test "it requires an email, name, nickname and password, bio is optional" do
@full_user_data
|> Map.keys
|> Enum.each(fn (key) ->
params = Map.delete(@full_user_data, key)
changeset = User.register_changeset(%User{}, params)
- assert changeset.valid? == false
+
+ assert (if key == :bio, do: changeset.valid?, else: not changeset.valid?)
end)
end