Merge branch 'master' of github.com:swanson/stringer

Matt Swanson · Matt Swanson · commit 2ecc679ec088 · 2013-08-12T17:56:45.000-04:00
diff --git a/app/commands/stories/mark_group_as_read.rb b/app/commands/stories/mark_group_as_read.rb
@@ -1,14 +1,19 @@
 require_relative "../../repositories/story_repository"
 
 class MarkGroupAsRead
+  KINDLING_GROUP_ID            = 1
+  SPARKS_AND_KINDLING_GROUP_ID = 0
+
   def initialize(group_id, timestamp, repository = StoryRepository)
     @group_id  = group_id.to_i
     @repo      = repository
     @timestamp = timestamp
   end
 
   def mark_group_as_read
-    @repo.fetch_unread_by_timestamp(@timestamp).update_all(is_read: true) if @group_id == 1
+    if [SPARKS_AND_KINDLING_GROUP_ID, KINDLING_GROUP_ID].include? @group_id
+      @repo.fetch_unread_by_timestamp(@timestamp).update_all(is_read: true)
+    end
   end
 end
 
diff --git a/spec/commands/stories/mark_group_as_read_spec.rb b/spec/commands/stories/mark_group_as_read_spec.rb
@@ -7,6 +7,12 @@
     let(:stories) { stub }
     let(:repo){ stub(fetch_unread_by_timestamp: stories) }
 
+    it "marks group 0 as read" do
+      command = MarkGroupAsRead.new(0, Time.now.to_i, repo)
+      stories.should_receive(:update_all).with(is_read: true)
+      command.mark_group_as_read
+    end
+
     it "marks group 1 as read" do
       command = MarkGroupAsRead.new(1, Time.now.to_i, repo)
       stories.should_receive(:update_all).with(is_read: true)
