From 203e0e0e4a863a25fedfe6985b371c3f4cfc6839 Mon Sep 17 00:00:00 2001 From: Dan McClain Date: Mon, 25 Mar 2013 22:41:29 -0400 Subject: Checks :algorithm argument for valid values --- activerecord/test/cases/adapters/mysql/active_schema_test.rb | 4 ++++ activerecord/test/cases/adapters/mysql2/active_schema_test.rb | 4 ++++ activerecord/test/cases/adapters/postgresql/active_schema_test.rb | 3 +++ 3 files changed, 11 insertions(+) (limited to 'activerecord/test/cases/adapters') diff --git a/activerecord/test/cases/adapters/mysql/active_schema_test.rb b/activerecord/test/cases/adapters/mysql/active_schema_test.rb index ca6478f6ac..9050ae3fe3 100644 --- a/activerecord/test/cases/adapters/mysql/active_schema_test.rb +++ b/activerecord/test/cases/adapters/mysql/active_schema_test.rb @@ -47,6 +47,10 @@ class ActiveSchemaTest < ActiveRecord::TestCase expected = "CREATE INDEX `index_people_on_last_name` USING btree ON `people` (`last_name`(10)) ALGORITHM = COPY" assert_equal expected, add_index(:people, :last_name, :length => 10, using: :btree, algorithm: :copy) + assert_raise ArgumentError do + add_index(:people, :last_name, algorithm: :coyp) + end + expected = "CREATE INDEX `index_people_on_last_name_and_first_name` USING btree ON `people` (`last_name`(15), `first_name`(15)) " assert_equal expected, add_index(:people, [:last_name, :first_name], :length => 15, :using => :btree) diff --git a/activerecord/test/cases/adapters/mysql2/active_schema_test.rb b/activerecord/test/cases/adapters/mysql2/active_schema_test.rb index 9b10d23465..48d63aeef5 100644 --- a/activerecord/test/cases/adapters/mysql2/active_schema_test.rb +++ b/activerecord/test/cases/adapters/mysql2/active_schema_test.rb @@ -47,6 +47,10 @@ class ActiveSchemaTest < ActiveRecord::TestCase expected = "CREATE INDEX `index_people_on_last_name` USING btree ON `people` (`last_name`(10)) ALGORITHM = COPY" assert_equal expected, add_index(:people, :last_name, :length => 10, using: :btree, algorithm: :copy) + assert_raise ArgumentError do + add_index(:people, :last_name, algorithm: :coyp) + end + expected = "CREATE INDEX `index_people_on_last_name_and_first_name` USING btree ON `people` (`last_name`(15), `first_name`(15)) " assert_equal expected, add_index(:people, [:last_name, :first_name], :length => 15, :using => :btree) diff --git a/activerecord/test/cases/adapters/postgresql/active_schema_test.rb b/activerecord/test/cases/adapters/postgresql/active_schema_test.rb index 41e9bb912e..16329689c0 100644 --- a/activerecord/test/cases/adapters/postgresql/active_schema_test.rb +++ b/activerecord/test/cases/adapters/postgresql/active_schema_test.rb @@ -43,6 +43,9 @@ class PostgresqlActiveSchemaTest < ActiveRecord::TestCase assert_equal expected, add_index(:people, :last_name, using: type, algorithm: :concurrently) end + assert_raise ArgumentError do + add_index(:people, :last_name, algorithm: :copy) + end expected = %(CREATE UNIQUE INDEX "index_people_on_last_name" ON "people" USING gist ("last_name")) assert_equal expected, add_index(:people, :last_name, :unique => true, :using => :gist) -- cgit v1.2.3