From b18f899bfafd5516f4b960772b4c903ead58d4ad Mon Sep 17 00:00:00 2001
From: Joenio Costa <joenio@colivre.coop.br>
Date: Sun, 22 Aug 2010 00:53:42 -0300
Subject: [PATCH] Storing time when external feed was fetched as datetime instead of date

---
 db/migrate/20100822034415_change_external_feed_fetched_at_column_type.rb | 13 +++++++++++++
 db/schema.rb                                                             |  4 ++--
 test/unit/external_feed_test.rb                                          | 13 +++++++++++++
 3 files changed, 28 insertions(+), 2 deletions(-)
 create mode 100644 db/migrate/20100822034415_change_external_feed_fetched_at_column_type.rb

diff --git a/db/migrate/20100822034415_change_external_feed_fetched_at_column_type.rb b/db/migrate/20100822034415_change_external_feed_fetched_at_column_type.rb
new file mode 100644
index 0000000..d7f23cf
--- /dev/null
+++ b/db/migrate/20100822034415_change_external_feed_fetched_at_column_type.rb
@@ -0,0 +1,13 @@
+class ChangeExternalFeedFetchedAtColumnType < ActiveRecord::Migration
+  def self.up
+    change_table :external_feeds do |t|
+      t.change :fetched_at, :datetime
+    end
+  end
+
+  def self.down
+    change_table :external_feeds do |t|
+      t.change :fetched_at, :date
+    end
+  end
+end
diff --git a/db/schema.rb b/db/schema.rb
index be31aa4..218b18c 100644
--- a/db/schema.rb
+++ b/db/schema.rb
@@ -9,7 +9,7 @@
 #
 # It's strongly recommended to check this file into your version control system.
 
-ActiveRecord::Schema.define(:version => 20100730141134) do
+ActiveRecord::Schema.define(:version => 20100822034415) do
 
   create_table "article_versions", :force => true do |t|
     t.integer  "article_id"
@@ -190,7 +190,7 @@ ActiveRecord::Schema.define(:version => 20100730141134) do
 
   create_table "external_feeds", :force => true do |t|
     t.string   "feed_title"
-    t.date     "fetched_at"
+    t.datetime "fetched_at"
     t.string   "address"
     t.integer  "blog_id",                         :null => false
     t.boolean  "enabled",       :default => true, :null => false
diff --git a/test/unit/external_feed_test.rb b/test/unit/external_feed_test.rb
index b4fa25a..5edb19e 100644
--- a/test/unit/external_feed_test.rb
+++ b/test/unit/external_feed_test.rb
@@ -128,4 +128,17 @@ class ExternalFeedTest < ActiveSupport::TestCase
     assert_equal 0, ExternalFeed.new.update_errors
   end
 
+  should 'save hour when feed was fetched' do
+    external_feed = create(:external_feed)
+
+    now = Time.parse('2009-01-23 09:35')
+    Time.stubs(:now).returns(now)
+
+    external_feed.finish_fetch
+    external_feed.reload
+
+    assert_equal 9, external_feed.fetched_at.hour
+    assert_equal 35, external_feed.fetched_at.min
+  end
+
 end
--
libgit2 0.21.2