Fixes issue Seneca-CDOT#462

c3ho · Feb 6, 2020 · c6f9f11 · c6f9f11
1 parent 4260501
commit c6f9f11
Show file tree

Hide file tree

Showing 2 changed files with 22 additions and 1 deletion.
diff --git a/src/backend/data/feed.js b/src/backend/data/feed.js
@@ -1,6 +1,6 @@
 const normalizeUrl = require('normalize-url');
 
-const { getFeed, addFeed } = require('../utils/storage');
+const { getFeed, addFeed, addInvalidFeed } = require('../utils/storage');
 const hash = require('./hash');
 
 const urlToId = url => hash(normalizeUrl(url));
@@ -30,6 +30,10 @@ class Feed {
     addFeed(this);
   }
 
+  saveInvalid() {
+    addInvalidFeed(this);
+  }
+
   /**
    * Creates a Feed by extracting data from the given feed-like object.
    * @param {Object} o - an Object containing the necessary fields for a feed

diff --git a/src/backend/utils/storage.js b/src/backend/utils/storage.js
@@ -3,6 +3,7 @@ const { redis } = require('../lib/redis');
 // Redis Keys
 const feedsKey = 't:feeds';
 const postsKey = 't:posts';
+const invalidKey = 't:feeds:invalid';
 
 // Namespaces
 const feedNamespace = 't:feed:';
@@ -47,6 +48,22 @@ module.exports = {
 
   getFeedsCount: () => redis.scard(feedsKey),
 
+  addInvalidFeed: async feed => {
+    const key = createFeedKey(feed.id);
+    await redis
+      .multi()
+      // Using hmset() until hset() fully supports multiple fields:
+      // https://github.com/stipsan/ioredis-mock/issues/345
+      // https://github.com/luin/ioredis/issues/551
+      .hmset(key, 'id', feed.id, 'author', feed.author, 'url', feed.url)
+      .sadd(invalidKey, feed.id)
+      .exec();
+  },
+
+  getInvalidFeeds: () => redis.smembers(invalidKey),
+
+  getInvalidCount: () => redis.scard(invalidKey),
+
   /**
    * Posts
    */