test: fix tests & update tag/regression test

Author: zachdaniel

Diff for: priv/resource_snapshots/test_repo/users/20250321142835.json

@@ -0,0 +1,130 @@
+{
+  "attributes": [
+    {
+      "allow_nil?": false,
+      "default": "fragment(\"gen_random_uuid()\")",
+      "generated?": false,
+      "primary_key?": true,
+      "references": null,
+      "size": null,
+      "source": "id",
+      "type": "uuid"
+    },
+    {
+      "allow_nil?": true,
+      "default": "nil",
+      "generated?": false,
+      "primary_key?": false,
+      "references": null,
+      "size": null,
+      "source": "is_active",
+      "type": "boolean"
+    },
+    {
+      "allow_nil?": true,
+      "default": "nil",
+      "generated?": false,
+      "primary_key?": false,
+      "references": null,
+      "size": null,
+      "source": "name",
+      "type": "text"
+    },
+    {
+      "allow_nil?": true,
+      "default": "\"user\"",
+      "generated?": false,
+      "primary_key?": false,
+      "references": null,
+      "size": null,
+      "source": "role",
+      "type": "text"
+    },
+    {
+      "allow_nil?": true,
+      "default": "[]",
+      "generated?": false,
+      "primary_key?": false,
+      "references": null,
+      "size": null,
+      "source": "role_list",
+      "type": [
+        "array",
+        "text"
+      ]
+    },
+    {
+      "allow_nil?": true,
+      "default": "nil",
+      "generated?": false,
+      "primary_key?": false,
+      "references": {
+        "deferrable": false,
+        "destination_attribute": "id",
+        "destination_attribute_default": null,
+        "destination_attribute_generated": null,
+        "index?": false,
+        "match_type": null,
+        "match_with": null,
+        "multitenancy": {
+          "attribute": null,
+          "global": null,
+          "strategy": null
+        },
+        "name": "users_organization_id_fkey",
+        "on_delete": null,
+        "on_update": null,
+        "primary_key?": true,
+        "schema": "public",
+        "table": "orgs"
+      },
+      "size": null,
+      "source": "organization_id",
+      "type": "uuid"
+    },
+    {
+      "allow_nil?": true,
+      "default": "nil",
+      "generated?": false,
+      "primary_key?": false,
+      "references": {
+        "deferrable": false,
+        "destination_attribute": "id",
+        "destination_attribute_default": null,
+        "destination_attribute_generated": null,
+        "index?": false,
+        "match_type": null,
+        "match_with": null,
+        "multitenancy": {
+          "attribute": "id",
+          "global": true,
+          "strategy": "attribute"
+        },
+        "name": "users_org_id_fkey",
+        "on_delete": null,
+        "on_update": null,
+        "primary_key?": true,
+        "schema": "public",
+        "table": "multitenant_orgs"
+      },
+      "size": null,
+      "source": "org_id",
+      "type": "uuid"
+    }
+  ],
+  "base_filter": null,
+  "check_constraints": [],
+  "custom_indexes": [],
+  "custom_statements": [],
+  "has_create_action": true,
+  "hash": "8DC96004D53169EA4AA3721AD7836DB2A230F30C55EE1B06DDB1D7505AE50210",
+  "identities": [],
+  "multitenancy": {
+    "attribute": null,
+    "global": null,
+    "strategy": null
+  },
+  "repo": "Elixir.AshPostgres.TestRepo",
+  "schema": null,
+  "table": "users"
+}

Diff for: priv/test_repo/migrations/20250321142835_migrate_resources52.exs

@@ -0,0 +1,36 @@
+defmodule AshPostgres.TestRepo.Migrations.MigrateResources52 do
+  @moduledoc """
+  Updates resources based on their most recent snapshots.
+
+  This file was autogenerated with `mix ash_postgres.generate_migrations`
+  """
+
+  use Ecto.Migration
+
+  def up do
+    alter table(:users) do
+      modify(:role_list, {:array, :text}, null: true)
+      modify(:role, :text, null: true)
+
+      add(
+        :org_id,
+        references(:multitenant_orgs,
+          column: :id,
+          name: "users_org_id_fkey",
+          type: :uuid,
+          prefix: "public"
+        )
+      )
+    end
+  end
+
+  def down do
+    drop(constraint(:users, "users_org_id_fkey"))
+
+    alter table(:users) do
+      remove(:org_id)
+      modify(:role, :text, null: false)
+      modify(:role_list, {:array, :text}, null: false)
+    end
+  end
+end

Diff for: test/bulk_update_test.exs

@@ -264,7 +264,7 @@ defmodule AshPostgres.BulkUpdateTest do
              )
   end
 
-  @tag :wip
+  @tag :regression
   test "jsonb[] attribute with embedded Resource definition can be created and updated" do
     %{status: :success} =
       Ash.bulk_create!(
@@ -275,10 +275,9 @@ defmodule AshPostgres.BulkUpdateTest do
         return_errors?: true
       )
 
-    %{status: :success} =
-      Ash.bulk_update(CSV, :update, %{
-        column_mapping_embedded: [%{column: 1, attribute: "foo"}],
-        column_mapping_new_type: [%{column: 1, attribute: "foo"}]
-      })
+    Ash.bulk_update!(CSV, :update, %{
+      column_mapping_embedded: [%{column: 1, attribute: "foo"}],
+      column_mapping_new_type: [%{column: 1, attribute: "foo"}]
+    })
   end
 end

Diff for: test/support/resources/csv.ex

@@ -50,7 +50,6 @@ defmodule AshPostgres.Test.CSV do
     table "csv"
     repo(AshPostgres.TestRepo)
 
-    storage_types column_mapping_embedded: :jsonb
-    storage_types column_mapping_new_type: :jsonb
+    storage_types column_mapping_embedded: :jsonb, column_mapping_new_type: :jsonb
   end
 end