@applitools/core-base - npm Package Compare versions

Comparing version 1.7.1 to 1.7.2

CHANGELOG.md

 # Changelog
 
+## [1.7.2](https://github.com/applitools/eyes.sdk.javascript1/compare/js/core-base@1.7.1...js/core-base@1.7.2) (2023-10-17)
+
+
+### Code Refactoring
+
+* disallow usage of global Buffer ([#1957](https://github.com/applitools/eyes.sdk.javascript1/issues/1957)) ([adcc082](https://github.com/applitools/eyes.sdk.javascript1/commit/adcc082f20f6b92e819b96424e995d9a69d99758))
+
+
+### Dependencies
+
+* @applitools/utils bumped to 1.6.2
+  #### Code Refactoring
+
+  * disallow usage of global Buffer ([#1957](https://github.com/applitools/eyes.sdk.javascript1/issues/1957)) ([adcc082](https://github.com/applitools/eyes.sdk.javascript1/commit/adcc082f20f6b92e819b96424e995d9a69d99758))
+* @applitools/logger bumped to 2.0.12
+  #### Code Refactoring
+
+  * disallow usage of global Buffer ([#1957](https://github.com/applitools/eyes.sdk.javascript1/issues/1957)) ([adcc082](https://github.com/applitools/eyes.sdk.javascript1/commit/adcc082f20f6b92e819b96424e995d9a69d99758))
+
+
+
+* @applitools/req bumped to 1.6.2
+  #### Code Refactoring
+
+  * disallow usage of global Buffer ([#1957](https://github.com/applitools/eyes.sdk.javascript1/issues/1957)) ([adcc082](https://github.com/applitools/eyes.sdk.javascript1/commit/adcc082f20f6b92e819b96424e995d9a69d99758))
+
+
+
+* @applitools/image bumped to 1.1.7
+  #### Bug Fixes
+
+  * avoid using ascii text decoder ([1b68d39](https://github.com/applitools/eyes.sdk.javascript1/commit/1b68d3945d96b17b9ab1f1a87d352206fd0c81d6))
+
+
+  #### Code Refactoring
+
+  * disallow usage of global Buffer ([#1957](https://github.com/applitools/eyes.sdk.javascript1/issues/1957)) ([adcc082](https://github.com/applitools/eyes.sdk.javascript1/commit/adcc082f20f6b92e819b96424e995d9a69d99758))
+
+
+
+
 ## [1.7.1](https://github.com/applitools/eyes.sdk.javascript1/compare/js/core-base@1.7.0...js/core-base@1.7.1) (2023-09-29)
@@ -4,0 +45,0 @@

dist/server/req-eyes.js

@@ -105,3 +105,4 @@ "use strict";
                     : options.expected !== response.status)) {
-                throw new Error(`Request "${options === null || options === void 0 ? void 0 : options.name}" that was sent to the address "[${request.method}]${request.url}" failed due to unexpected status ${response.statusText}(${response.status})`);
+                const requestId = request.headers.get('x-applitools-eyes-client-request-id');
+                throw new Error(`Request "${options === null || options === void 0 ? void 0 : options.name}" [${requestId}] that was sent to the address "[${request.method}]${request.url}" failed due to unexpected status ${response.statusText}(${response.status})`);
             }
@@ -108,0 +109,0 @@ },

dist/server/requests.js

@@ -86,6 +86,8 @@ "use strict";
                             os: settings.environment.os,
-                            osInfo: settings.environment.osInfo,
+                            osInfo: settings.environment.displayOs,
                             hostingApp: settings.environment.hostingApp,
-                            hostingAppInfo: settings.environment.hostingAppInfo,
-                            displaySize: settings.environment.viewportSize && utils.geometry.round(settings.environment.viewportSize),
+                            hostingAppInfo: settings.environment.displayHostingApp,
+                            displaySize: settings.environment.viewportSize
+                                ? utils.geometry.round(settings.environment.viewportSize)
+                                : { width: 0, height: 0 },
                             inferred: settings.environment.userAgent && `useragent:${settings.environment.userAgent}`,
@@ -110,6 +112,8 @@ }),
         const test = await response.json().then(async (result) => {
-            var _a, _b, _c, _d, _e;
+            var _a, _b, _c;
             const account = await accountPromise;
             return {
                 testId: result.id,
+                // TODO revisit the need for it
+                testName: settings.testName,
                 userTestId: settings.userTestId,
@@ -119,7 +123,5 @@ batchId: (_b = (_a = settings.batch) === null || _a === void 0 ? void 0 : _a.id) !== null && _b !== void 0 ? _b : result.batchId,
                 sessionId: result.sessionId,
-                renderEnvironmentId: (_c = settings.environment) === null || _c === void 0 ? void 0 : _c.renderEnvironmentId,
-                renderer: (_d = settings.environment) === null || _d === void 0 ? void 0 : _d.renderer,
                 initializedAt,
                 appId: settings.appName,
-                isNew: (_e = result.isNew) !== null && _e !== void 0 ? _e : response.status === 201,
+                isNew: (_c = result.isNew) !== null && _c !== void 0 ? _c : response.status === 201,
                 keepBatchOpen: !!settings.keepBatchOpen,
@@ -131,6 +133,6 @@ keepIfDuplicate: !!settings.baselineEnvName,
                 uploadUrl: account.uploadUrl,
-                renderEnvironmentsUrl: account.renderEnvironmentsUrl,
+                supportedEnvironmentsUrl: account.supportedEnvironmentsUrl,
                 stitchingServiceUrl: account.stitchingServiceUrl,
-                testName: settings.testName,
                 account,
+                environment: settings.environment,
             };
@@ -176,6 +178,8 @@ });
                             os: settings.environment.os,
-                            osInfo: settings.environment.osInfo,
+                            osInfo: settings.environment.displayOs,
                             hostingApp: settings.environment.hostingApp,
-                            hostingAppInfo: settings.environment.hostingAppInfo,
-                            displaySize: settings.environment.viewportSize && utils.geometry.round(settings.environment.viewportSize),
+                            hostingAppInfo: settings.environment.displayHostingApp,
+                            displaySize: settings.environment.viewportSize
+                                ? utils.geometry.round(settings.environment.viewportSize)
+                                : { width: 0, height: 0 },
                             inferred: settings.environment.userAgent && `useragent:${settings.environment.userAgent}`,
@@ -205,2 +209,3 @@ }),
                 account,
+                environment: settings.environment,
             };
@@ -317,3 +322,3 @@ });
         const result = await response.json().then((result) => {
-            const { serviceUrl: ufgServerUrl, accessToken, mobileDevicesListUrl: renderEnvironmentsUrl, resultsUrl: uploadUrl, ...rest } = result;
+            const { serviceUrl: ufgServerUrl, accessToken, mobileDevicesListUrl: supportedEnvironmentsUrl, resultsUrl: uploadUrl, ...rest } = result;
             return {
@@ -334,3 +339,3 @@ eyesServer: {
                 },
-                renderEnvironmentsUrl,
+                supportedEnvironmentsUrl,
                 uploadUrl,
@@ -337,0 +342,0 @@ ...rest,

dist/utils/transform-dom-mapping.js

@@ -49,3 +49,3 @@ "use strict";
             const response = await (0, req_1.req)(str, { proxy: settings.autProxy });
-            settings.domMapping = Buffer.from(await response.arrayBuffer());
+            settings.domMapping = new Uint8Array(await response.arrayBuffer());
         }
@@ -52,0 +52,0 @@ else {

package.json

 {
   "name": "@applitools/core-base",
-  "version": "1.7.1",
+  "version": "1.7.2",
   "homepage": "https://applitools.com",
@@ -45,6 +45,6 @@ "bugs": {
   "dependencies": {
-    "@applitools/image": "1.1.6",
-    "@applitools/logger": "2.0.11",
-    "@applitools/req": "1.6.1",
-    "@applitools/utils": "1.6.1",
+    "@applitools/image": "1.1.7",
+    "@applitools/logger": "2.0.12",
+    "@applitools/req": "1.6.2",
+    "@applitools/utils": "1.6.2",
     "abort-controller": "3.0.0",
@@ -51,0 +51,0 @@ "throat": "6.0.2"

types/types.d.ts

@@ -1,2 +0,1 @@
-/// <reference types="node" />
 import { type MaybeArray, type Region, type Size, type Location } from '@applitools/utils';
@@ -105,2 +104,3 @@ import { type Logger } from '@applitools/logger';
     testId: string;
+    testName: string;
     userTestId: string;
@@ -111,4 +111,2 @@ batchId: string;
     appId: string;
-    renderEnvironmentId?: string;
-    renderer?: Record<string, any>;
     initializedAt: string;
@@ -120,8 +118,8 @@ isNew: boolean;
     ufgServer: UFGServerSettings;
+    supportedEnvironmentsUrl: string;
     uploadUrl: string;
-    renderEnvironmentsUrl: string;
     stitchingServiceUrl: string;
     resultsUrl: string;
     account: Account;
-    testName: string;
+    environment?: Environment;
 }
@@ -140,2 +138,3 @@ export interface FunctionalTest {
     account: Account;
+    environment?: Environment;
 }
@@ -145,4 +144,4 @@ export interface Account {
     ufgServer: UFGServerSettings;
+    supportedEnvironmentsUrl: string;
     stitchingServiceUrl: string;
-    renderEnvironmentsUrl: string;
     uploadUrl: string;
@@ -182,12 +181,11 @@ maxImageHeight: number;
 export type Environment = {
-    renderEnvironmentId?: string;
-    ecSessionId?: string;
+    environmentId?: string;
     os?: string;
-    osInfo?: string;
+    displayOs?: string;
     hostingApp?: string;
-    hostingAppInfo?: string;
+    displayHostingApp?: string;
     deviceName?: string;
     viewportSize?: Size;
     userAgent?: string;
-    renderer?: Record<string, any>;
+    ecSessionId?: string;
     rawEnvironment?: Record<string, any>;
@@ -328,3 +326,3 @@ properties?: CustomProperty[];
     };
-    domMapping?: Buffer | URL | string;
+    domMapping?: Uint8Array | URL | string;
     pageId?: string;
@@ -331,0 +329,0 @@ /** @internal */

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

149423

increased by1.26%

Lines of code

2564

increased by0.16%

Dependency changes

• + Added@applitools/image@1.1.7(transitive)

• + Added@applitools/logger@2.0.12(transitive)

• + Added@applitools/req@1.6.2(transitive)

• + Added@applitools/utils@1.6.2(transitive)

• - Removed@applitools/image@1.1.6(transitive)

• - Removed@applitools/logger@2.0.11(transitive)

• - Removed@applitools/req@1.6.1(transitive)

• - Removed@applitools/utils@1.6.1(transitive)

• Updated@applitools/image@1.1.7

• Updated@applitools/logger@2.0.12

• Updated@applitools/req@1.6.2

• Updated@applitools/utils@1.6.2