From 648410248c744476ffb769fafe7723adf45f15d0 Mon Sep 17 00:00:00 2001 From: Ken Collins Date: Mon, 11 May 2009 22:33:45 -0400 Subject: [PATCH] Reimplement Fixtures.identify so that it consistently generates identities across ruby versions. --- activerecord/lib/active_record/fixtures.rb | 7 ++++++- activerecord/test/cases/fixtures_test.rb | 5 +++++ 2 files changed, 11 insertions(+), 1 deletions(-) diff --git a/activerecord/lib/active_record/fixtures.rb b/activerecord/lib/active_record/fixtures.rb index c650111..45aada9 100644 --- a/activerecord/lib/active_record/fixtures.rb +++ b/activerecord/lib/active_record/fixtures.rb @@ -1,6 +1,7 @@ require 'erb' require 'yaml' require 'csv' +require 'zlib' require 'active_support/dependencies' require 'active_support/test_case' @@ -528,7 +529,11 @@ class Fixtures < (RUBY_VERSION < '1.9' ? YAML::Omap : Hash) # be a positive integer, and will always be the same for a given # label, assuming the same OS, platform, and version of Ruby. def self.identify(label) - label.to_s.hash.abs + crc = Zlib.crc32(label.to_s) + while (crc > 2147483646) do + crc -= 1000000000 + end + return crc end attr_reader :table_name, :name diff --git a/activerecord/test/cases/fixtures_test.rb b/activerecord/test/cases/fixtures_test.rb index 252bf4f..c8a3c51 100644 --- a/activerecord/test/cases/fixtures_test.rb +++ b/activerecord/test/cases/fixtures_test.rb @@ -517,6 +517,11 @@ class FoxyFixturesTest < ActiveRecord::TestCase def test_identifies_symbols assert_equal(Fixtures.identify(:foo), Fixtures.identify(:foo)) end + + def test_identifies_consistently + assert_equal 1428506893, Fixtures.identify(:ruby) + assert_equal 1287589245, Fixtures.identify(:sapphire_2) + end TIMESTAMP_COLUMNS = %w(created_at created_on updated_at updated_on) -- 1.6.1