refactor(typed-channel): use real aggregate expressions in test calc transforms

Author: Torkan

test/ash_typescript/typed_channel/codegen_test.exs

@@ -313,40 +313,39 @@ defmodule AshTypescript.TypedChannel.CodegenTest do
                "export type TrackerSnapshotPayload = {id: UUID, name: string | null, status: string | null};"
     end
 
-    test "map calc transform payload type with different field types", %{
+    test "map calc transform payload type with relationship traversal", %{
       types_content: content
     } do
-      # %{id: id, name: name, description: status}
-      # id non-null (PK), name/description nullable (attributes)
+      # %{id: id, name: name, latest_entry_body: first(entries, :body)}
+      # id non-null, name nullable, first() nullable
       assert content =~
-               "export type TrackerDetailPayload = {id: UUID, name: string | null, description: string | null};"
+               "export type TrackerDetailPayload = {id: UUID, name: string | null, latestEntryBody: string | null};"
     end
 
-    test "integer calc payload type from attribute", %{types_content: content} do
+    test "integer calc payload type (count aggregate)", %{types_content: content} do
       assert content =~ "export type TrackerEntryCountPayload = number;"
     end
 
     test "boolean expression calc transform payload type", %{types_content: content} do
       assert content =~ "export type TrackerIsActivePayload = boolean;"
     end
 
-    test "integer calc payload type from attribute (alt)", %{types_content: content} do
+    test "integer aggregate calc payload type (max)", %{types_content: content} do
       assert content =~ "export type TrackerTopScorePayload = number;"
     end
 
-    test "map with various field types", %{types_content: content} do
-      # id non-null (PK), name/status nullable, currentPriority nullable (integer attr)
+    test "map with nested relationship fields (first on related FK)", %{types_content: content} do
+      # id non-null (PK), name nullable, aggregates (first) nullable
       assert content =~
-               "export type TrackerDeepDetailPayload = {id: UUID, name: string | null, status: string | null, currentPriority: number | null};"
+               "export type TrackerDeepDetailPayload = {id: UUID, name: string | null, latestAuthor: UUID | null, latestBody: string | null, latestScore: number | null};"
     end
 
-    test "map mixing expressions, booleans, and attributes with correct nullability", %{
+    test "map mixing aggregates, booleans, and strings with correct nullability", %{
       types_content: content
     } do
-      # boolean expr → nullable (operands are nullable), attributes → nullable, string concat → nullable
-      # Fields are alphabetically ordered in output
+      # all fields nullable: attributes nullable, aggregates nullable, boolean expr with nullable operands nullable
       assert content =~
-               "export type TrackerReportPayload = {label: string | null, name: string | null, status: string | null, isActive: boolean | null, currentPriority: number | null};"
+               "export type TrackerReportPayload = {name: string | null, status: string | null, entryCount: number | null, isActive: boolean | null, topScore: number | null, latestBody: string | null};"
     end
 
     test "no payload types are unknown", %{types_content: content} do

test/support/resources/channel_tracker.ex

@@ -9,18 +9,19 @@ defmodule AshTypescript.Test.ChannelTracker do
   All publications use `transform: :calc_name`. Ash auto-populates the
   `returns` type from the calculation, which AshTypescript reads directly.
 
-  Expression types covered (all :auto unless noted):
+  Expression types covered:
   - String concat (explicit and :auto)
-  - Map with local attribute fields
-  - Map with mixed field types (string, integer, boolean)
-  - Integer from attribute reference
-  - Boolean expression
-
-  Note: Avoids aggregate expressions (first, count, max) because
-  Ash.DataLayer.Simple doesn't support aggregate type resolution for :auto.
+  - Map with local fields (:auto)
+  - Map with relationship traversal via first() (:auto)
+  - Map with nested relationship traversal via first() with dot access (:auto)
+  - Map mixing aggregates, booleans, strings, and relationship fields (:auto)
+  - Count aggregate (:auto)
+  - Max aggregate (:auto)
+  - Boolean expression (:auto)
   """
   use Ash.Resource,
     domain: nil,
+    data_layer: Ash.DataLayer.Ets,
     notifiers: [Ash.Notifier.PubSub]
 
   pub_sub do
@@ -51,13 +52,13 @@ defmodule AshTypescript.Test.ChannelTracker do
       public?: true,
       transform: :snapshot
 
-    # Map calc with different field types (:auto typed)
+    # Map calc with relationship traversal (:auto typed)
     publish :detail_snapshot, [:id],
       event: "tracker_detail",
       public?: true,
       transform: :detail
 
-    # Integer from attribute (:auto typed)
+    # Count aggregate (:auto typed)
     publish :count_entries, [:id],
       event: "tracker_entry_count",
       public?: true,
@@ -69,19 +70,19 @@ defmodule AshTypescript.Test.ChannelTracker do
       public?: true,
       transform: :is_active
 
-    # Integer from attribute (:auto typed)
+    # Max aggregate (:auto typed)
     publish :top_score, [:id],
       event: "tracker_top_score",
       public?: true,
       transform: :top_entry_score
 
-    # Map with various field types (:auto typed)
+    # Map with nested relationship traversal (:auto typed)
     publish :deep_snapshot, [:id],
       event: "tracker_deep_detail",
       public?: true,
       transform: :deep_detail
 
-    # Map mixing expressions and attributes (:auto typed)
+    # Map mixing aggregates, booleans, and strings (:auto typed)
     publish :full_report, [:id],
       event: "tracker_report",
       public?: true,
@@ -122,15 +123,15 @@ defmodule AshTypescript.Test.ChannelTracker do
       public?(true)
     end
 
-    # :auto map calc with different field types (no aggregates)
+    # :auto map calc with relationship traversal (first aggregate)
     calculate :detail,
               :auto,
-              expr(%{id: id, name: name, description: status}) do
+              expr(%{id: id, name: name, latest_entry_body: first(entries, field: :body)}) do
       public?(true)
     end
 
-    # :auto integer — type inferred from integer attribute
-    calculate :entry_count, :auto, expr(priority) do
+    # :auto count aggregate — should resolve to integer
+    calculate :entry_count, :auto, expr(count(entries)) do
       public?(true)
     end
 
@@ -139,19 +140,20 @@ defmodule AshTypescript.Test.ChannelTracker do
       public?(true)
     end
 
-    # :auto integer — type inferred from integer attribute
-    calculate :top_entry_score, :auto, expr(priority) do
+    # :auto max aggregate on related field — should resolve to integer
+    calculate :top_entry_score, :auto, expr(max(entries, field: :score)) do
       public?(true)
     end
 
-    # :auto map with various field types
+    # :auto map with nested relationship traversal — entries -> author -> username
     calculate :deep_detail,
               :auto,
               expr(%{
                 id: id,
                 name: name,
-                status: status,
-                current_priority: priority
+                latest_author: first(entries, field: :channel_tracker_author_id),
+                latest_body: first(entries, field: :body),
+                latest_score: first(entries, field: :score)
               }) do
       public?(true)
     end
@@ -163,8 +165,9 @@ defmodule AshTypescript.Test.ChannelTracker do
                 name: name,
                 status: status,
                 is_active: status == "active" and priority > 0,
-                current_priority: priority,
-                label: name <> " tracker"
+                entry_count: count(entries),
+                top_score: max(entries, field: :score),
+                latest_body: first(entries, field: :body)
               }) do
       public?(true)
     end

test/support/resources/channel_tracker_author.ex

@@ -6,7 +6,7 @@ defmodule AshTypescript.Test.ChannelTrackerAuthor do
   @moduledoc """
   Author resource for nested relationship traversal testing in :auto calculations.
   """
-  use Ash.Resource, domain: nil
+  use Ash.Resource, domain: nil, data_layer: Ash.DataLayer.Ets
 
   attributes do
     uuid_primary_key :id

test/support/resources/channel_tracker_entry.ex

@@ -9,7 +9,7 @@ defmodule AshTypescript.Test.ChannelTrackerEntry do
 
   Has a belongs_to :author for nested relationship traversal tests.
   """
-  use Ash.Resource, domain: nil
+  use Ash.Resource, domain: nil, data_layer: Ash.DataLayer.Ets
 
   attributes do
     uuid_primary_key :id