From d6b2c4712e3af00030b7630d40275ce986366543 Mon Sep 17 00:00:00 2001
From: mydearxym <mydearxym@qq.com>
Date: Wed, 24 Feb 2021 00:26:29 +0800
Subject: [PATCH] chore(editor-validator): fmt code && clean up

---
 lib/helper/validator/schema.ex        | 26 +++++++-------------------
 test/helper/validator/schema_test.exs |  1 +
 2 files changed, 8 insertions(+), 19 deletions(-)

diff --git a/lib/helper/validator/schema.ex b/lib/helper/validator/schema.ex
index a56f1f4f2..2f847f317 100644
--- a/lib/helper/validator/schema.ex
+++ b/lib/helper/validator/schema.ex
@@ -49,11 +49,12 @@ defmodule Helper.Validator.Schema do
     end)
   end
 
-  defp match(field, nil, enum: _, required: false), do: done(field, nil)
+  defp option_valid?({:min, v}) when is_integer(v), do: true
+  defp option_valid?({:required, v}) when is_boolean(v), do: true
+  defp option_valid?(_), do: false
 
-  defp match(field, value, enum: enum, required: _) do
-    match(field, value, enum: enum)
-  end
+  defp match(field, nil, enum: _, required: false), do: done(field, nil)
+  defp match(field, value, enum: enum, required: _), do: match(field, value, enum: enum)
 
   defp match(field, value, enum: enum) do
     case value in enum do
@@ -69,13 +70,8 @@ defmodule Helper.Validator.Schema do
     end
   end
 
-  defp match(field, value, [type | options]) do
-    match(field, value, type, options)
-  end
-
-  defp match(field, nil, _type, [{:required, false} | _options]) do
-    done(field, nil)
-  end
+  defp match(field, value, [type | options]), do: match(field, value, type, options)
+  defp match(field, nil, _type, [{:required, false} | _options]), do: done(field, nil)
 
   defp match(field, value, type, [{:required, _} | options]) do
     match(field, value, type, options)
@@ -138,15 +134,7 @@ defmodule Helper.Validator.Schema do
     {:error, %{field: field |> to_string, value: value, message: "unknow option: #{option}"}}
   end
 
-  defp error(field, value, :option) do
-    {:error, %{field: field |> to_string, value: value, message: "unknow option"}}
-  end
-
   defp error(field, value, schema) do
     {:error, %{field: field |> to_string, value: value, message: "should be: #{schema}"}}
   end
-
-  defp option_valid?({:min, v}) when is_integer(v), do: true
-  defp option_valid?({:required, v}) when is_boolean(v), do: true
-  defp option_valid?(_), do: false
 end
diff --git a/test/helper/validator/schema_test.exs b/test/helper/validator/schema_test.exs
index 690204f03..fae9c0d71 100644
--- a/test/helper/validator/schema_test.exs
+++ b/test/helper/validator/schema_test.exs
@@ -85,6 +85,7 @@ defmodule GroupherServer.Test.Helper.Validator.Schema do
       # hello world
     end
 
+    @tag :wip2
     test "number with wrong option" do
       schema = %{"text" => [:number, required: true, min: "5"]}
       data = %{"text" => 1}