fix: move to kv for configs

Author: maxakuru

src/config.js

@@ -10,29 +10,6 @@
  * governing permissions and limitations under the License.
  */
 
-/**
- * @type {Record<string, Record<string, Config>>}
- */
-// const TENANT_CONFIGS = {
-//   visualcomfort: {
-//     base: {
-//       apiKey: '59878b5d8af24fe9a354f523f5a0bb62',
-//       magentoEnvironmentId: '97034e45-43a5-48ab-91ab-c9b5a98623a8',
-//       magentoWebsiteCode: 'base',
-//       magentoStoreViewCode: 'default',
-//       coreEndpoint: 'https://www.visualcomfort.com/graphql',
-//     },
-//     '/us/p/{{urlkey}}/{{sku}}': {
-//       pageType: 'product',
-//       apiKey: '59878b5d8af24fe9a354f523f5a0bb62',
-//       magentoEnvironmentId: '97034e45-43a5-48ab-91ab-c9b5a98623a8',
-//       magentoWebsiteCode: 'base',
-//       magentoStoreViewCode: 'default',
-//       coreEndpoint: 'https://www.visualcomfort.com/graphql',
-//     },
-//   },
-// };
-
 /**
  * @param {string[]} patterns
  * @param {string} path
@@ -63,11 +40,13 @@ function extractPathParams(pattern, path) {
  */
 export async function resolveConfig(ctx, tenant, overrides = {}) {
   const confMap = await ctx.env.CONFIGS.get(tenant, 'json');
-  console.log('confMap: ', tenant, confMap, confMap);
-  console.log('config list: ', await ctx.env.CONFIGS.list());
   if (!confMap) {
     return null;
   }
+  if (typeof confMap !== 'object') {
+    ctx.log.warn('invalid config for tenant', tenant);
+    return null;
+  }
 
   // order paths by preference
   const suffix = `/${ctx.url.pathname.split('/').slice(3).join('/')}`;

test/config.test.js

@@ -15,10 +15,15 @@ import { resolveConfig } from '../src/config.js';
 
 /**
  * @param {string} path
+ * @param {Record<string, Config>} configMap
  * @returns {Context}
  */
-const TEST_CONTEXT = (path) => ({
-  env: {},
+const TEST_CONTEXT = (path, configMap) => ({
+  env: {
+    CONFIGS: {
+      get: async (tenant) => configMap[tenant],
+    },
+  },
   log: console,
   url: new URL(`https://www.example.com/tenant/content${path}`),
   info: {
@@ -28,7 +33,7 @@ const TEST_CONTEXT = (path) => ({
 });
 
 describe('config tests', () => {
-  it('should extract path params', () => {
+  it('should extract path params', async () => {
     const tenantConfigs = {
       'test-tenant': {
         base: {
@@ -40,15 +45,18 @@ describe('config tests', () => {
         },
       },
     };
-    const config = resolveConfig(TEST_CONTEXT('/us/p/my-url-key/some-sku'), 'test-tenant', undefined, tenantConfigs);
+    const config = await resolveConfig(
+      TEST_CONTEXT('/us/p/my-url-key/some-sku', tenantConfigs),
+      'test-tenant',
+    );
     assert.deepStrictEqual(config, {
       apiKey: 'good',
       params: { urlkey: 'my-url-key', sku: 'some-sku' },
       pageType: 'product',
     });
   });
 
-  it('should allow wildcard path segments', () => {
+  it('should allow wildcard path segments', async () => {
     const tenantConfigs = {
       'test-tenant': {
         base: {
@@ -60,15 +68,18 @@ describe('config tests', () => {
         },
       },
     };
-    const config = resolveConfig(TEST_CONTEXT('/us/p/something-here/some-sku'), 'test-tenant', undefined, tenantConfigs);
+    const config = await resolveConfig(
+      TEST_CONTEXT('/us/p/something-here/some-sku', tenantConfigs),
+      'test-tenant',
+    );
     assert.deepStrictEqual(config, {
       apiKey: 'good',
       params: { sku: 'some-sku' },
       pageType: 'product',
     });
   });
 
-  it('should allow overrides', () => {
+  it('should allow overrides', async () => {
     const tenantConfigs = {
       'test-tenant': {
         base: {
@@ -80,7 +91,11 @@ describe('config tests', () => {
         },
       },
     };
-    const config = resolveConfig(TEST_CONTEXT('/us/p/some-sku'), 'test-tenant', { apiKey: 'good' }, tenantConfigs);
+    const config = await resolveConfig(
+      TEST_CONTEXT('/us/p/some-sku', tenantConfigs),
+      'test-tenant',
+      { apiKey: 'good' },
+    );
     assert.deepStrictEqual(config, {
       apiKey: 'good',
       params: { sku: 'some-sku' },