chore: fixing credo warning

Author: srikanthkyatham

lib/ash_admin/components/resource/show.ex

@@ -9,7 +9,7 @@ defmodule AshAdmin.Components.Resource.Show do
 
   require Logger
 
-  alias AshAdmin.Components.Resource.{Helpers.FormatHelper, SensitiveAttribute, CinderTable}
+  alias AshAdmin.Components.Resource.{CinderTable, Helpers.FormatHelper, SensitiveAttribute}
   import AshAdmin.Helpers
   import AshAdmin.CoreComponents
 

lib/ash_admin/pages/page_live.ex

@@ -455,7 +455,7 @@ defmodule AshAdmin.PageLive do
   def handle_event("clear_all_filters", _params, socket), do: {:noreply, socket}
   def handle_event("toggle_sort", _params, socket), do: {:noreply, socket}
   def handle_event("goto_page", _params, socket), do: {:noreply, socket}
-  def handle_event("filter_change", params, socket), do: {:noreply, socket}
+  def handle_event("filter_change", _params, socket), do: {:noreply, socket}
 
   @impl true
   def handle_info({:filter_builder_value, _filter, filter_query}, socket) do

mix.exs

@@ -123,11 +123,12 @@ defmodule AshAdmin.MixProject do
     ]
   end
 
-  if Mix.env() == :test do
+  if Mix.env() == :text do
     # Run "mix help compile.app" to learn about applications.
     def application do
       [
-        extra_applications: [:logger]
+        extra_applications: [:logger],
+        mod: {AshAdmin, []}
       ]
     end
   end

test/components/resource/data_table_filtering_test.exs

@@ -18,14 +18,8 @@ defmodule AshAdmin.Test.Components.Resource.DataTableFilteringTest do
     {view, render(view)}
   end
 
-  # Helper function to wait for Cinder table to finish loading (for existing view)
-  defp wait_for_table_load(view) do
-    refute_eventually(fn -> render(view) =~ "Loading..." end, timeout: 5000, interval: 100)
-    view
-  end
-
   # Helper to retry a condition until it's false or timeout
-  defp refute_eventually(func, opts \\ []) do
+  defp refute_eventually(func, opts) do
     timeout = Keyword.get(opts, :timeout, 2000)
     interval = Keyword.get(opts, :interval, 50)
     end_time = System.monotonic_time(:millisecond) + timeout
@@ -89,7 +83,7 @@ defmodule AshAdmin.Test.Components.Resource.DataTableFilteringTest do
 
   describe "basic filter functionality" do
     test "renders data table instead of schema view", %{conn: conn} do
-      {view, html} =
+      {_view, html} =
         live_and_wait(conn, "/api/admin?domain=Domain&resource=Post&action_type=read&action=read")
 
       # Should NOT show the schema view (Attributes/Relationships tables)
@@ -174,7 +168,6 @@ defmodule AshAdmin.Test.Components.Resource.DataTableFilteringTest do
       assert html =~ "First post about Elixir"
     end
 
-    @skip true
     test "clears all filters at once", %{conn: conn} do
       {view, _html} =
         live_and_wait(conn, "/api/admin?domain=Domain&resource=Post&action_type=read&action=read")

test/support/resources/comment.ex

@@ -19,9 +19,10 @@ defmodule AshAdmin.Test.Comment do
 
   actions do
     default_accept(:*)
-    defaults([:create, :read, :update, :destroy])
+    defaults([:read, :update, :destroy])
 
-    create :create_with_photo do
+    create :create do
+      primary?(true)
       argument(:photo, :file)
     end
   end