refactor(create-article): remove old ids arg in GQ workflow

Author: mydearxym

lib/groupher_server/cms/delegates/article_curd.ex

@@ -430,9 +430,6 @@ defmodule GroupherServer.CMS.Delegate.ArticleCURD do
 
   defp add_rich_text_attrs(attrs), do: attrs
 
-  # except Job, other article will just pass, should use set_article_tags function instead
-  # defp exec_update_tags(_, _tags_ids), do: {:ok, :pass}
-
   defp update_viewed_user_list(%{meta: nil} = article, user_id) do
     new_ids = Enum.uniq([user_id] ++ @default_article_meta.viewed_user_ids)
     meta = @default_article_meta |> Map.merge(%{viewed_user_ids: new_ids})

lib/groupher_server/cms/delegates/article_tag.ex

@@ -72,12 +72,10 @@ defmodule GroupherServer.CMS.Delegate.ArticleTag do
   end
 
   # check if the tag to be set is in same community & thread
-  defp is_article_tag_in_some_thread?(article_tags, filter) do
+  defp is_article_tag_in_some_thread?(article_tag_ids, filter) do
     with {:ok, paged_article_tags} <- paged_article_tags(filter) do
       domain_tags_ids = Enum.map(paged_article_tags.entries, &to_string(&1.id))
-      cur_tags_ids = Enum.map(article_tags, &to_string(&1.id))
-
-      Enum.all?(cur_tags_ids, &Enum.member?(domain_tags_ids, &1))
+      Enum.all?(article_tag_ids, &Enum.member?(domain_tags_ids, &1))
     end
   end
 
@@ -86,15 +84,11 @@ defmodule GroupherServer.CMS.Delegate.ArticleTag do
 
   used for create article with article_tags in args
   """
-  def set_article_tags(%Community{id: cid}, thread, article, %{article_tags: article_tags}) do
+  def set_article_tags(%Community{id: cid}, thread, article, %{article_tags: article_tag_ids}) do
     check_filter = %{page: 1, size: 100, community_id: cid, thread: thread}
 
-    with true <- is_article_tag_in_some_thread?(article_tags, check_filter) do
-      Enum.each(article_tags, fn article_tag ->
-        set_article_tag(thread, article, article_tag.id)
-      end)
-
-      {:ok, :pass}
+    with true <- is_article_tag_in_some_thread?(article_tag_ids, check_filter) do
+      Enum.each(article_tag_ids, &set_article_tag(thread, article, &1)) |> done
     else
       false -> raise_error(:invalid_domain_tag, "tag not in same community & thread")
     end

lib/groupher_server_web/schema/Helper/metrics.ex

@@ -27,10 +27,6 @@ defmodule GroupherServerWeb.Schema.Helper.Metrics do
     pagination_fields()
   end
 
-  input_object :ids do
-    field(:id, :id)
-  end
-
   input_object :common_paged_filter do
     pagination_args()
     field(:sort, :inserted_sort_enum, default_value: :desc_inserted)

lib/groupher_server_web/schema/cms/mutations/blog.ex

@@ -15,7 +15,7 @@ defmodule GroupherServerWeb.Schema.CMS.Mutations.Blog do
       arg(:community_id, non_null(:id))
       arg(:link_addr, :string)
       arg(:thread, :thread, default_value: :blog)
-      arg(:article_tags, list_of(:ids))
+      arg(:article_tags, list_of(:id))
 
       middleware(M.Authorize, :login)
       middleware(M.PublishThrottle)
@@ -34,7 +34,7 @@ defmodule GroupherServerWeb.Schema.CMS.Mutations.Blog do
 
       arg(:company, :string)
       arg(:company_link, :string)
-      arg(:article_tags, list_of(:ids))
+      arg(:article_tags, list_of(:id))
 
       # ...
 

lib/groupher_server_web/schema/cms/mutations/job.ex

@@ -21,7 +21,7 @@ defmodule GroupherServerWeb.Schema.CMS.Mutations.Job do
       arg(:copy_right, :string)
 
       arg(:thread, :thread, default_value: :job)
-      arg(:article_tags, list_of(:ids))
+      arg(:article_tags, list_of(:id))
 
       middleware(M.Authorize, :login)
       middleware(M.PublishThrottle)
@@ -42,7 +42,7 @@ defmodule GroupherServerWeb.Schema.CMS.Mutations.Job do
 
       arg(:company, :string)
       arg(:company_link, :string)
-      arg(:article_tags, list_of(:ids))
+      arg(:article_tags, list_of(:id))
 
       # ...
 

lib/groupher_server_web/schema/cms/mutations/post.ex

@@ -17,7 +17,7 @@ defmodule GroupherServerWeb.Schema.CMS.Mutations.Post do
       arg(:copy_right, :string)
       arg(:community_id, non_null(:id))
       arg(:thread, :thread, default_value: :post)
-      arg(:article_tags, list_of(:ids))
+      arg(:article_tags, list_of(:id))
 
       middleware(M.Authorize, :login)
       # middleware(M.PublishThrottle)
@@ -34,7 +34,7 @@ defmodule GroupherServerWeb.Schema.CMS.Mutations.Post do
       arg(:digest, :string)
       arg(:copy_right, :string)
       arg(:link_addr, :string)
-      arg(:article_tags, list_of(:ids))
+      arg(:article_tags, list_of(:id))
 
       middleware(M.Authorize, :login)
       middleware(M.PassportLoader, source: :post)

lib/groupher_server_web/schema/cms/mutations/repo.ex

@@ -31,7 +31,7 @@ defmodule GroupherServerWeb.Schema.CMS.Mutations.Repo do
 
       arg(:community_id, non_null(:id))
       arg(:thread, :thread, default_value: :repo)
-      arg(:article_tags, list_of(:ids))
+      arg(:article_tags, list_of(:id))
 
       middleware(M.Authorize, :login)
       middleware(M.PublishThrottle)

test/groupher_server_web/mutation/accounts/mailbox_test.exs

@@ -1,4 +1,4 @@
-defmodule GroupherServer.Test.Query.Accounts.Mailbox do
+defmodule GroupherServer.Test.Mutaion.Accounts.Mailbox do
   use GroupherServer.TestTools
 
   alias GroupherServer.Delivery
@@ -30,7 +30,7 @@ defmodule GroupherServer.Test.Query.Accounts.Mailbox do
       mention = mentions.entries |> List.first()
 
       variables = %{ids: [mention.id], type: "MENTION"}
-      result = user_conn |> mutation_result(@query, variables, "markRead")
+      user_conn |> mutation_result(@query, variables, "markRead")
 
       {:ok, mentions} = Delivery.fetch(:mention, user, %{page: 1, size: 10, read: true})
       mention = mentions.entries |> List.first()
@@ -44,7 +44,7 @@ defmodule GroupherServer.Test.Query.Accounts.Mailbox do
       notify = notifications.entries |> List.first()
 
       variables = %{ids: [notify.id], type: "NOTIFICATION"}
-      result = user_conn |> mutation_result(@query, variables, "markRead")
+      user_conn |> mutation_result(@query, variables, "markRead")
 
       {:ok, notifications} = Delivery.fetch(:notification, user, %{page: 1, size: 10, read: true})
       notify = notifications.entries |> List.first()
@@ -68,7 +68,7 @@ defmodule GroupherServer.Test.Query.Accounts.Mailbox do
       assert mentions.total_count == 2
 
       variables = %{type: "MENTION"}
-      result = user_conn |> mutation_result(@query, variables, "markReadAll")
+      user_conn |> mutation_result(@query, variables, "markReadAll")
 
       {:ok, mentions} = Delivery.fetch(:mention, user, %{page: 1, size: 10, read: true})
       assert mentions.total_count == 2
@@ -81,7 +81,7 @@ defmodule GroupherServer.Test.Query.Accounts.Mailbox do
       assert notifications.total_count == 1
 
       variables = %{type: "NOTIFICATION"}
-      result = user_conn |> mutation_result(@query, variables, "markReadAll")
+      user_conn |> mutation_result(@query, variables, "markReadAll")
 
       {:ok, notifications} = Delivery.fetch(:notification, user, %{page: 1, size: 10, read: true})
       assert notifications.total_count == 1

test/groupher_server_web/mutation/cms/articles/blog_test.exs

@@ -9,12 +9,13 @@ defmodule GroupherServer.Test.Mutation.Articles.Blog do
   setup do
     {:ok, blog} = db_insert(:blog)
     {:ok, user} = db_insert(:user)
+    {:ok, community} = db_insert(:community)
 
     guest_conn = simu_conn(:guest)
     user_conn = simu_conn(:user)
     owner_conn = simu_conn(:owner, blog)
 
-    {:ok, ~m(user_conn guest_conn owner_conn user blog)a}
+    {:ok, ~m(user_conn guest_conn owner_conn community user blog)a}
   end
 
   describe "[mutation blog curd]" do
@@ -25,7 +26,7 @@ defmodule GroupherServer.Test.Mutation.Articles.Blog do
       $digest: String!,
       $length: Int,
       $communityId: ID!,
-      $articleTags: [Ids]
+      $articleTags: [Id]
      ) {
       createBlog(
         title: $title,
@@ -67,6 +68,21 @@ defmodule GroupherServer.Test.Mutation.Articles.Blog do
       assert created["id"] == to_string(found.id)
     end
 
+    test "create blog with valid tags id list", ~m(user_conn user community)a do
+      article_tag_attrs = mock_attrs(:article_tag)
+      {:ok, article_tag} = CMS.create_article_tag(community, :blog, article_tag_attrs, user)
+
+      blog_attr = mock_attrs(:blog)
+
+      variables =
+        blog_attr |> Map.merge(%{communityId: community.id, articleTags: [article_tag.id]})
+
+      created = user_conn |> mutation_result(@create_blog_query, variables, "createBlog")
+      {:ok, blog} = ORM.find(Blog, created["id"], preload: :article_tags)
+
+      assert exist_in?(%{id: article_tag.id}, blog.article_tags)
+    end
+
     test "create blog should excape xss attracts" do
       {:ok, user} = db_insert(:user)
       user_conn = simu_conn(:user, user)

test/groupher_server_web/mutation/cms/articles/job_test.exs

@@ -9,12 +9,13 @@ defmodule GroupherServer.Test.Mutation.Articles.Job do
   setup do
     {:ok, job} = db_insert(:job)
     {:ok, user} = db_insert(:user)
+    {:ok, community} = db_insert(:community)
 
     guest_conn = simu_conn(:guest)
     user_conn = simu_conn(:user)
     owner_conn = simu_conn(:owner, job)
 
-    {:ok, ~m(user_conn guest_conn owner_conn user job)a}
+    {:ok, ~m(user_conn guest_conn owner_conn user community job)a}
   end
 
   describe "[mutation job curd]" do
@@ -26,7 +27,7 @@ defmodule GroupherServer.Test.Mutation.Articles.Job do
       $length: Int!,
       $communityId: ID!,
       $company: String!,
-      $articleTags: [Ids]
+      $articleTags: [Id]
      ) {
       createJob(
         title: $title,
@@ -50,7 +51,6 @@ defmodule GroupherServer.Test.Mutation.Articles.Job do
       }
     }
     """
-
     test "create job with valid attrs and make sure author exsit" do
       {:ok, user} = db_insert(:user)
       user_conn = simu_conn(:user, user)
@@ -70,6 +70,21 @@ defmodule GroupherServer.Test.Mutation.Articles.Job do
       assert created["id"] == to_string(found.id)
     end
 
+    test "create job with valid tags id list.", ~m(user_conn user community)a do
+      article_tag_attrs = mock_attrs(:article_tag)
+      {:ok, article_tag} = CMS.create_article_tag(community, :job, article_tag_attrs, user)
+
+      job_attr = mock_attrs(:job)
+
+      variables =
+        job_attr |> Map.merge(%{communityId: community.id, articleTags: [article_tag.id]})
+
+      created = user_conn |> mutation_result(@create_job_query, variables, "createJob")
+      {:ok, job} = ORM.find(Job, created["id"], preload: :article_tags)
+
+      assert exist_in?(%{id: article_tag.id}, job.article_tags)
+    end
+
     test "create job should excape xss attracts" do
       {:ok, user} = db_insert(:user)
       user_conn = simu_conn(:user, user)