testing loaded? method and reducing funcalls

activerecord/lib/active_record/session_store.rb

@@ -259,7 +259,7 @@ def data
       end
 
       def loaded?
-        !!@data
+        @data
       end
 
       def save

activerecord/test/cases/session_store/sql_bypass.rb

@@ -22,6 +22,16 @@ def test_new_record?
         s = SqlBypass.new :data => 'foo', :session_id => 10
         assert s.new_record?, 'this is a new record!'
       end
+
+      def test_not_loaded?
+        s = SqlBypass.new({})
+        assert !s.loaded?, 'it is not loaded'
+      end
+
+      def test_loaded?
+        s = SqlBypass.new :data => 'hello'
+        assert s.loaded?, 'it is loaded'
+      end
     end
   end
 end