Merge branch 'c11n' into dev

Author: mydearxym

lib/mastani_server_web/context.ex

@@ -42,7 +42,7 @@ defmodule MastaniServerWeb.Context do
 
   defp authorize(token) do
     with {:ok, claims, _info} <- Guardian.jwt_decode(token) do
-      case ORM.find(Accounts.User, claims.id) do
+      case ORM.find(Accounts.User, claims.id, preload: :customization) do
         {:ok, user} ->
           check_passport(user)
 

lib/mastani_server_web/middleware/pagesize_proof.ex

@@ -16,6 +16,16 @@ defmodule MastaniServerWeb.Middleware.PageSizeProof do
   # 3. large size should trigger error
   def call(%{errors: errors} = resolution, _) when length(errors) > 0, do: resolution
 
+  def call(%{context: %{cur_user: %{customization: customization}}} = resolution, _info)
+      when not is_nil(customization) do
+    size = String.to_integer(customization.display_density)
+
+    filter = resolution.arguments.filter |> Map.merge(%{size: size})
+    arguments = resolution.arguments |> Map.merge(%{filter: filter})
+
+    %{resolution | arguments: sort_desc_by_default(arguments)}
+  end
+
   def call(resolution, _) do
     case valid_size(resolution.arguments) do
       {:error, msg} ->

test/mastani_server_web/mutation/accounts/customization_test.exs

@@ -3,6 +3,9 @@ defmodule MastaniServer.Test.Mutation.Account.Customization do
 
   # alias MastaniServer.{Accounts}
   # alias Helper.ORM
+  import Helper.Utils, only: [get_config: 2]
+
+  @max_page_size get_config(:general, :page_size)
 
   setup do
     {:ok, user} = db_insert(:user)
@@ -55,6 +58,48 @@ defmodule MastaniServer.Test.Mutation.Account.Customization do
       assert result["customization"]["displayDensity"] == "25"
     end
 
+    @paged_post_query """
+    query($filter: PagedArticleFilter!) {
+      pagedPosts(filter: $filter) {
+        pageSize
+        pageNumber
+      }
+    }
+    """
+    test "PageSizeProof middleware should load items based on c11n settings", ~m(user)a do
+      user_conn = simu_conn(:user, user)
+      db_insert_multi(:post, 50)
+
+      variables = %{filter: %{page: 1}}
+      results = user_conn |> query_result(@paged_post_query, variables, "pagedPosts")
+      assert results["pageSize"] == @max_page_size
+
+      variables = %{
+        customization: %{
+          displayDensity: "25"
+        }
+      }
+
+      user_conn |> mutation_result(@query, variables, "setCustomization")
+
+      variables = %{filter: %{page: 1}}
+      results = user_conn |> query_result(@paged_post_query, variables, "pagedPosts")
+      assert results["pageSize"] == 25
+
+      variables = %{
+        customization: %{
+          displayDensity: "20"
+        }
+      }
+
+      user_conn |> mutation_result(@query, variables, "setCustomization")
+
+      variables = %{filter: %{page: 2}}
+      results = user_conn |> query_result(@paged_post_query, variables, "pagedPosts")
+      assert results["pageSize"] == 20
+      assert results["pageNumber"] == 2
+    end
+
     test "set single customization should merge not overwright other settings", ~m(user_conn)a do
       variables = %{
         customization: %{