fix: use correct key format for listing name owner tables (#688)

Author: sborrazas

lib/ae_mdw/names.ex

@@ -101,21 +101,23 @@ defmodule AeMdw.Names do
 
   defp build_name_streamer(%{owned_by: owner_pk, state: "active"}, cursor) do
     cursor = if cursor, do: {owner_pk, cursor}
+    scope = {{owner_pk, Util.min_bin()}, {owner_pk, Util.max_256bit_bin()}}
 
     fn direction ->
       @table_active_owner
-      |> Collection.stream(direction, nil, cursor)
-      |> Stream.map(fn key -> {key, :active} end)
+      |> Collection.stream(direction, scope, cursor)
+      |> Stream.map(fn {_owner_pk, plain_name} -> {plain_name, :active} end)
     end
   end
 
   defp build_name_streamer(%{owned_by: owner_pk, state: "inactive"}, cursor) do
     cursor = if cursor, do: {owner_pk, cursor}
+    scope = {{owner_pk, Util.min_bin()}, {owner_pk, Util.max_256bit_bin()}}
 
     fn direction ->
       @table_inactive_owner
-      |> Collection.stream(direction, nil, cursor)
-      |> Stream.map(fn key -> {key, :inactive} end)
+      |> Collection.stream(direction, scope, cursor)
+      |> Stream.map(fn {_owner_pk, plain_name} -> {plain_name, :inactive} end)
     end
   end
 

test/integration/ae_mdw_web/controllers/name_controller_test.exs

@@ -636,6 +636,22 @@ defmodule Integration.AeMdwWeb.NameControllerTest do
       assert ^plain_names = Enum.sort(plain_names)
     end
 
+    test "when filtering by owned_by, it returns names owned by that owner", %{conn: conn} do
+      # first, retrieve any name to get the owner
+      assert %{"data" => [%{"info" => %{"ownership" => %{"current" => owner_pk}}} | _rest]} =
+               conn |> get("/v2/names", state: "active", limit: 1) |> json_response(200)
+
+      assert %{"data" => names} =
+               conn
+               |> get("/v2/names", state: "active", by: "name", owned_by: owner_pk, limit: 3)
+               |> json_response(200)
+
+      assert Enum.all?(
+               names,
+               &match?(%{"info" => %{"ownership" => %{"current" => ^owner_pk}}}, &1)
+             )
+    end
+
     test "renders error when parameter by is invalid", %{conn: conn} do
       by = "invalid_by"
       error_msg = "invalid query: by=#{by}"