diff options
Diffstat (limited to 'test')
-rw-r--r-- | test/fixtures/lambadalambda.atom | 2 | ||||
-rw-r--r-- | test/support/factory.ex | 4 | ||||
-rw-r--r-- | test/web/ostatus/activity_representer_test.exs | 57 | ||||
-rw-r--r-- | test/web/ostatus/ostatus_test.exs | 9 | ||||
-rw-r--r-- | test/web/twitter_api/twitter_api_test.exs | 2 | ||||
-rw-r--r-- | test/web/websub/websub_test.exs | 1 |
6 files changed, 25 insertions, 50 deletions
diff --git a/test/fixtures/lambadalambda.atom b/test/fixtures/lambadalambda.atom index 35e506420..964a416f7 100644 --- a/test/fixtures/lambadalambda.atom +++ b/test/fixtures/lambadalambda.atom @@ -11,7 +11,7 @@ <uri>https://mastodon.social/users/lambadalambda</uri> <name>lambadalambda</name> <email>lambadalambda@mastodon.social</email> - <summary></summary> + <summary>a cool dude.</summary> <link rel="alternate" type="text/html" href="https://mastodon.social/@lambadalambda"/> <link rel="avatar" type="image/gif" media:width="120" media:height="120" href="https://files.mastodon.social/accounts/avatars/000/000/264/original/1429214160519.gif?1492379244"/> <link rel="header" type="" media:width="700" media:height="335" href="/headers/original/missing.png"/> diff --git a/test/support/factory.ex b/test/support/factory.ex index 3cf35e3bc..bb3bdb02c 100644 --- a/test/support/factory.ex +++ b/test/support/factory.ex @@ -22,7 +22,7 @@ defmodule Pleroma.Factory do "id" => Pleroma.Web.ActivityPub.Utils.generate_object_id, "actor" => user.ap_id, "to" => ["https://www.w3.org/ns/activitystreams#Public"], - "published_at" => DateTime.utc_now() |> DateTime.to_iso8601, + "published" => DateTime.utc_now() |> DateTime.to_iso8601, "likes" => [], "like_count" => 0, "context" => "2hu", @@ -42,7 +42,7 @@ defmodule Pleroma.Factory do "actor" => note.data["actor"], "to" => note.data["to"], "object" => note.data, - "published_at" => DateTime.utc_now() |> DateTime.to_iso8601, + "published" => DateTime.utc_now() |> DateTime.to_iso8601, "context" => note.data["context"] } diff --git a/test/web/ostatus/activity_representer_test.exs b/test/web/ostatus/activity_representer_test.exs index 06ffbdf69..ec75fc765 100644 --- a/test/web/ostatus/activity_representer_test.exs +++ b/test/web/ostatus/activity_representer_test.exs @@ -9,10 +9,6 @@ defmodule Pleroma.Web.OStatus.ActivityRepresenterTest do test "a note activity" do note_activity = insert(:note_activity) - updated_at = note_activity.updated_at - |> NaiveDateTime.to_iso8601 - inserted_at = note_activity.inserted_at - |> NaiveDateTime.to_iso8601 user = User.get_cached_by_ap_id(note_activity.data["actor"]) @@ -22,8 +18,8 @@ defmodule Pleroma.Web.OStatus.ActivityRepresenterTest do <id>#{note_activity.data["object"]["id"]}</id> <title>New note by #{user.nickname}</title> <content type="html">#{note_activity.data["object"]["content"]}</content> - <published>#{inserted_at}</published> - <updated>#{updated_at}</updated> + <published>#{note_activity.data["published"]}</published> + <updated>#{note_activity.data["published"]}</updated> <ostatus:conversation>#{note_activity.data["context"]}</ostatus:conversation> <link ref="#{note_activity.data["context"]}" rel="ostatus:conversation" /> <link type="application/atom+xml" href="#{note_activity.data["object"]["id"]}" rel="self" /> @@ -47,11 +43,6 @@ defmodule Pleroma.Web.OStatus.ActivityRepresenterTest do data = %{answer.data | "object" => object} answer = %{answer | data: data} - updated_at = answer.updated_at - |> NaiveDateTime.to_iso8601 - inserted_at = answer.inserted_at - |> NaiveDateTime.to_iso8601 - user = User.get_cached_by_ap_id(answer.data["actor"]) expected = """ @@ -60,8 +51,8 @@ defmodule Pleroma.Web.OStatus.ActivityRepresenterTest do <id>#{answer.data["object"]["id"]}</id> <title>New note by #{user.nickname}</title> <content type="html">#{answer.data["object"]["content"]}</content> - <published>#{inserted_at}</published> - <updated>#{updated_at}</updated> + <published>#{answer.data["published"]}</published> + <updated>#{answer.data["published"]}</updated> <ostatus:conversation>#{answer.data["context"]}</ostatus:conversation> <link ref="#{answer.data["context"]}" rel="ostatus:conversation" /> <link type="application/atom+xml" href="#{answer.data["object"]["id"]}" rel="self" /> @@ -92,19 +83,14 @@ defmodule Pleroma.Web.OStatus.ActivityRepresenterTest do |> :xmerl.export_simple_content(:xmerl_xml) |> to_string - updated_at = announce.updated_at - |> NaiveDateTime.to_iso8601 - inserted_at = announce.inserted_at - |> NaiveDateTime.to_iso8601 - expected = """ <activity:object-type>http://activitystrea.ms/schema/1.0/activity</activity:object-type> <activity:verb>http://activitystrea.ms/schema/1.0/share</activity:verb> <id>#{announce.data["id"]}</id> <title>#{user.nickname} repeated a notice</title> <content type="html">RT #{note.data["object"]["content"]}</content> - <published>#{inserted_at}</published> - <updated>#{updated_at}</updated> + <published>#{announce.data["published"]}</published> + <updated>#{announce.data["published"]}</updated> <ostatus:conversation>#{announce.data["context"]}</ostatus:conversation> <link ref="#{announce.data["context"]}" rel="ostatus:conversation" /> <link rel="self" type="application/atom+xml" href="#{announce.data["id"]}"/> @@ -126,12 +112,6 @@ defmodule Pleroma.Web.OStatus.ActivityRepresenterTest do user = insert(:user) {:ok, like, _note} = ActivityPub.like(user, note) - # TODO: Are these the correct dates? - updated_at = like.updated_at - |> NaiveDateTime.to_iso8601 - inserted_at = like.inserted_at - |> NaiveDateTime.to_iso8601 - tuple = ActivityRepresenter.to_simple_form(like, user) refute is_nil(tuple) @@ -142,8 +122,8 @@ defmodule Pleroma.Web.OStatus.ActivityRepresenterTest do <id>#{like.data["id"]}</id> <title>New favorite by #{user.nickname}</title> <content type="html">#{user.nickname} favorited something</content> - <published>#{inserted_at}</published> - <updated>#{updated_at}</updated> + <published>#{like.data["published"]}</published> + <updated>#{like.data["published"]}</updated> <activity:object> <activity:object-type>http://activitystrea.ms/schema/1.0/note</activity:object-type> <id>#{note.data["id"]}</id> @@ -168,13 +148,6 @@ defmodule Pleroma.Web.OStatus.ActivityRepresenterTest do "to" => [followed.ap_id] }) - - # TODO: Are these the correct dates? - updated_at = activity.updated_at - |> NaiveDateTime.to_iso8601 - inserted_at = activity.inserted_at - |> NaiveDateTime.to_iso8601 - tuple = ActivityRepresenter.to_simple_form(activity, follower) refute is_nil(tuple) @@ -187,8 +160,8 @@ defmodule Pleroma.Web.OStatus.ActivityRepresenterTest do <id>#{activity.data["id"]}</id> <title>#{follower.nickname} started following #{activity.data["object"]}</title> <content type="html"> #{follower.nickname} started following #{activity.data["object"]}</content> - <published>#{inserted_at}</published> - <updated>#{updated_at}</updated> + <published>#{activity.data["published"]}</published> + <updated>#{activity.data["published"]}</updated> <activity:object> <activity:object-type>http://activitystrea.ms/schema/1.0/person</activity:object-type> <id>#{activity.data["object"]}</id> @@ -207,12 +180,6 @@ defmodule Pleroma.Web.OStatus.ActivityRepresenterTest do {:ok, _activity} = ActivityPub.follow(follower, followed) {:ok, activity} = ActivityPub.unfollow(follower, followed) - # TODO: Are these the correct dates? - updated_at = activity.updated_at - |> NaiveDateTime.to_iso8601 - inserted_at = activity.inserted_at - |> NaiveDateTime.to_iso8601 - tuple = ActivityRepresenter.to_simple_form(activity, follower) refute is_nil(tuple) @@ -225,8 +192,8 @@ defmodule Pleroma.Web.OStatus.ActivityRepresenterTest do <id>#{activity.data["id"]}</id> <title>#{follower.nickname} stopped following #{followed.ap_id}</title> <content type="html"> #{follower.nickname} stopped following #{followed.ap_id}</content> - <published>#{inserted_at}</published> - <updated>#{updated_at}</updated> + <published>#{activity.data["published"]}</published> + <updated>#{activity.data["published"]}</updated> <activity:object> <activity:object-type>http://activitystrea.ms/schema/1.0/person</activity:object-type> <id>#{followed.ap_id}</id> diff --git a/test/web/ostatus/ostatus_test.exs b/test/web/ostatus/ostatus_test.exs index 6599b00b3..e40fae78c 100644 --- a/test/web/ostatus/ostatus_test.exs +++ b/test/web/ostatus/ostatus_test.exs @@ -233,6 +233,7 @@ defmodule Pleroma.Web.OStatusTest do assert user.local == false assert user.info["uri"] == uri assert user.ap_id == uri + assert user.bio == "Call me Deacon Blues." assert user.avatar["type"] == "Image" {:ok, user_again} = OStatus.find_or_make_user(uri) @@ -244,7 +245,9 @@ defmodule Pleroma.Web.OStatusTest do uri = "https://social.heldscal.la/user/23211" {:ok, user} = OStatus.find_or_make_user(uri) - change = Ecto.Changeset.change(user, %{avatar: nil}) + old_name = user.name + old_bio = user.bio + change = Ecto.Changeset.change(user, %{avatar: nil, bio: nil, old_name: nil}) {:ok, user} = Repo.update(change) refute user.avatar @@ -253,6 +256,8 @@ defmodule Pleroma.Web.OStatusTest do [author] = :xmerl_xpath.string('//author[1]', doc) {:ok, user} = OStatus.find_make_or_update_user(author) assert user.avatar["type"] == "Image" + assert user.name == old_name + assert user.bio == old_bio {:ok, user_again} = OStatus.find_make_or_update_user(author) assert user_again == user @@ -277,6 +282,7 @@ defmodule Pleroma.Web.OStatusTest do "uri" => "https://social.heldscal.la/user/29191", "host" => "social.heldscal.la", "fqn" => user, + "bio" => "cofe", "avatar" => %{"type" => "Image", "url" => [%{"href" => "https://social.heldscal.la/avatar/29191-original-20170421154949.jpeg", "mediaType" => "image/jpeg", "type" => "Link"}]} } assert data == expected @@ -299,6 +305,7 @@ defmodule Pleroma.Web.OStatusTest do "uri" => "https://social.heldscal.la/user/29191", "host" => "social.heldscal.la", "fqn" => user, + "bio" => "cofe", "avatar" => %{"type" => "Image", "url" => [%{"href" => "https://social.heldscal.la/avatar/29191-original-20170421154949.jpeg", "mediaType" => "image/jpeg", "type" => "Link"}]} } assert data == expected diff --git a/test/web/twitter_api/twitter_api_test.exs b/test/web/twitter_api/twitter_api_test.exs index 4df1aba9e..df1022d6b 100644 --- a/test/web/twitter_api/twitter_api_test.exs +++ b/test/web/twitter_api/twitter_api_test.exs @@ -263,7 +263,7 @@ defmodule Pleroma.Web.TwitterAPI.TwitterAPITest do archaeme_remote = insert(:user, %{nickname: "archaeme@archae.me"}) mentions = TwitterAPI.parse_mentions(text) - expected_text = "<a href='#{gsimg.ap_id}'>@gsimg</a> According to <a href='#{archaeme.ap_id}'>@archaeme</a>, that is @daggsy. Also hello <a href='#{archaeme_remote.ap_id}'>@archaeme@archae.me</a>" + expected_text = "<a href='#{gsimg.ap_id}'>@gsimg</a> According to <a href='#{archaeme.ap_id}'>@archaeme</a>, that is @daggsy. Also hello <a href='#{archaeme_remote.ap_id}'>@archaeme</a>" assert Utils.add_user_links(text, mentions) == expected_text end diff --git a/test/web/websub/websub_test.exs b/test/web/websub/websub_test.exs index aaed9226e..1ca573d66 100644 --- a/test/web/websub/websub_test.exs +++ b/test/web/websub/websub_test.exs @@ -120,6 +120,7 @@ defmodule Pleroma.Web.WebsubTest do "nickname" => "lambadalambda", "name" => "Critical Value", "host" => "mastodon.social", + "bio" => "a cool dude.", "avatar" => %{"type" => "Image", "url" => [%{"href" => "https://files.mastodon.social/accounts/avatars/000/000/264/original/1429214160519.gif?1492379244", "mediaType" => "image/gif", "type" => "Link"}]} } |