@sapui5/sap.fe.test 1.102.3 → 1.105.0

package/src/sap/fe/test/api/HeaderAssertions.js

@@ -45,7 +45,6 @@ sap.ui.define(
 		 * @param {string | sap.fe.test.api.ActionIdentifier} vActionIdentifier The identifier of the action
 		 * @param {object} [mState] Defines the expected state of the button
 		 * @returns {object} The result of the {@link sap.ui.test.Opa5#waitFor} function, to be used for chained statements
-		 *
 		 * @public
 		 */
 		HeaderAssertions.prototype.iCheckAction = function (vActionIdentifier, mState) {
@@ -63,7 +62,6 @@ sap.ui.define(
 		 *
 		 * @param {object} [mState] Defines the expected state of the button
 		 * @returns {object} The result of the {@link sap.ui.test.Opa5#waitFor} function, to be used for chained statements
-		 *
 		 * @public
 		 */
 		HeaderAssertions.prototype.iCheckEdit = function (mState) {
@@ -75,7 +73,6 @@ sap.ui.define(
 		 *
 		 * @param {object} [mState] Defines the expected state of the button
 		 * @returns {object} The result of the {@link sap.ui.test.Opa5#waitFor} function, to be used for chained statements
-		 *
 		 * @public
 		 */
 		HeaderAssertions.prototype.iCheckDelete = function (mState) {
@@ -87,7 +84,6 @@ sap.ui.define(
 		 *
 		 * @param {object} [mState] Defines the expected state of the button
 		 * @returns {object} The result of the {@link sap.ui.test.Opa5#waitFor} function, to be used for chained statements
-		 *
 		 * @public
 		 */
 		HeaderAssertions.prototype.iCheckRelatedApps = function (mState) {
@@ -99,7 +95,6 @@ sap.ui.define(
 		 *
 		 * @param {object | string} vAction The state map or label of the action
 		 * @returns {object} The result of the {@link sap.ui.test.Opa5#waitFor} function, to be used for chained statements
-		 *
 		 * @public
 		 */
 		HeaderAssertions.prototype.iCheckMenuAction = function (vAction) {
@@ -112,7 +107,6 @@ sap.ui.define(
 		 * @param {number} iNumberOfItems The expected number of items
 		 * @param {boolean} [bIncludeHidden] Defines whether non-visible items should be counted
 		 * @returns {object} The result of the {@link sap.ui.test.Opa5#waitFor} function, to be used for chained statements
-		 *
 		 * @ui5-restricted
 		 */
 		HeaderAssertions.prototype.iCheckNumberOfHeaderContentItems = function (iNumberOfItems, bIncludeHidden) {
@@ -148,7 +142,6 @@ sap.ui.define(
 		 * </pre></code>
 		 * @param {object} [mState] Defines the expected state of the field
 		 * @returns {object} The result of the {@link sap.ui.test.Opa5#waitFor} function, to be used for chained statements
-		 *
 		 * @public
 		 */
 		HeaderAssertions.prototype.iCheckFieldInFieldGroup = function (vFieldIdentifier, vValue, mState) {
@@ -181,7 +174,6 @@ sap.ui.define(
 		 * @param {string} sTitle The title of the data point
 		 * @param {string} sValue The expected value of the data point
 		 * @returns {object} The result of the {@link sap.ui.test.Opa5#waitFor} function, to be used for chained statements
-		 *
 		 * @ui5-restricted
 		 */
 		HeaderAssertions.prototype.iCheckDataPoint = function (sTitle, sValue) {
@@ -191,13 +183,22 @@ sap.ui.define(
 				oHeaderContentBuilder
 					.has(
 						OpaBuilder.Matchers.childrenMatcher(
-							OpaBuilder.create(this)
-								.hasType("sap.m.ObjectNumber")
-								.hasProperties({ number: sValue })
-								.has(function (oObjectNumber) {
-									return oObjectNumber.getParent();
-								})
-								.hasAggregationProperties("items", { text: sTitle })
+							OpaBuilder.Matchers.some([
+								OpaBuilder.create(this)
+									.hasType("sap.m.ObjectNumber")
+									.hasProperties({ number: sValue })
+									.has(function (oObjectNumber) {
+										return oObjectNumber.getParent();
+									})
+									.hasAggregationProperties("items", { text: sTitle }),
+								OpaBuilder.create(this)
+									.hasType("sap.m.ObjectStatus")
+									.hasProperties({ text: sValue })
+									.has(function (oObjectNumber) {
+										return oObjectNumber.getParent();
+									})
+									.hasAggregationProperties("items", { text: sTitle })
+							])
 						)
 					)
 					.description(Utils.formatMessage("Seeing header data point '{0}' with value '{1}'", sTitle, sValue))
@@ -213,7 +214,6 @@ sap.ui.define(
 		 * @param {string} [sTitle] Title of the object page header
 		 * @param {string} [sDescription] Description of the object page header
 		 * @returns {object} The result of the {@link sap.ui.test.Opa5#waitFor} function, to be used for chained statements
-		 *
 		 * @public
 		 */
 		HeaderAssertions.prototype.iCheckTitle = function (sTitle, sDescription) {
@@ -244,7 +244,6 @@ sap.ui.define(
 		 *
 		 * @param {object} mState Defines the expected state of the button
 		 * @returns {object} The result of the {@link sap.ui.test.Opa5#waitFor} function, to be used for chained statements
-		 *
 		 * @public
 		 */
 		HeaderAssertions.prototype.iCheckPaginatorDown = function (mState) {
@@ -264,7 +263,6 @@ sap.ui.define(
 		 *
 		 * @param {object} mState Defines the expected state of the button
 		 * @returns {object} The result of the {@link sap.ui.test.Opa5#waitFor} function, to be used for chained statements
-		 *
 		 * @public
 		 */
 		HeaderAssertions.prototype.iCheckPaginatorUp = function (mState) {
@@ -283,8 +281,9 @@ sap.ui.define(
 		 * Checks a MicroChart shown in the header of an object page.
 		 *
 		 * TODO this function will not be public yet: Its signature doesn't fit the framework.
+		 *
 		 * @param {object|string} vMicroChartIdentifier Id/Type or Title of MicroChart
-		 * @param sUoMLabel UoM label of the MicroChart
+		 * @param {string} sUoMLabel UoM label of the MicroChart
 		 * @returns {object} The result of the {@link sap.ui.test.Opa5#waitFor} function, to be used for chained statements
 		 * @ui5-restricted
 		 */
@@ -340,7 +339,6 @@ sap.ui.define(
 		 * @param {sap.fe.test.api.HeaderFacetIdentifier} vFacetIdentifier The Identifier of the header facet
 		 * @param {object} [mState] Defines the expected state
 		 * @returns {object} The result of the {@link sap.ui.test.Opa5#waitFor} function, to be used for chained statements
-		 *
 		 * @public
 		 */
 		HeaderAssertions.prototype.iCheckHeaderFacet = function (vFacetIdentifier, mState) {
@@ -373,9 +371,7 @@ sap.ui.define(
 		 * This is the case for the main object page, which does not show breadcrumb links.
 		 * If <code>sLink</code> is not provided, a check is executed to ensure the breadcrumb control does not exist at all. This is the case for the flexible column layout
 		 * showing multiple floorplans at the same time.
-		 *
 		 * @returns {object} The result of the {@link sap.ui.test.Opa5#waitFor} function, to be used for chained statements
-		 *
 		 * @ui5-restricted
 		 */
 		HeaderAssertions.prototype.iCheckBreadCrumb = function (sLink) {
@@ -401,7 +397,6 @@ sap.ui.define(
 		 *
 		 * @param {object} [mState] Defines the expected state of the button
 		 * @returns {object} The result of the {@link sap.ui.test.Opa5#waitFor} function, to be used for chained statements
-		 *
 		 * @public
 		 */
 		HeaderAssertions.prototype.iCheckSaveAsTile = function (mState) {
@@ -427,7 +422,6 @@ sap.ui.define(
 		 *
 		 * @param {object} [mState] Defines the expected state of the button
 		 * @returns {object} The result of the {@link sap.ui.test.Opa5#waitFor} function, to be used for chained statements
-		 *
 		 * @public
 		 */
 		HeaderAssertions.prototype.iCheckSendEmail = function (mState) {
@@ -456,7 +450,6 @@ sap.ui.define(
 		 * @param {object | string} vLinkIdentifier The identifier of the field
 		 * @param {object} [mState] Defines the expected state of the link
 		 * @returns {object} The result of the {@link sap.ui.test.Opa5#waitFor} function, to be used for chained statements
-		 *
 		 * @private
 		 */
 		HeaderAssertions.prototype.iCheckLink = function (vLinkIdentifier, mState) {
@@ -480,7 +473,6 @@ sap.ui.define(
 		 *
 		 * @param {number} iExpectedNumber The expected number of users
 		 * @returns {object} The result of the {@link sap.ui.test.Opa5#waitFor} function, to be used for chained statements
-		 *
 		 * @ui5-restricted
 		 */
 		HeaderAssertions.prototype.iCheckNumberOfCollaboratingUsers = function (iExpectedNumber) {
@@ -507,7 +499,6 @@ sap.ui.define(
 		 *
 		 * @param {string} sInitials The initials of the user (displayed in the avatar)
 		 * @returns {object} The result of the {@link sap.ui.test.Opa5#waitFor} function, to be used for chained statements
-		 *
 		 * @ui5-restricted
 		 */
 		HeaderAssertions.prototype.iCheckCollaboratingUser = function (sInitials) {

package/src/sap/fe/test/api/HeaderAssertionsLR.js

@@ -12,7 +12,7 @@ sap.ui.define(
 		"sap/fe/test/builder/MdcFieldBuilder",
 		"./APIHelper"
 	],
-	function(HeaderLR, Utils, OpaBuilder, FEBuilder, FieldBuilder, APIHelper) {
+	function (HeaderLR, Utils, OpaBuilder, FEBuilder, FieldBuilder, APIHelper) {
 		"use strict";
 
 		/**
@@ -26,7 +26,7 @@ sap.ui.define(
 		 * @hideconstructor
 		 * @public
 		 */
-		var HeaderAssertionsLR = function(oHeaderBuilder, vHeaderDescription) {
+		var HeaderAssertionsLR = function (oHeaderBuilder, vHeaderDescription) {
 			return HeaderLR.call(this, oHeaderBuilder, vHeaderDescription);
 		};
 		HeaderAssertionsLR.prototype = Object.create(HeaderLR.prototype);
@@ -39,10 +39,9 @@ sap.ui.define(
 		 * @param {string | sap.fe.test.api.ActionIdentifier} [vActionIdentifier] The identifier of the action, or its label
 		 * @param {object} [mState] Defines the expected state of the button
 		 * @returns {object} The result of the {@link sap.ui.test.Opa5#waitFor} function, to be used for chained statements
-		 *
 		 * @public
 		 */
-		HeaderAssertionsLR.prototype.iCheckAction = function(vActionIdentifier, mState) {
+		HeaderAssertionsLR.prototype.iCheckAction = function (vActionIdentifier, mState) {
 			var aArguments = Utils.parseArguments([[Object, String], Object], arguments),
 				oOverflowToolbarBuilder = this.createOverflowToolbarBuilder(this._sPageId);
 			return this.prepareResult(
@@ -58,10 +57,9 @@ sap.ui.define(
 		 *
 		 * @param {object} [mState] Defines the expected state of the button
 		 * @returns {object} The result of the {@link sap.ui.test.Opa5#waitFor} function, to be used for chained statements
-		 *
 		 * @public
 		 */
-		HeaderAssertionsLR.prototype.iCheckSaveAsTile = function(mState) {
+		HeaderAssertionsLR.prototype.iCheckSaveAsTile = function (mState) {
 			var oOverflowToolbarBuilder = this.createOverflowToolbarBuilder(this._sPageId),
 				sShareId = "fe::Share";
 
@@ -84,10 +82,9 @@ sap.ui.define(
 		 *
 		 * @param {object} [mState] Defines the expected state of the button
 		 * @returns {object} The result of the {@link sap.ui.test.Opa5#waitFor} function, to be used for chained statements
-		 *
 		 * @public
 		 */
-		HeaderAssertionsLR.prototype.iCheckSendEmail = function(mState) {
+		HeaderAssertionsLR.prototype.iCheckSendEmail = function (mState) {
 			var oOverflowToolbarBuilder = this.createOverflowToolbarBuilder(this._sPageId),
 				sShareId = "fe::Share";
 

package/src/sap/fe/test/api/HeaderLR.js

@@ -12,7 +12,7 @@ sap.ui.define(
 		"sap/fe/test/builder/OverflowToolbarBuilder",
 		"sap/fe/core/helpers/StableIdHelper"
 	],
-	function(BaseAPI, Utils, OpaBuilder, FEBuilder, OverflowToolbarBuilder, StableIdHelper) {
+	function (BaseAPI, Utils, OpaBuilder, FEBuilder, OverflowToolbarBuilder, StableIdHelper) {
 		"use strict";
 
 		/**
@@ -26,7 +26,7 @@ sap.ui.define(
 		 * @hideconstructor
 		 * @private
 		 */
-		var HeaderLR = function(oHeaderBuilder, vHeaderDescription) {
+		var HeaderLR = function (oHeaderBuilder, vHeaderDescription) {
 			if (!Utils.isOfType(oHeaderBuilder, FEBuilder)) {
 				throw new Error("oHeaderBuilder parameter must be a FEBuilder instance");
 			}
@@ -43,28 +43,16 @@ sap.ui.define(
 		 *
 		 * @param {string} sPageId The id of page control.
 		 * @returns {object} OverflowToolbarBuilder object
-		 *
 		 * @ui5-restricted
 		 */
-		HeaderLR.prototype.createOverflowToolbarBuilder = function(sPageId) {
+		HeaderLR.prototype.createOverflowToolbarBuilder = function (sPageId) {
 			return OverflowToolbarBuilder.create(this.getOpaInstance())
 				.hasType("sap.m.OverflowToolbar")
-				.has(function(oOverflowToolbar) {
+				.has(function (oOverflowToolbar) {
 					return (
-						oOverflowToolbar
-							.getParent()
-							.getMetadata()
-							.getName() === "sap.f.DynamicPageTitle" &&
-						oOverflowToolbar
-							.getParent()
-							.getParent()
-							.getMetadata()
-							.getName() === "sap.f.DynamicPage" &&
-						oOverflowToolbar
-							.getParent()
-							.getParent()
-							.getId()
-							.endsWith(sPageId)
+						oOverflowToolbar.getParent().getMetadata().getName() === "sap.f.DynamicPageTitle" &&
+						oOverflowToolbar.getParent().getParent().getMetadata().getName() === "sap.f.DynamicPage" &&
+						oOverflowToolbar.getParent().getParent().getId().endsWith(sPageId)
 					);
 				});
 		};

package/src/sap/fe/test/api/KPICardAssertions.js

@@ -3,7 +3,7 @@
         (c) Copyright 2009-2021 SAP SE. All rights reserved
     
  */
-sap.ui.define(["./KPICardAPI", "sap/fe/test/Utils", "sap/fe/test/builder/FEBuilder"], function(KPICardAPI, Utils, FEBuilder) {
+sap.ui.define(["./KPICardAPI", "sap/fe/test/Utils", "sap/fe/test/builder/FEBuilder"], function (KPICardAPI, Utils, FEBuilder) {
 	"use strict";
 
 	/**
@@ -16,7 +16,7 @@ sap.ui.define(["./KPICardAPI", "sap/fe/test/Utils", "sap/fe/test/builder/FEBuild
 	 * @hideconstructor
 	 * @public
 	 */
-	var KPICardAssertions = function(oBuilderInstance) {
+	var KPICardAssertions = function (oBuilderInstance) {
 		return KPICardAPI.call(this, oBuilderInstance);
 	};
 	KPICardAssertions.prototype = Object.create(KPICardAPI.prototype);
@@ -29,11 +29,11 @@ sap.ui.define(["./KPICardAPI", "sap/fe/test/Utils", "sap/fe/test/builder/FEBuild
 	 * @param {string} sTitle The expected title
 	 * @returns {object} The result of the {@link sap.ui.test.Opa5#waitFor} function, to be used for chained statements
 	 */
-	KPICardAssertions.prototype.iSeeHeaderTitle = function(sTitle) {
+	KPICardAssertions.prototype.iSeeHeaderTitle = function (sTitle) {
 		var oKPIBuilder = this.getBuilder();
 		var vTitleMatcher = FEBuilder.create(this)
 			.hasType("sap.m.Text")
-			.has(function(oControl) {
+			.has(function (oControl) {
 				return oControl.getId().endsWith("-title");
 			})
 			.hasProperties({ text: sTitle });
@@ -52,11 +52,11 @@ sap.ui.define(["./KPICardAPI", "sap/fe/test/Utils", "sap/fe/test/builder/FEBuild
 	 * @param {string} sSubtitle The expected title
 	 * @returns {object} The result of the {@link sap.ui.test.Opa5#waitFor} function, to be used for chained statements
 	 */
-	KPICardAssertions.prototype.iSeeHeaderSubtitle = function(sSubtitle) {
+	KPICardAssertions.prototype.iSeeHeaderSubtitle = function (sSubtitle) {
 		var oKPIBuilder = this.getBuilder();
 		var vTitleMatcher = FEBuilder.create(this)
 			.hasType("sap.m.Text")
-			.has(function(oControl) {
+			.has(function (oControl) {
 				return oControl.getId().endsWith("-subtitle");
 			})
 			.hasProperties({ text: sSubtitle });
@@ -75,11 +75,11 @@ sap.ui.define(["./KPICardAPI", "sap/fe/test/Utils", "sap/fe/test/builder/FEBuild
 	 * @param {string} sText The expected unit
 	 * @returns {object} The result of the {@link sap.ui.test.Opa5#waitFor} function, to be used for chained statements
 	 */
-	KPICardAssertions.prototype.iSeeHeaderUnit = function(sText) {
+	KPICardAssertions.prototype.iSeeHeaderUnit = function (sText) {
 		var oKPIBuilder = this.getBuilder();
 		var vTitleMatcher = FEBuilder.create(this)
 			.hasType("sap.m.Text")
-			.has(function(oControl) {
+			.has(function (oControl) {
 				return oControl.getId().endsWith("-unitOfMeasurement");
 			})
 			.hasProperties({ text: sText });
@@ -98,11 +98,11 @@ sap.ui.define(["./KPICardAPI", "sap/fe/test/Utils", "sap/fe/test/builder/FEBuild
 	 * @param {string} sText The expected detail value
 	 * @returns {object} The result of the {@link sap.ui.test.Opa5#waitFor} function, to be used for chained statements
 	 */
-	KPICardAssertions.prototype.iSeeHeaderDetails = function(sText) {
+	KPICardAssertions.prototype.iSeeHeaderDetails = function (sText) {
 		var oKPIBuilder = this.getBuilder();
 		var vTitleMatcher = FEBuilder.create(this)
 			.hasType("sap.m.Text")
-			.has(function(oControl) {
+			.has(function (oControl) {
 				return oControl.getId().endsWith("-details");
 			})
 			.hasProperties({ text: sText });
@@ -122,14 +122,12 @@ sap.ui.define(["./KPICardAPI", "sap/fe/test/Utils", "sap/fe/test/builder/FEBuild
 	 * @param {object} mStates Additionnal properties to check (indicator, valueColor, scale)
 	 * @returns {object} The result of the {@link sap.ui.test.Opa5#waitFor} function, to be used for chained statements
 	 */
-	KPICardAssertions.prototype.iSeeHeaderValue = function(sValue, mStates) {
+	KPICardAssertions.prototype.iSeeHeaderValue = function (sValue, mStates) {
 		var oKPIBuilder = this.getBuilder();
 		var oProperties = mStates || {};
 		oProperties.number = sValue;
 
-		var vTitleMatcher = FEBuilder.create(this)
-			.hasType("sap.f.cards.NumericIndicators")
-			.hasProperties(oProperties);
+		var vTitleMatcher = FEBuilder.create(this).hasType("sap.f.cards.NumericIndicators").hasProperties(oProperties);
 
 		var sDescription = mStates
 			? Utils.formatMessage("Checking card header value '{0}' in state '{1}'", sValue, mStates)
@@ -150,7 +148,7 @@ sap.ui.define(["./KPICardAPI", "sap/fe/test/Utils", "sap/fe/test/builder/FEBuild
 	 * @param {string} sUnit The expected unit of the side indicator
 	 * @returns {object} The result of the {@link sap.ui.test.Opa5#waitFor} function, to be used for chained statements
 	 */
-	KPICardAssertions.prototype.iSeeSideIndicator = function(sTitle, sValue, sUnit) {
+	KPICardAssertions.prototype.iSeeSideIndicator = function (sTitle, sValue, sUnit) {
 		var oKPIBuilder = this.getBuilder();
 
 		var vIndicatorMatcher = FEBuilder.create(this)
@@ -158,21 +156,9 @@ sap.ui.define(["./KPICardAPI", "sap/fe/test/Utils", "sap/fe/test/builder/FEBuild
 			.hasChildren(
 				FEBuilder.create(this)
 					.hasType("sap.f.cards.NumericSideIndicator")
-					.hasChildren(
-						FEBuilder.create(this)
-							.hasType("sap.m.Text")
-							.hasProperties({ text: sTitle })
-					)
-					.hasChildren(
-						FEBuilder.create(this)
-							.hasType("sap.m.Text")
-							.hasProperties({ text: sTitle })
-					)
-					.hasChildren(
-						FEBuilder.create(this)
-							.hasType("sap.m.Text")
-							.hasProperties({ text: sValue })
-					)
+					.hasChildren(FEBuilder.create(this).hasType("sap.m.Text").hasProperties({ text: sTitle }))
+					.hasChildren(FEBuilder.create(this).hasType("sap.m.Text").hasProperties({ text: sTitle }))
+					.hasChildren(FEBuilder.create(this).hasType("sap.m.Text").hasProperties({ text: sValue }))
 			);
 
 		return this.prepareResult(
@@ -189,7 +175,7 @@ sap.ui.define(["./KPICardAPI", "sap/fe/test/Utils", "sap/fe/test/builder/FEBuild
 	 * @param {object} mProperties The properties to be checked on the chart (vizType). If null or empty, we just check if the chart is visible.
 	 * @returns {object} The result of the {@link sap.ui.test.Opa5#waitFor} function, to be used for chained statements
 	 */
-	KPICardAssertions.prototype.iSeeChart = function(mProperties) {
+	KPICardAssertions.prototype.iSeeChart = function (mProperties) {
 		var oKPIBuilder = this.getBuilder();
 
 		var vChartMatcher = mProperties && Object.keys(mProperties).length ? FEBuilder.create(this).hasProperties(mProperties) : undefined;
@@ -198,12 +184,7 @@ sap.ui.define(["./KPICardAPI", "sap/fe/test/Utils", "sap/fe/test/builder/FEBuild
 			? Utils.formatMessage("Checking card chart with properties '{0}'", mProperties)
 			: Utils.formatMessage("Checking card chart");
 
-		return this.prepareResult(
-			oKPIBuilder
-				.doOnKPICardChart(vChartMatcher)
-				.description(sDescription)
-				.execute()
-		);
+		return this.prepareResult(oKPIBuilder.doOnKPICardChart(vChartMatcher).description(sDescription).execute());
 	};
 	return KPICardAssertions;
 });

package/src/sap/fe/test/api/TableAPI.js

@@ -22,7 +22,6 @@ sap.ui.define(
 		 *
 		 * @typedef {object} ColumnIdentifier
 		 * @property {string} name The technical name of the column
-		 *
 		 * @name sap.fe.test.api.ColumnIdentifier
 		 * @public
 		 */
@@ -81,7 +80,6 @@ sap.ui.define(
 		 * Opens the column adaptation dialog of the table.
 		 *
 		 * @returns {object} The result of the {@link sap.ui.test.Opa5#waitFor} function, to be used for chained statements
-		 *
 		 * @public
 		 */
 		TableAPI.prototype.iOpenColumnAdaptation = function () {
@@ -100,7 +98,6 @@ sap.ui.define(
 		 * Confirms and closes the adaptation dialog of the table.
 		 *
 		 * @returns {object} The result of the {@link sap.ui.test.Opa5#waitFor} function, to be used for chained statements
-		 *
 		 * @public
 		 */
 		TableAPI.prototype.iConfirmColumnAdaptation = function () {
@@ -118,7 +115,6 @@ sap.ui.define(
 		 * Opens the sorting dialog of the table.
 		 *
 		 * @returns {object} The result of the {@link sap.ui.test.Opa5#waitFor} function, to be used for chained statements
-		 *
 		 * @public
 		 */
 		TableAPI.prototype.iOpenColumnSorting = function () {
@@ -135,7 +131,6 @@ sap.ui.define(
 		 * Confirms and closes the sorting dialog of the table.
 		 *
 		 * @returns {object} The result of the {@link sap.ui.test.Opa5#waitFor} function, to be used for chained statements
-		 *
 		 * @public
 		 */
 		TableAPI.prototype.iConfirmColumnSorting = function () {
@@ -153,7 +148,6 @@ sap.ui.define(
 		 * Opens the filtering dialog of the table.
 		 *
 		 * @returns {object} The result of the {@link sap.ui.test.Opa5#waitFor} function, to be used for chained statements
-		 *
 		 * @public
 		 */
 		TableAPI.prototype.iOpenFilterDialog = function () {
@@ -170,7 +164,6 @@ sap.ui.define(
 		 * Confirms and closes the filtering dialog of the table.
 		 *
 		 * @returns {object} The result of the {@link sap.ui.test.Opa5#waitFor} function, to be used for chained statements
-		 *
 		 * @public
 		 */
 		TableAPI.prototype.iConfirmFilterDialog = function () {
@@ -200,7 +193,6 @@ sap.ui.define(
 		 * @param {Function} fnOpenDialog A callback for opening the dialog
 		 * @param {Function} fnCloseDialog A callback for closing the dialog
 		 * @returns {object} The result of the {@link sap.ui.test.Opa5#waitFor} function, to be used for chained statements
-		 *
 		 * @ui5-restricted
 		 */
 		TableAPI.prototype.personalization = function (
@@ -291,7 +283,6 @@ sap.ui.define(
 		 * @param {Function} fnOpenDialog A callback for opening the dialog
 		 * @param {Function} fnCloseDialog A callback for closing the dialog
 		 * @returns {object} The result of the {@link sap.ui.test.Opa5#waitFor} function, to be used for chained statements
-		 *
 		 * @ui5-restricted
 		 */
 		TableAPI.prototype.sortingPersonalization = function (
@@ -338,6 +329,9 @@ sap.ui.define(
 				);
 			}
 
+			mState = aArguments[1];
+			var bCheckForNotVisible = mState && mState.visible === false;
+
 			oDialogBuilder
 				.has(OpaBuilder.Matchers.children(oAdaptColumnBuilder))
 				.has(function (aCustomListItems) {
@@ -354,8 +348,6 @@ sap.ui.define(
 					return FEBuilder.Matchers.atIndex(0)(aFoundAdaptationColumns);
 				});
 
-			mState = aArguments[1];
-			var bCheckForNotVisible = mState && mState.visible === false;
 			if (!bCheckForNotVisible && !Utils.isOfType(mState, [null, undefined])) {
 				if (mState.sortOrder) {
 					var sSortOrder = mState.sortOrder;
@@ -403,7 +395,6 @@ sap.ui.define(
 		 * @param {Function|Array|sap.ui.test.actions.Action} [vActions] The actions to be executed on found adaptation field
 		 * @param {string} sDescription The description of the check or adaptation
 		 * @returns {object} The result of the {@link sap.ui.test.Opa5#waitFor} function, to be used for chained statements
-		 *
 		 * @ui5-restricted
 		 */
 		TableAPI.prototype.columnAdaptation = function (vColumnIdentifier, mState, vActions, sDescription) {
@@ -428,7 +419,6 @@ sap.ui.define(
 		 * @param {Function|Array|sap.ui.test.actions.Action} [vActions] The actions to be executed on found adaptation field
 		 * @param {string} sDescription The description of the check or adaptation
 		 * @returns {object} The result of the {@link sap.ui.test.Opa5#waitFor} function, to be used for chained statements
-		 *
 		 * @ui5-restricted
 		 */
 		TableAPI.prototype.columnSorting = function (vColumnIdentifier, mState, vActions, sDescription) {