Merge branch 'radar-source' into dev

Author: mydearxym

lib/mastani_server/cms/post.ex

@@ -19,14 +19,15 @@ defmodule MastaniServer.CMS.Post do
 
   @timestamps_opts [type: :utc_datetime_usec]
   @required_fields ~w(title body digest length)a
-  @optional_fields ~w(link_addr copy_right link_addr)a
+  @optional_fields ~w(link_addr copy_right link_addr link_icon)a
 
   @type t :: %Post{}
   schema "cms_posts" do
     field(:body, :string)
     field(:title, :string)
     field(:digest, :string)
     field(:link_addr, :string)
+    field(:link_icon, :string)
     field(:copy_right, :string)
     field(:length, :integer)
     field(:views, :integer, default: 0)

lib/mastani_server_web/middleware/add_source_icon.ex

@@ -0,0 +1,41 @@
+# ---
+# parse and add source icon to radar posts
+# ---
+defmodule MastaniServerWeb.Middleware.AddSourceIcon do
+  @moduledoc """
+  parse and add source icon to radar posts
+  """
+  @behaviour Absinthe.Middleware
+
+  @parse_error "https://cps-oss.oss-cn-shanghai.aliyuncs.com/icons/radar_source/url_parse_waning.png"
+  @wanqu "https://cps-oss.oss-cn-shanghai.aliyuncs.com/icons/radar_source/wanqu.png"
+  @solidot "https://cps-oss.oss-cn-shanghai.aliyuncs.com/icons/radar_source/solidot.png"
+  @techcrunch "https://cps-oss.oss-cn-shanghai.aliyuncs.com/icons/radar_source/techcrunch.png"
+  @default_radar "https://cps-oss.oss-cn-shanghai.aliyuncs.com/icons/radar_source/default_radar.png"
+
+  def call(%{arguments: %{topic: "radar"} = arguments} = resolution, _) do
+    link_icon = parse_source_addr(arguments)
+    %{resolution | arguments: Map.merge(arguments, %{link_icon: link_icon})}
+  end
+
+  def call(resolution, _), do: resolution
+
+  defp parse_source_addr(%{link_addr: link_addr}) do
+    result = URI.parse(link_addr)
+
+    case Map.get(result, :host) do
+      nil -> source_addr(:error)
+      host -> source_addr(host)
+    end
+  end
+
+  defp parse_source_addr(_), do: @default_radar
+
+  defp source_addr("wanqu.co"), do: @wanqu
+  defp source_addr("www.solidot.org"), do: @solidot
+  defp source_addr("techcrunch.cn"), do: @techcrunch
+  defp source_addr("techcrunch.com"), do: @techcrunch
+
+  defp source_addr(:error), do: @parse_error
+  defp source_addr(_), do: @default_radar
+end

lib/mastani_server_web/schema/cms/cms_types.ex

@@ -25,6 +25,7 @@ defmodule MastaniServerWeb.Schema.CMS.Types do
     field(:digest, :string)
     field(:length, :integer)
     field(:link_addr, :string)
+    field(:link_icon, :string)
     field(:copy_right, :string)
     field(:body, :string)
     field(:views, :integer)

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

@@ -21,6 +21,7 @@ defmodule MastaniServerWeb.Schema.CMS.Mutations.Post do
 
       middleware(M.Authorize, :login)
       middleware(M.PublishThrottle)
+      middleware(M.AddSourceIcon)
       # middleware(M.PublishThrottle, interval: 3, hour_limit: 15, day_limit: 30)
       resolve(&R.CMS.create_content/3)
       middleware(M.Statistics.MakeContribute, for: :user)

priv/repo/migrations/20190118035531_add_link_icon_to_posts.exs

@@ -0,0 +1,9 @@
+defmodule MastaniServer.Repo.Migrations.AddLinkIconToPosts do
+  use Ecto.Migration
+
+  def change do
+    alter table(:cms_posts) do
+      add(:link_icon, :string)
+    end
+  end
+end

test/mastani_server_web/mutation/cms/radar_test.exs

@@ -0,0 +1,111 @@
+defmodule MastaniServer.Test.Mutation.Radar do
+  use MastaniServer.TestTools
+
+  # alias Helper.ORM
+
+  setup do
+    {:ok, post} = db_insert(:post)
+
+    guest_conn = simu_conn(:guest)
+    user_conn = simu_conn(:user)
+    owner_conn = simu_conn(:owner, post)
+
+    {:ok, ~m(user_conn guest_conn owner_conn post)a}
+  end
+
+  describe "[mutation radar posts]" do
+    @create_post_query """
+    mutation(
+      $title: String!
+      $body: String!
+      $digest: String!
+      $length: Int!
+      $communityId: ID!
+      $tags: [Ids]
+      $mentionUsers: [Ids]
+      $linkAddr: String
+      $topic: String
+    ) {
+      createPost(
+        title: $title
+        body: $body
+        digest: $digest
+        length: $length
+        communityId: $communityId
+        tags: $tags
+        mentionUsers: $mentionUsers
+        linkAddr: $linkAddr
+        topic: $topic
+      ) {
+        title
+        id
+        linkAddr
+        linkIcon
+      }
+    }
+    """
+    @wanqu "https://cps-oss.oss-cn-shanghai.aliyuncs.com/icons/radar_source/wanqu.png"
+    @default_radar "https://cps-oss.oss-cn-shanghai.aliyuncs.com/icons/radar_source/default_radar.png"
+    @parse_error "https://cps-oss.oss-cn-shanghai.aliyuncs.com/icons/radar_source/url_parse_waning.png"
+
+    test "create radar with known source will auto add link icon addr" do
+      {:ok, user} = db_insert(:user)
+      user_conn = simu_conn(:user, user)
+
+      {:ok, community} = db_insert(:community)
+      post_attr = mock_attrs(:post)
+
+      variables =
+        post_attr
+        |> Map.merge(%{
+          communityId: community.id,
+          linkAddr: "https://wanqu.co/a/7237/",
+          topic: "radar"
+        })
+
+      created = user_conn |> mutation_result(@create_post_query, variables, "createPost")
+      assert created["linkIcon"] == @wanqu
+      assert created["linkAddr"] == "https://wanqu.co/a/7237/"
+    end
+
+    test "create radar with unknown source will add default link icon addr" do
+      {:ok, user} = db_insert(:user)
+      user_conn = simu_conn(:user, user)
+
+      {:ok, community} = db_insert(:community)
+      post_attr = mock_attrs(:post)
+
+      variables =
+        post_attr
+        |> Map.merge(%{
+          communityId: community.id,
+          linkAddr: "https://unknown.com",
+          topic: "radar"
+        })
+
+      created = user_conn |> mutation_result(@create_post_query, variables, "createPost")
+      assert created["linkIcon"] == @default_radar
+      assert created["linkAddr"] == "https://unknown.com"
+    end
+
+    test "create radar with invalid link addr will add error link icon addr" do
+      {:ok, user} = db_insert(:user)
+      user_conn = simu_conn(:user, user)
+
+      {:ok, community} = db_insert(:community)
+      post_attr = mock_attrs(:post)
+
+      variables =
+        post_attr
+        |> Map.merge(%{
+          communityId: community.id,
+          linkAddr: "watdjfiefejife",
+          topic: "radar"
+        })
+
+      created = user_conn |> mutation_result(@create_post_query, variables, "createPost")
+      assert created["linkIcon"] == @parse_error
+      assert created["linkAddr"] == "watdjfiefejife"
+    end
+  end
+end