fixes regression from #192 and adds specs

Author: catmando

ruby/hyper-model/lib/reactive_record/active_record/class_methods.rb

@@ -342,6 +342,7 @@ def server_method(name, default: nil)
     def define_attribute_methods
       columns_hash.each do |name, column_hash|
         next if name == primary_key
+        column_hash[:serialized?] = ReactiveRecord::Base.serialized?[self][name]
         define_method(name) { @backing_record.get_attr_value(name, nil) }
         define_method("#{name}!") { @backing_record.get_attr_value(name, true) }
         define_method("#{name}=") { |val| @backing_record.set_attr_value(name, val) }

ruby/hyper-model/lib/reactive_record/active_record/reactive_record/dummy_value.rb

@@ -29,11 +29,11 @@ def initialize(column_hash = nil)
         column_type = Base.column_type(@column_hash) || 'nil'
         default_value_method = "build_default_value_for_#{column_type}"
         @object = __send__ default_value_method
-      rescue ::Exception => e
+      rescue ::Exception
       end
 
       def build_default_value_for_nil
-        @column_hash.key?(:default) ? @column_hash[:default] :  nil
+        @column_hash[:default] || nil
       end
 
       def build_default_value_for_datetime
@@ -56,23 +56,24 @@ def build_default_value_for_date
       end
 
       def build_default_value_for_boolean
-        @column_hash.key?(:default) ? @column_hash[:default] :  false
+        @column_hash[:default] || false
       end
 
       def build_default_value_for_float
-        @column_hash.key?(:default) ? @column_hash[:default] :  Float(0.0)
+        @column_hash[:default] || Float(0.0)
       end
 
       alias build_default_value_for_decimal build_default_value_for_float
 
       def build_default_value_for_integer
-        @column_hash.key?(:default) ? @column_hash[:default] :  Integer(0)
+        @column_hash[:default] || Integer(0)
       end
 
       alias build_default_value_for_bigint build_default_value_for_integer
 
       def build_default_value_for_string
-        @column_hash.key?(:default) ? @column_hash[:default] :  ''
+        return @column_hash[:default] if @column_hash[:serialized?]
+        @column_hash[:default] || ''
       end
 
       alias build_default_value_for_text build_default_value_for_string

ruby/hyper-model/spec/batch1/column_types/column_type_spec.rb

@@ -145,6 +145,16 @@ class DefaultTest < ActiveRecord::Base
     check_errors
   end
 
+  it 'will not override a default of nil if the attribute is serialized' do
+    isomorphic do
+      TypeTest.serialize :string
+      TypeTest.serialize :text
+    end
+    [:string, :text].each do |attr|
+      expect_evaluate_ruby("TypeTest.find(1).#{attr}.class").to eq('NilClass')
+    end
+  end
+
   it 'while loading the dummy value delegates the correct type with operators etc' do
     t = Time.parse('1/2/2003')
     TypeTest.create(
@@ -293,6 +303,15 @@ class DefaultTest < ActiveRecord::Base
     check_errors
   end
 
+  it 'uses the default value if even if the attribute is serialized' do
+    isomorphic do
+      DefaultTest.serialize :string
+    end
+    expect_evaluate_ruby do
+      DefaultTest.find(1).string
+    end.to eq("I'm a string!")
+  end
+
   it 'uses the default value if specified when initializing a new record' do
     expect_evaluate_ruby do
       DefaultTest.new.string