[IMP] types: correctly support Set type for props-validation

Previously, having a Set as a type in the static props description
of a component would would lead to a type error.

This commit adds Set to the list of supported types in the Schema

docs/owl.js

@@ -5538,7 +5538,7 @@ function compile(template, options = {}) {
 }
 
 // do not modify manually. This file is generated by the release script.
-const version = "2.2.10";
+const version = "2.2.9";
 
 // -----------------------------------------------------------------------------
 //  Scheduler
@@ -5967,6 +5967,6 @@ TemplateSet.prototype._compileTemplate = function _compileTemplate(name, templat
 export { App, Component, EventBus, OwlError, __info__, blockDom, loadFile, markRaw, markup, mount, onError, onMounted, onPatched, onRendered, onWillDestroy, onWillPatch, onWillRender, onWillStart, onWillUnmount, onWillUpdateProps, reactive, status, toRaw, useChildSubEnv, useComponent, useEffect, useEnv, useExternalListener, useRef, useState, useSubEnv, validate, validateType, whenReady, xml };
 
 
-__info__.date = '2024-04-02T10:25:32.577Z';
-__info__.hash = '97b69f1';
+__info__.date = '2024-01-12T14:43:56.804Z';
+__info__.hash = '7b3e39b';
 __info__.url = 'https://github.com/odoo/owl';

package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "@odoo/owl",
-  "version": "2.2.10",
+  "version": "2.2.9",
   "lockfileVersion": 1,
   "requires": true,
   "dependencies": {

package.json

@@ -1,6 +1,6 @@
 {
   "name": "@odoo/owl",
-  "version": "2.2.10",
+  "version": "2.2.9",
   "description": "Odoo Web Library (OWL)",
   "main": "dist/owl.cjs.js",
   "module": "dist/owl.es.js",

src/runtime/validation.ts

@@ -1,7 +1,17 @@
 import { OwlError } from "../common/owl_error";
 import { toRaw } from "./reactivity";
 
-type BaseType = { new (...args: any[]): any } | true | "*";
+type BaseType =
+  | typeof String
+  | typeof Boolean
+  | typeof Number
+  | typeof Date
+  | typeof Object
+  | typeof Array
+  | typeof Function
+  | typeof Set
+  | true
+  | "*";
 
 interface TypeInfo {
   type?: TypeDescription;

src/version.ts

@@ -1,2 +1,2 @@
 // do not modify manually. This file is generated by the release script.
-export const version = "2.2.10";
+export const version = "2.2.9";

tests/components/__snapshots__/props_validation.test.ts.snap

@@ -167,45 +167,6 @@ exports[`props validation can specify that additional props are allowed (object)
 }"
 `;
 
-exports[`props validation can use custom class as type 1`] = `
-"function anonymous(app, bdom, helpers
-) {
-  let { text, createBlock, list, multi, html, toggler, comment } = bdom;
-  const comp1 = app.createComponent(\`Child\`, true, false, false, [\\"customObj\\"]);
-  
-  return function template(ctx, node, key = \\"\\") {
-    const props1 = {customObj: ctx['customObj']};
-    helpers.validateProps(\`Child\`, props1, this);
-    return comp1(props1, key + \`__1\`, node, this, null);
-  }
-}"
-`;
-
-exports[`props validation can use custom class as type 2`] = `
-"function anonymous(app, bdom, helpers
-) {
-  let { text, createBlock, list, multi, html, toggler, comment } = bdom;
-  
-  return function template(ctx, node, key = \\"\\") {
-    return text(ctx['props'].customObj.val);
-  }
-}"
-`;
-
-exports[`props validation can use custom class as type: validation failure 1`] = `
-"function anonymous(app, bdom, helpers
-) {
-  let { text, createBlock, list, multi, html, toggler, comment } = bdom;
-  const comp1 = app.createComponent(\`Child\`, true, false, false, [\\"customObj\\"]);
-  
-  return function template(ctx, node, key = \\"\\") {
-    const props1 = {customObj: ctx['customObj']};
-    helpers.validateProps(\`Child\`, props1, this);
-    return comp1(props1, key + \`__1\`, node, this, null);
-  }
-}"
-`;
-
 exports[`props validation can validate a prop with multiple types 1`] = `
 "function anonymous(app, bdom, helpers
 ) {

tests/components/props_validation.test.ts

@@ -829,52 +829,6 @@ describe("props validation", () => {
     expect(error!).toBeDefined();
     expect(error!.message).toBe("Invalid props for component 'Child': 'message' is missing");
   });
-
-  test("can use custom class as type", async () => {
-    class CustomClass {
-      val = "hey";
-    }
-    class Child extends Component {
-      static props = { customObj: CustomClass };
-      static template = xml`<t t-esc="props.customObj.val"/>`;
-    }
-
-    class Parent extends Component {
-      static components = { Child };
-      static template = xml`<Child customObj="customObj" />`;
-      customObj = new CustomClass();
-    }
-
-    const app = new App(Parent, { test: true });
-    await app.mount(fixture);
-    expect(fixture.innerHTML).toBe("hey");
-  });
-
-  test("can use custom class as type: validation failure", async () => {
-    class CustomClass {}
-    class Child extends Component {
-      static props = { customObj: CustomClass };
-      static template = xml`<div>hey</div>`;
-    }
-
-    class Parent extends Component {
-      static components = { Child };
-      static template = xml`<Child customObj="customObj" />`;
-      customObj = {};
-    }
-
-    const app = new App(Parent, { test: true });
-    let error: OwlError | undefined;
-    const mountProm = app.mount(fixture).catch((e: Error) => (error = e));
-    await expect(nextAppError(app)).resolves.toThrow(
-      "Invalid props for component 'Child': 'customObj' is not a customclass"
-    );
-    await mountProm;
-    expect(error!).toBeDefined();
-    expect(error!.message).toBe(
-      "Invalid props for component 'Child': 'customObj' is not a customclass"
-    );
-  });
 });
 
 //------------------------------------------------------------------------------