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>
Wed, 16 Oct 2019 12:09:39 +0000 (15:09 +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 e413f08935e1cdb8ff0335669a94f5ac475e52e6..78a02d5364c5434fc2dda5f3c83f5caa96512daf 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"})