fixed bug that didnt add feeds when they were deleted once

2 files changed, 28 insertions, 2 deletions

diff --git a/js/tests/services/models/feedmodelSpec.coffee b/js/tests/services/models/feedmodelSpec.coffee
index 2ae72eec2..08ddabc86 100644
--- a/js/tests/services/models/feedmodelSpec.coffee
+++ b/js/tests/services/models/feedmodelSpec.coffee
@@ -62,4 +62,18 @@ describe '_FeedModel', ->
 		model.add({id: 3, faviconLink: 'hey', urlHash: 'hi4'})
 		expect(model.size()).toBe(1)
 		expect(model.getById(2)).toBe(undefined)
-		expect(model.getById(3).faviconLink).toBe('hey')
\ No newline at end of file
+		expect(model.getById(3).faviconLink).toBe('hey')
+
+
+	it 'should also remove the feed from the urlHash cache when its removed', =>
+		utils =
+			imagePath: jasmine.createSpy('utils')
+		model = new @_FeedModel(utils)
+
+		item = {id: 2, faviconLink: null, urlHash: 'hi'}
+		model.add(item)
+
+		expect(model.getByUrlHash('hi')).toBe(item)
+
+		model.removeById(2)
+		expect(model.getByUrlHash('hi')).toBe(undefined)
\ No newline at end of file
diff --git a/js/tests/services/models/itemmodelSpec.coffee b/js/tests/services/models/itemmodelSpec.coffee
index 7d929b223..364c07b80 100644
--- a/js/tests/services/models/itemmodelSpec.coffee
+++ b/js/tests/services/models/itemmodelSpec.coffee
@@ -56,4 +56,16 @@ describe '_ItemModel', ->
 		expect(model.getById(3).feedId).toBe(item4.feedId)
 		expect(model.getById(3).id).toBe(item4.id)
 		expect(model.getById(5)).toBe(undefined)
-		expect(model.size()).toBe(2)
\ No newline at end of file
+		expect(model.size()).toBe(2)
+
+
+	it 'should also remove the feed from the urlHash cache when its removed', =>
+		model = new @_ItemModel()
+		item = {id: 4, guidHash: 'abc', feedId: 3}
+		model.add(item)
+
+		expect(model.getById(4)).toBe(item)
+		expect(model.getByGuidHashAndFeedId('abc', 3)).toBe(item)
+
+		model.removeById(4)
+		expect(model.getByGuidHashAndFeedId('abc', 3)).toBe(undefined)
\ No newline at end of file