feat: CXSPA-6890 Create toggle and optimization options for propagati…

…ng errors to the server (#19021)

core-libs/setup/ssr/error-handling/multi-error-handlers/propagating-to-server-error-handler.spec.ts

@@ -1,39 +1,56 @@
 import { TestBed } from '@angular/core/testing';
+import { FeatureConfigService } from '@spartacus/core';
 import { PROPAGATE_ERROR_TO_SERVER } from '../error-response/propagate-error-to-server';
 import { PropagatingToServerErrorHandler } from './propagating-to-server-error-handler';
 
 describe('PropagatingToServerErrorHandler', () => {
  describe('default factories', () => {
- let serverRespondingErrorHandler: PropagatingToServerErrorHandler;
+ let propagatingToServerErrorHandler: PropagatingToServerErrorHandler;
+ let featureConfigService: FeatureConfigService;
  let propagateErrorResponse: any;
 
  beforeEach(() => {
  TestBed.configureTestingModule({
  providers: [
  PropagatingToServerErrorHandler,
+ FeatureConfigService,
  {
  provide: PROPAGATE_ERROR_TO_SERVER,
  useValue: jest.fn(),
  },
  ],
  });
-
- serverRespondingErrorHandler = TestBed.inject(
+ propagatingToServerErrorHandler = TestBed.inject(
  PropagatingToServerErrorHandler
  );
  propagateErrorResponse = TestBed.inject(PROPAGATE_ERROR_TO_SERVER);
+ featureConfigService = TestBed.inject(FeatureConfigService);
  });
 
  afterEach(() => {
  jest.clearAllMocks();
  });
 
- it('should propagate error', () => {
+ it('should propagate error when propagateErrorsToServer is enabled', () => {
+ jest
+ .spyOn(featureConfigService, 'isEnabled')
+ .mockImplementationOnce((val) => val === 'propagateErrorsToServer');
  const error = new Error('test error');
 
- serverRespondingErrorHandler.handleError(error);
+ propagatingToServerErrorHandler.handleError(error);
 
  expect(propagateErrorResponse as jest.Mock).toHaveBeenCalledWith(error);
  });
+
+ it('should not propagate error when propagateErrorsToServer is disabled', () => {
+ jest
+ .spyOn(featureConfigService, 'isEnabled')
+ .mockImplementationOnce((val) => !(val === 'propagateErrorsToServer'));
+ const error = new Error('test error');
+
+ propagatingToServerErrorHandler.handleError(error);
+
+ expect(propagateErrorResponse as jest.Mock).not.toHaveBeenCalled();
+ });
  });
 });

core-libs/setup/ssr/error-handling/multi-error-handlers/propagating-to-server-error-handler.ts

@@ -5,7 +5,7 @@
  */
 
 import { Injectable, inject } from '@angular/core';
-import { MultiErrorHandler } from '@spartacus/core';
+import { FeatureConfigService, MultiErrorHandler } from '@spartacus/core';
 import { PROPAGATE_ERROR_TO_SERVER } from '../error-response/propagate-error-to-server';
 
 /**
@@ -28,8 +28,13 @@ import { PROPAGATE_ERROR_TO_SERVER } from '../error-response/propagate-error-to-
 })
 export class PropagatingToServerErrorHandler implements MultiErrorHandler {
  protected propagateErrorToServer = inject(PROPAGATE_ERROR_TO_SERVER);
+ private featureConfigService: FeatureConfigService =
+ inject(FeatureConfigService);
 
  handleError(error: unknown): void {
+ if (!this.featureConfigService.isEnabled('propagateErrorsToServer')) {
+ return;
+ }
  this.propagateErrorToServer(error);
  }
 }

core-libs/setup/ssr/optimized-engine/index.ts

@@ -6,6 +6,7 @@
 
 export * from './optimized-ssr-engine';
 export * from './rendering-cache';
+export * from './rendering-cache.model';
 export * from './rendering-strategy-resolver';
 export * from './rendering-strategy-resolver-options';
 export { RequestContext, getRequestContext } from './request-context';