[sitecore-jss-nextjs] Reject SDK promise when init was rejected

packages/create-sitecore-jss/src/templates/nextjs-xmcloud/src/lib/context/sdk/events.ts

@@ -6,8 +6,11 @@ const sdkModule: SDK<typeof Events> = {
   init: async (props) => {
     // Events module can't be initialized on the server side
     // We also don't want to initialize it in development mode
-    if (typeof window === 'undefined' || process.env.NODE_ENV === 'development') return;
-
+    if (typeof window === 'undefined') 
+      throw "Events SDK can't be initialized in server context";
+    if (process.env.NODE_ENV === 'development')
+      throw 'Events SDK is not initialized in development environment';
+
     await Events.init({
       siteName: props.siteName,
       sitecoreEdgeUrl: props.sitecoreEdgeUrl,

packages/sitecore-jss-nextjs/src/context/context.test.ts

@@ -28,6 +28,17 @@ describe('Context', () => {
     },
   };
 
+  const errorSdk = {
+    Error: {
+      sdk: { error: 'yes' },
+      init: () => {
+        return new Promise<void>((_, reject) => {
+          reject('Cannot init Error');
+        });
+      },
+    },
+  };
+
   const fooInitSpy = sinon.spy(sdks.Foo, 'init');
   const barInitSpy = sinon.spy(sdks.Bar, 'init');
 
@@ -159,6 +170,32 @@ describe('Context', () => {
 
       expect(context.siteName).to.equal('website-1');
     });
+
+    it('should catch and log when SDK initialization rejects', () => {
+      const consoleSpy = sinon.spy(console, 'log');
+      const localProps = { ...props, sdks: errorSdk };
+      const context = new Context<typeof errorSdk>(localProps);
+      context.init();
+      expect(
+        consoleSpy.calledWith('Initialization for SDK Error skipped. Reason: \n Cannot init Error')
+      );
+      consoleSpy.restore();
+    });
+
+    it('should reject when getting SDK that rejected initialization', (done) => {
+      const localProps = { ...props, sdks: errorSdk };
+      const context = new Context<typeof errorSdk>(localProps);
+      context.init();
+      context
+        .getSDK('Error')
+        .then(() => {
+          throw new Error('should not resolve');
+        })
+        .catch((e) => {
+          expect(e).to.be.equal('Cannot init Error');
+          done();
+        });
+    });
   });
 
   describe('getSDK', () => {

packages/sitecore-jss-nextjs/src/context/context.ts

@@ -137,12 +137,17 @@ export class Context<SDKModules extends SDKModulesType> {
    * @returns {void}
    */
   protected initSDK<T extends keyof SDKModules>(name: T): void {
-    this.sdkPromises[name] = new Promise((resolve) => {
-      this.props.sdks[name].init(this).then(() => {
-        this.sdks[name] = this.props.sdks[name].sdk;
-
-        resolve(this.sdks[name]);
-      });
+    this.sdkPromises[name] = new Promise((resolve, reject) => {
+      this.props.sdks[name]
+        .init(this)
+        .then(() => {
+          this.sdks[name] = this.props.sdks[name].sdk;
+          resolve(this.sdks[name]);
+        })
+        .catch((e) => {
+          console.log(`Initialization for SDK ${name.toString()} skipped. Reason: \n ${e}`);
+          reject(e);
+        });
     });
   }
 }