cypress 14.5.1 → 14.5.3

package/angular/angular/dist/index.js

@@ -8,7 +8,7 @@
 import 'zone.js';
 import 'zone.js/testing';
 import { CommonModule } from '@angular/common';
-import { assertInInjectionContext, inject, Injector, effect, untracked, DestroyRef, Injectable, Component, EventEmitter, ErrorHandler, SimpleChange, signal } from '@angular/core';
+import { assertInInjectionContext, inject, Injector, effect, untracked, DestroyRef, Injectable, Component, EventEmitter, ErrorHandler, SimpleChange } from '@angular/core';
 import { getTestBed, TestComponentRenderer, TestBed } from '@angular/core/testing';
 import { BrowserDynamicTestingModule, platformBrowserDynamicTesting } from '@angular/platform-browser-dynamic/testing';
 
@@ -1046,55 +1046,30 @@ function isModelSignal(prop) {
 function isWritableSignal(prop) {
     return isSignal(prop) && typeof prop === 'function' && typeof prop.set === 'function';
 }
-function convertPropertyToSignalIfApplicable(propValue, componentValue, injector) {
-    const isComponentValueAnInputSignal = isInputSignal(componentValue);
-    const isComponentValueAModelSignal = isModelSignal(componentValue);
-    let convertedValueIfApplicable = propValue;
-    // If the component has the property defined as an InputSignal, we need to detect whether a non signal value or not was passed into the component as a prop
-    // and attempt to merge the value in correctly.
-    // We don't want to expose the primitive created signal as it should really be one-way binding from within the component.
-    // However, to make CT testing easier, a user can technically pass in a signal to an input component and assert on the signal itself and pass in updates
-    // down to the component as 1 way binding is supported by the test harness
-    if (isComponentValueAnInputSignal) {
-        const isPassedInValueNotASignal = !isSignal(propValue);
-        if (isPassedInValueNotASignal) {
-            // Input signals require an injection context to set initial values.
-            // Because of this, we cannot create them outside the scope of the component.
-            // Options for input signals also don't allow the passing of an injection contexts, so in order to work around this,
-            // we convert the non signal input passed into the input to a writable signal
-            convertedValueIfApplicable = signal(propValue);
-        }
-        // If the component has the property defined as a ModelSignal, we need to detect whether a signal value or not was passed into the component as a prop.
-        // If a non signal property is passed into the component model (primitive, object, array, etc), we need to set the model to that value and propagate changes of that model through the output spy.
-        // Since the non signal type likely lives outside the context of Angular, the non signal type will NOT be updated outside of this context. Instead, the output spy will allow you
-        // to see this change.
-        // If the value passed into the property is in fact a signal, we need to set up two-way binding between the signals to make sure changes from one propagate to the other.
+function registerSignalEventsIfNeeded(propKey, propValue, componentValue, injector, fixture) {
+    const isPropValueASignal = isSignal(propValue);
+    if (isPropValueASignal) {
+        // propValue -> componentValue
+        const convertedToObservable = toObservable(propValue, {
+            injector,
+        });
+        // push the subscription into an array to be cleaned up at the end of the test
+        // to prevent a memory leak
+        activeInternalSubscriptions.push(convertedToObservable.subscribe((value) => {
+            // keep the component up to date as prop signal changes
+            fixture.componentRef.setInput(propKey, value);
+        }));
     }
-    else if (isComponentValueAModelSignal) {
-        const isPassedInValueLikelyARegularSignal = isWritableSignal(propValue);
-        // if the value passed into the component is a signal, set up two-way binding
-        if (isPassedInValueLikelyARegularSignal) {
-            // update the passed in value with the models updates
-            componentValue.subscribe((value) => {
-                propValue.set(value);
-            });
-            // update the model signal with the properties updates
-            const convertedToObservable = toObservable(propValue, {
-                injector,
-            });
-            // push the subscription into an array to be cleaned up at the end of the test
-            // to prevent a memory leak
-            activeInternalSubscriptions.push(convertedToObservable.subscribe((value) => {
-                componentValue.set(value);
-            }));
-        }
-        else {
-            // it's a non signal type, set it as we only need to handle updating the model signal and emit changes on this through the output spy.
-            componentValue.set(propValue);
-            convertedValueIfApplicable = componentValue;
-        }
+    const isComponentValueAModelSignal = isModelSignal(componentValue);
+    if (isPropValueASignal && isComponentValueAModelSignal) {
+        // propValue <- componentValue
+        const modelChanged$ = toObservable(componentValue, {
+            injector,
+        });
+        activeInternalSubscriptions.push(modelChanged$.subscribe((value) => {
+            propValue.set(value);
+        }));
     }
-    return convertedValueIfApplicable;
 }
 // In the case of signals, if we need to create an output spy, we need to check first whether or not a user has one defined first or has it created through
 // autoSpyOutputs. If so, we need to subscribe to the writable signal to push updates into the event emitter. We do NOT observe input signals and output spies will not
@@ -1103,6 +1078,11 @@ function detectAndRegisterOutputSpyToSignal(config, component, key, injector) {
     if (config.componentProperties) {
         const expectedChangeKey = `${key}Change`;
         let changeKeyIfExists = !!Object.keys(config.componentProperties).find((componentKey) => componentKey === expectedChangeKey);
+        if (changeKeyIfExists) {
+            component[expectedChangeKey] =
+                // @ts-expect-error
+                config.componentProperties[expectedChangeKey];
+        }
         // since spies do NOT make change handlers by default, similar to the Output() decorator, we need to create the spy and subscribe to the signal
         if (!changeKeyIfExists && config.autoSpyOutputs) {
             component[expectedChangeKey] = createOutputSpy(`${expectedChangeKey}Spy`);
@@ -1135,21 +1115,32 @@ function setupComponent(config, fixture) {
     let component = fixture.componentInstance;
     const injector = fixture.componentRef.injector;
     if (config === null || config === void 0 ? void 0 : config.componentProperties) {
-        // convert primitives to signals if passed in type is a primitive but expected type is signal
-        // a bit of magic. need to move to another function
-        Object.keys(component).forEach((key) => {
+        if (component instanceof WrapperComponent) {
+            component = Object.assign(component, config.componentProperties);
+        }
+        getComponentInputs(fixture.componentRef.componentType).forEach((key) => {
             var _a;
             // only assign props if they are passed into the component
             if ((_a = config === null || config === void 0 ? void 0 : config.componentProperties) === null || _a === void 0 ? void 0 : _a.hasOwnProperty(key)) {
                 // @ts-expect-error
                 const passedInValue = config === null || config === void 0 ? void 0 : config.componentProperties[key];
-                const componentValue = component[key];
-                // @ts-expect-error
-                config.componentProperties[key] = convertPropertyToSignalIfApplicable(passedInValue, componentValue, injector);
+                registerSignalEventsIfNeeded(key, passedInValue, component[key], injector, fixture);
                 detectAndRegisterOutputSpyToSignal(config, component, key, injector);
+                fixture.componentRef.setInput(key, isSignal(passedInValue) ? passedInValue() : passedInValue);
+            }
+        });
+        getComponentOutputs(fixture.componentRef.componentType).forEach((key) => {
+            var _a;
+            const property = component[key];
+            if (property instanceof EventEmitter) {
+                // only assign props if they are passed into the component
+                if ((_a = config === null || config === void 0 ? void 0 : config.componentProperties) === null || _a === void 0 ? void 0 : _a.hasOwnProperty(key)) {
+                    // @ts-expect-error
+                    const passedInValue = config === null || config === void 0 ? void 0 : config.componentProperties[key];
+                    component[key] = passedInValue;
+                }
             }
         });
-        component = Object.assign(component, config.componentProperties);
     }
     if (config.autoSpyOutputs) {
         Object.keys(component).forEach((key) => {
@@ -1173,6 +1164,24 @@ function setupComponent(config, fixture) {
         }
     }
 }
+/**
+ * Gets the input properties of a component - cannot rely on Object.keys() because inclusion of optional properties depends on useDefineForClassFields=true
+ *   Since Angular 15, useDefineForClassFields=false
+ * @param componentType
+ * @returns array of input property names
+ */
+function getComponentInputs(componentType) {
+    var _a;
+    // Access Angular's metadata to get input properties
+    const propMetadata = ((_a = componentType.ɵcmp) === null || _a === void 0 ? void 0 : _a.inputs) || {};
+    return Object.keys(propMetadata);
+}
+function getComponentOutputs(componentType) {
+    var _a;
+    // Access Angular's metadata to get output properties
+    const propMetadata = ((_a = componentType.ɵcmp) === null || _a === void 0 ? void 0 : _a.outputs) || {};
+    return Object.keys(propMetadata);
+}
 /**
  * Mounts an Angular component inside Cypress browser
  *

package/angular/dist/index.js

@@ -8,7 +8,7 @@
 import 'zone.js';
 import 'zone.js/testing';
 import { CommonModule } from '@angular/common';
-import { assertInInjectionContext, inject, Injector, effect, untracked, DestroyRef, Injectable, Component, EventEmitter, ErrorHandler, SimpleChange, signal } from '@angular/core';
+import { assertInInjectionContext, inject, Injector, effect, untracked, DestroyRef, Injectable, Component, EventEmitter, ErrorHandler, SimpleChange } from '@angular/core';
 import { getTestBed, TestComponentRenderer, TestBed } from '@angular/core/testing';
 import { BrowserDynamicTestingModule, platformBrowserDynamicTesting } from '@angular/platform-browser-dynamic/testing';
 
@@ -1046,55 +1046,30 @@ function isModelSignal(prop) {
 function isWritableSignal(prop) {
     return isSignal(prop) && typeof prop === 'function' && typeof prop.set === 'function';
 }
-function convertPropertyToSignalIfApplicable(propValue, componentValue, injector) {
-    const isComponentValueAnInputSignal = isInputSignal(componentValue);
-    const isComponentValueAModelSignal = isModelSignal(componentValue);
-    let convertedValueIfApplicable = propValue;
-    // If the component has the property defined as an InputSignal, we need to detect whether a non signal value or not was passed into the component as a prop
-    // and attempt to merge the value in correctly.
-    // We don't want to expose the primitive created signal as it should really be one-way binding from within the component.
-    // However, to make CT testing easier, a user can technically pass in a signal to an input component and assert on the signal itself and pass in updates
-    // down to the component as 1 way binding is supported by the test harness
-    if (isComponentValueAnInputSignal) {
-        const isPassedInValueNotASignal = !isSignal(propValue);
-        if (isPassedInValueNotASignal) {
-            // Input signals require an injection context to set initial values.
-            // Because of this, we cannot create them outside the scope of the component.
-            // Options for input signals also don't allow the passing of an injection contexts, so in order to work around this,
-            // we convert the non signal input passed into the input to a writable signal
-            convertedValueIfApplicable = signal(propValue);
-        }
-        // If the component has the property defined as a ModelSignal, we need to detect whether a signal value or not was passed into the component as a prop.
-        // If a non signal property is passed into the component model (primitive, object, array, etc), we need to set the model to that value and propagate changes of that model through the output spy.
-        // Since the non signal type likely lives outside the context of Angular, the non signal type will NOT be updated outside of this context. Instead, the output spy will allow you
-        // to see this change.
-        // If the value passed into the property is in fact a signal, we need to set up two-way binding between the signals to make sure changes from one propagate to the other.
+function registerSignalEventsIfNeeded(propKey, propValue, componentValue, injector, fixture) {
+    const isPropValueASignal = isSignal(propValue);
+    if (isPropValueASignal) {
+        // propValue -> componentValue
+        const convertedToObservable = toObservable(propValue, {
+            injector,
+        });
+        // push the subscription into an array to be cleaned up at the end of the test
+        // to prevent a memory leak
+        activeInternalSubscriptions.push(convertedToObservable.subscribe((value) => {
+            // keep the component up to date as prop signal changes
+            fixture.componentRef.setInput(propKey, value);
+        }));
     }
-    else if (isComponentValueAModelSignal) {
-        const isPassedInValueLikelyARegularSignal = isWritableSignal(propValue);
-        // if the value passed into the component is a signal, set up two-way binding
-        if (isPassedInValueLikelyARegularSignal) {
-            // update the passed in value with the models updates
-            componentValue.subscribe((value) => {
-                propValue.set(value);
-            });
-            // update the model signal with the properties updates
-            const convertedToObservable = toObservable(propValue, {
-                injector,
-            });
-            // push the subscription into an array to be cleaned up at the end of the test
-            // to prevent a memory leak
-            activeInternalSubscriptions.push(convertedToObservable.subscribe((value) => {
-                componentValue.set(value);
-            }));
-        }
-        else {
-            // it's a non signal type, set it as we only need to handle updating the model signal and emit changes on this through the output spy.
-            componentValue.set(propValue);
-            convertedValueIfApplicable = componentValue;
-        }
+    const isComponentValueAModelSignal = isModelSignal(componentValue);
+    if (isPropValueASignal && isComponentValueAModelSignal) {
+        // propValue <- componentValue
+        const modelChanged$ = toObservable(componentValue, {
+            injector,
+        });
+        activeInternalSubscriptions.push(modelChanged$.subscribe((value) => {
+            propValue.set(value);
+        }));
     }
-    return convertedValueIfApplicable;
 }
 // In the case of signals, if we need to create an output spy, we need to check first whether or not a user has one defined first or has it created through
 // autoSpyOutputs. If so, we need to subscribe to the writable signal to push updates into the event emitter. We do NOT observe input signals and output spies will not
@@ -1103,6 +1078,11 @@ function detectAndRegisterOutputSpyToSignal(config, component, key, injector) {
     if (config.componentProperties) {
         const expectedChangeKey = `${key}Change`;
         let changeKeyIfExists = !!Object.keys(config.componentProperties).find((componentKey) => componentKey === expectedChangeKey);
+        if (changeKeyIfExists) {
+            component[expectedChangeKey] =
+                // @ts-expect-error
+                config.componentProperties[expectedChangeKey];
+        }
         // since spies do NOT make change handlers by default, similar to the Output() decorator, we need to create the spy and subscribe to the signal
         if (!changeKeyIfExists && config.autoSpyOutputs) {
             component[expectedChangeKey] = createOutputSpy(`${expectedChangeKey}Spy`);
@@ -1135,21 +1115,32 @@ function setupComponent(config, fixture) {
     let component = fixture.componentInstance;
     const injector = fixture.componentRef.injector;
     if (config === null || config === void 0 ? void 0 : config.componentProperties) {
-        // convert primitives to signals if passed in type is a primitive but expected type is signal
-        // a bit of magic. need to move to another function
-        Object.keys(component).forEach((key) => {
+        if (component instanceof WrapperComponent) {
+            component = Object.assign(component, config.componentProperties);
+        }
+        getComponentInputs(fixture.componentRef.componentType).forEach((key) => {
             var _a;
             // only assign props if they are passed into the component
             if ((_a = config === null || config === void 0 ? void 0 : config.componentProperties) === null || _a === void 0 ? void 0 : _a.hasOwnProperty(key)) {
                 // @ts-expect-error
                 const passedInValue = config === null || config === void 0 ? void 0 : config.componentProperties[key];
-                const componentValue = component[key];
-                // @ts-expect-error
-                config.componentProperties[key] = convertPropertyToSignalIfApplicable(passedInValue, componentValue, injector);
+                registerSignalEventsIfNeeded(key, passedInValue, component[key], injector, fixture);
                 detectAndRegisterOutputSpyToSignal(config, component, key, injector);
+                fixture.componentRef.setInput(key, isSignal(passedInValue) ? passedInValue() : passedInValue);
+            }
+        });
+        getComponentOutputs(fixture.componentRef.componentType).forEach((key) => {
+            var _a;
+            const property = component[key];
+            if (property instanceof EventEmitter) {
+                // only assign props if they are passed into the component
+                if ((_a = config === null || config === void 0 ? void 0 : config.componentProperties) === null || _a === void 0 ? void 0 : _a.hasOwnProperty(key)) {
+                    // @ts-expect-error
+                    const passedInValue = config === null || config === void 0 ? void 0 : config.componentProperties[key];
+                    component[key] = passedInValue;
+                }
             }
         });
-        component = Object.assign(component, config.componentProperties);
     }
     if (config.autoSpyOutputs) {
         Object.keys(component).forEach((key) => {
@@ -1173,6 +1164,24 @@ function setupComponent(config, fixture) {
         }
     }
 }
+/**
+ * Gets the input properties of a component - cannot rely on Object.keys() because inclusion of optional properties depends on useDefineForClassFields=true
+ *   Since Angular 15, useDefineForClassFields=false
+ * @param componentType
+ * @returns array of input property names
+ */
+function getComponentInputs(componentType) {
+    var _a;
+    // Access Angular's metadata to get input properties
+    const propMetadata = ((_a = componentType.ɵcmp) === null || _a === void 0 ? void 0 : _a.inputs) || {};
+    return Object.keys(propMetadata);
+}
+function getComponentOutputs(componentType) {
+    var _a;
+    // Access Angular's metadata to get output properties
+    const propMetadata = ((_a = componentType.ɵcmp) === null || _a === void 0 ? void 0 : _a.outputs) || {};
+    return Object.keys(propMetadata);
+}
 /**
  * Mounts an Angular component inside Cypress browser
  *

package/package.json

@@ -1,13 +1,13 @@
 {
   "name": "cypress",
-  "version": "14.5.1",
+  "version": "14.5.3",
   "main": "index.js",
   "scripts": {
     "postinstall": "node index.js --exec install",
     "size": "t=\"$(npm pack .)\"; wc -c \"${t}\"; tar tvf \"${t}\"; rm \"${t}\";"
   },
   "dependencies": {
-    "@cypress/request": "^3.0.8",
+    "@cypress/request": "^3.0.9",
     "@cypress/xvfb": "^1.2.4",
     "@types/sinonjs__fake-timers": "8.1.1",
     "@types/sizzle": "^2.3.2",
@@ -124,8 +124,8 @@
   },
   "buildInfo": {
     "commitBranch": "develop",
-    "commitSha": "fa25461cb0ab04487f487aa5ee8d9b5e849a7de4",
-    "commitDate": "2025-07-01T19:14:05.000Z",
+    "commitSha": "d1478d8961a69dc3c0304d3d3d85ef68fbb5e2eb",
+    "commitDate": "2025-07-24T19:13:32.000Z",
     "stable": true
   },
   "description": "Cypress is a next generation front end testing tool built for the modern web",