@quintype/framework 7.10.2 → 7.10.5-infinite-scroll-tests.0

package/CHANGELOG.md

@@ -2,7 +2,22 @@
 
 All notable changes to this project will be documented in this file. See [standard-version](https://github.com/conventional-changelog/standard-version) for commit guidelines.
 
-### [7.10.2](https://github.com/quintype/quintype-node-framework/compare/v7.8.1...v7.10.2) (2022-10-04)
+### [7.10.4](https://github.com/quintype/quintype-node-framework/compare/v7.10.3...v7.10.4) (2022-10-04)
+
+## [7.10.0](https://github.com/quintype/quintype-node-framework/compare/v7.7.7...v7.10.0) (2022-10-04)
+
+
+### Features
+
+* add video support to web stories ([#322](https://github.com/quintype/quintype-node-framework/issues/322)) ([f41b82b](https://github.com/quintype/quintype-node-framework/commit/f41b82be8285b555970527204ec6478f21c47782))
+* **SW:** Disable sw shell  ([#321](https://github.com/quintype/quintype-node-framework/issues/321)) ([6d04bc8](https://github.com/quintype/quintype-node-framework/commit/6d04bc87dd42c647ec9763eca430ada2f4a4d8ea))
+
+## [7.9.0](https://github.com/quintype/quintype-node-framework/compare/v7.8.0...v7.9.0) (2022-10-03)
+
+
+### Features
+
+* **SW:** Disable sw shell  ([#321](https://github.com/quintype/quintype-node-framework/issues/321)) ([6d04bc8](https://github.com/quintype/quintype-node-framework/commit/6d04bc87dd42c647ec9763eca430ada2f4a4d8ea))
 
 ## [7.8.0](https://github.com/quintype/quintype-node-framework/compare/v7.7.7...v7.8.0) (2022-09-19)
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@quintype/framework",
-  "version": "7.10.2",
+  "version": "7.10.5-infinite-scroll-tests.0",
   "description": "Libraries to help build Quintype Node.js apps",
   "main": "index.js",
   "engines": {

package/server/amp/helpers/infinite-scroll.js

@@ -57,7 +57,7 @@ class InfiniteScrollAmp {
     };
     if (this.infiniteScrollSource === "relatedStoriesApi") {
       const relatedStoriesList = await this.client.getRelatedStories(storyId, null, params);
-      if (!relatedStoriesList)
+      if (!relatedStoriesList || (relatedStoriesList["related-stories"] && !relatedStoriesList["related-stories"].length) || relatedStoriesList["related-stories"].error || relatedStoriesList === null)
         return new Error();
       return filteredItems = this.getFilteredApiItems(relatedStoriesList["related-stories"]);
     } else {

package/test/unit/amp/infinite-scroll-helper.js

@@ -3,17 +3,8 @@
 
 const assert = require("assert");
 const InfiniteScrollAmp = require("../../../server/amp/helpers/infinite-scroll");
-const { getTextStory } = require("../../data/amp-test-data");
 
 function getClientStub({
-  getStoryById = (id) =>
-    new Promise((resolve) => {
-      if (id === "4444")
-        resolve({
-          story: getTextStory({ "story-content-id": "7f3d5bdb-ec52-4047-ac0d-df4036ec974b" }),
-        });
-      resolve(null);
-    }),
   getCollectionBySlug = (slug) =>
     new Promise((resolve) => {
       if (slug === "amp-infinite-scroll")
@@ -122,7 +113,156 @@ function getClientStub({
 } = {}) {
   return {
     getCollectionBySlug,
-    getStoryById
+  };
+}
+
+function getRelatedStoriesClientStub({
+  getRelatedStories = () =>
+    new Promise((resolve) => {
+      resolve({
+        "related-stories": [
+          {
+            "story-template": "text",
+            headline: "aaa",
+            "story-content-id": 1111,
+            slug: "sports/aa",
+            "hero-image-s3-key": "aa/a.jpg",
+            access: "public",
+          },
+          {
+            "story-template": "visual-story",
+            headline: "bbb",
+            "story-content-id": 2222,
+            slug: "sports/bb",
+            "hero-image-s3-key": "bb/b.jpg",
+            access: "public",
+          },
+          {
+            "story-template": "text",
+            headline: "ccc",
+            "story-content-id": 3333,
+            slug: "sports/cc",
+            "hero-image-s3-key": "cc/c.jpg",
+            access: "public",
+          },
+          {
+            "story-template": "text",
+            headline: "ddd",
+            "story-content-id": 4444,
+            slug: "sports/dd",
+            "hero-image-s3-key": "dd/d.jpg",
+            access: "public",
+          },
+          {
+            "story-template": "text",
+            headline: "eee",
+            "story-content-id": 5555,
+            slug: "politics/ee",
+            "hero-image-s3-key": "ee/e.jpg",
+            access: "public",
+          },
+          {
+            "story-template": "text",
+            headline: "fff",
+            "story-content-id": 6666,
+            slug: "politics/ff",
+            "hero-image-s3-key": "ff/f.jpg",
+            access: "public",
+          },
+          {
+            "story-template": "text",
+            headline: "ggg",
+            "story-content-id": 7777,
+            slug: "politics/gg",
+            "hero-image-s3-key": "gg/g.jpg",
+            access: "public",
+          },
+          {
+            "story-template": "text",
+            headline: "hhh",
+            "story-content-id": 8888,
+            slug: "politics/hh",
+            "hero-image-s3-key": "hh/h.jpg",
+            access: "public",
+          },
+        ],
+      });
+    }),
+} = {}) {
+  return {
+    getRelatedStories,
+  };
+}
+
+function getCustomApiStoriesClientStub({
+  getCustomApiStories = () =>
+    new Promise((resolve) => {
+      resolve(
+        [
+          {
+            title: "aaa",
+            url: "sports/aa",
+            image: "aa/a.jpg",
+          },
+          {
+            title: "bbb",
+            url: "sports/bb",
+            image: "bb/b.jpg",
+          },
+          {
+            title: "ccc",
+            url: "sports/cc",
+            image: "cc/c.jpg",
+          },
+          {
+            title: "ddd",
+            url: "sports/dd",
+            image: "dd/d.jpg",
+          },
+          {
+            title: "eee",
+            url: "politics/ee",
+            image: "ee/e.jpg",
+          },
+          {
+            title: "fff",
+            url: "politics/ff",
+            image: "ff/f.jpg",
+          },
+          {
+            title: "ggg",
+            url: "politics/gg",
+            image: "gg/g.jpg",
+          },
+          {
+            title: "hhh",
+            url: "politics/hh",
+            image: "hh/h.jpg",
+          },
+        ]
+      );
+    }),
+} = {}) {
+  return {
+    getCustomApiStories,
+  };
+}
+class CustomInfiniteScrollAmp {
+  constructor({ client, publisherConfig, queryParams, infiniteScrollSource }) {
+    this.client = client;
+    this.publisherConfig = publisherConfig;
+    this.queryParams = queryParams;
+    this.infiniteScrollSource = infiniteScrollSource;
+  }
+
+  async getCustomStoryList({ type = "inlineConfig" }) {
+    const customStories = await this.client.getCustomApiStories();
+    if (!customStories || customStories === null || customStories.length === 0)
+      return new Error();
+    if (type === "remoteConfig") {
+      return JSON.stringify({ pages: customStories });
+    }
+    return JSON.stringify(customStories);
   };
 }
 const dummyPublisherConfig = {
@@ -130,6 +270,7 @@ const dummyPublisherConfig = {
 };
 
 describe("getInitialInlineConfig method of InfiniteScrollAmp helper function", function () {
+  // collection
   it("should throw err if storyId isn't passed", async function () {
     const infiniteScrollAmp = new InfiniteScrollAmp({
       ampConfig: {},
@@ -155,7 +296,7 @@ describe("getInitialInlineConfig method of InfiniteScrollAmp helper function", f
       publisherConfig: dummyPublisherConfig,
     });
     const inlineConfig = await infiniteScrollAmp.getInitialInlineConfig({
-      storyId: 1111,
+      storyId: 2222,
     });
     assert.strictEqual(inlineConfig, null);
   });
@@ -174,7 +315,7 @@ describe("getInitialInlineConfig method of InfiniteScrollAmp helper function", f
       publisherConfig: dummyPublisherConfig,
     });
     const inlineConfig = await infiniteScrollAmp.getInitialInlineConfig({
-      storyId: 1111,
+      storyId: 2222,
     });
     assert.strictEqual(inlineConfig, null);
   });
@@ -186,7 +327,7 @@ describe("getInitialInlineConfig method of InfiniteScrollAmp helper function", f
       publisherConfig: dummyPublisherConfig,
     });
     const inlineConfig = await infiniteScrollAmp.getInitialInlineConfig({
-      storyId: 1111,
+      storyId: 2222,
     });
     assert.strictEqual(false, /sports\/bb/.test(inlineConfig));
     assert.strictEqual(false, /bb\/b.jpg/.test(inlineConfig));
@@ -199,7 +340,7 @@ describe("getInitialInlineConfig method of InfiniteScrollAmp helper function", f
       publisherConfig: dummyPublisherConfig,
     });
     const inlineConfig = await infiniteScrollAmp.getInitialInlineConfig({
-      storyId: 1111,
+      storyId: 3333,
     });
     assert.strictEqual(false, /sports\/bb/.test(inlineConfig));
     assert.strictEqual(false, /bb\/b.jpg/.test(inlineConfig));
@@ -213,7 +354,7 @@ describe("getInitialInlineConfig method of InfiniteScrollAmp helper function", f
       publisherConfig: dummyPublisherConfig,
     });
     const inlineConfig = await infiniteScrollAmp.getInitialInlineConfig({
-      storyId: 1111,
+      storyId: 2222,
     });
     function isInlineConfigStructureValid(jsonStr) {
       const stories = JSON.parse(jsonStr);
@@ -230,9 +371,166 @@ describe("getInitialInlineConfig method of InfiniteScrollAmp helper function", f
   // it("should take the first 'n' stories", async function() {
   //   // this test needs to be written after https://github.com/quintype/quintype-node-framework/pull/202 is merged
   // })
+  /// related-stories
+  it("should throw err if storyId isn't passed for relatedStoriesApi", async function () {
+    const infiniteScrollAmp = new InfiniteScrollAmp({
+      ampConfig: {},
+      client: getRelatedStoriesClientStub(),
+      publisherConfig: dummyPublisherConfig,
+      infiniteScrollSource: "relatedStoriesApi",
+    });
+    const inlineConfig = await infiniteScrollAmp.getInitialInlineConfig({ storyId: "" });
+    assert.strictEqual(inlineConfig instanceof Error, true);
+    assert.throws(() => {
+      throw new Error("Required params for getInitialInlineConfig missing");
+    }, inlineConfig);
+  });
+
+  it("should return null if relatedStoriesApi infinite scroll collection doesn't exist", async function () {
+    const clientStub = getRelatedStoriesClientStub({
+      getRelatedStories: () =>
+        new Promise((resolve) => {
+          resolve(null);
+        }),
+    });
+    const infiniteScrollAmp = new InfiniteScrollAmp({
+      ampConfig: {},
+      client: clientStub,
+      publisherConfig: dummyPublisherConfig,
+      infiniteScrollSource: "relatedStoriesApi",
+    });
+    const inlineConfig = await infiniteScrollAmp.getInitialInlineConfig({
+      storyId: 2222,
+    });
+    assert.strictEqual(inlineConfig, null);
+  });
+
+  it("should return null if relatedStoriesApi infinite scroll collection contains no stories", async function () {
+    const clientStub = getRelatedStoriesClientStub({
+      getRelatedStories: () =>
+        new Promise((resolve) => {
+          resolve({
+            "related-stories": [],
+          });
+        }),
+    });
+    const infiniteScrollAmp = new InfiniteScrollAmp({
+      ampConfig: {},
+      client: clientStub,
+      publisherConfig: dummyPublisherConfig,
+      infiniteScrollSource: "relatedStoriesApi",
+    });
+    const inlineConfig = await infiniteScrollAmp.getInitialInlineConfig({
+      storyId: 2222,
+    });
+    assert.strictEqual(inlineConfig, null);
+  });
+
+  it("should remove visual stories from relatedStoriesApi infinite scroll", async function () {
+    const clientStub = getRelatedStoriesClientStub();
+    const infiniteScrollAmp = new InfiniteScrollAmp({
+      ampConfig: {},
+      client: clientStub,
+      publisherConfig: dummyPublisherConfig,
+      infiniteScrollSource: "relatedStoriesApi",
+    });
+    const inlineConfig = await infiniteScrollAmp.getInitialInlineConfig({
+      storyId: 3333,
+    });
+
+    assert.strictEqual(false, /sports\/bb/.test(inlineConfig));
+    assert.strictEqual(false, /bb\/b.jpg/.test(inlineConfig));
+  });
+
+  it("Related Stories Response should be in JSON format as per AMP spec", async function () {
+    // https://amp.dev/documentation/components/amp-next-page/
+    const clientStub = getRelatedStoriesClientStub();
+    const infiniteScrollAmp = new InfiniteScrollAmp({
+      ampConfig: {},
+      client: clientStub,
+      publisherConfig: dummyPublisherConfig,
+      infiniteScrollSource: "relatedStoriesApi",
+    });
+    const inlineConfig = await infiniteScrollAmp.getInitialInlineConfig({
+      storyId: 2222,
+    });
+    function isInlineConfigStructureValid(jsonStr) {
+      const stories = JSON.parse(jsonStr);
+      if (!stories.length) throw new Error("Can't verify empty array!");
+      stories.forEach((story) => {
+        const keys = Object.keys(story);
+        if (keys.includes("image") && keys.includes("url") && keys.includes("title") && keys.length === 3) return;
+        throw new Error("Invalid InlineConfigStructure");
+      });
+      return true;
+    }
+    assert.strictEqual(isInlineConfigStructureValid(inlineConfig), true);
+  });
+
+  /// custom-stories
+  it("should return null if CustomApi Stories infinite scroll collection doesn't exist", async function () {
+    const clientStub = getCustomApiStoriesClientStub({
+      getCustomApiStories: () =>
+        new Promise((resolve) => {
+          resolve(null);
+        }),
+    });
+    const customInfiniteScrollAmp = new CustomInfiniteScrollAmp({
+      client: clientStub,
+      publisherConfig: dummyPublisherConfig,
+      infiniteScrollSource: "custom",
+    });
+    const inlineConfig = await customInfiniteScrollAmp.getCustomStoryList({
+      type: "inlineConfig",
+    });
+    assert.strictEqual(inlineConfig instanceof Error, true);
+  });
+
+  it("should return null if CustomApi Stories infinite scroll collection contains no stories", async function () {
+    const clientStub = getCustomApiStoriesClientStub({
+      getCustomApiStories: () =>
+        new Promise((resolve) => {
+          resolve([]);
+        }),
+    });
+    const customInfiniteScrollAmp = new CustomInfiniteScrollAmp({
+      client: clientStub,
+      publisherConfig: dummyPublisherConfig,
+      infiniteScrollSource: "custom",
+    });
+    const inlineConfig = await customInfiniteScrollAmp.getCustomStoryList({
+      type: "inlineConfig",
+    });
+    assert.strictEqual(inlineConfig instanceof Error, true);
+  });
+
+  it("CustomApi Stories Response should be in JSON format as per AMP spec", async function () {
+    // https://amp.dev/documentation/components/amp-next-page/
+    const clientStub = getCustomApiStoriesClientStub();
+    const customInfiniteScrollAmp = new CustomInfiniteScrollAmp({
+      client: clientStub,
+      publisherConfig: dummyPublisherConfig,
+      infiniteScrollSource: "custom",
+    });
+    const inlineConfig = await customInfiniteScrollAmp.getCustomStoryList({
+      type: "inlineConfig",
+    });
+    function isInlineConfigStructureValid(jsonStr) {
+      const stories = JSON.parse(jsonStr);
+      if (!stories.length) throw new Error("Can't verify empty array!");
+      stories.forEach((story) => {
+        const keys = Object.keys(story);
+        if (keys.includes("image") && keys.includes("url") && keys.includes("title") && keys.length === 3) return;
+        throw new Error("Invalid InlineConfigStructure");
+      });
+      return true;
+    }
+    assert.strictEqual(isInlineConfigStructureValid(inlineConfig), true);
+  });
 });
 
 describe("getResponse method of InfiniteScrollAmp helper function", function () {
+  // collection
   it("should throw an error if 'story-id' isn't passed as query param", async function () {
     const infiniteScrollAmp = new InfiniteScrollAmp({
       ampConfig: {},
@@ -296,7 +594,6 @@ describe("getResponse method of InfiniteScrollAmp helper function", function ()
       queryParams: { "story-id": 4444 },
     });
     const jsonResponse = await infiniteScrollAmp.getResponse();
-
     function isJsonConfigStructureValid(jsonStr) {
       const parsed = JSON.parse(jsonStr);
       const stories = parsed.pages;
@@ -313,4 +610,120 @@ describe("getResponse method of InfiniteScrollAmp helper function", function ()
   // it("should omit the first 'n' stories, take the rest", async function() {
   //   // this test needs to be written after https://github.com/quintype/quintype-node-framework/pull/202 is merged
   // })
+  // related-stories
+  it("should throw an error if 'story-id' isn't passed as query param in relatedStoriesApi", async function () {
+    const infiniteScrollAmp = new InfiniteScrollAmp({
+      ampConfig: {},
+      client: getRelatedStoriesClientStub(),
+      publisherConfig: dummyPublisherConfig,
+      queryParams: { foo: "bar" },
+      infiniteScrollSource: "relatedStoriesApi",
+    });
+    const jsonResponse = await infiniteScrollAmp.getResponse();
+    assert.strictEqual(jsonResponse instanceof Error, true);
+    assert.throws(() => {
+      throw new Error(`Query param "story-id" missing`);
+    }, jsonResponse);
+  });
+
+  it("should throw an error if relatedStoriesApi infinite scroll collection doesn't exist", async function () {
+    const clientStub = getRelatedStoriesClientStub({
+      getRelatedStories: () =>
+        new Promise((resolve) => {
+          resolve(null);
+        }),
+    });
+    const infiniteScrollAmp = new InfiniteScrollAmp({
+      ampConfig: {},
+      client: clientStub,
+      publisherConfig: dummyPublisherConfig,
+      queryParams: { "story-id": 2222 },
+      infiniteScrollSource: "relatedStoriesApi",
+    });
+    const jsonResponse = await infiniteScrollAmp.getResponse();
+    assert.strictEqual(jsonResponse instanceof Error, true);
+  });
+
+  it("should remove visual stories from relatedStoriesApi response", async function () {
+    const clientStub = getRelatedStoriesClientStub();
+    const infiniteScrollAmp = new InfiniteScrollAmp({
+      ampConfig: {},
+      client: clientStub,
+      publisherConfig: dummyPublisherConfig,
+      queryParams: { "story-id": 4444 },
+      infiniteScrollSource: "relatedStoriesApi",
+    });
+    const jsonResponse = await infiniteScrollAmp.getResponse();
+    assert.strictEqual(false, /sports\/bb/.test(jsonResponse));
+    assert.strictEqual(false, /bb\/b.jpg/.test(jsonResponse));
+  });
+
+  it("should format JSON as per AMP spec in relatedStoriesApi", async function () {
+    // https://amp.dev/documentation/components/amp-next-page/
+    const clientStub = getRelatedStoriesClientStub();
+    const infiniteScrollAmp = new InfiniteScrollAmp({
+      ampConfig: {},
+      client: clientStub,
+      publisherConfig: dummyPublisherConfig,
+      queryParams: { "story-id": 4444 },
+      infiniteScrollSource: "relatedStoriesApi",
+    });
+    const jsonResponse = await infiniteScrollAmp.getResponse();
+    function isJsonConfigStructureValid(jsonStr) {
+      const parsed = JSON.parse(jsonStr);
+      const stories = parsed.pages;
+      if (!stories.length) throw new Error("Can't verify empty array!");
+      stories.forEach((story) => {
+        const keys = Object.keys(story);
+        if (keys.includes("image") && keys.includes("url") && keys.includes("title") && keys.length === 3) return;
+        throw new Error("Invalid InlineConfigStructure");
+      });
+      return true;
+    }
+    assert.strictEqual(isJsonConfigStructureValid(jsonResponse), true);
+  });
+
+  // custom-stories
+  it("should throw an error if customApi infinite scroll collection doesn't exist", async function () {
+    const clientStub = getCustomApiStoriesClientStub({
+      getCustomApiStories: () =>
+        new Promise((resolve) => {
+          resolve(null);
+        }),
+    });
+    const customInfiniteScrollAmp = new CustomInfiniteScrollAmp({
+      client: clientStub,
+      publisherConfig: dummyPublisherConfig,
+      infiniteScrollSource: "custom",
+    });
+    const jsonResponse = await customInfiniteScrollAmp.getCustomStoryList({
+      type: "remoteConfig",
+    });
+    assert.strictEqual(jsonResponse instanceof Error, true);
+  });
+
+  it("should format JSON as per AMP spec in customApi list", async function () {
+    // https://amp.dev/documentation/components/amp-next-page/
+    const clientStub = getCustomApiStoriesClientStub();
+    const customInfiniteScrollAmp = new CustomInfiniteScrollAmp({
+      client: clientStub,
+      publisherConfig: dummyPublisherConfig,
+      infiniteScrollSource: "custom",
+    });
+    const jsonResponse = await customInfiniteScrollAmp.getCustomStoryList({
+      type: "remoteConfig",
+    });
+    function isJsonConfigStructureValid(jsonStr) {
+      const parsed = JSON.parse(jsonStr);
+      const stories = parsed.pages;
+      if (!stories.length) throw new Error("Can't verify empty array!");
+      stories.forEach((story) => {
+        const keys = Object.keys(story);
+        if (keys.includes("image") && keys.includes("url") && keys.includes("title") && keys.length === 3) return;
+        throw new Error("Invalid InlineConfigStructure");
+      });
+      return true;
+    }
+    assert.strictEqual(isJsonConfigStructureValid(jsonResponse), true);
+  });
 });