Add integer casts.

This commit is contained in:
lain 2019-01-09 18:17:23 +01:00
parent 7ac152ed38
commit a99e156f2c
2 changed files with 3 additions and 3 deletions

View file

@ -472,7 +472,7 @@ def external_profile(%{assigns: %{user: current_user}} = conn, %{"profileurl" =>
end end
def followers(%{assigns: %{user: for_user}} = conn, params) do def followers(%{assigns: %{user: for_user}} = conn, params) do
page = params["page"] || 1 {:ok, page} = Ecto.Type.cast(:integer, params["page"] || 1)
with {:ok, user} <- TwitterAPI.get_user(for_user, params), with {:ok, user} <- TwitterAPI.get_user(for_user, params),
{:ok, followers} <- User.get_followers(user, page) do {:ok, followers} <- User.get_followers(user, page) do
@ -492,7 +492,7 @@ def followers(%{assigns: %{user: for_user}} = conn, params) do
end end
def friends(%{assigns: %{user: for_user}} = conn, params) do def friends(%{assigns: %{user: for_user}} = conn, params) do
page = params["page"] || 1 {:ok, page} = Ecto.Type.cast(:integer, params["page"] || 1)
with {:ok, user} <- TwitterAPI.get_user(conn.assigns[:user], params), with {:ok, user} <- TwitterAPI.get_user(conn.assigns[:user], params),
{:ok, friends} <- User.get_friends(user, page) do {:ok, friends} <- User.get_friends(user, page) do

View file

@ -1101,7 +1101,7 @@ test "it returns 20 followers per page", %{conn: conn} do
res_conn = res_conn =
conn conn
|> assign(:user, user) |> assign(:user, user)
|> get("/api/statuses/followers", %{page: 2}) |> get("/api/statuses/followers?page=2")
result = json_response(res_conn, 200) result = json_response(res_conn, 200)
assert length(result) == 1 assert length(result) == 1