refactor(mention): clean up

Author: mydearxym

test/groupher_server/cms/hooks/cite_blog_test.exs

@@ -31,9 +31,9 @@ defmodule GroupherServer.Test.CMS.Hooks.CiteBlog do
     test "cited multi blog should work", ~m(user community blog2 blog3 blog4 blog5 blog_attrs)a do
       body =
         mock_rich_text(
-          ~s(the <a href=#{@site_host}/blog/#{blog2.id} /> and <a href=#{@site_host}/blog/#{blog2.id}>same la</a> is awesome, the <a href=#{
-            @site_host
-          }/blog/#{blog3.id}></a> is awesome too.),
+          ~s(the <a href=#{@site_host}/blog/#{blog2.id} /> and <a href=#{@site_host}/blog/#{
+            blog2.id
+          }>same la</a> is awesome, the <a href=#{@site_host}/blog/#{blog3.id}></a> is awesome too.),
           # second paragraph
           ~s(the paragraph 2 <a href=#{@site_host}/blog/#{blog2.id} class=#{blog2.title}> again</a>, the paragraph 2 <a href=#{
             @site_host
@@ -90,7 +90,7 @@ defmodule GroupherServer.Test.CMS.Hooks.CiteBlog do
       assert cited_comment.meta.citing_count == 0
     end
 
-    @tag :wip
+    @tag :wip2
     test "can cite blog's comment in blog", ~m(community user blog blog2 blog_attrs)a do
       {:ok, comment} = CMS.create_comment(:blog, blog.id, mock_rich_text("hello"), user)
 
@@ -136,9 +136,9 @@ defmodule GroupherServer.Test.CMS.Hooks.CiteBlog do
     test "can cited blog inside a comment", ~m(user blog blog2 blog3 blog4 blog5)a do
       comment_body =
         mock_rich_text(
-          ~s(the <a href=#{@site_host}/blog/#{blog2.id} /> and <a href=#{@site_host}/blog/#{blog2.id}>same la</a> is awesome, the <a href=#{
-            @site_host
-          }/blog/#{blog3.id}></a> is awesome too.),
+          ~s(the <a href=#{@site_host}/blog/#{blog2.id} /> and <a href=#{@site_host}/blog/#{
+            blog2.id
+          }>same la</a> is awesome, the <a href=#{@site_host}/blog/#{blog3.id}></a> is awesome too.),
           # second paragraph
           ~s(the paragraph 2 <a href=#{@site_host}/blog/#{blog2.id} class=#{blog2.title}> again</a>, the paragraph 2 <a href=#{
             @site_host
@@ -221,60 +221,4 @@ defmodule GroupherServer.Test.CMS.Hooks.CiteBlog do
       assert result.total_count == 3
     end
   end
-
-  describe "[cross cite]" do
-    test "can citing multi type thread and comment in one time", ~m(user community blog2)a do
-      blog_attrs = mock_attrs(:blog, %{community_id: community.id})
-      blog_attrs = mock_attrs(:blog, %{community_id: community.id})
-      blog_attrs = mock_attrs(:blog, %{community_id: community.id})
-
-      body = mock_rich_text(~s(the <a href=#{@site_host}/blog/#{blog2.id} />))
-
-      {:ok, blog} = CMS.create_article(community, :blog, Map.merge(blog_attrs, %{body: body}), user)
-
-      Hooks.Cite.handle(blog)
-
-      Process.sleep(1000)
-
-      {:ok, blog} = CMS.create_article(community, :blog, Map.merge(blog_attrs, %{body: body}), user)
-      Hooks.Cite.handle(blog)
-
-      Process.sleep(1000)
-
-      comment_body = mock_comment(~s(the <a href=#{@site_host}/blog/#{blog2.id} />))
-      {:ok, comment} = CMS.create_comment(:blog, blog.id, comment_body, user)
-
-      Hooks.Cite.handle(comment)
-
-      Process.sleep(1000)
-
-      {:ok, blog} =
-        CMS.create_article(community, :blog, Map.merge(blog_attrs, %{body: body}), user)
-
-      Hooks.Cite.handle(blog)
-
-      {:ok, result} = CMS.paged_citing_contents("BLOG", blog2.id, %{page: 1, size: 10})
-      # IO.inspect(result, label: "the result")
-
-      assert result.total_count == 4
-
-      result_blog = result.entries |> List.first()
-      result_blog = result.entries |> Enum.at(1)
-      result_comment = result.entries |> Enum.at(2)
-      result_blog = result.entries |> List.last()
-
-      assert result_blog.id == blog.id
-      assert result_blog.thread == :blog
-
-      assert result_blog.id == blog.id
-      assert result_blog.thread == :blog
-
-      assert result_comment.id == blog.id
-      assert result_comment.thread == :blog
-      assert result_comment.comment_id == comment.id
-
-      assert result_blog.id == blog.id
-      assert result_blog.thread == :blog
-    end
-  end
 end

test/groupher_server/cms/hooks/cite_job_test.exs

@@ -90,7 +90,7 @@ defmodule GroupherServer.Test.CMS.Hooks.CiteJob do
       assert cited_comment.meta.citing_count == 0
     end
 
-    @tag :wip
+    @tag :wip2
     test "can cite job's comment in job", ~m(community user job job2 job_attrs)a do
       {:ok, comment} = CMS.create_comment(:job, job.id, mock_rich_text("hello"), user)
 
@@ -221,60 +221,4 @@ defmodule GroupherServer.Test.CMS.Hooks.CiteJob do
       assert result.total_count == 3
     end
   end
-
-  describe "[cross cite]" do
-    test "can citing multi type thread and comment in one time", ~m(user community job2)a do
-      job_attrs = mock_attrs(:job, %{community_id: community.id})
-      job_attrs = mock_attrs(:job, %{community_id: community.id})
-      blog_attrs = mock_attrs(:blog, %{community_id: community.id})
-
-      body = mock_rich_text(~s(the <a href=#{@site_host}/job/#{job2.id} />))
-
-      {:ok, job} = CMS.create_article(community, :job, Map.merge(job_attrs, %{body: body}), user)
-
-      Hooks.Cite.handle(job)
-
-      Process.sleep(1000)
-
-      {:ok, job} = CMS.create_article(community, :job, Map.merge(job_attrs, %{body: body}), user)
-      Hooks.Cite.handle(job)
-
-      Process.sleep(1000)
-
-      comment_body = mock_comment(~s(the <a href=#{@site_host}/job/#{job2.id} />))
-      {:ok, comment} = CMS.create_comment(:job, job.id, comment_body, user)
-
-      Hooks.Cite.handle(comment)
-
-      Process.sleep(1000)
-
-      {:ok, blog} =
-        CMS.create_article(community, :blog, Map.merge(blog_attrs, %{body: body}), user)
-
-      Hooks.Cite.handle(blog)
-
-      {:ok, result} = CMS.paged_citing_contents("JOB", job2.id, %{page: 1, size: 10})
-      # IO.inspect(result, label: "the result")
-
-      assert result.total_count == 4
-
-      result_job = result.entries |> List.first()
-      result_job = result.entries |> Enum.at(1)
-      result_comment = result.entries |> Enum.at(2)
-      result_blog = result.entries |> List.last()
-
-      assert result_job.id == job.id
-      assert result_job.thread == :job
-
-      assert result_job.id == job.id
-      assert result_job.thread == :job
-
-      assert result_comment.id == job.id
-      assert result_comment.thread == :job
-      assert result_comment.comment_id == comment.id
-
-      assert result_blog.id == blog.id
-      assert result_blog.thread == :blog
-    end
-  end
 end

test/groupher_server/cms/hooks/cite_post_test.exs

@@ -90,7 +90,7 @@ defmodule GroupherServer.Test.CMS.Hooks.CitePost do
       assert cited_comment.meta.citing_count == 0
     end
 
-    @tag :wip
+    @tag :wip2
     test "can cite post's comment in post", ~m(community user post post2 post_attrs)a do
       {:ok, comment} = CMS.create_comment(:post, post.id, mock_rich_text("hello"), user)
 

test/groupher_server/cms/hooks/notify_blog_test.exs

@@ -21,7 +21,7 @@ defmodule GroupherServer.Test.CMS.Hooks.NotifyBlog do
   end
 
   describe "[upvote notify]" do
-    @tag :wip2
+    @tag :wip22
     test "upvote hook should work on blog", ~m(user2 blog)a do
       {:ok, blog} = preload_author(blog)
 
@@ -41,7 +41,7 @@ defmodule GroupherServer.Test.CMS.Hooks.NotifyBlog do
       assert user_exist_in?(user2, notify.from_users)
     end
 
-    @tag :wip2
+    @tag :wip22
     test "upvote hook should work on blog comment", ~m(user2 blog comment)a do
       {:ok, comment} = CMS.upvote_comment(comment.id, user2)
       {:ok, comment} = preload_author(comment)
@@ -62,7 +62,7 @@ defmodule GroupherServer.Test.CMS.Hooks.NotifyBlog do
       assert user_exist_in?(user2, notify.from_users)
     end
 
-    @tag :wip2
+    @tag :wip22
     test "undo upvote hook should work on blog", ~m(user2 blog)a do
       {:ok, blog} = preload_author(blog)
 
@@ -78,7 +78,7 @@ defmodule GroupherServer.Test.CMS.Hooks.NotifyBlog do
       assert notifications.total_count == 0
     end
 
-    @tag :wip2
+    @tag :wip22
     test "undo upvote hook should work on blog comment", ~m(user2 comment)a do
       {:ok, comment} = CMS.upvote_comment(comment.id, user2)
 
@@ -97,7 +97,7 @@ defmodule GroupherServer.Test.CMS.Hooks.NotifyBlog do
   end
 
   describe "[collect notify]" do
-    @tag :wip2
+    @tag :wip22
     test "collect hook should work on blog", ~m(user2 blog)a do
       {:ok, blog} = preload_author(blog)
 
@@ -117,7 +117,7 @@ defmodule GroupherServer.Test.CMS.Hooks.NotifyBlog do
       assert user_exist_in?(user2, notify.from_users)
     end
 
-    @tag :wip2
+    @tag :wip22
     test "undo collect hook should work on blog", ~m(user2 blog)a do
       {:ok, blog} = preload_author(blog)
 
@@ -135,7 +135,7 @@ defmodule GroupherServer.Test.CMS.Hooks.NotifyBlog do
   end
 
   describe "[comment notify]" do
-    @tag :wip2
+    @tag :wip22
     test "blog author should get notify after some one comment on it", ~m(user2 blog)a do
       {:ok, blog} = preload_author(blog)
 
@@ -155,7 +155,7 @@ defmodule GroupherServer.Test.CMS.Hooks.NotifyBlog do
       assert user_exist_in?(user2, notify.from_users)
     end
 
-    @tag :wip2
+    @tag :wip22
     test "blog comment author should get notify after some one reply it", ~m(user2 user3 blog)a do
       {:ok, blog} = preload_author(blog)
 

test/groupher_server/cms/hooks/notify_job_test.exs

@@ -21,7 +21,7 @@ defmodule GroupherServer.Test.CMS.Hooks.NotifyJob do
   end
 
   describe "[upvote notify]" do
-    @tag :wip2
+    @tag :wip22
     test "upvote hook should work on job", ~m(user2 job)a do
       {:ok, job} = preload_author(job)
 
@@ -41,7 +41,7 @@ defmodule GroupherServer.Test.CMS.Hooks.NotifyJob do
       assert user_exist_in?(user2, notify.from_users)
     end
 
-    @tag :wip2
+    @tag :wip22
     test "upvote hook should work on job comment", ~m(user2 job comment)a do
       {:ok, comment} = CMS.upvote_comment(comment.id, user2)
       {:ok, comment} = preload_author(comment)
@@ -62,7 +62,7 @@ defmodule GroupherServer.Test.CMS.Hooks.NotifyJob do
       assert user_exist_in?(user2, notify.from_users)
     end
 
-    @tag :wip2
+    @tag :wip22
     test "undo upvote hook should work on job", ~m(user2 job)a do
       {:ok, job} = preload_author(job)
 
@@ -78,7 +78,7 @@ defmodule GroupherServer.Test.CMS.Hooks.NotifyJob do
       assert notifications.total_count == 0
     end
 
-    @tag :wip2
+    @tag :wip22
     test "undo upvote hook should work on job comment", ~m(user2 comment)a do
       {:ok, comment} = CMS.upvote_comment(comment.id, user2)
 
@@ -97,7 +97,7 @@ defmodule GroupherServer.Test.CMS.Hooks.NotifyJob do
   end
 
   describe "[collect notify]" do
-    @tag :wip2
+    @tag :wip22
     test "collect hook should work on job", ~m(user2 job)a do
       {:ok, job} = preload_author(job)
 
@@ -117,7 +117,7 @@ defmodule GroupherServer.Test.CMS.Hooks.NotifyJob do
       assert user_exist_in?(user2, notify.from_users)
     end
 
-    @tag :wip2
+    @tag :wip22
     test "undo collect hook should work on job", ~m(user2 job)a do
       {:ok, job} = preload_author(job)
 
@@ -135,7 +135,7 @@ defmodule GroupherServer.Test.CMS.Hooks.NotifyJob do
   end
 
   describe "[comment notify]" do
-    @tag :wip2
+    @tag :wip22
     test "job author should get notify after some one comment on it", ~m(user2 job)a do
       {:ok, job} = preload_author(job)
 
@@ -155,7 +155,7 @@ defmodule GroupherServer.Test.CMS.Hooks.NotifyJob do
       assert user_exist_in?(user2, notify.from_users)
     end
 
-    @tag :wip2
+    @tag :wip22
     test "job comment author should get notify after some one reply it", ~m(user2 user3 job)a do
       {:ok, job} = preload_author(job)
 

test/groupher_server/cms/hooks/notify_post_test.exs

@@ -21,7 +21,7 @@ defmodule GroupherServer.Test.CMS.Hooks.NotifyPost do
   end
 
   describe "[upvote notify]" do
-    @tag :wip2
+    @tag :wip22
     test "upvote hook should work on post", ~m(user2 post)a do
       {:ok, post} = preload_author(post)
 
@@ -41,7 +41,7 @@ defmodule GroupherServer.Test.CMS.Hooks.NotifyPost do
       assert user_exist_in?(user2, notify.from_users)
     end
 
-    @tag :wip2
+    @tag :wip22
     test "upvote hook should work on post comment", ~m(user2 post comment)a do
       {:ok, comment} = CMS.upvote_comment(comment.id, user2)
       {:ok, comment} = preload_author(comment)
@@ -62,7 +62,7 @@ defmodule GroupherServer.Test.CMS.Hooks.NotifyPost do
       assert user_exist_in?(user2, notify.from_users)
     end
 
-    @tag :wip2
+    @tag :wip22
     test "undo upvote hook should work on post", ~m(user2 post)a do
       {:ok, post} = preload_author(post)
 
@@ -78,7 +78,7 @@ defmodule GroupherServer.Test.CMS.Hooks.NotifyPost do
       assert notifications.total_count == 0
     end
 
-    @tag :wip2
+    @tag :wip22
     test "undo upvote hook should work on post comment", ~m(user2 comment)a do
       {:ok, comment} = CMS.upvote_comment(comment.id, user2)
 
@@ -97,7 +97,7 @@ defmodule GroupherServer.Test.CMS.Hooks.NotifyPost do
   end
 
   describe "[collect notify]" do
-    @tag :wip2
+    @tag :wip22
     test "collect hook should work on post", ~m(user2 post)a do
       {:ok, post} = preload_author(post)
 
@@ -117,7 +117,7 @@ defmodule GroupherServer.Test.CMS.Hooks.NotifyPost do
       assert user_exist_in?(user2, notify.from_users)
     end
 
-    @tag :wip2
+    @tag :wip22
     test "undo collect hook should work on post", ~m(user2 post)a do
       {:ok, post} = preload_author(post)
 
@@ -135,7 +135,7 @@ defmodule GroupherServer.Test.CMS.Hooks.NotifyPost do
   end
 
   describe "[comment notify]" do
-    @tag :wip2
+    @tag :wip22
     test "post author should get notify after some one comment on it", ~m(user2 post)a do
       {:ok, post} = preload_author(post)
 
@@ -155,7 +155,7 @@ defmodule GroupherServer.Test.CMS.Hooks.NotifyPost do
       assert user_exist_in?(user2, notify.from_users)
     end
 
-    @tag :wip2
+    @tag :wip22
     test "post comment author should get notify after some one reply it", ~m(user2 user3 post)a do
       {:ok, post} = preload_author(post)