Merge pull request #413 from kbrock/pr_412

Fix primary_key integer check

Author: kbrock

lib/ancestry/class_methods.rb

@@ -221,7 +221,7 @@ def primary_key_is_an_integer?
         if defined?(@primary_key_is_an_integer)
           @primary_key_is_an_integer
         else
-          @primary_key_is_an_integer = !ANCESTRY_UNCAST_TYPES.include?(type_for_attribute(primary_key))
+          @primary_key_is_an_integer = !ANCESTRY_UNCAST_TYPES.include?(type_for_attribute(primary_key).type)
         end
       end
     end

test/concerns/has_ancestry_test.rb

@@ -123,4 +123,16 @@ def test_setup_test_nodes
       end
     end
   end
+
+  def test_primary_key_is_an_integer
+    AncestryTestDatabase.with_model(extra_columns: { string_id: :string }) do |model|
+      model.primary_key = :string_id
+
+      assert !model.primary_key_is_an_integer?
+    end
+
+    AncestryTestDatabase.with_model do |model|
+      assert model.primary_key_is_an_integer?
+    end
+  end
 end