Partial indexes using :where in schema dumper. Fixes #153

I have altered the deal 🙏 I dont alter it any further.
Furthermore, I wish you to wear this 👗 and 👶 bonnet.

cc @hmadison

Author: metaskills

CHANGELOG.md

@@ -21,6 +21,7 @@
 
 * Every time datatype has perfect micro/nano second handling.
 * All supported datatypes dump defaults properly to schema.rb
+* Partial indexes using `:where` in schema dumper. Fixes #153
 
 
 ## v4.2.6

lib/active_record/connection_adapters/sqlserver/schema_statements.rb

@@ -32,12 +32,13 @@ def indexes(table_name, name = nil)
             else
               name    = index[:index_name]
               unique  = index[:index_description] =~ /unique/
+              where   = select_value("SELECT [filter_definition] FROM sys.indexes WHERE name = #{quote(name)}")
               columns = index[:index_keys].split(',').map do |column|
                 column.strip!
                 column.gsub! '(-)', '' if column.ends_with?('(-)')
                 column
               end
-              indexes << IndexDefinition.new(table_name, name, unique, columns)
+              indexes << IndexDefinition.new(table_name, name, unique, columns, nil, nil, where)
             end
           end
         end

test/cases/coerced_tests.rb

@@ -601,6 +601,13 @@ def test_schema_dump_keeps_large_precision_integer_columns_as_decimal_coerced
   # This is a poorly written test and really does not catch the bottom'ness it is meant too. Ours throw it off.
   coerce_tests! :test_foreign_keys_are_dumped_at_the_bottom_to_circumvent_dependency_issues
 
+  # Fall through false positive with no filter.
+  coerce_tests! :test_schema_dumps_partial_indices
+  def test_schema_dumps_partial_indices_coerced
+    index_definition = standard_dump.split(/\n/).grep(/add_index.*company_partial_index/).first.strip
+    assert_equal 'add_index "companies", ["firm_id", "type"], name: "company_partial_index", where: "([rating]>(10))"', index_definition
+  end
+
 end
 
 class SchemaDumperDefaultsTest < ActiveRecord::TestCase