@angular/core 16.0.1 → 16.0.3

package/fesm2022/testing.mjs

@@ -1,5 +1,5 @@
 /**
- * @license Angular v16.0.1
+ * @license Angular v16.0.3
  * (c) 2010-2022 Google LLC. https://angular.io/
  * License: MIT
  */
@@ -656,9 +656,36 @@ const __forward_ref__ = getClosureSafeProperty({ __forward_ref__: getClosureSafe
  * DI is declared, but not yet defined. It is also used when the `token` which we use when creating
  * a query is not yet defined.
  *
+ * `forwardRef` is also used to break circularities in standalone components imports.
+ *
  * @usageNotes
- * ### Example
+ * ### Circular dependency example
  * {@example core/di/ts/forward_ref/forward_ref_spec.ts region='forward_ref'}
+ *
+ * ### Circular standalone reference import example
+ * ```ts
+ * @Component({
+ *   standalone: true,
+ *   imports: [ChildComponent],
+ *   selector: 'app-parent',
+ *   template: `<app-child [hideParent]="hideParent"></app-child>`,
+ * })
+ * export class ParentComponent {
+ *   @Input() hideParent: boolean;
+ * }
+ *
+ *
+ * @Component({
+ *   standalone: true,
+ *   imports: [CommonModule, forwardRef(() => ParentComponent)],
+ *   selector: 'app-child',
+ *   template: `<app-parent *ngIf="!hideParent"></app-parent>`,
+ * })
+ * export class ChildComponent {
+ *   @Input() hideParent: boolean;
+ * }
+ * ```
+ *
  * @publicApi
  */
 function forwardRef(forwardRefFn) {
@@ -3084,7 +3111,7 @@ function getPipeDef$1(type) {
 /**
  * Checks whether a given Component, Directive or Pipe is marked as standalone.
  * This will return false if passed anything other than a Component, Directive, or Pipe class
- * See this guide for additional information: https://angular.io/guide/standalone-components
+ * See [this guide](/guide/standalone-components) for additional information:
  *
  * @param type A reference to a Component, Directive or Pipe.
  * @publicApi
@@ -3154,7 +3181,7 @@ function getComponentId(componentDef) {
     // Example:
     // https://github.com/angular/components/blob/d9f82c8f95309e77a6d82fd574c65871e91354c2/src/material/core/option/option.ts#L248
     // https://github.com/angular/components/blob/285f46dc2b4c5b127d356cb7c4714b221f03ce50/src/material/legacy-core/option/option.ts#L32
-    const hashSelectors = JSON.stringify([
+    const hashSelectors = [
         componentDef.selectors,
         componentDef.ngContentSelectors,
         componentDef.hostVars,
@@ -3165,14 +3192,14 @@ function getComponentId(componentDef) {
         componentDef.encapsulation,
         componentDef.standalone,
         componentDef.exportAs,
-        componentDef.inputs,
-        componentDef.outputs,
+        JSON.stringify(componentDef.inputs),
+        JSON.stringify(componentDef.outputs),
         // We cannot use 'componentDef.type.name' as the name of the symbol will change and will not
         // match in the server and browser bundles.
         Object.getOwnPropertyNames(componentDef.type.prototype),
         !!componentDef.contentQueries,
         !!componentDef.viewQuery,
-    ]);
+    ].join('|');
     for (const char of hashSelectors) {
         hash = Math.imul(31, hash) + char.charCodeAt(0) << 0;
     }
@@ -3430,10 +3457,12 @@ function getFactoryDef(type, throwNotFound) {
  */
 const SIGNAL = Symbol('SIGNAL');
 /**
- * Checks if the given `value` function is a reactive `Signal`.
+ * Checks if the given `value` is a reactive `Signal`.
+ *
+ * @developerPreview
  */
 function isSignal(value) {
-    return value[SIGNAL] !== undefined;
+    return typeof value === 'function' && value[SIGNAL] !== undefined;
 }
 /**
  * Converts `fn` into a marked signal function (where `isSignal(fn)` will be `true`), and
@@ -5681,7 +5710,10 @@ function getOrCreateInjectable(tNode, lView, token, flags = InjectFlags.Default,
     if (tNode !== null) {
         // If the view or any of its ancestors have an embedded
         // view injector, we have to look it up there first.
-        if (lView[FLAGS] & 2048 /* LViewFlags.HasEmbeddedViewInjector */) {
+        if (lView[FLAGS] & 2048 /* LViewFlags.HasEmbeddedViewInjector */ &&
+            // The token must be present on the current node injector when the `Self`
+            // flag is set, so the lookup on embedded view injector(s) can be skipped.
+            !(flags & InjectFlags.Self)) {
             const embeddedInjectorValue = lookupTokenUsingEmbeddedInjector(tNode, lView, token, flags, NOT_FOUND);
             if (embeddedInjectorValue !== NOT_FOUND) {
                 return embeddedInjectorValue;
@@ -7459,12 +7491,14 @@ function processCleanups(tView, lView) {
     }
     const destroyHooks = lView[ON_DESTROY_HOOKS];
     if (destroyHooks !== null) {
+        // Reset the ON_DESTROY_HOOKS array before iterating over it to prevent hooks that unregister
+        // themselves from mutating the array during iteration.
+        lView[ON_DESTROY_HOOKS] = null;
         for (let i = 0; i < destroyHooks.length; i++) {
             const destroyHooksFn = destroyHooks[i];
             ngDevMode && assertFunction(destroyHooksFn, 'Expecting destroy hook to be a function.');
             destroyHooksFn();
         }
-        lView[ON_DESTROY_HOOKS] = null;
     }
 }
 /** Calls onDestroy hooks for this view */
@@ -9461,7 +9495,11 @@ class R3Injector extends EnvironmentInjector {
             for (const service of this._ngOnDestroyHooks) {
                 service.ngOnDestroy();
             }
-            for (const hook of this._onDestroyHooks) {
+            const onDestroyHooks = this._onDestroyHooks;
+            // Reset the _onDestroyHooks array before iterating over it to prevent hooks that unregister
+            // themselves from mutating the array during iteration.
+            this._onDestroyHooks = [];
+            for (const hook of onDestroyHooks) {
                 hook();
             }
         }
@@ -9470,7 +9508,6 @@ class R3Injector extends EnvironmentInjector {
             this.records.clear();
             this._ngOnDestroyHooks.clear();
             this.injectorDefTypes.clear();
-            this._onDestroyHooks.length = 0;
         }
     }
     onDestroy(callback) {
@@ -9884,26 +9921,6 @@ const ENABLED_SSR_FEATURES = new InjectionToken((typeof ngDevMode === 'undefined
     factory: () => new Set(),
 });
 
-function escapeTransferStateContent(text) {
-    const escapedText = {
-        '&': '&a;',
-        '"': '&q;',
-        '\'': '&s;',
-        '<': '&l;',
-        '>': '&g;',
-    };
-    return text.replace(/[&"'<>]/g, s => escapedText[s]);
-}
-function unescapeTransferStateContent(text) {
-    const unescapedText = {
-        '&a;': '&',
-        '&q;': '"',
-        '&s;': '\'',
-        '&l;': '<',
-        '&g;': '>',
-    };
-    return text.replace(/&[^;]+;/g, s => unescapedText[s]);
-}
 /**
  * Create a `StateKey<T>` that can be used to store value of type T with `TransferState`.
  *
@@ -10007,7 +10024,9 @@ class TransferState {
                 }
             }
         }
-        return JSON.stringify(this.store);
+        // Escape script tag to avoid break out of <script> tag in serialized output.
+        // Encoding of `<` is the same behaviour as G3 script_builders.
+        return JSON.stringify(this.store).replace(/</g, '\\u003C');
     }
 }
 function retrieveTransferredState(doc, appId) {
@@ -10017,7 +10036,9 @@ function retrieveTransferredState(doc, appId) {
     if (script?.textContent) {
         try {
             // Avoid using any here as it triggers lint errors in google3 (any is not allowed).
-            return JSON.parse(unescapeTransferStateContent(script.textContent));
+            // Decoding of `<` is done of the box by browsers and node.js, same behaviour as G3
+            // script_builders.
+            return JSON.parse(script.textContent);
         }
         catch (e) {
             console.warn('Exception while restoring TransferState for app ' + appId, e);
@@ -10443,7 +10464,7 @@ class Version {
 /**
  * @publicApi
  */
-const VERSION = new Version('16.0.1');
+const VERSION = new Version('16.0.3');
 
 // This default value is when checking the hierarchy for a token.
 //
@@ -10585,6 +10606,10 @@ function normalizeDebugBindingValue(value) {
     }
 }
 
+/**
+ * The max length of the string representation of a value in an error message
+ */
+const VALUE_STRING_LENGTH_LIMIT = 200;
 /** Verifies that a given type is a Standalone Component. */
 function assertStandaloneComponentType(type) {
     assertComponentDef(type);
@@ -10610,9 +10635,11 @@ function throwMultipleComponentError(tNode, first, second) {
         `${stringifyForError(second)}`);
 }
 /** Throws an ExpressionChangedAfterChecked error if checkNoChanges mode is on. */
-function throwErrorIfNoChangesMode(creationMode, oldValue, currValue, propName) {
+function throwErrorIfNoChangesMode(creationMode, oldValue, currValue, propName, lView) {
+    const hostComponentDef = getDeclarationComponentDef(lView);
+    const componentClassName = hostComponentDef?.type?.name;
     const field = propName ? ` for '${propName}'` : '';
-    let msg = `ExpressionChangedAfterItHasBeenCheckedError: Expression has changed after it was checked. Previous value${field}: '${oldValue}'. Current value: '${currValue}'.`;
+    let msg = `ExpressionChangedAfterItHasBeenCheckedError: Expression has changed after it was checked. Previous value${field}: '${formatValue(oldValue)}'. Current value: '${formatValue(currValue)}'.${componentClassName ? ` Expression location: ${componentClassName} component` : ''}`;
     if (creationMode) {
         msg +=
             ` It seems like the view has been created after its parent and its children have been dirty checked.` +
@@ -10620,6 +10647,20 @@ function throwErrorIfNoChangesMode(creationMode, oldValue, currValue, propName)
     }
     throw new RuntimeError(-100 /* RuntimeErrorCode.EXPRESSION_CHANGED_AFTER_CHECKED */, msg);
 }
+function formatValue(value) {
+    let strValue = String(value);
+    // JSON.stringify will throw on circular references
+    try {
+        if (Array.isArray(value) || strValue === '[object Object]') {
+            strValue = JSON.stringify(value);
+        }
+    }
+    catch (error) {
+    }
+    return strValue.length > VALUE_STRING_LENGTH_LIMIT ?
+        (strValue.substring(0, VALUE_STRING_LENGTH_LIMIT) + '…') :
+        strValue;
+}
 function constructDetailsForInterpolation(lView, rootIndex, expressionIndex, meta, changedValue) {
     const [propName, prefix, ...chunks] = meta.split(INTERPOLATION_DELIMITER);
     let oldValue = prefix, newValue = prefix;
@@ -13955,7 +13996,7 @@ function validateMappings(bindingType, def, hostDirectiveBindings) {
                 throw new RuntimeError(311 /* RuntimeErrorCode.HOST_DIRECTIVE_UNDEFINED_BINDING */, `Directive ${className} does not have an ${bindingType} with a public name of ${publicName}.`);
             }
             const remappedPublicName = hostDirectiveBindings[publicName];
-            if (bindings.hasOwnProperty(remappedPublicName) &&
+            if (bindings.hasOwnProperty(remappedPublicName) && remappedPublicName !== publicName &&
                 bindings[remappedPublicName] !== publicName) {
                 throw new RuntimeError(312 /* RuntimeErrorCode.HOST_DIRECTIVE_CONFLICTING_ALIAS */, `Cannot alias ${bindingType} ${publicName} of host directive ${className} to ${remappedPublicName}, because it already has a different ${bindingType} with the same public name.`);
             }
@@ -14063,7 +14104,7 @@ function bindingUpdated(lView, bindingIndex, value) {
             const oldValueToCompare = oldValue !== NO_CHANGE ? oldValue : undefined;
             if (!devModeEqual(oldValueToCompare, value)) {
                 const details = getExpressionChangedErrorDetails(lView, bindingIndex, oldValueToCompare, value);
-                throwErrorIfNoChangesMode(oldValue === NO_CHANGE, details.oldValue, details.newValue, details.propName);
+                throwErrorIfNoChangesMode(oldValue === NO_CHANGE, details.oldValue, details.newValue, details.propName, lView);
             }
             // There was a change, but the `devModeEqual` decided that the change is exempt from an error.
             // For this reason we exit as if no change. The early exit is needed to prevent the changed
@@ -16048,8 +16089,6 @@ function wrapListener(tNode, lView, context, listenerFn, wrapWithPreventDefault)
         }
         if (wrapWithPreventDefault && result === false) {
             e.preventDefault();
-            // Necessary for legacy browsers that don't support preventDefault (e.g. IE)
-            e.returnValue = false;
         }
         return result;
     };
@@ -19582,7 +19621,7 @@ function getLocaleCurrencyCode(locale) {
  * @param locale A locale code for the locale format rules to use.
  * @returns The plural function for the locale.
  * @see `NgPlural`
- * @see [Internationalization (i18n) Guide](https://angular.io/guide/i18n-overview)
+ * @see [Internationalization (i18n) Guide](/guide/i18n-overview)
  */
 function getLocalePluralCase(locale) {
     const data = findLocaleData(locale);
@@ -24761,16 +24800,16 @@ function computeCombinedExports(type) {
     if (ngModuleDef === null) {
         return [type];
     }
-    return [...flatten$1(maybeUnwrapFn$1(ngModuleDef.exports).map((type) => {
-            const ngModuleDef = getNgModuleDef(type);
-            if (ngModuleDef) {
-                verifySemanticsOfNgModuleDef(type, false);
-                return computeCombinedExports(type);
-            }
-            else {
-                return type;
-            }
-        }))];
+    return flatten$1(maybeUnwrapFn$1(ngModuleDef.exports).map((type) => {
+        const ngModuleDef = getNgModuleDef(type);
+        if (ngModuleDef) {
+            verifySemanticsOfNgModuleDef(type, false);
+            return computeCombinedExports(type);
+        }
+        else {
+            return type;
+        }
+    }));
 }
 /**
  * Some declared components may be compiled asynchronously, and thus may not have their