とんでもない見落としをしていた。こんなテストコード書いてた。
fixtures :pay_types def test_record o = Order.new(:pay_type_id => pay_types(:one)) assert_equal o.pay_type, pay_types(:one) ... end
このテストは通りますが、やってはいけない書き方です。何がいけないかというと、「:pay_type_id => pay_types(:one)」の部分。
以下の例をみるとおそろしさが分かる。
Loading development environment. >> o = Order.new(:pay_type_id => PayType.find(2)) => #<Order:0x22f8080 @new_record=true, @attributes={"name"=>nil, "pay_type_id"=>#<PayType:0x22f8008 @attributes={"updated_at"=>"2008-01-12 12:20:07", "id"=>"2", "pay_type"=>"cc"}>, "address"=>nil, "email"=>nil}> >> o.pay_type_id => 1
「pay_type_id => PayType.find(2)」ってやったから、この結果は2になると思ってました。
type_castで変換される
何故直前の例の最後が2にならないかと言うと、キャストに関係があります。「o.pay_type_id」のように、カラムのデータへのアクセサを呼ぶと、内部的には「ActiveRecord::Base#read_attribute」メソッドが使われ、その中でキャストが行われます。
キャストはこんな感じ。
- ActiveRecord::ConnectionAdapters::Column#type_cast
# Casts value (which is a String) to an appropriate instance. def type_cast(value) return nil if value.nil? case type when :string then value when :text then value when :integer then value.to_i rescue value ? 1 : 0 when :float then value.to_f when :decimal then self.class.value_to_decimal(value) when :datetime then self.class.string_to_time(value) when :timestamp then self.class.string_to_time(value) when :time then self.class.string_to_dummy_time(value) when :date then self.class.string_to_date(value) when :binary then self.class.binary_to_string(value) when :boolean then self.class.value_to_boolean(value) else value end end
注目は「when :integer then value.to_i rescue value ? 1 : 0」の部分。pay_type_idの型はinteger(MySQLで)です。よって、キャストされる際（先程のテストの例で言えば、o.pay_type_idの際にキャストを受ける）には「to_i」の変換を受けます。しかし、value(この例では、PayTypeオブジェクト）は「to_i」を持たないので、例外が発生し、結果、1が格納されます。
先程のテストの改善
そもそも、先程のテストが何故通るかというと、「pay_types(:one).id」の結果がたまたま1だったからです。しかし、この1は「pay_types(:one).id」の1ではなく、キャストで発生した例外の結果でした。
実際は、次の様に書きます。
o = Order.new(:pay_type_id => pay_types(:one).id)
次のようにも書けます（あまりおすすめはしない）。
o = Order.new(:pay_type => pay_types(:one))
これは、内部では「o.pay_type = pay_types(:one)」と処理されるので、結果は上の書き方と同じになります。
まぁ、実際のコードでは「o = Order.new(:pay_type_id => pay_types(:one))」のように、オブジェクトをHashの形で指定することはないと思うのですが（Order.new(params[:order])とかが多いかと）、テストコードや、または、PayType(支払い方法)のように、決められた値だけを持つデータベースを使う場合は、このような書き方をしてしまうかもしれません。
