npm - @openui5/sap.ui.core - Versions diffs - 1.119.1 → 1.120.1 - Mend

@openui5/sap.ui.core 1.119.1 → 1.120.1

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.

Files changed (968) hide show

package/test/sap/ui/core/qunit/jquery.sap.global-config.qunit.html ADDED Viewed

@@ -0,0 +1,24 @@
+<!DOCTYPE HTML>
+<html>
+<head>
+	<meta charset="utf-8">
+	<!-- Initialization -->
+	<script src="./jquery.sap.global-config_beforeBootstrap.qunit.js"></script>
+	<script id="sap-ui-bootstrap"
+			src="../../../../../resources/sap-ui-core.js"
+			data-sap-ui-async="true"'></script>
+	<link  href="../../../../../resources/sap/ui/thirdparty/qunit-2.css" rel="stylesheet" media="screen">
+	<script src="../../../../../resources/sap/ui/thirdparty/qunit-2.js"></script>
+	<script src="../../../../../resources/sap/ui/qunit/qunit-junit.js"></script>
+	<script src="../../../../../resources/sap/ui/thirdparty/sinon-4.js"></script>
+	<script src="../../../../../resources/sap/ui/qunit/sinon-qunit-bridge.js"></script>
+	<!-- Test functions -->
+	<script src="jquery.sap.global-config.qunit.js"></script>
+</head>
+<body>
+	<div id="qunit"></div>
+</body>
+</html>

package/test/sap/ui/core/qunit/jquery.sap.global-config.qunit.js CHANGED Viewed

@@ -1,13 +1,19 @@
 /* global QUnit */
-sap.ui.define(["sap/ui/core/Theming"], function(Theming) {
+sap.ui.require([
+	"sap/ui/core/Core",
+	"sap/ui/core/Theming"
+], function(
+	Core,
+	Theming
+) {
 	"use strict";
 	QUnit.module("Basic");
 	QUnit.test("ExternalConfig", function(assert) {
-		assert.equal(Theming.getTheme(), "sap_bluecrystal", "Theme must be sap_blucrystal");
-		assert.ok(sap.ui.commons !== undefined, "sap.ui.commons must be preloaded");
-		assert.ok(sap.m === undefined, "sap.m must not be preloaded");
+		return Core.ready().then(() => {
+			assert.equal(Theming.getTheme(), "sap_bluecrystal", "Theme must be sap_blucrystal");
+			assert.ok(sap.ui.commons !== undefined, "sap.ui.commons must be preloaded");
+			assert.ok(sap.m === undefined, "sap.m must not be preloaded");
+		});
 	});
 });

package/test/sap/ui/core/qunit/jquery.sap.global-config_beforeBootstrap.qunit.js CHANGED Viewed

	@@ -1 +1 @@
1	- window["sap-ui-config"] = ~~sap.ui.require.toUrl(~~"~~testdata/core/") + "~~sap-ui-config.json";
1	+ window["sap-ui-config"] = "./sap-ui-config.json";

package/test/sap/ui/core/qunit/json/JSONListBinding.qunit.js CHANGED Viewed

@@ -140,6 +140,9 @@ sap.ui.define([
 	}
 	QUnit.module("sap.ui.model.json.JSONListbinding: getContexts", {
+		before() {
+			this.__ignoreIsolatedCoverage__ = true;
+		},
 		beforeEach: function() {
 			setup();
 			createListBinding("/teamMembers", "");
@@ -253,6 +256,9 @@ sap.ui.define([
 	});
 	QUnit.module("sap.ui.model.json.JSONListbinding: refresh", {
+		before() {
+			this.__ignoreIsolatedCoverage__ = true;
+		},
 		beforeEach: function() {
 			setup();
 			createListBinding("/teamMembers", "");
@@ -324,6 +330,9 @@ sap.ui.define([
 	});
 	QUnit.module("sap.ui.model.json.JSONListbinding: sort", {
+		before() {
+			this.__ignoreIsolatedCoverage__ = true;
+		},
 		beforeEach: function() {
 			setup();
 		}
@@ -638,6 +647,9 @@ sap.ui.define([
 	});
 	QUnit.module("sap.ui.model.json.JSONListbinding: filter", {
+		before() {
+			this.__ignoreIsolatedCoverage__ = true;
+		},
 		beforeEach: function() {
 			setup();
 			createListBinding("/teamMembers", "");
@@ -991,6 +1003,9 @@ sap.ui.define([
 	}
 	QUnit.module("sap.ui.model.json.JSONListbinding: filter case sensitive", {
+		before() {
+			this.__ignoreIsolatedCoverage__ = true;
+		},
 		beforeEach: function() {
 			setup();
 			createListBinding("/caseSensitive", "");
@@ -1040,6 +1055,9 @@ sap.ui.define([
 	});
 	QUnit.module("sap.ui.model.json.JSONListbinding: Unsupported Filters", {
+		before() {
+			this.__ignoreIsolatedCoverage__ = true;
+		},
 		getErrorWithMessage: function(sFilter) {
 			return new Error("Filter instances contain an unsupported FilterOperator: " + sFilter);
 		}
@@ -1190,6 +1208,9 @@ sap.ui.define([
 	QUnit.module("sap.ui.model.json.JSONListbinding: general", {
+		before() {
+			this.__ignoreIsolatedCoverage__ = true;
+		},
 		beforeEach: function() {
 			setup();
 		}
@@ -1357,6 +1378,9 @@ sap.ui.define([
 	});
 	QUnit.module("sap.ui.model.json.JSONListbinding: diff calculation", {
+		before() {
+			this.__ignoreIsolatedCoverage__ = true;
+		},
 		beforeEach: function() {
 			setup();
 			this.oModel = new JSONModel(testData.teamMembers);
@@ -1439,6 +1463,9 @@ sap.ui.define([
 	QUnit.module("sap.ui.model.json.JSONListbinding: suspend/resume", {
+		before() {
+			this.__ignoreIsolatedCoverage__ = true;
+		},
 		beforeEach: function() {
 			setup();
 			this.oData = testData.teamMembers.slice(0);
@@ -1520,6 +1547,9 @@ sap.ui.define([
 	//*********************************************************************************************
 	QUnit.module("sap.ui.model.json.JSONListBinding", {
+		before() {
+			this.__ignoreIsolatedCoverage__ = true;
+		},
 		beforeEach : function () {
 			this.oLogMock = this.mock(Log);
 			this.oLogMock.expects("error").never();

package/test/sap/ui/core/qunit/loader/exposeAsAMDLoader.qunit.html CHANGED Viewed

@@ -13,6 +13,7 @@
 	<script src="exposeAsAMDLoader.beforeBootstrap.qunit.js"></script>
 	<script src="../../shared-config.js"></script>
 	<script id="sap-ui-bootstrap"
+			data-sap-ui-async="true"
 			src="../../../../../../resources/sap-ui-core.js">
 	</script>
 	<script src="exposeAsAMDLoader.qunit.js"></script>

package/test/sap/ui/core/qunit/model/Binding.qunit.js CHANGED Viewed

@@ -13,6 +13,9 @@ sap.ui.define([
 	//*********************************************************************************************
 	QUnit.module("sap.ui.model.Binding", {
+		before() {
+			this.__ignoreIsolatedCoverage__ = true;
+		},
 		beforeEach : function () {
 			this.oLogMock = this.mock(Log);
 			this.oLogMock.expects("error").never();

package/test/sap/ui/core/qunit/model/ClientListBinding.qunit.js CHANGED Viewed

@@ -19,6 +19,9 @@ sap.ui.define([
 	//*********************************************************************************************
 	QUnit.module("sap.ui.model.ClientListBinding", {
+		before() {
+			this.__ignoreIsolatedCoverage__ = true;
+		},
 		beforeEach : function () {
 			this.oLogMock = this.mock(Log);
 			this.oLogMock.expects("error").never();
@@ -31,10 +34,10 @@ sap.ui.define([
 		var oBinding,
 			aFilters = [],
 			oModel = {
-				checkFilterOperation : function () {}
+				checkFilter : function () {}
 			};
-		this.mock(oModel).expects("checkFilterOperation").withExactArgs(sinon.match.same(aFilters));
+		this.mock(oModel).expects("checkFilter").withExactArgs(sinon.match.same(aFilters));
 		this.mock(FilterProcessor).expects("combineFilters")
 			.withExactArgs([], sinon.match.same(aFilters))
 			.returns("~combinedFilters");

package/test/sap/ui/core/qunit/model/ClientPropertyBinding.qunit.js CHANGED Viewed

@@ -10,6 +10,9 @@ sap.ui.define([
 	//*********************************************************************************************
 	QUnit.module("sap.ui.model.ClientPropertyBinding", {
+		before() {
+			this.__ignoreIsolatedCoverage__ = true;
+		},
 		beforeEach() {
 			this.oLogMock = this.mock(Log);
 			this.oLogMock.expects("error").never();

package/test/sap/ui/core/qunit/model/ClientTreeBinding.qunit.js CHANGED Viewed

@@ -12,6 +12,9 @@ sap.ui.define([
 	//*********************************************************************************************
 	QUnit.module("ClientTreeBinding", {
+		before() {
+			this.__ignoreIsolatedCoverage__ = true;
+		},
 		beforeEach : function () {
 			this.oLogMock = this.mock(Log);
 			this.oLogMock.expects("error").never();
@@ -175,10 +178,10 @@ sap.ui.define([
 			aFilters = ["~anyFilter"],
 			oModel = {
 				_getObject : function () {},
-				checkFilterOperation : function () {}
+				checkFilter : function () {}
 			};
-		this.mock(oModel).expects("checkFilterOperation").withExactArgs(sinon.match.same(aFilters));
+		this.mock(oModel).expects("checkFilter").withExactArgs(sinon.match.same(aFilters));
 		this.mock(oModel).expects("_getObject")
 			.withExactArgs("~path", "~oContext")
 			.returns({/*any data*/});

package/test/sap/ui/core/qunit/model/ClientTreeBindingAdapter.qunit.js CHANGED Viewed

@@ -164,6 +164,9 @@ sap.ui.define([
 	};
 	QUnit.module("ClientTreeBindingAdapter", {
+		before() {
+			this.__ignoreIsolatedCoverage__ = true;
+		},
 		beforeEach: function() {
 			oModel = new JSONModel();
 			oModel.setData(deepExtend({}, oData));
@@ -1136,6 +1139,9 @@ sap.ui.define([
 	};
 	QUnit.module("ClientTreeBindingAdapter - Model Data 2", {
+		before() {
+			this.__ignoreIsolatedCoverage__ = true;
+		},
 		beforeEach: function() {
 			oModel = new JSONModel();
 			oModel.setData(deepExtend({}, oData2));

package/test/sap/ui/core/qunit/model/CompositeDataState.qunit.js CHANGED Viewed

@@ -7,6 +7,9 @@ sap.ui.define([
 	//*********************************************************************************************
 	QUnit.module("sap.ui.model.CompositeDataState", {
+		before() {
+			this.__ignoreIsolatedCoverage__ = true;
+		},
 		beforeEach : function () {
 			this.oLogMock = this.mock(Log);
 			this.oLogMock.expects("error").never();

package/test/sap/ui/core/qunit/model/Context.qunit.js CHANGED Viewed

@@ -10,6 +10,9 @@ sap.ui.define([
 	//*********************************************************************************************
 	QUnit.module("sap.ui.model.Context", {
+		before() {
+			this.__ignoreIsolatedCoverage__ = true;
+		},
 		beforeEach : function () {
 			this.oLogMock = this.mock(Log);
 			this.oLogMock.expects("error").never();

package/test/sap/ui/core/qunit/model/ContextBinding.qunit.js CHANGED Viewed

@@ -10,6 +10,9 @@ sap.ui.define([
 	//*********************************************************************************************
 	QUnit.module("sap.ui.model.ContextBinding", {
+		before() {
+			this.__ignoreIsolatedCoverage__ = true;
+		},
 		beforeEach : function () {
 			this.oLogMock = this.mock(Log);
 			this.oLogMock.expects("error").never();

package/test/sap/ui/core/qunit/model/Filter.qunit.js CHANGED Viewed

@@ -9,6 +9,9 @@ sap.ui.define([
 	var sDefaultLanguage = Configuration.getLanguage();
 	QUnit.module("sap.ui.model.Filter", {
+		before() {
+			this.__ignoreIsolatedCoverage__ = true;
+		},
 		beforeEach : function () {
 			Configuration.setLanguage("en-US");
 		},
@@ -748,4 +751,50 @@ sap.ui.define([
 	});
+	//*********************************************************************************************
+	QUnit.test("Static never fulfilled filter", function(assert) {
+		assert.ok(Filter.NONE instanceof Filter);
+		assert.strictEqual(Filter.NONE.getPath(), "/");
+		assert.strictEqual(typeof Filter.NONE.getTest(), "function");
+		assert.strictEqual(Filter.NONE.getTest()(), false);
+	});
+	//*********************************************************************************************
+[
+	{filters : [{}, Filter.NONE]},
+	[{}, Filter.NONE],
+	{condition : Filter.NONE}
+].forEach((oFixture, i) => {
+	QUnit.test("Filter.NONE passed to constructor, " + i, function(assert) {
+		assert.throws(() => {
+			// code under test
+			new Filter(oFixture);
+		}, new Error("Filter.NONE not allowed "
+			+ (oFixture.condition ? "as condition" : "in multiple filter")));
+	});
+});
+	//*********************************************************************************************
+	QUnit.test("checkFilterNone", function(assert) {
+		// code under test
+		Filter.checkFilterNone();
+		// code under test
+		Filter.checkFilterNone(Filter.NONE);
+		// code under test
+		Filter.checkFilterNone([Filter.NONE]);
+		const oFilter = new Filter("path", FilterOperator.EQ, "value");
+		oFilter.length = 2;
+		// code under test - no error in case of tagged filter or Filter subclass with length property
+		Filter.checkFilterNone(oFilter);
+		// code under test
+		assert.throws(function () {
+			Filter.checkFilterNone([new Filter("path", FilterOperator.EQ, "value"), Filter.NONE]);
+		}, new Error("Filter.NONE cannot be used together with other filters"));
+	});
 });

package/test/sap/ui/core/qunit/model/FilterProcessor.qunit.js CHANGED Viewed

@@ -6,7 +6,11 @@ sap.ui.define([
 ], function(FilterProcessor, Filter, FilterOperator) {
 	"use strict";
-	QUnit.module("sap.ui.model.FilterProcessor");
+	QUnit.module("sap.ui.model.FilterProcessor", {
+		before() {
+			this.__ignoreIsolatedCoverage__ = true;
+		}
+	});
 	QUnit.test("Contains", function(assert) {
@@ -397,6 +401,69 @@ sap.ui.define([
 		assert.notOk(oGroupedFilter.aFilters[1]._bMultiFilter, "Second Filter should not be a MultiFilter");
 	});
+	//*********************************************************************************************
+	QUnit.test("groupFilters: early exit wíthout grouping, check Filter.NONE", function (assert) {
+		// code under test: early exit
+		assert.strictEqual(FilterProcessor.groupFilters(), undefined);
+		assert.strictEqual(FilterProcessor.groupFilters([]), undefined);
+		// code under test
+		assert.strictEqual(FilterProcessor.groupFilters(["~oFilter"]), "~oFilter");
+		const aFilters = ["~oFilter", Filter.NONE];
+		const oError = new Error("~Filter.NONE error");
+		this.mock(Filter).expects("checkFilterNone").withExactArgs(sinon.match.same(aFilters)).throws(oError);
+		// code under test
+		assert.throws(() => {
+			FilterProcessor.groupFilters(aFilters);
+		}, oError);
+	});
+	//*********************************************************************************************
+	QUnit.test("combineFilters: filter c'tor throws error", function (assert) {
+		// code under test
+		assert.throws(() => {
+			FilterProcessor.combineFilters([Filter.NONE], [Filter.NONE]);
+		}, new Error("Filter.NONE not allowed in multiple filter"));
+	});
+	//*********************************************************************************************
+	QUnit.test("combineFilters: groupFilter throws error", function (assert) {
+		const aApplicationFilters = "~ApplicationFilters";
+		const oError = new Error("~Filter.NONE error");
+		const aFilters = "~Filters";
+		const oFilterProcessorMock = this.mock(FilterProcessor);
+		oFilterProcessorMock.expects("groupFilters").withExactArgs(aFilters).throws(oError);
+		// code under test
+		assert.throws(() => {
+			FilterProcessor.combineFilters(aFilters, aApplicationFilters);
+		}, oError);
+		oFilterProcessorMock.expects("groupFilters").withExactArgs(aFilters).returns("~someGroupedFilter");
+		oFilterProcessorMock.expects("groupFilters").withExactArgs(aApplicationFilters).throws(oError);
+		// code under test
+		assert.throws(() => {
+			FilterProcessor.combineFilters(aFilters, aApplicationFilters);
+		}, oError);
+	});
+	//*********************************************************************************************
+	QUnit.test("apply: propagates error from groupFilters", function (assert) {
+		const aFilters = ["~oFilter", Filter.NONE];
+		const oError = new Error("~Filter.NONE error");
+		this.mock(FilterProcessor).expects("groupFilters").withExactArgs(sinon.match.same(aFilters)).throws(oError);
+		// code under test
+		assert.throws(() => {
+			FilterProcessor.apply([/*aData*/], aFilters);
+		}, oError);
+	});
 	QUnit.test("apply: values contain 'toString' value", function (assert) {
 		var aData = ["foo", "toString", "bar", "foo bar"],
 			oFilter = new Filter({path: "name", operator: FilterOperator.Contains, value1: "foo"});

package/test/sap/ui/core/qunit/model/ListBinding.qunit.js CHANGED Viewed

@@ -7,11 +7,14 @@ sap.ui.define([
 	"sap/ui/model/ListBinding",
 	"sap/ui/model/Sorter"
 ], function (Log, Filter, ListBinding, Sorter) {
-	/*global QUnit*/
+	/*global QUnit, sinon*/
 	"use strict";
 	//*********************************************************************************************
 	QUnit.module("sap.ui.model.ListBinding", {
+		before() {
+			this.__ignoreIsolatedCoverage__ = true;
+		},
 		beforeEach : function () {
 			this.oLogMock = this.mock(Log);
 			this.oLogMock.expects("error").never();
@@ -60,23 +63,38 @@ sap.ui.define([
 	//*********************************************************************************************
 	QUnit.test("constructor, application filters", function (assert) {
-		var aApplicationFilters = ["~filter"],
-			// code under test
-			oBinding = new ListBinding("~oModel", "~sPath", "~oContext", undefined, aApplicationFilters);
+		const aApplicationFilters = ["~filter"];
+		let oBinding;
+		const oFilterMock = this.mock(Filter);
+		oFilterMock.expects("checkFilterNone").withExactArgs(sinon.match.same(aApplicationFilters));
+		// code under test
+		oBinding = new ListBinding("~oModel", "~sPath", "~oContext", undefined, aApplicationFilters);
 		assert.strictEqual(oBinding.aApplicationFilters, aApplicationFilters, "filter: any array is kept as is");
-		var oFilter = new Filter({path: "~filterPath", test : function () {}});
+		const oFilter = new Filter({path: "~filterPath", test : function () {}});
+		oFilterMock.expects("checkFilterNone").withExactArgs(sinon.match.same(oFilter));
 		// code under test
 		oBinding = new ListBinding("~oModel", "~sPath", "~oContext", undefined, oFilter);
 		assert.deepEqual(oBinding.aApplicationFilters, [oFilter], "filter: single Filter is put into array");
+		oFilterMock.expects("checkFilterNone").withExactArgs("~noFilter");
 		// code under test
 		oBinding = new ListBinding("~oModel", "~sPath", "~oContext", undefined, "~noFilter");
 		assert.deepEqual(oBinding.aApplicationFilters, [], "filter: non-Filter object");
+		const oError = new Error("~Filter.NONE error");
+		oFilterMock.expects("checkFilterNone").withExactArgs("~invalidFilter").throws(oError);
+		// code under test
+		assert.throws(() => {
+			oBinding = new ListBinding("~oModel", "~sPath", "~oContext", undefined, "~invalidFilter");
+		}, oError);
 	});
 	//*********************************************************************************************

package/test/sap/ui/core/qunit/model/Model.qunit.js CHANGED Viewed

@@ -5,14 +5,19 @@ sap.ui.define([
 	"sap/base/Log",
 	"sap/ui/core/message/Message",
 	"sap/ui/model/BindingMode",
+	"sap/ui/model/Filter",
+	"sap/ui/model/FilterOperator",
 	"sap/ui/model/Model"
-], function (Log, Message, BindingMode, Model) {
-	/*global QUnit, Set*/
+], function (Log, Message, BindingMode, Filter, FilterOperator, Model) {
+	/*global QUnit, sinon*/
 	/*eslint max-nested-callbacks: 0*/
 	"use strict";
 	//*********************************************************************************************
 	QUnit.module("sap.ui.model.Model", {
+		before() {
+			this.__ignoreIsolatedCoverage__ = true;
+		},
 		beforeEach : function () {
 			this.oLogMock = this.mock(Log);
 			this.oLogMock.expects("error").never();
@@ -238,4 +243,43 @@ sap.ui.define([
 		assert.strictEqual(oModel.bForceUpdate, undefined);
 		assert.strictEqual(oModel.sUpdateTimer, null);
 	});
+	//*********************************************************************************************
+	QUnit.test("checkFilter: success", function (assert) {
+		const oFilter = new Filter({path: 'Price', operator: FilterOperator.EQ, value1: 100});
+		this.mock(Filter).expects("checkFilterNone").withExactArgs(sinon.match.same(oFilter));
+		const oModel = new Model();
+		this.mock(Model).expects("_traverseFilter").withExactArgs(sinon.match.same(oFilter), sinon.match.func)
+			.callsArgOnWith(1, oModel, oFilter);
+		// code under test
+		oModel.checkFilter(oFilter);
+	});
+	//*********************************************************************************************
+	QUnit.test("checkFilter: throws error", function (assert) {
+		const oError = new Error("~Error");
+		const oFilter = new Filter({path: 'Price', operator: FilterOperator.EQ, value1: 100});
+		const oFilterMock = this.mock(Filter);
+		oFilterMock.expects("checkFilterNone").withExactArgs(sinon.match.same(oFilter))
+			.throws(oError);
+		const oModelMock = this.mock(Model);
+		oModelMock.expects("_traverseFilter").never();
+		const oModel = new Model();
+		// code under test
+		assert.throws(function () {
+			oModel.checkFilter(oFilter);
+		}, oError);
+		oModel.mUnsupportedFilterOperators["EQ"] = true;
+		oFilterMock.expects("checkFilterNone").withExactArgs(sinon.match.same(oFilter));
+		oModelMock.expects("_traverseFilter").withExactArgs(oFilter, sinon.match.func)
+			.callsArgOnWith(1, oModel, oFilter);
+		// code under test
+		assert.throws(function () {
+			oModel.checkFilter(oFilter);
+		}, new Error("Filter instances contain an unsupported FilterOperator: EQ"));
+	});
 });

package/test/sap/ui/core/qunit/model/PropertyBinding.qunit.js CHANGED Viewed

@@ -11,6 +11,9 @@ sap.ui.define([
 	//*********************************************************************************************
 	QUnit.module("sap.ui.model.PropertyBinding", {
+		before() {
+			this.__ignoreIsolatedCoverage__ = true;
+		},
 		beforeEach : function () {
 			this.oLogMock = this.mock(Log);
 			this.oLogMock.expects("error").never();

package/test/sap/ui/core/qunit/model/Sorter.qunit.js CHANGED Viewed

@@ -8,6 +8,9 @@ sap.ui.define([
 	var sDefaultLanguage = Configuration.getLanguage();
 	QUnit.module("sap.ui.model.Sorter", {
+		before() {
+			this.__ignoreIsolatedCoverage__ = true;
+		},
 		beforeEach : function () {
 			Configuration.setLanguage("en-US");
 		},

package/test/sap/ui/core/qunit/model/TreeBinding.qunit.js CHANGED Viewed

@@ -3,13 +3,17 @@
  */
 sap.ui.define([
 	"sap/base/Log",
+	"sap/ui/model/Filter",
 	"sap/ui/model/TreeBinding"
-], function (Log, TreeBinding) {
+], function (Log, Filter, TreeBinding) {
 	"use strict";
-	/*global QUnit */
+	/*global QUnit, sinon */
 	//*********************************************************************************************
-	QUnit.module("TreeBinding", {
+	QUnit.module("sap.ui.model.TreeBinding", {
+		before() {
+			this.__ignoreIsolatedCoverage__ = true;
+		},
 		beforeEach : function () {
 			this.oLogMock = this.mock(Log);
 			this.oLogMock.expects("error").never();
@@ -22,4 +26,40 @@ sap.ui.define([
 		// code under test
 		assert.strictEqual(TreeBinding.prototype.getCount.call(), undefined);
 	});
+	//*********************************************************************************************
+	QUnit.test("constructor, application filters", function (assert) {
+		const aApplicationFilters = ["~filter"];
+		let oBinding;
+		const oFilterMock = this.mock(Filter);
+		oFilterMock.expects("checkFilterNone").withExactArgs(sinon.match.same(aApplicationFilters));
+		// code under test
+		oBinding = new TreeBinding("~oModel", "~sPath", "~oContext", aApplicationFilters);
+		assert.strictEqual(oBinding.aApplicationFilters, aApplicationFilters, "filter: any array is kept as is");
+		const oFilter = new Filter({path: "~filterPath", test : function () {}});
+		oFilterMock.expects("checkFilterNone").withExactArgs(sinon.match.same(oFilter));
+		// code under test
+		oBinding = new TreeBinding("~oModel", "~sPath", "~oContext", oFilter);
+		assert.deepEqual(oBinding.aApplicationFilters, [oFilter], "filter: single Filter is put into array");
+		oFilterMock.expects("checkFilterNone").withExactArgs("~noFilter");
+		// code under test
+		oBinding = new TreeBinding("~oModel", "~sPath", "~oContext", "~noFilter");
+		assert.deepEqual(oBinding.aApplicationFilters, [], "filter: non-Filter object");
+		const oError = new Error("~Filter.NONE error");
+		oFilterMock.expects("checkFilterNone").withExactArgs("~invalidFilter").throws(oError);
+		// code under test
+		assert.throws(() => {
+			oBinding = new TreeBinding("~oModel", "~sPath", "~oContext", "~invalidFilter");
+		}, oError);
+	});
 });

package/test/sap/ui/core/qunit/model/TreeBindingAdapter.qunit.js CHANGED Viewed

@@ -13,6 +13,9 @@ sap.ui.define([
 	//*********************************************************************************************
 	QUnit.module("sap.ui.model.TreeBindingAdapter", {
+		before() {
+			this.__ignoreIsolatedCoverage__ = true;
+		},
 		beforeEach : function () {
 			this.oLogMock = this.mock(Log);
 			this.oLogMock.expects("error").never();