@qooxdoo/framework 7.9.2 → 8.0.0-beta.2

package/source/class/qx/test/Theme.js

@@ -48,7 +48,18 @@ qx.Class.define("qx.test.Theme", {
       });
 
       var decorationManager = qx.theme.manager.Decoration.getInstance();
+
+      // djl notes:
+      //
+      // This fires a changeTheme event which causes the testtapper
+      // app to try to rerender based on the new theme, but the new
+      // thime is minimal: it contains only a single decoration; it
+      // does not contain required elements such as "inset" needed by
+      // Widget. Thus, an exception is thrown during this call.
+      //
+      // Why/how does it work in the v7 code?
       decorationManager.setTheme(qx.test.Theme.themes.B);
+
       this.assertInstance(
         decorationManager.resolve("main"),
         qx.ui.decoration.Decorator

package/source/class/qx/test/bom/Font.js

@@ -192,12 +192,9 @@ qx.Class.define("qx.test.bom.Font", {
       this.getRoot().add(label);
       this.flush();
 
-      var checkValue = "blue";
       var color = label.getContentElement().getDomElement().style["color"];
-
-      // the current implementation has a higher priority for the color which is
-      // set using the color theme. So this default color should show up and not
-      // the defined color of the font.
+      var checkValue = "blue";
+      
       this.assertEquals(
         checkValue,
         color,

package/source/class/qx/test/bom/Template.js

@@ -40,7 +40,7 @@ qx.Class.define("qx.test.bom.Template", {
       this.assertEquals(expected, result);
     },
 
-    testFunc() {
+    testFunction() {
       var template = "{{name}} xyz";
       var view = {
         name() {

package/source/class/qx/test/compiler/ClassFile.js

@@ -17,6 +17,7 @@ qx.Class.define("qx.test.compiler.ClassFile", {
   extend: qx.dev.unit.TestCase,
 
   construct() {
+    super();
     this.__lib = new qx.tool.compiler.app.Library();
     this.__analyser = new qx.tool.compiler.Analyser();
     this.__analyser.setOutputDir("tmp/unittest");
@@ -91,6 +92,19 @@ qx.Class.define("qx.test.compiler.ClassFile", {
       this.assert(!dbClassInfo.unresolved);
     },
 
+    async "test issue 10623"() {
+      var classFile = new qx.tool.compiler.ClassFile(
+        this.__analyser,
+        "classIssue10623",
+        this.__lib
+      );
+
+      await qx.tool.utils.Promisify.call(cb => classFile.load(cb));
+      var dbClassInfo = {};
+      classFile.writeDbInfo(dbClassInfo);
+      this.assert(!dbClassInfo.unresolved);
+    },
+
     __lib: null,
     __analyser: null
   }

package/source/class/qx/test/core/Assert.js

@@ -18,9 +18,19 @@
 ************************************************************************ */
 qx.Class.define("qx.test.core.Assert", {
   extend: qx.dev.unit.TestCase,
+  include: [qx.dev.unit.MRequirements],
 
   members: {
+    hasUi() {
+      return typeof qx.ui !== "undefined";
+    },
+
+    hasDocument() {
+      return typeof document !== "undefined" && document.body;
+    },
+
     testQxWidget() {
+      this.require(["ui"]);
       // working widget
       var widget = new qx.ui.core.Widget();
       this.assertQxWidget(widget);
@@ -63,6 +73,7 @@ qx.Class.define("qx.test.core.Assert", {
     },
 
     testQxObject() {
+      this.require(["ui"]);
       // working widget
       var o = new qx.core.Object();
       this.assertQxObject(o);
@@ -148,6 +159,7 @@ qx.Class.define("qx.test.core.Assert", {
     },
 
     testAssertElement() {
+      this.require(["document"]);
       // working
       this.assertElement(document.body);
       this.assertElement(document.createElement("div"));

package/source/class/qx/test/core/Environment.js

@@ -160,7 +160,10 @@ qx.Class.define("qx.test.core.Environment", {
         return "AFFE";
       });
 
-      this.assertEquals("AFFE", qx.core.Environment.get("qx.test.core.Environment.affe"));
+      this.assertEquals(
+        "AFFE",
+        qx.core.Environment.get("qx.test.core.Environment.affe")
+      );
 
       // clear the check
       delete qx.core.Environment.getChecks()["qx.test.core.Environment.affe"];
@@ -170,7 +173,10 @@ qx.Class.define("qx.test.core.Environment", {
     testAddValue() {
       qx.core.Environment.add("qx.test.core.Environment.affe", "AFFE");
 
-      this.assertEquals("AFFE", qx.core.Environment.get("qx.test.core.Environment.affe"));
+      this.assertEquals(
+        "AFFE",
+        qx.core.Environment.get("qx.test.core.Environment.affe")
+      );
 
       // clear the check
       delete qx.core.Environment.getChecks()["qx.test.core.Environment.affe"];
@@ -251,7 +257,15 @@ qx.Class.define("qx.test.core.Environment", {
     },
 
     testLocale() {
-      this.assertNotEquals("", qx.core.Environment.get("locale"));
+      if (
+        qx.core.Environment.get("browser.name") === "firefox" &&
+        parseFloat(qx.core.Environment.get("browser.version")) >= 146
+      ) {
+        this.assertEquals("", qx.core.Environment.get("locale"));
+      }
+      else {
+        this.assertNotEquals("", qx.core.Environment.get("locale"));
+      }
     },
 
     testVariant() {

package/source/class/qx/test/core/InheritanceDummy.js

@@ -22,10 +22,11 @@ qx.Class.define("qx.test.core.InheritanceDummy", {
   construct() {
     super();
     this.children = [];
+    this.addListener("changeEnabled", () => this.eventCount++);
   },
 
   properties: {
-    enabled: { inheritable: true },
+    enabled: { inheritable: true, apply: "_applyEnabled", event: "changeEnabled" },
 
     width_: {
       inheritable: true,
@@ -49,6 +50,14 @@ qx.Class.define("qx.test.core.InheritanceDummy", {
   },
 
   members: {
+    applyCount: 0,
+    eventCount: 0,
+    lastApply: null,
+    _applyEnabled(value, old) {
+      this.applyCount++;
+      this.lastApply = [value, old];
+    },
+
     add(child) {
       this.children.push(child);
       child.parent = this;

package/source/class/qx/test/core/Object.js

@@ -94,12 +94,7 @@ qx.Class.define("qx.test.core.Object", {
     },
 
     testRemoveListenerById() {
-      var id = this.addListener(
-        "testRemoveListenerById",
-        function () {},
-        this,
-        false
-      );
+      var id = this.addListener("testRemoveListenerById", function () {}, this, false);
 
       this.assertTrue(this.hasListener("testRemoveListenerById", false));
       this.removeListenerById(id);
@@ -107,12 +102,7 @@ qx.Class.define("qx.test.core.Object", {
     },
 
     testRemoveListenerOnceById() {
-      var id = this.addListenerOnce(
-        "testRemoveListenerOnceById",
-        function () {},
-        this,
-        false
-      );
+      var id = this.addListenerOnce("testRemoveListenerOnceById", function () {}, this, false);
 
       this.assertTrue(this.hasListener("testRemoveListenerOnceById", false));
       this.removeListenerById(id);
@@ -226,6 +216,7 @@ qx.Class.define("qx.test.core.Object", {
       o._disposeObjects("x");
 
       // object dispose with a singleton
+      qx.Class.undefine("qx.test.Single");
       qx.Class.define("qx.test.Single", {
         extend: qx.core.Object,
         type: "singleton"
@@ -238,12 +229,12 @@ qx.Class.define("qx.test.core.Object", {
       this.assertException(function () {
         o._disposeObjects("s");
       });
-      qx.Class.undefine("qx.test.Single");
       o.dispose();
     },
 
     testDisposeBindingWithChain() {
       // object dispose with a singleton
+      qx.Class.undefine("qx.test.Single");
       qx.Class.define("qx.test.Single", {
         extend: qx.core.Object,
         properties: {
@@ -269,12 +260,11 @@ qx.Class.define("qx.test.core.Object", {
 
       o2.dispose();
       o3.dispose();
-
-      qx.Class.undefine("qx.test.Single");
     },
 
     testDisposeBindingWithSelfChain() {
       // object dispose with a singleton
+      qx.Class.undefine("qx.test.Single");
       qx.Class.define("qx.test.Single", {
         extend: qx.core.Object,
         properties: {
@@ -283,7 +273,8 @@ qx.Class.define("qx.test.core.Object", {
         },
 
         members: {
-          applyB() {},
+          applyB() {
+          },
           init() {
             this.bind("a.a", this, "b");
           }
@@ -306,12 +297,11 @@ qx.Class.define("qx.test.core.Object", {
       this.assertCalledOnce(spy);
 
       o2.dispose();
-
-      qx.Class.undefine("qx.test.Single");
     },
 
     testDisposeBinding() {
       // object dispose with a singleton
+      qx.Class.undefine("qx.test.Single");
       qx.Class.define("qx.test.Single", {
         extend: qx.core.Object,
         properties: {
@@ -333,14 +323,13 @@ qx.Class.define("qx.test.core.Object", {
       o.dispose();
       o2.dispose();
 
-      qx.Class.undefine("qx.test.Single");
-
       this.assertEquals(0, o.getBindings().length);
       this.assertEquals(0, o2.getBindings().length);
     },
 
     testDisposeSingletonObject() {
       // object dispose with a singleton and an object
+      qx.Class.undefine("qx.test.Single");
       qx.Class.define("qx.test.Single", {
         extend: qx.core.Object,
         type: "singleton"
@@ -353,7 +342,6 @@ qx.Class.define("qx.test.core.Object", {
       this.assertTrue(o.o == null);
       this.assertTrue(o.s == null);
 
-      qx.Class.undefine("qx.test.Single");
       o.dispose();
     },
 
@@ -371,13 +359,52 @@ qx.Class.define("qx.test.core.Object", {
 
       var o = new qx.test.MyClass();
 
-      this.assertFalse(o.isPropertyInitialized("a"));
+      this.assertFalse(qx.Class.isPropertyInitialized(o, "a"));
       o.setA(false);
-      this.assertTrue(o.isPropertyInitialized("a"));
-      this.assertTrue(o.isPropertyInitialized("b"));
+      this.assertTrue(qx.Class.isPropertyInitialized(o, "a"));
+      this.assertTrue(qx.Class.isPropertyInitialized(o, "b"));
 
       qx.Class.undefine("qx.test.MyClass");
       o.dispose();
+    },
+
+    /**
+     * @ignore(qx.test.CloneClass)
+     */
+    testClone() {
+      qx.Class.define("qx.test.CloneClass", {
+        extend: qx.core.Object,
+        properties: {
+          name: { init: null, nullable: true },
+          value: { init: 0 },
+          enabled: { init: true }
+        }
+      });
+
+      var original = new qx.test.CloneClass();
+      original.setName("test");
+      original.setValue(42);
+      original.setEnabled(false);
+
+      var clone = original.clone();
+
+      // Check that clone is a different instance
+      this.assertNotIdentical(original, clone);
+      this.assertInstance(clone, qx.test.CloneClass);
+
+      // Check that properties were copied correctly
+      this.assertEquals("test", clone.getName());
+      this.assertEquals(42, clone.getValue());
+      this.assertEquals(false, clone.getEnabled());
+
+      // Check that modifying clone doesn't affect original
+      clone.setName("modified");
+      this.assertEquals("test", original.getName());
+      this.assertEquals("modified", clone.getName());
+
+      qx.Class.undefine("qx.test.CloneClass");
+      original.dispose();
+      clone.dispose();
     }
   }
 });

package/source/class/qx/test/core/ObjectId.js

@@ -26,10 +26,18 @@
 
 qx.Class.define("qx.test.core.ObjectId", {
   extend: qx.dev.unit.TestCase,
-  include: qx.dev.unit.MMock,
+  include: [qx.dev.unit.MMock, 
+            qx.dev.unit.MRequirements
+  ],
 
   members: {
+
+    hasUi() {
+      return typeof qx.ui !== "undefined";
+    },
+
     testGetObject() {
+      this.require(["ui"]);
       var clazz = qx.Class.define("demo.MyClass", {
         extend: qx.core.Object,
         members: {
@@ -72,6 +80,7 @@ qx.Class.define("qx.test.core.ObjectId", {
     },
 
     testObjectsSection() {
+      qx.Class.undefine("demo.SuperClass");
       const SuperClass = qx.Class.define("demo.SuperClass", {
         extend: qx.core.Object,
         objects: {