perf(component): do not schedule render for synchronous events (#3487)

ngrx · Jul 25, 2022 · bb9071c · bb9071c
1 parent 398fbed
commit bb9071c
Show file tree

Hide file tree

Showing 6 changed files with 244 additions and 30 deletions.
diff --git a/modules/component/spec/core/render-event/handlers.spec.ts b/modules/component/spec/core/render-event/handlers.spec.ts
@@ -30,26 +30,30 @@ describe('combineRenderEventHandlers', () => {
   const suspenseEvent: SuspenseRenderEvent = {
     type: 'suspense',
     reset: true,
+    synchronous: true,
   };
   testRenderEvent(suspenseEvent);
 
   const nextEvent: NextRenderEvent<number> = {
     type: 'next',
     value: 1,
     reset: true,
+    synchronous: false,
   };
   testRenderEvent(nextEvent);
 
   const errorEvent: ErrorRenderEvent = {
     type: 'error',
     error: 'ERROR!',
     reset: false,
+    synchronous: true,
   };
   testRenderEvent(errorEvent);
 
   const completeEvent: CompleteRenderEvent = {
     type: 'complete',
     reset: false,
+    synchronous: false,
   };
   testRenderEvent(completeEvent);
 });