Lead by example: lint PORO model

test/fixtures/poro.rb

@@ -13,12 +13,8 @@ class Model
     "#{self.class.name.downcase}/#{self.id}-#{self.updated_at.strftime("%Y%m%d%H%M%S%9N")}"
   end
 
-  def cache_key_with_digest
-    "#{cache_key}/#{FILE_DIGEST}"
-  end
-
-  def updated_at
-    @attributes[:updated_at] ||= DateTime.now.to_time
+  def serializable_hash(options = nil)
+    @attributes
   end
 
   def read_attribute_for_serialization(name)
@@ -33,6 +29,9 @@ class Model
     @attributes[:id] || @attributes['id'] || object_id
   end
 
+  ### Helper methods, not required to be serializable
+  #
+  # Convenience for adding @attributes readers and writers
   def method_missing(meth, *args)
     if meth.to_s =~ /^(.*)=$/
       @attributes[$1.to_sym] = args[0]
@@ -42,6 +41,14 @@ class Model
       super
     end
   end
+
+  def cache_key_with_digest
+    "#{cache_key}/#{FILE_DIGEST}"
+  end
+
+  def updated_at
+    @attributes[:updated_at] ||= DateTime.now.to_time
+  end
 end
 
 class Profile < Model