Add uuid_test to re-implement the ActiveRecord test where an invalid UUID is passed to the database.

Author: keithdoggett

test/cases/adapters/postgresql/uuid_test.rb

@@ -0,0 +1,54 @@
+# frozen_string_literal: true
+
+require "cases/helper"
+require "support/schema_dumping_helper"
+
+module PostgresqlUUIDHelper
+  def connection
+    @connection ||= ActiveRecord::Base.connection
+  end
+
+  def drop_table(name)
+    connection.drop_table name, if_exists: true
+  end
+
+  def uuid_function
+    connection.supports_pgcrypto_uuid? ? "gen_random_uuid()" : "uuid_generate_v4()"
+  end
+
+  def uuid_default
+    connection.supports_pgcrypto_uuid? ? {} : { default: uuid_function }
+  end
+end
+
+class PostgresqlUUIDTest < ActiveRecord::PostgreSQLTestCase
+  include PostgresqlUUIDHelper
+  include SchemaDumpingHelper
+
+  class UUIDType < ActiveRecord::Base
+    self.table_name = "uuid_data_type"
+  end
+
+  setup do
+    enable_extension!("uuid-ossp", connection)
+    enable_extension!("pgcrypto",  connection) if connection.supports_pgcrypto_uuid?
+
+    connection.create_table "uuid_data_type" do |t|
+      t.uuid "guid"
+    end
+  end
+
+  teardown do
+    drop_table "uuid_data_type"
+  end
+
+
+  # This test case is nearly identical to the ActiveRecord test, except that
+  # the input guid is a valid UUID since CockroachDB will raise an exception
+  # if the UUID is invalid whereas Postgres won't.
+  def test_uuid_change_case_does_not_mark_dirty
+    model = UUIDType.create!(guid: "A0EEBC99-9C0B-4EF8-BB6D-6BB9BD380A11")
+    model.guid = model.guid.swapcase
+    assert_not_predicate model, :changed?
+  end
+end

test/excludes/ActiveRecord/ConnectionAdapters/PostgreSQLAdapterPreventWritesLegacyTest.rb

@@ -1 +1 @@
-exclude :test_doesnt_error_when_a_read_query_with_cursors_is_called_while_preventing_writes, "CockroachDB does not currently support declaring a cursor."
+exclude :test_doesnt_error_when_a_read_query_with_cursors_is_called_while_preventing_writes, "CockroachDB does not currently support declaring a cursor. See https://github.com/cockroachdb/cockroach/issues/41412."

test/excludes/ActiveRecord/ConnectionAdapters/PostgreSQLAdapterPreventWritesTest.rb

@@ -1 +1 @@
-exclude :test_doesnt_error_when_a_read_query_with_cursors_is_called_while_preventing_writes, "CockroachDB does not currently support declaring a cursor."
+exclude :test_doesnt_error_when_a_read_query_with_cursors_is_called_while_preventing_writes, "CockroachDB does not currently support declaring a cursor. See https://github.com/cockroachdb/cockroach/issues/41412."

test/excludes/PostgresqlMoneyTest.rb

@@ -7,3 +7,5 @@
 exclude :test_update_all_with_money_string, "The money type is not implemented in CockroachDB. See https://github.com/cockroachdb/cockroach/issues/41578."
 exclude :test_update_all_with_money_big_decimal, "The money type is not implemented in CockroachDB. See https://github.com/cockroachdb/cockroach/issues/41578."
 exclude :test_update_all_with_money_numeric, "The money type is not implemented in CockroachDB. See https://github.com/cockroachdb/cockroach/issues/41578."
+exclude :test_sum_with_type_cast, "The money type is not implemented in CockroachDB. See https://github.com/cockroachdb/cockroach/issues/41578."
+exclude :test_pluck_with_type_cast, "The money type is not implemented in CockroachDB. See https://github.com/cockroachdb/cockroach/issues/41578."

test/excludes/PostgresqlUUIDTest.rb

@@ -1,3 +1,4 @@
 exclude :test_uniqueness_validation_ignores_uuid, "Skipping until we can triage further. See https://github.com/cockroachdb/activerecord-cockroachdb-adapter/issues/48"
 exclude :test_change_column_default, "Skipping until we can triage further. See https://github.com/cockroachdb/activerecord-cockroachdb-adapter/issues/48"
 exclude :test_uuid_formats, "Skipping until we can triage further. See https://github.com/cockroachdb/activerecord-cockroachdb-adapter/issues/48"
+exclude :test_uuid_change_case_does_not_mark_dirty, "This test is re-implemented by us. The original version tests with an invalid UUID, which causes CockroachDB to raise an exception."