@applitools/eyes-cypress 3.44.8 → 3.44.9

package/CHANGELOG.md

@@ -1,5 +1,62 @@
 # Changelog
 
+## [3.44.9](https://github.com/Applitools-Dev/sdk/compare/js/eyes-cypress@3.44.8...js/eyes-cypress@3.44.9) (2024-09-16)
+
+
+### Bug Fixes
+
+* stale layout breakpoints elements ([#2479](https://github.com/Applitools-Dev/sdk/issues/2479)) ([f5e4b5a](https://github.com/Applitools-Dev/sdk/commit/f5e4b5ac8077b8c4b7ba67a38c4e58050a55ce75))
+
+
+### Dependencies
+
+* @applitools/snippets bumped to 2.5.0
+  #### Features
+
+  * add playwright fixture ([#2412](https://github.com/Applitools-Dev/sdk/issues/2412)) ([0632e93](https://github.com/Applitools-Dev/sdk/commit/0632e93dd7f53029a8c5f4230d22a05abf5fefd4))
+
+
+  #### Bug Fixes
+
+  * stale layout breakpoints elements ([#2479](https://github.com/Applitools-Dev/sdk/issues/2479)) ([f5e4b5a](https://github.com/Applitools-Dev/sdk/commit/f5e4b5ac8077b8c4b7ba67a38c4e58050a55ce75))
+* @applitools/dom-capture bumped to 11.5.0
+  #### Features
+
+  * ability to capture element properties ([#2525](https://github.com/Applitools-Dev/sdk/issues/2525)) ([ab35e1d](https://github.com/Applitools-Dev/sdk/commit/ab35e1dc929ba4bd82e99081527b277c8090f1e0))
+* @applitools/driver bumped to 1.19.1
+  #### Bug Fixes
+
+  * stale layout breakpoints elements ([#2479](https://github.com/Applitools-Dev/sdk/issues/2479)) ([f5e4b5a](https://github.com/Applitools-Dev/sdk/commit/f5e4b5ac8077b8c4b7ba67a38c4e58050a55ce75))
+
+
+
+* @applitools/spec-driver-webdriver bumped to 1.1.13
+
+* @applitools/spec-driver-selenium bumped to 1.5.84
+
+* @applitools/spec-driver-puppeteer bumped to 1.4.13
+
+* @applitools/screenshoter bumped to 3.8.37
+
+* @applitools/nml-client bumped to 1.8.11
+
+* @applitools/ec-client bumped to 1.9.5
+
+* @applitools/core bumped to 4.19.0
+  #### Features
+
+  * allow passing arguments to dom-capture from core ([#2531](https://github.com/Applitools-Dev/sdk/issues/2531)) ([9cfc9a5](https://github.com/Applitools-Dev/sdk/commit/9cfc9a5b5d6650afd1e7daf2714d6779a7dc2724))
+
+
+  #### Bug Fixes
+
+  * stale layout breakpoints elements ([#2479](https://github.com/Applitools-Dev/sdk/issues/2479)) ([f5e4b5a](https://github.com/Applitools-Dev/sdk/commit/f5e4b5ac8077b8c4b7ba67a38c4e58050a55ce75))
+
+
+
+* @applitools/eyes bumped to 1.22.3
+
+
 ## [3.44.8](https://github.com/Applitools-Dev/sdk/compare/js/eyes-cypress@3.44.7...js/eyes-cypress@3.44.8) (2024-09-10)
 
 

package/dist/browser/spec-driver.js

@@ -11,7 +11,12 @@ function executeScript(context, script, arg) {
         scriptToExecute = prepScirpt.concat(' return func(arg)');
     }
     const executor = new context.defaultView.Function('arg', scriptToExecute);
-    return executor(arg);
+    try {
+        return executor(arg);
+    }
+    catch (e) {
+        throw new Error(`Failed to execute script: ${e}`); // stringifiying prevents [object Object] errors
+    }
 }
 exports.executeScript = executeScript;
 function mainContext() {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@applitools/eyes-cypress",
-  "version": "3.44.8",
+  "version": "3.44.9",
   "homepage": "https://applitools.com/docs/api-ref/sdk-api/cypress/",
   "license": "SEE LICENSE IN LICENSE",
   "main": "./index.js",
@@ -22,7 +22,7 @@
     "build": "run --top-level tspc --project ./tsconfig.build.json",
     "generate:tests": "NODE_OPTIONS='--experimental-import-meta-resolve --experimental-loader=@applitools/generic/dist/code-loader.js' generic ./test/generic/config.mjs",
     "test": "yarn test:unit && yarn test:it && yarn test:ts:compile && yarn test:setup && yarn test:components && yarn test:e2e",
-    "test:unit": "run --top-level mocha --no-timeouts 'test/unit/**/*.test.js'",
+    "test:unit": "run --top-level mocha --no-timeouts 'test/unit/**/*.test.{js,ts}'",
     "test:it": "run --top-level mocha --no-timeouts 'test/it/**/*.test.js'",
     "test:setup": "run --top-level mocha --no-timeouts 'test/setup/**/*.test.js'",
     "test:ts": "yarn test:ts:compile && yarn test:ts:run",
@@ -54,7 +54,7 @@
     "setup": "run --top-level xvfb:setup"
   },
   "dependencies": {
-    "@applitools/core": "4.18.2",
+    "@applitools/core": "4.19.0",
     "@applitools/eyes": "1.22.2",
     "@applitools/functional-commons": "1.6.0",
     "@applitools/logger": "2.0.18",

package/src/browser/derefArgs.js

@@ -0,0 +1,28 @@
+const derefArgsCallStack = []
+function derefArgs(refer, arg) {
+  if (derefArgsCallStack.includes(arg)) return arg // prevent circular reference
+  try {
+    derefArgsCallStack.push(arg)
+    if (refer.isRef(arg)) return refer.deref(arg)
+    else if (Array.isArray(arg)) {
+      const derefArg = []
+      for (const argument of arg) {
+        derefArg.push(derefArgs(refer, argument))
+      }
+      return derefArg
+    } else if (typeof arg === 'object') {
+      const derefArg = {}
+      for (const [key, value] of Object.entries(arg)) {
+        derefArg[key] = derefArgs(refer, value)
+      }
+      return derefArg
+    }
+    return arg
+  } finally {
+    derefArgsCallStack.pop()
+  }
+}
+
+module.exports = {
+  derefArgs,
+}

package/src/browser/refer.js

@@ -3,16 +3,28 @@
 const uuid = require('uuid')
 const REF_ID = 'applitools-ref-id'
 
+// type RefType = 'cypress-driver' | 'element' | 'context' | 'driver'
+
+// type Ref = {
+//   [REF_ID]: string
+//   type?: RefType
+// }
+
 class Refer {
+  // store: Map<string, any>
+  // relation: Map<string, Set<Ref>>
+
   constructor() {
     this.store = new Map()
     this.relation = new Map()
   }
 
+  // isRef(ref: any): ref is Ref {
   isRef(ref) {
     return Boolean(ref && ref[REF_ID])
   }
 
+  // check(value: any): RefType | undefined {
   check(value) {
     if (!value) return
     if (value === '__CYPRESS_DRIVER__') {
@@ -26,6 +38,7 @@ class Refer {
     }
   }
 
+  // ref(value: any, parentRef?: Ref) {
   ref(value, parentRef) {
     const refType = this.check(value)
     if (refType) {

package/src/browser/socketCommands.js

@@ -1,8 +1,9 @@
 const spec = require('../../dist/browser/spec-driver')
+const {derefArgs} = require('./derefArgs')
 
 function socketCommands(socket, refer) {
   socket.command('Driver.executeScript', ({context, script, arg = []}) => {
-    const res = spec.executeScript(refer.deref(context), script, derefArgs(arg))
+    const res = spec.executeScript(refer.deref(context), script, derefArgs(refer, arg))
     return res ? refer.ref(res) : res
   })
 
@@ -48,29 +49,6 @@ function socketCommands(socket, refer) {
   socket.command('Driver.reload', async () => {
     await spec.reload()
   })
-
-  // utils
-
-  function derefArgs(arg) {
-    const derefArg = []
-    if (Array.isArray(arg)) {
-      for (const argument of arg) {
-        if (Array.isArray(argument)) {
-          derefArg.push(derefArgs(argument))
-        } else {
-          derefArg.push(refer.deref(argument))
-        }
-      }
-      return derefArg
-    } else if (typeof arg === 'object') {
-      for (const [key, value] of Object.entries(arg)) {
-        derefArg[key] = refer.deref(value)
-      }
-      return derefArg
-    } else {
-      return arg
-    }
-  }
 }
 
 module.exports = {socketCommands}

package/src/browser/spec-driver.ts

@@ -13,7 +13,11 @@ export function executeScript(context: Context, script: string, arg: any): any {
   }
 
   const executor = new context.defaultView.Function('arg', scriptToExecute)
-  return executor(arg)
+  try {
+    return executor(arg)
+  } catch (e) {
+    throw new Error(`Failed to execute script: ${e}`) // stringifiying prevents [object Object] errors
+  }
 }
 
 export function mainContext(): Context {