From: Maksim Pechnikov <parallel588@gmail.com>
Date: Tue, 14 Jan 2020 13:26:56 +0000 (+0300)
Subject: Merge branch 'develop' into feature/tag_feed
X-Git-Url: https://git.squeep.com/?a=commitdiff_plain;h=c9f45edeac2cc3ed262bb4b30dc7e1120c6fa047;p=akkoma

Merge branch 'develop' into feature/tag_feed
---

c9f45edeac2cc3ed262bb4b30dc7e1120c6fa047
diff --cc lib/pleroma/web/metadata/utils.ex
index c0dae1b7e,589d11901..000bd9f66
--- a/lib/pleroma/web/metadata/utils.ex
+++ b/lib/pleroma/web/metadata/utils.ex
@@@ -19,13 -20,6 +20,14 @@@ defmodule Pleroma.Web.Metadata.Utils d
    end
  
    def scrub_html_and_truncate(content, max_length \\ 200) when is_binary(content) do
 +    content
 +    |> scrub_html
 +    |> Emoji.Formatter.demojify()
++    |> HtmlEntities.decode()
 +    |> Formatter.truncate(max_length)
 +  end
 +
 +  def scrub_html(content) when is_binary(content) do
      content
      # html content comes from DB already encoded, decode first and scrub after
      |> HtmlEntities.decode()
diff --cc test/web/feed/tag_controller_test.exs
index efc588070,000000000..a56a18738
mode 100644,000000..100644
--- a/test/web/feed/tag_controller_test.exs
+++ b/test/web/feed/tag_controller_test.exs
@@@ -1,89 -1,0 +1,101 @@@
 +# Pleroma: A lightweight social networking server
 +# Copyright © 2017-2019 Pleroma Authors <https://pleroma.social/>
 +# SPDX-License-Identifier: AGPL-3.0-only
 +
 +defmodule Pleroma.Web.Feed.TagControllerTest do
 +  use Pleroma.Web.ConnCase
 +
 +  import Pleroma.Factory
 +  import SweetXml
 +
 +  alias Pleroma.Web.Feed.FeedView
 +
 +  clear_config([:feed])
 +
 +  test "gets a feed", %{conn: conn} do
 +    Pleroma.Config.put(
 +      [:feed, :post_title],
 +      %{max_length: 25, omission: "..."}
 +    )
 +
 +    user = insert(:user)
 +    {:ok, activity1} = Pleroma.Web.CommonAPI.post(user, %{"status" => "yeah #PleromaArt"})
 +
 +    object = Pleroma.Object.normalize(activity1)
 +
 +    object_data =
 +      Map.put(object.data, "attachment", [
 +        %{
 +          "url" => [
 +            %{
 +              "href" =>
 +                "https://peertube.moe/static/webseed/df5f464b-be8d-46fb-ad81-2d4c2d1630e3-480.mp4",
 +              "mediaType" => "video/mp4",
 +              "type" => "Link"
 +            }
 +          ]
 +        }
 +      ])
 +
 +    object
 +    |> Ecto.Changeset.change(data: object_data)
 +    |> Pleroma.Repo.update()
 +
 +    {:ok, activity2} =
 +      Pleroma.Web.CommonAPI.post(user, %{"status" => "42 This is :moominmamma #PleromaArt"})
 +
 +    {:ok, _activity3} = Pleroma.Web.CommonAPI.post(user, %{"status" => "This is :moominmamma"})
 +
 +    response =
 +      conn
 +      |> put_req_header("content-type", "application/atom+xml")
 +      |> get(tag_feed_path(conn, :feed, "pleromaart.rss"))
 +      |> response(200)
 +
 +    xml = parse(response)
 +    assert xpath(xml, ~x"//channel/title/text()") == '#pleromaart'
 +
 +    assert xpath(xml, ~x"//channel/description/text()"s) ==
 +             "These are public toots tagged with #pleromaart. You can interact with them if you have an account anywhere in the fediverse."
 +
 +    assert xpath(xml, ~x"//channel/link/text()") ==
 +             '#{Pleroma.Web.base_url()}/tags/pleromaart.rss'
 +
 +    assert xpath(xml, ~x"//channel/webfeeds:logo/text()") ==
 +             '#{Pleroma.Web.base_url()}/static/logo.png'
 +
 +    assert xpath(xml, ~x"//channel/item/title/text()"l) == [
 +             '42 This is :moominmamm...',
 +             'yeah #PleromaArt'
 +           ]
 +
 +    assert xpath(xml, ~x"//channel/item/pubDate/text()"sl) == [
 +             FeedView.pub_date(activity1.data["published"]),
 +             FeedView.pub_date(activity2.data["published"])
 +           ]
 +
 +    assert xpath(xml, ~x"//channel/item/enclosure/@url"sl) == [
 +             "https://peertube.moe/static/webseed/df5f464b-be8d-46fb-ad81-2d4c2d1630e3-480.mp4"
 +           ]
 +
 +    obj1 = Pleroma.Object.normalize(activity1)
 +    obj2 = Pleroma.Object.normalize(activity2)
 +
 +    assert xpath(xml, ~x"//channel/item/description/text()"sl) == [
 +             HtmlEntities.decode(FeedView.activity_content(obj2)),
 +             HtmlEntities.decode(FeedView.activity_content(obj1))
 +           ]
++
++    response =
++      conn
++      |> put_req_header("content-type", "application/atom+xml")
++      |> get(tag_feed_path(conn, :feed, "pleromaart"))
++      |> response(200)
++
++    xml = parse(response)
++    assert xpath(xml, ~x"//channel/title/text()") == '#pleromaart'
++
++    assert xpath(xml, ~x"//channel/description/text()"s) ==
++             "These are public toots tagged with #pleromaart. You can interact with them if you have an account anywhere in the fediverse."
 +  end
 +end
diff --cc test/web/twitter_api/util_controller_test.exs
index 43299e147,8418fd071..5d60c0d51
--- a/test/web/twitter_api/util_controller_test.exs
+++ b/test/web/twitter_api/util_controller_test.exs
@@@ -6,11 -6,11 +6,9 @@@ defmodule Pleroma.Web.TwitterAPI.UtilCo
    use Pleroma.Web.ConnCase
    use Oban.Testing, repo: Pleroma.Repo
  
-   alias Pleroma.Repo
    alias Pleroma.Tests.ObanHelpers
    alias Pleroma.User
--  alias Pleroma.Web.CommonAPI
-   import ExUnit.CaptureLog
+ 
 -  import ExUnit.CaptureLog
    import Pleroma.Factory
    import Mock