feat(MockBuilder): mocks root providers via inject function help-me-m…

…om#4282

.eslintrc.yml

@@ -93,12 +93,14 @@ overrides:
 
       '@typescript-eslint/no-explicit-any': off
       '@typescript-eslint/no-namespace': off
+      '@typescript-eslint/no-this-alias': off
       '@typescript-eslint/no-unused-vars': error
 
       unicorn/no-array-callback-reference: off
       unicorn/no-array-method-this-argument: off
       unicorn/no-for-loop: off
       unicorn/no-null: off
+      unicorn/no-this-assignment: off
       unicorn/no-useless-undefined: off
       unicorn/prefer-array-flat: off
       unicorn/prefer-event-target: off

libs/ng-mocks/src/lib/common/core.config.ts

@@ -11,6 +11,7 @@ export default {
     // https://github.com/help-me-mom/ng-mocks/issues/538
     'Sanitizer',
     'DomSanitizer',
+    'DomSanitizerImpl',
 
     // ApplicationModule, A14 made them global at root level
     'ApplicationInitStatus',

libs/ng-mocks/src/lib/common/func.is-mock.ts

@@ -10,5 +10,5 @@ export default <T>(
     __template?: TemplateRef<any>;
     __vcr?: ViewContainerRef;
   } => {
-  return value && typeof value === 'object' && !!(value as any).__ngMocksConfig;
+  return value && typeof value === 'object' && !!(value as any).__ngMocks;
 };

libs/ng-mocks/src/lib/common/ng-mocks-global-overrides.ts

@@ -1,8 +1,9 @@
-import { ViewContainerRef } from '@angular/core';
+import { Injector, ViewContainerRef } from '@angular/core';
 import { getTestBed, MetadataOverride, TestBed, TestBedStatic, TestModuleMetadata } from '@angular/core/testing';
 
 import funcExtractTokens from '../mock-builder/func.extract-tokens';
 import getOverrideDef from '../mock-builder/promise/get-override-def';
+import skipDep from '../mock-builder/promise/skip-dep';
 import { ngMocks } from '../mock-helper/mock-helper';
 import mockHelperFasterInstall from '../mock-helper/mock-helper.faster-install';
 import { MockProvider } from '../mock-provider/mock-provider';
@@ -15,8 +16,9 @@ import coreInjector from './core.injector';
 import coreReflectMeta from './core.reflect.meta';
 import coreReflectModuleResolve from './core.reflect.module-resolve';
 import coreReflectProvidedIn from './core.reflect.provided-in';
-import { NG_MOCKS, NG_MOCKS_TOUCHES } from './core.tokens';
+import { NG_MOCKS, NG_MOCKS_ROOT_PROVIDERS, NG_MOCKS_TOUCHES } from './core.tokens';
 import { AnyType, dependencyKeys } from './core.types';
+import funcGetName from './func.get-name';
 import funcGetProvider from './func.get-provider';
 import { isNgDef } from './func.is-ng-def';
 import { isNgModuleDefWithProviders } from './func.is-ng-module-def-with-providers';
@@ -264,6 +266,87 @@ const viewContainerInstall = () => {
   }
 };
 
+// this function monkey-patches Angular injectors.
+const installInjector = (injector: Injector & { __ngMocksInjector?: any }): Injector => {
+  // skipping the matched injector
+  if (injector.constructor.prototype.__ngMocksInjector || !injector.constructor.prototype.get) {
+    return injector;
+  }
+
+  // marking the injector as patched
+  coreDefineProperty(injector.constructor.prototype, '__ngMocksInjector', true);
+  const injectorGet = injector.constructor.prototype.get;
+
+  // patch
+  injector.constructor.prototype.get = helperCreateClone(
+    injectorGet,
+    undefined,
+    undefined,
+    function (token: any, ...argsGet: any) {
+      // eslint-disable-next-line @typescript-eslint/ban-ts-comment
+      // @ts-ignore
+      const binding: any = this;
+
+      // Some tokens are declaration of providers and should be normalized.
+      let ngMocksToken = funcGetProvider(token);
+      let touched = true;
+      try {
+        // First, we check if we should skip the token.
+        // If the token is a root provider which nobody touched or kept, we can mock it.
+        if (
+          !skipDep(ngMocksToken) &&
+          coreReflectProvidedIn(ngMocksToken) === 'root' &&
+          ngMocksUniverse.getResolution(ngMocksToken) !== 'keep'
+        ) {
+          // Now, we check if the token hasn't been used yet.
+          // NG_MOCKS is present in MockBuilder setup only.
+          touched =
+            injectorGet.call(binding, NG_MOCKS).get(NG_MOCKS_ROOT_PROVIDERS) === NG_MOCKS_ROOT_PROVIDERS ||
+            injectorGet.call(binding, NG_MOCKS).has(ngMocksToken) ||
+            injectorGet.call(binding, NG_MOCKS_TOUCHES).has(ngMocksToken);
+        }
+      } catch {
+        // nothing to do
+      }
+
+      // This means that the token hasn't been used yet and is suitable for mocking.
+      if (!touched) {
+        const def = MockProvider(ngMocksToken);
+        ngMocksToken = (() => {
+          const declaration = /* istanbul ignore next */ () => undefined;
+          coreDefineProperty(declaration, 'name', funcGetName(token), true);
+          coreDefineProperty(declaration, 'ngInjectableDef', {
+            ...def,
+            factory: def.useFactory,
+            providedIn: 'root',
+          });
+
+          return declaration;
+        })();
+      } else {
+        // If we don't mock, we should keep the original token.
+        ngMocksToken = undefined;
+      }
+
+      const result = injectorGet.call(binding, ngMocksToken ?? token, ...argsGet);
+      // If the result is an injector, we should patch it too.
+      if (
+        result &&
+        typeof result === 'object' &&
+        typeof result.constructor === 'function' &&
+        typeof result.constructor.name === 'string' &&
+        result.constructor.name.slice(-8) === 'Injector'
+      ) {
+        installInjector(result);
+      }
+
+      return result;
+    },
+  );
+
+  return injector;
+};
+
 const install = () => {
   // istanbul ignore else
   if (!(TestBed as any).ngMocksOverridesInstalled) {
@@ -280,6 +363,10 @@ const install = () => {
     }
 
     coreDefineProperty(TestBed, 'ngMocksOverridesInstalled', true);
+    const injectorCreate = Injector.create;
+    Injector.create = helperCreateClone(injectorCreate, undefined, undefined, (...argsCreate: any) =>
+      installInjector(injectorCreate.apply(Injector, argsCreate)),
+    );
   }
 };
 

libs/ng-mocks/src/lib/mock-builder/promise/create-ng-mocks-token.ts

@@ -1,6 +1,6 @@
 import { ValueProvider } from '@angular/core';
 
-import { mapEntries } from '../../common/core.helpers';
+import { mapEntries, mapValues } from '../../common/core.helpers';
 import { NG_MOCKS } from '../../common/core.tokens';
 import ngMocksUniverse from '../../common/ng-mocks-universe';
 
@@ -17,6 +17,12 @@ export default (): ValueProvider => {
     }
     mocks.set(key, value);
   }
+  for (const key of mapValues(ngMocksUniverse.config.get('ngMocksDepsSkip'))) {
+    if (mocks.has(key)) {
+      continue;
+    }
+    mocks.set(key, undefined);
+  }
 
   return {
     provide: NG_MOCKS,

package.json

@@ -99,8 +99,8 @@
     "clean:nx": "rm -Rf e2e/nx/node_modules/ng-mocks && rm -Rf e2e/nx/apps/a-nx/src/test",
     "s:test:e2e": "npm run s:test:a5 && npm run s:test:a6 && npm run s:test:a7 && npm run s:test:a8 && npm run s:test:a9 && npm run s:test:a10 && npm run s:test:a11 && npm run s:test:a12 && npm run s:test:a13 && npm run s:test:a14 && npm run s:test:a15 && npm run s:test:nx",
     "s:test:a5": "npm run s:test:a5es5 && npm run s:test:a5es2015",
-    "s:test:a5es5": "P=e2e/a5es5/src/test && rm -Rf $P && mkdir -p $P && cp -R tests $P && cp -R examples $P && rm $P/examples/TestRoutingGuard/test.spec.ts && rm $P/examples/TestRoutingResolver/test.spec.ts",
-    "s:test:a5es2015": "P=e2e/a5es2015/src/test && rm -Rf $P && mkdir -p $P && cp -R tests $P && cp -R examples $P && rm $P/examples/TestRoutingGuard/test.spec.ts && rm $P/examples/TestRoutingResolver/test.spec.ts",
+    "s:test:a5es5": "P=e2e/a5es5/src/test && rm -Rf $P && mkdir -p $P && cp -R tests $P && cp -R examples $P && rm $P/examples/TestRoutingGuard/test.spec.ts && rm $P/examples/TestRoutingResolver/test.spec.ts && rm $P/tests/issue-4282/test.spec.ts",
+    "s:test:a5es2015": "P=e2e/a5es2015/src/test && rm -Rf $P && mkdir -p $P && cp -R tests $P && cp -R examples $P && rm $P/examples/TestRoutingGuard/test.spec.ts && rm $P/examples/TestRoutingResolver/test.spec.ts && rm $P/tests/issue-4282/test.spec.ts",
     "s:test:a6": "P=e2e/a6/src/test && rm -Rf $P && mkdir -p $P && cp -R tests $P && cp -R examples $P",
     "s:test:a7": "P=e2e/a7/src/test && rm -Rf $P && mkdir -p $P && cp -R tests $P && cp -R examples $P",
     "s:test:a8": "P=e2e/a8/src/test && rm -Rf $P && mkdir -p $P && cp -R tests $P && cp -R examples $P",

tests/issue-4282/test.spec.ts

@@ -0,0 +1,53 @@
+import {
+  Component,
+  inject,
+  Injectable,
+  VERSION,
+} from '@angular/core';
+
+import { isMockOf, MockBuilder, MockRender, ngMocks } from 'ng-mocks';
+
+// @TODO remove with A5 support
+const injectableTargetServiceArgs = [
+  {
+    providedIn: 'root',
+  } as never,
+];
+
+@Injectable(...injectableTargetServiceArgs)
+export class TargetService {
+  name = 'real';
+}
+
+@Component({
+  selector: 'target',
+  template: `{{ service.name }}`,
+})
+export class TargetComponent {
+  readonly service = inject(TargetService);
+}
+
+// @see https://github.com/help-me-mom/ng-mocks/issues/4282
+describe('issue-4282', () => {
+  if (Number.parseInt(VERSION.major, 10) < 14) {
+    it('a14', () => {
+      // pending('Need Angular >= 14');
+      expect(true).toBeTruthy();
+    });
+
+    return;
+  }
+
+  beforeEach(() => MockBuilder(TargetComponent));
+
+  it('should create', () => {
+    const fixture = MockRender(TargetComponent);
+    expect(ngMocks.formatText(fixture)).toEqual('');
+    expect(
+      isMockOf(
+        fixture.point.componentInstance.service,
+        TargetService,
+      ),
+    ).toEqual(true);
+  });
+});