Remove a failing search test due to it being unrealistic
authorrinpatch <rinpatch@sdf.org>
Wed, 16 Oct 2019 12:09:39 +0000 (15:09 +0300)
committerrinpatch <rinpatch@sdf.org>
Fri, 18 Oct 2019 11:56:27 +0000 (14:56 +0300)
While the case tested (`lain@ple`) gives wrong ordering due to
using only trigram to order, it almost never happens in reality. In reality
it would be either `lain` (in which case it's fine to list the user with `lain`
in display name first), or `@lain@pleroma.soykaf.com`/`lain@pleroma.soykaf.com` (which is
handled fine as well)

test/user_search_test.exs

index 341074031cc764855b3240360dc88743e5a7c90a..6ee624929fd7f1acd021bfd50af580b7ab481fce 100644 (file)
@@ -65,15 +65,6 @@ defmodule Pleroma.UserSearchTest do
       assert [u2.id, u1.id] == Enum.map(User.search("bar word"), & &1.id)
     end
 
-    test "finds users, ranking by similarity" do
-      u1 = insert(:user, %{name: "lain"})
-      _u2 = insert(:user, %{name: "ean"})
-      u3 = insert(:user, %{name: "ebn", nickname: "lain@mastodon.social"})
-      u4 = insert(:user, %{nickname: "lain@pleroma.soykaf.com"})
-
-      assert [u4.id, u3.id, u1.id] == Enum.map(User.search("lain@ple", for_user: u1), & &1.id)
-    end
-
     test "finds users, boosting ranks of friends and followers" do
       u1 = insert(:user)
       u2 = insert(:user, %{name: "Doe"})