@itwin/rpcinterface-full-stack-tests 5.3.0-dev.3 → 5.3.0-dev.4

package/lib/dist/bundled-tests.js

@@ -185298,10 +185298,14 @@ class StandardQuantityTypeDefinition {
  */
 class QuantityTypeFormatsProvider {
     onFormatsChanged = new _itwin_core_bentley__WEBPACK_IMPORTED_MODULE_0__.BeEvent();
+    _removeListeners = [];
     constructor() {
-        _IModelApp__WEBPACK_IMPORTED_MODULE_3__.IModelApp.quantityFormatter.onActiveFormattingUnitSystemChanged.addListener(() => {
+        this._removeListeners.push(_IModelApp__WEBPACK_IMPORTED_MODULE_3__.IModelApp.quantityFormatter.onActiveFormattingUnitSystemChanged.addListener(() => {
             this.onFormatsChanged.raiseEvent({ formatsChanged: "all" });
-        });
+        }));
+    }
+    [Symbol.dispose]() {
+        this._removeListeners.forEach(listener => listener());
     }
     _kindOfQuantityMap = new Map([
         ["AecUnits.LENGTH", QuantityType.Length],
@@ -334943,8 +334947,8 @@ class Formatter {
      */
     static formatComposite(magnitude, spec) {
         const compositeStrings = [];
-        // Caller will deal with appending +||-||() value sign as specified by formatting options so just format positive value
-        let posMagnitude = Math.abs(magnitude);
+        let isNegative = false;
+        let remainingMagnitude = magnitude;
         for (let i = 0; i < spec.unitConversions.length; i++) {
             const currentLabel = spec.unitConversions[i].label;
             const unitConversion = spec.unitConversions[i].conversion;
@@ -334954,40 +334958,43 @@ class Formatter {
                 throw new _Exception__WEBPACK_IMPORTED_MODULE_1__.QuantityError(_Exception__WEBPACK_IMPORTED_MODULE_1__.QuantityStatus.InvalidCompositeFormat, `The Format ${spec.format.name} has a invalid unit specification.`);
             let unitValue = 0.0;
             if (spec.format.type === _FormatEnums__WEBPACK_IMPORTED_MODULE_2__.FormatType.Ratio) {
-                if (1 !== spec.format.units.length)
+                if (1 !== (spec.format.units?.length ?? 0))
                     throw new _Exception__WEBPACK_IMPORTED_MODULE_1__.QuantityError(_Exception__WEBPACK_IMPORTED_MODULE_1__.QuantityStatus.InvalidCompositeFormat, `The Format '${spec.format.name}' with type 'ratio' must have exactly one unit.`);
                 try {
-                    unitValue = (0,_Quantity__WEBPACK_IMPORTED_MODULE_3__.applyConversion)(posMagnitude, unitConversion) + this.FPV_MINTHRESHOLD;
+                    unitValue = (0,_Quantity__WEBPACK_IMPORTED_MODULE_3__.applyConversion)(remainingMagnitude, unitConversion) + this.FPV_MINTHRESHOLD;
                 }
                 catch (e) {
                     // The "InvertingZero" error is thrown when the value is zero and the conversion factor is inverted.
                     // For ratio, we actually want to support this corner case and return "1:0" as the formatted value.
                     if (e instanceof _Exception__WEBPACK_IMPORTED_MODULE_1__.QuantityError && e.errorNumber === _Exception__WEBPACK_IMPORTED_MODULE_1__.QuantityStatus.InvertingZero) {
-                        return "1:0";
+                        return { componentText: "1:0", isNegative: false };
                     }
                 }
                 compositeStrings.push(this.formatRatio(unitValue, spec));
+                isNegative = unitValue < 0;
                 continue;
             }
-            unitValue = (0,_Quantity__WEBPACK_IMPORTED_MODULE_3__.applyConversion)(posMagnitude, unitConversion) + this.FPV_MINTHRESHOLD;
+            unitValue = (0,_Quantity__WEBPACK_IMPORTED_MODULE_3__.applyConversion)(remainingMagnitude, unitConversion) + this.FPV_MINTHRESHOLD;
             if (0 === i) {
+                // Only set isNegative from the first (major) unit conversion
+                isNegative = unitValue < 0;
                 const precisionScale = Math.pow(10, 8); // use a fixed round off precision of 8 to avoid loss of precision in actual magnitude
                 unitValue = Math.floor(unitValue * precisionScale + FPV_ROUNDFACTOR) / precisionScale;
                 if ((Math.abs(unitValue) < 0.0001) && spec.format.hasFormatTraitSet(_FormatEnums__WEBPACK_IMPORTED_MODULE_2__.FormatTraits.ZeroEmpty))
-                    return "";
+                    return { componentText: "", isNegative: false };
             }
-            if (i < spec.format.units.length - 1) {
-                const wholePart = Math.floor(unitValue);
-                const componentText = Formatter.formatCompositePart(wholePart, false, currentLabel, spec);
-                posMagnitude = unitValue - wholePart;
+            if (i < (spec.format.units?.length ?? 0) - 1) {
+                const wholePart = Math.trunc(unitValue);
+                const componentText = Formatter.formatCompositePart(Math.abs(wholePart), false, currentLabel, spec);
+                remainingMagnitude = unitValue - wholePart;
                 compositeStrings.push(componentText);
             }
             else {
-                const componentText = Formatter.formatCompositePart(unitValue, true, currentLabel, spec);
+                const componentText = Formatter.formatCompositePart(Math.abs(unitValue), true, currentLabel, spec);
                 compositeStrings.push(componentText);
             }
         }
-        return compositeStrings.join(spec.format.spacerOrDefault);
+        return { componentText: compositeStrings.join(spec.format.spacerOrDefault), isNegative };
     }
     /** Format a quantity value into a single text string. Imitate how formatting done by server method NumericFormatSpec::FormatDouble.
      *  @param magnitude   quantity value
@@ -335116,45 +335123,58 @@ class Formatter {
         }
         return value;
     }
-    /** Format a quantity value into a single text string based on the current format specification of this class.
-     *  @param magnitude   defines the value to spec.format.
-     *  @param spec      A FormatterSpec object the defines specification for the magnitude and unit conversions for the formatter.
+    /** Helper function to apply sign formatting based on showSignOption
+     *  @param isNegative   whether the value should be treated as negative
+     *  @param showSignOption   the sign display option
+     *  @param formatType   the format type (to handle bearing/azimuth exceptions)
+     *  @returns object containing prefix and suffix strings
      */
-    static formatQuantity(magnitude, spec) {
-        const valueIsNegative = magnitude < 0.0;
+    static applySignFormatting(isNegative, showSignOption, formatType) {
         let prefix = "";
         let suffix = "";
-        let formattedValue = "";
-        if (spec.format.type === _FormatEnums__WEBPACK_IMPORTED_MODULE_2__.FormatType.Bearing || spec.format.type === _FormatEnums__WEBPACK_IMPORTED_MODULE_2__.FormatType.Azimuth) {
-            const result = this.processBearingAndAzimuth(magnitude, spec);
-            magnitude = result.magnitude;
-            prefix = result.prefix ?? "";
-            suffix = result.suffix ?? "";
-        }
-        switch (spec.format.showSignOption) {
+        switch (showSignOption) {
             case _FormatEnums__WEBPACK_IMPORTED_MODULE_2__.ShowSignOption.NegativeParentheses:
-                if (valueIsNegative) {
-                    prefix += "(";
-                    suffix = `)${suffix}`;
+                if (isNegative) {
+                    prefix = "(";
+                    suffix = ")";
                 }
                 break;
             case _FormatEnums__WEBPACK_IMPORTED_MODULE_2__.ShowSignOption.OnlyNegative:
-                if (valueIsNegative && spec.format.type !== _FormatEnums__WEBPACK_IMPORTED_MODULE_2__.FormatType.Bearing && spec.format.type !== _FormatEnums__WEBPACK_IMPORTED_MODULE_2__.FormatType.Azimuth)
-                    prefix += "-";
+                if (isNegative && formatType !== _FormatEnums__WEBPACK_IMPORTED_MODULE_2__.FormatType.Bearing && formatType !== _FormatEnums__WEBPACK_IMPORTED_MODULE_2__.FormatType.Azimuth) {
+                    prefix = "-";
+                }
                 break;
             case _FormatEnums__WEBPACK_IMPORTED_MODULE_2__.ShowSignOption.SignAlways:
-                if (valueIsNegative)
-                    prefix += "-";
-                else
-                    prefix += "+";
+                prefix = isNegative ? "-" : "+";
                 break;
             case _FormatEnums__WEBPACK_IMPORTED_MODULE_2__.ShowSignOption.NoSign:
             default:
                 break;
         }
+        return { prefix, suffix };
+    }
+    /** Format a quantity value into a single text string based on the current format specification of this class.
+     *  @param magnitude   defines the value to spec.format.
+     *  @param spec      A FormatterSpec object the defines specification for the magnitude and unit conversions for the formatter.
+     */
+    static formatQuantity(magnitude, spec) {
+        let valueIsNegative = magnitude < 0.0;
+        let prefix = "";
+        let suffix = "";
+        let formattedValue = "";
+        // Handle bearing/azimuth special formatting
+        if (spec.format.type === _FormatEnums__WEBPACK_IMPORTED_MODULE_2__.FormatType.Bearing || spec.format.type === _FormatEnums__WEBPACK_IMPORTED_MODULE_2__.FormatType.Azimuth) {
+            const result = this.processBearingAndAzimuth(magnitude, spec);
+            magnitude = result.magnitude;
+            prefix = result.prefix ?? "";
+            suffix = result.suffix ?? "";
+        }
         let formattedMagnitude = "";
         if (spec.format.hasUnits) {
-            formattedMagnitude = Formatter.formatComposite(magnitude, spec);
+            const compositeResult = Formatter.formatComposite(magnitude, spec);
+            formattedMagnitude = compositeResult.componentText;
+            // Override the sign detection with the composite conversion result
+            valueIsNegative = compositeResult.isNegative;
         }
         else {
             // unitless quantity
@@ -335165,7 +335185,12 @@ class Formatter {
                 else
                     formattedMagnitude = formattedMagnitude + spec.format.uomSeparator + spec.unitConversions[0].label;
             }
+            // For unitless quantities, keep original sign detection
         }
+        // Apply sign formatting based on the final determined sign
+        const signFormatting = this.applySignFormatting(valueIsNegative, spec.format.showSignOption, spec.format.type);
+        prefix += signFormatting.prefix;
+        suffix = signFormatting.suffix + suffix;
         // add Sign prefix and suffix as necessary
         if ((prefix.length > 0 || suffix.length > 0) && formattedMagnitude.length > 0)
             formattedValue = prefix + formattedMagnitude + suffix;
@@ -335186,7 +335211,8 @@ class Formatter {
                 magnitude -= quarterRevolution;
                 quadrant++;
             }
-            let prefix, suffix;
+            let prefix = "";
+            let suffix = "";
             // Quadrants are
             // 3 0
             // 2 1
@@ -335213,7 +335239,7 @@ class Formatter {
                     prefix = "N";
                 }
             }
-            return { magnitude, prefix, suffix: suffix };
+            return { magnitude, prefix, suffix };
         }
         if (type === _FormatEnums__WEBPACK_IMPORTED_MODULE_2__.FormatType.Azimuth) {
             let azimuthBase = 0; // default base is North
@@ -337680,7 +337706,7 @@ class TestContext {
         this.initializeRpcInterfaces({ title: this.settings.Backend.name, version: this.settings.Backend.version });
         const iModelClient = new imodels_client_management_1.IModelsClient({ api: { baseUrl: `https://${process.env.IMJS_URL_PREFIX ?? ""}api.bentley.com/imodels` } });
         await core_frontend_1.NoRenderApp.startup({
-            applicationVersion: "5.3.0-dev.3",
+            applicationVersion: "5.3.0-dev.4",
             applicationId: this.settings.gprid,
             authorizationClient: new frontend_1.TestFrontendAuthorizationClient(this.serviceAuthToken),
             hubAccess: new imodels_access_frontend_1.FrontendIModelsAccess(iModelClient),
@@ -362947,7 +362973,7 @@ var loadLanguages = instance.loadLanguages;
 /***/ ((module) => {
 
 "use strict";
-module.exports = /*#__PURE__*/JSON.parse('{"name":"@itwin/core-frontend","version":"5.3.0-dev.3","description":"iTwin.js frontend components","main":"lib/cjs/core-frontend.js","module":"lib/esm/core-frontend.js","typings":"lib/cjs/core-frontend","license":"MIT","scripts":{"build":"npm run -s copy:public && npm run -s build:cjs && npm run -s build:esm && npm run -s webpackWorkers && npm run -s copy:workers && npm run -s copy:draco","build:cjs":"npm run -s copy:js:cjs && tsc 1>&2 --outDir lib/cjs","build:esm":"npm run -s copy:js:esm && tsc 1>&2 --module ES2022 --outDir lib/esm","clean":"rimraf -g lib .rush/temp/package-deps*.json","copy:public":"cpx \\"./src/public/**/*\\" ./lib/public","copy:js:cjs":"cpx \\"./src/**/*.js\\" ./lib/cjs","copy:js:esm":"cpx \\"./src/**/*.js\\" ./lib/esm","copy:workers":"cpx \\"./lib/workers/webpack/parse-imdl-worker.js\\" ./lib/public/scripts","copy:draco":"cpx \\"./node_modules/@loaders.gl/draco/dist/libs/*\\" ./lib/public/scripts","docs":"betools docs --json=../../generated-docs/core/core-frontend/file.json --tsIndexFile=./core-frontend.ts --onlyJson --excludes=webgl/**/*,**/map/*.d.ts,**/tile/*.d.ts,**/*-css.ts","extract-api":"betools extract-api --entry=core-frontend && npm run extract-extension-api","extract-extension-api":"eslint --no-inline-config -c extraction.eslint.config.js  \\"./src/**/*.ts\\" 1>&2","lint":"eslint \\"./src/**/*.ts\\" 1>&2","lint-fix":"eslint --fix -f visualstudio \\"./src/**/*.ts\\" 1>&2","lint-deprecation":"eslint --fix -f visualstudio --no-inline-config -c ../../common/config/eslint/eslint.config.deprecation-policy.js \\"./src/**/*.ts\\"","pseudolocalize":"betools pseudolocalize --englishDir ./src/public/locales/en --out ./public/locales/en-PSEUDO","test":"npm run webpackTestWorker && vitest --run","cover":"npm run webpackTestWorker && vitest --run","webpackTests":"webpack --config ./src/test/utils/webpack.config.js 1>&2 && npm run -s webpackTestWorker","webpackTestWorker":"webpack --config ./src/test/worker/webpack.config.js 1>&2 && cpx \\"./lib/test/test-worker.js\\" ./lib/test","webpackWorkers":"webpack --config ./src/workers/ImdlParser/webpack.config.js 1>&2"},"repository":{"type":"git","url":"https://github.com/iTwin/itwinjs-core.git","directory":"core/frontend"},"keywords":["Bentley","BIM","iModel","digital-twin","iTwin"],"author":{"name":"Bentley Systems, Inc.","url":"http://www.bentley.com"},"peerDependencies":{"@itwin/appui-abstract":"workspace:*","@itwin/core-bentley":"workspace:*","@itwin/core-common":"workspace:*","@itwin/core-geometry":"workspace:*","@itwin/core-orbitgt":"workspace:*","@itwin/core-quantity":"workspace:*","@itwin/ecschema-metadata":"workspace:*","@itwin/ecschema-rpcinterface-common":"workspace:*"},"//devDependencies":["NOTE: All peerDependencies should also be listed as devDependencies since peerDependencies are not considered by npm install","NOTE: All tools used by scripts in this package must be listed as devDependencies"],"devDependencies":{"@itwin/appui-abstract":"workspace:*","@itwin/build-tools":"workspace:*","@itwin/core-bentley":"workspace:*","@itwin/core-common":"workspace:*","@itwin/core-geometry":"workspace:*","@itwin/core-orbitgt":"workspace:*","@itwin/core-quantity":"workspace:*","@itwin/ecschema-metadata":"workspace:*","@itwin/ecschema-rpcinterface-common":"workspace:*","@itwin/eslint-plugin":"5.2.2-dev.2","@types/chai-as-promised":"^7","@types/draco3d":"^1.4.10","@types/sinon":"^17.0.2","@vitest/browser":"^3.0.6","@vitest/coverage-v8":"^3.0.6","cpx2":"^8.0.0","eslint":"^9.31.0","glob":"^10.3.12","playwright":"~1.47.1","rimraf":"^6.0.1","sinon":"^17.0.2","source-map-loader":"^5.0.0","typescript":"~5.6.2","typemoq":"^2.1.0","vitest":"^3.0.6","vite-multiple-assets":"^1.3.1","vite-plugin-static-copy":"2.2.0","webpack":"^5.97.1"},"//dependencies":["NOTE: these dependencies should be only for things that DO NOT APPEAR IN THE API","NOTE: core-frontend should remain UI technology agnostic, so no react/angular dependencies are allowed"],"dependencies":{"@itwin/cloud-agnostic-core":"^2.2.4","@itwin/object-storage-core":"^2.3.0","@itwin/core-i18n":"workspace:*","@itwin/webgl-compatibility":"workspace:*","@loaders.gl/core":"^4.3.4","@loaders.gl/draco":"^4.3.4","fuse.js":"^3.3.0","wms-capabilities":"0.4.0"}}');
+module.exports = /*#__PURE__*/JSON.parse('{"name":"@itwin/core-frontend","version":"5.3.0-dev.4","description":"iTwin.js frontend components","main":"lib/cjs/core-frontend.js","module":"lib/esm/core-frontend.js","typings":"lib/cjs/core-frontend","license":"MIT","scripts":{"build":"npm run -s copy:public && npm run -s build:cjs && npm run -s build:esm && npm run -s webpackWorkers && npm run -s copy:workers && npm run -s copy:draco","build:cjs":"npm run -s copy:js:cjs && tsc 1>&2 --outDir lib/cjs","build:esm":"npm run -s copy:js:esm && tsc 1>&2 --module ES2022 --outDir lib/esm","clean":"rimraf -g lib .rush/temp/package-deps*.json","copy:public":"cpx \\"./src/public/**/*\\" ./lib/public","copy:js:cjs":"cpx \\"./src/**/*.js\\" ./lib/cjs","copy:js:esm":"cpx \\"./src/**/*.js\\" ./lib/esm","copy:workers":"cpx \\"./lib/workers/webpack/parse-imdl-worker.js\\" ./lib/public/scripts","copy:draco":"cpx \\"./node_modules/@loaders.gl/draco/dist/libs/*\\" ./lib/public/scripts","docs":"betools docs --json=../../generated-docs/core/core-frontend/file.json --tsIndexFile=./core-frontend.ts --onlyJson --excludes=webgl/**/*,**/map/*.d.ts,**/tile/*.d.ts,**/*-css.ts","extract-api":"betools extract-api --entry=core-frontend && npm run extract-extension-api","extract-extension-api":"eslint --no-inline-config -c extraction.eslint.config.js  \\"./src/**/*.ts\\" 1>&2","lint":"eslint \\"./src/**/*.ts\\" 1>&2","lint-fix":"eslint --fix -f visualstudio \\"./src/**/*.ts\\" 1>&2","lint-deprecation":"eslint --fix -f visualstudio --no-inline-config -c ../../common/config/eslint/eslint.config.deprecation-policy.js \\"./src/**/*.ts\\"","pseudolocalize":"betools pseudolocalize --englishDir ./src/public/locales/en --out ./public/locales/en-PSEUDO","test":"npm run webpackTestWorker && vitest --run","cover":"npm run webpackTestWorker && vitest --run","webpackTests":"webpack --config ./src/test/utils/webpack.config.js 1>&2 && npm run -s webpackTestWorker","webpackTestWorker":"webpack --config ./src/test/worker/webpack.config.js 1>&2 && cpx \\"./lib/test/test-worker.js\\" ./lib/test","webpackWorkers":"webpack --config ./src/workers/ImdlParser/webpack.config.js 1>&2"},"repository":{"type":"git","url":"https://github.com/iTwin/itwinjs-core.git","directory":"core/frontend"},"keywords":["Bentley","BIM","iModel","digital-twin","iTwin"],"author":{"name":"Bentley Systems, Inc.","url":"http://www.bentley.com"},"peerDependencies":{"@itwin/appui-abstract":"workspace:*","@itwin/core-bentley":"workspace:*","@itwin/core-common":"workspace:*","@itwin/core-geometry":"workspace:*","@itwin/core-orbitgt":"workspace:*","@itwin/core-quantity":"workspace:*","@itwin/ecschema-metadata":"workspace:*","@itwin/ecschema-rpcinterface-common":"workspace:*"},"//devDependencies":["NOTE: All peerDependencies should also be listed as devDependencies since peerDependencies are not considered by npm install","NOTE: All tools used by scripts in this package must be listed as devDependencies"],"devDependencies":{"@itwin/appui-abstract":"workspace:*","@itwin/build-tools":"workspace:*","@itwin/core-bentley":"workspace:*","@itwin/core-common":"workspace:*","@itwin/core-geometry":"workspace:*","@itwin/core-orbitgt":"workspace:*","@itwin/core-quantity":"workspace:*","@itwin/ecschema-metadata":"workspace:*","@itwin/ecschema-rpcinterface-common":"workspace:*","@itwin/eslint-plugin":"5.2.2-dev.2","@types/chai-as-promised":"^7","@types/draco3d":"^1.4.10","@types/sinon":"^17.0.2","@vitest/browser":"^3.0.6","@vitest/coverage-v8":"^3.0.6","cpx2":"^8.0.0","eslint":"^9.31.0","glob":"^10.3.12","playwright":"~1.47.1","rimraf":"^6.0.1","sinon":"^17.0.2","source-map-loader":"^5.0.0","typescript":"~5.6.2","typemoq":"^2.1.0","vitest":"^3.0.6","vite-multiple-assets":"^1.3.1","vite-plugin-static-copy":"2.2.0","webpack":"^5.97.1"},"//dependencies":["NOTE: these dependencies should be only for things that DO NOT APPEAR IN THE API","NOTE: core-frontend should remain UI technology agnostic, so no react/angular dependencies are allowed"],"dependencies":{"@itwin/cloud-agnostic-core":"^2.2.4","@itwin/object-storage-core":"^2.3.0","@itwin/core-i18n":"workspace:*","@itwin/webgl-compatibility":"workspace:*","@loaders.gl/core":"^4.3.4","@loaders.gl/draco":"^4.3.4","fuse.js":"^3.3.0","wms-capabilities":"0.4.0"}}');
 
 /***/ }),