Gate getTopics on deviceAccess (#162)

Author: zapo

lib/addons/topics-api.test.js

@@ -2,6 +2,31 @@ import OptableSDK from "../sdk";
 import { TEST_HOST, TEST_SITE } from "../test/mocks.ts";
 import "./topics-api.ts";
 
+describe("OptableSDK - getTopics", () => {
+  let SDK;
+
+  beforeEach(() => {
+    SDK = new OptableSDK({ host: TEST_HOST, site: TEST_SITE });
+    // Mock the profile method
+    SDK.profile = jest.fn();
+  });
+
+  test("fails when browsing topics is not supported", async () => {
+    expect.assertions(1);
+    const expected = "browsing-topics not supported";
+    await expect(SDK.getTopics()).rejects.toBe(expected);
+  });
+
+  test("fails when device access is not granted", async () => {
+    expect.assertions(1);
+    const expected = "consent not granted for reading browsing topics";
+    document.browsingTopics = {};
+
+    SDK.dcn.consent.deviceAccess = false;
+    await expect(SDK.getTopics()).rejects.toBe(expected);
+  });
+});
+
 describe("OptableSDK - ingestTopics", () => {
   let SDK;
 

lib/addons/topics-api.ts

@@ -23,6 +23,10 @@ OptableSDK.prototype.getTopics = async function (): Promise<BrowsingTopic[]> {
     throw "browsing-topics not supported";
   }
 
+  if (!this.dcn.consent.deviceAccess) {
+    throw "consent not granted for reading browsing topics";
+  }
+
   const siteConfig = await this.site();
   if (!siteConfig.getTopicsURL) {
     throw "origin not enabled for topics api";