update: some elements for the tests

npm/ng-packs/packages/core/src/lib/components/dynamic-layout.component.ts

@@ -3,9 +3,6 @@ import {
   inject,
   input,
   isDevMode,
-  OnInit,
-  Optional,
-  SkipSelf,
   Type,
 } from '@angular/core';
 import { ActivatedRoute, Router } from '@angular/router';

npm/ng-packs/packages/core/src/lib/directives/for.directive.ts

@@ -1,16 +1,16 @@
-import { 
+import {
-  Directive, 
+  Directive,
-  EmbeddedViewRef, 
+  EmbeddedViewRef,
-  Input, 
+  Input,
-  IterableChangeRecord, 
+  IterableChangeRecord,
-  IterableChanges, 
+  IterableChanges,
-  IterableDiffer, 
+  IterableDiffer,
-  IterableDiffers, 
+  IterableDiffers,
-  OnChanges, 
+  OnChanges,
-  TemplateRef, 
+  TemplateRef,
-  TrackByFunction, 
+  TrackByFunction,
-  ViewContainerRef, 
+  ViewContainerRef,
-  inject 
+  inject,
 } from '@angular/core';
 import clone from 'just-clone';
 import compare from 'just-compare';
@@ -67,6 +67,7 @@ export class ForDirective implements OnChanges {
   emptyRef?: TemplateRef<any>;
 
   private differ!: IterableDiffer<any> | null;
+  private lastItemsRef: any[] | null = null;
 
   private isShowEmptyRef!: boolean;
 
@@ -136,6 +137,7 @@ export class ForDirective implements OnChanges {
       this.vcRef.createEmbeddedView(this.emptyRef).rootNodes;
       this.isShowEmptyRef = true;
       this.differ = null;
+      this.lastItemsRef = null;
 
       return;
     }
@@ -169,6 +171,14 @@ export class ForDirective implements OnChanges {
   }
 
   ngOnChanges() {
+    if (!this.items) return;
+
+    // Recreate differ if items array reference changed
+    if (this.lastItemsRef !== this.items) {
+      this.differ = null;
+      this.lastItemsRef = this.items;
+    }
+
     let items = clone(this.items) as any[];
     if (!Array.isArray(items)) return;
 

npm/ng-packs/packages/core/src/lib/directives/permission.directive.ts

@@ -1,13 +1,13 @@
-import { 
+import {
-  AfterViewInit, 
+  AfterViewInit,
-  ChangeDetectorRef, 
+  ChangeDetectorRef,
-  Directive, 
+  Directive,
-  Input, 
+  Input,
-  OnChanges, 
+  OnChanges,
-  OnDestroy, 
+  OnDestroy,
-  TemplateRef, 
+  TemplateRef,
-  ViewContainerRef, 
+  ViewContainerRef,
-  inject 
+  inject,
 } from '@angular/core';
 import { ReplaySubject, Subscription } from 'rxjs';
 import { distinctUntilChanged, take } from 'rxjs/operators';
@@ -19,7 +19,7 @@ import { QueueManager } from '../utils/queue';
   selector: '[abpPermission]',
 })
 export class PermissionDirective implements OnDestroy, OnChanges, AfterViewInit {
-  private templateRef = inject<TemplateRef<any>>(TemplateRef, { optional: true })!;
+  private templateRef = inject<TemplateRef<any>>(TemplateRef, { optional: true });
   private vcRef = inject(ViewContainerRef);
   private permissionService = inject(PermissionService);
   private cdRef = inject(ChangeDetectorRef);
@@ -45,7 +45,9 @@ export class PermissionDirective implements OnDestroy, OnChanges, AfterViewInit
       .pipe(distinctUntilChanged())
       .subscribe(isGranted => {
         this.vcRef.clear();
-        if (isGranted) this.vcRef.createEmbeddedView(this.templateRef);
+        if (isGranted && this.templateRef) {
+          this.vcRef.createEmbeddedView(this.templateRef);
+        }
         if (this.runChangeDetection) {
           if (!this.rendered) {
             this.cdrSubject.next();

npm/ng-packs/packages/core/src/lib/services/list.service.ts

@@ -2,6 +2,7 @@ import { Injectable, Injector, OnDestroy, inject } from '@angular/core';
 import {
   EMPTY,
   BehaviorSubject,
+  defer,
   MonoTypeOperatorFunction,
   Observable,
   ReplaySubject,
@@ -134,7 +135,7 @@ export class ListService<QueryParamsType = ABP.PageQueryParams | any> implements
       tap(() => this._isLoading$.next(true)),
       tap(() => this._requestStatus.next('loading')),
       switchMap(query =>
-        streamCreatorCallback(query).pipe(
+        defer(() => streamCreatorCallback(query)).pipe(
           catchError(() => {
             this._requestStatus.next('error');
             return EMPTY;

npm/ng-packs/packages/core/src/lib/services/router-events.service.ts

@@ -34,7 +34,7 @@ export class RouterEvents {
 
   protected listenToNavigation(): void {
     const routerEvent$ = this.router.events.pipe(
-      filter(e => e instanceof NavigationEvent.End && !e.url.includes('error'))
+      filter(e => e instanceof NavigationEvent.End && e.url != null && !e.url.includes('error'))
     ) as Observable<NavigationEnd>;
     
     routerEvent$.subscribe(event => {

npm/ng-packs/packages/core/tsconfig.spec.json

@@ -6,7 +6,7 @@
   },
   "include": [
     "vite.config.ts",
-    "vite.config.mts",
+    "vitest.config.mts",
     "vitest.config.ts",
     "vitest.config.mts",
     "src/**/*.test.ts",

npm/ng-packs/packages/core/vite.config.ts

@@ -1,18 +0,0 @@
-/// <reference types="vitest" />
-import { defineConfig } from 'vitest/config';
-import { nxViteTsPaths } from '@nx/vite/plugins/nx-tsconfig-paths.plugin';
-
-export default defineConfig({
-  plugins: [nxViteTsPaths()],
-  test: {
-    globals: true,
-    environment: 'jsdom',
-    setupFiles: ['src/test-setup.ts'],
-    include: ['src/**/*.spec.ts'],
-    coverage: {
-      reportsDirectory: '../../coverage/packages/core',
-      provider: 'v8',
-      reporter: ['text', 'json', 'html'],
-    },
-  },
-});

npm/ng-packs/packages/core/vite.config.mts → npm/ng-packs/packages/core/vitest.config.mts

@@ -1,5 +1,4 @@
-/// <reference types='vitest' />
+import { defineConfig } from 'vitest/config';
-import { defineConfig } from 'vite';
 import { nxViteTsPaths } from '@nx/vite/plugins/nx-tsconfig-paths.plugin';
 import { nxCopyAssetsPlugin } from '@nx/vite/plugins/nx-copy-assets.plugin';
 
@@ -7,15 +6,12 @@ export default defineConfig(() => ({
   root: __dirname,
   cacheDir: '../../node_modules/.vite/packages/core',
   plugins: [nxViteTsPaths(), nxCopyAssetsPlugin(['*.md'])],
-  // Uncomment this if you are using workers.
-  // worker: {
-  //   plugins: () => [ nxViteTsPaths() ],
-  // },
   test: {
     name: 'core',
     watch: false,
     globals: true,
     environment: 'jsdom',
+    setupFiles: ['src/test-setup.ts'],
     include: ['{src,tests}/**/*.{test,spec}.{js,mjs,cjs,ts,mts,cts,jsx,tsx}'],
     reporters: ['default'],
     coverage: {