repository-provider 32.3.18 → 32.3.20

package/README.md

@@ -265,7 +265,6 @@ console.log(await readme.getString());
     *   [Properties](#properties-9)
     *   [attributeMapping](#attributemapping-1)
 *   [type](#type-1)
-*   [avatarURL](#avatarurl)
 *   [RepositoryOwner](#repositoryowner)
     *   [Parameters](#parameters-57)
 *   [Repository](#repository-3)
@@ -1565,14 +1564,6 @@ Map attributes between external and internal representation.
 
 Type of the repository group either User or Organization.
 
-Returns **[string](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/String)** 
-
-## avatarURL
-
-Avatar.
-
-Returns **[string](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/String)** 
-
 ## RepositoryOwner
 
 Mixin to define a class able to handle a collection of repositories.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "repository-provider",
-  "version": "32.3.18",
+  "version": "32.3.20",
   "publishConfig": {
     "access": "public"
   },
@@ -38,7 +38,7 @@
     "browser-ava": "^1.3.13",
     "c8": "^7.12.0",
     "documentation": "^14.0.1",
-    "repository-provider-test-support": "^2.2.25",
+    "repository-provider-test-support": "^2.2.27",
     "semantic-release": "^19.0.5",
     "typescript": "^4.9.4"
   },

package/src/attribute.mjs

@@ -135,6 +135,37 @@ export function defaultValues(attributes, object) {
   );
 }
 
+/**
+ * Split property path into tokens
+ * @param {string} string 
+ * @return {Iterator<string>}
+ */
+function* tokens(string) {
+  let identifier = "";
+
+  for (const c of string) {
+    switch (c) {
+      case "\t":
+      case " ": break;
+      case ".":
+      case "[":
+      case "]":
+        if (identifier.length) {
+          yield identifier;
+          identifier = "";
+        }
+        yield c;
+        break;
+      default:
+        identifier += c;
+    }
+  }
+
+  if (identifier.length) {
+    yield identifier;
+  }
+}
+
 /**
  * Set Object attribute.
  * The name may be a property path like 'a.b.c'.
@@ -143,7 +174,6 @@ export function defaultValues(attributes, object) {
  * @param {any} value
  */
 export function setAttribute(object, name, value) {
-
   let lastObject = object;
   let lastKey;
 
@@ -169,37 +199,6 @@ export function setAttribute(object, name, value) {
   lastObject[lastKey] = value;
 }
 
-/**
- * Split property path into tokens
- * @param {string} string 
- * @return {Iterator<string>}
- */
-function* tokens(string) {
-  let identifier = "";
-
-  for (const c of string) {
-    switch (c) {
-      case "\t":
-      case " ": break;
-      case ".":
-      case "[":
-      case "]":
-        if (identifier.length) {
-          yield identifier;
-          identifier = "";
-        }
-        yield c;
-        break;
-      default:
-        identifier += c;
-    }
-  }
-
-  if (identifier.length) {
-    yield identifier;
-  }
-}
-
 /**
  * Deliver attribute value.
  * The name may be a property path like 'a.b.c'.

package/src/base-object.mjs

@@ -37,6 +37,7 @@ export class BaseObject {
   }
 
   /**
+   * User modifyable attributes.
    * @return {Object} writable attributes
    */
   static get writableAttributes() {

package/src/branch.mjs

@@ -19,6 +19,10 @@ export class Branch extends Ref {
   static get addMethodName() {
     return "_addBranch";
   }
+
+  static get deleteMethodName() {
+    return "_deleteBranch";  	
+  }
   
   static get collectionName() {
     return "branches";

package/src/hook.mjs

@@ -20,4 +20,8 @@ export class Hook extends OwnedObject {
   static get addMethodName() {
     return "_addHook";
   }
+
+  static get delteteMethodName() {
+  	return "_deleteHook";
+  }
 }

package/src/multi-group-provider.mjs

@@ -47,7 +47,7 @@ export class MultiGroupProvider extends BaseProvider {
         if (rg !== undefined) {
           const r = await rg.repository(repository);
           if (r !== undefined) {
-            return r.branch( branch || r.defaultBranchName);
+            return r.branch( branch || r.defaultBranchName);
           }
         }
       }
@@ -104,7 +104,7 @@ export class MultiGroupProvider extends BaseProvider {
    * @return {RepositoryGroup}
    */
   addRepositoryGroup(name, options) {
-    return this.#repositoryGroups.get(this.normalizeGroupName(name, true)) || new this.repositoryGroupClass(this, name, options);
+    return this.#repositoryGroups.get(this.normalizeGroupName(name, true)) || new this.repositoryGroupClass(this, name, options);
   }
 
   _addRepositoryGroup(repositoryGroup) {

package/src/named-object.mjs

@@ -1,6 +1,6 @@
 import { optionJSON } from "./attribute.mjs";
 import { BaseObject } from "./base-object.mjs";
-import { name, url } from "./attributes.mjs";
+import { name, url, description, id, uuid } from "./attributes.mjs";
 
 /**
  * Object with a name.
@@ -16,9 +16,11 @@ export class NamedObject extends BaseObject {
    */
   static get attributes() {
     return {
-      ...super.attributes,
       name,
-
+      id,
+      uuid,
+      description,
+   
       /**
        * The url of home page.
        * @return {string}

package/src/owned-object.mjs

@@ -13,15 +13,23 @@ export class OwnedObject extends NamedObject {
     return "_add" + this.name;
   }
 
-  constructor(owner, name, options, additionalProperties) {
-    super(name, options, {
-      ...additionalProperties,
-      owner: { value: owner }
-    });
+  static get deleteMethodName() {
+    return "_delete" + this.name;
+  }
 
+  constructor(owner, name, options, additionalProperties) {
+    super(name, options, additionalProperties);
+    this.owner = owner;
     owner[this.constructor.addMethodName](this);
   }
 
+  /**
+   * Removes the receiver from the owner.
+   */
+  delete() {
+    this.owner[this.constructor.deleteMethodName](this);
+  }
+
   /**
    * Check for equality.
    * @param {OwnedObject} other

package/src/pull-request.mjs

@@ -31,6 +31,10 @@ export class PullRequest extends OwnedObject {
     return "_addPullRequest";
   }
 
+  static get deleteMethodName() {
+    return "_deletePullRequest";	
+  }
+  
   static get type() {
     return "pull-request";
   }
@@ -39,14 +43,6 @@ export class PullRequest extends OwnedObject {
     return "pullRequests";
   }
 
-  /**
-   * All valid states
-   * @return {Set<string>} valid states
-   */
-  static get validStates() {
-    return new Set(["OPEN", "MERGED", "CLOSED"]);
-  }
-
   /**
    * States to list pull request by default
    * @return {Set<string>} states to list by default
@@ -103,6 +99,7 @@ export class PullRequest extends OwnedObject {
       state: {
         ...state,
         default: "OPEN",
+        values: new Set(["OPEN", "MERGED", "CLOSED"]),
         writable: true
       },
 
@@ -143,7 +140,7 @@ export class PullRequest extends OwnedObject {
       state: {
         set(value) {
           value = value.toUpperCase();
-          if (this.constructor.validStates.has(value)) {
+          if (this.constructor.attributes.state.values.has(value)) {
             state = value;
           } else throw new Error(`Invalid Pull Request state ${value}`);
         },

package/src/ref.mjs

@@ -93,7 +93,7 @@ export class Ref extends OwnedObject {
    * Ref owner.
    * By default we provide the repository owner
    * @see {@link Repository#owner}
-   * @return {string}
+   * @return {Repository}
    */
   get repository() {
     return this.owner;

package/src/repository-group.mjs

@@ -23,6 +23,10 @@ export class RepositoryGroup extends RepositoryOwner(OwnedObject) {
     return "_addRepositoryGroup";
   }
 
+  static get deleteMethodName() {
+    return "_deleteRepositoryGroup";  	
+  }
+
   static get type() {
     return "repository-group";
   }

package/src/repository.mjs

@@ -8,13 +8,13 @@ import { url, size, language, boolean_attribute } from "./attributes.mjs";
 
 /**
  * Abstract repository
- * @param {Owner} owner
+ * @param {RepositoryOwner} owner
  * @param {string} name (#branch) will be removed
  * @param {Object} options
  * @param {string} [options.description] human readable description
  * @param {string} [options.id] internal id
  *
- * @property {Owner} owner
+ * @property {RepositoryOwner} owner
  * @property {string} name without (#branch)
  * @property {string} [description] from options.description
  * @property {string} [id] from options.id
@@ -28,6 +28,11 @@ export class Repository extends OwnedObject {
     return "_addRepository";
   }
 
+  static get deleteMethodName()
+  {
+     return "_deleteRepository";  	
+  }
+
   static get collectionName() {
     return "repositories";
   }

package/src/tag.mjs

@@ -9,6 +9,10 @@ export class Tag extends Ref {
     return "_addTag";
   }
 
+  static get deleteMethodName() {
+    return "_deleteTag";  	
+  }
+
   /**
    * @return {string} tags
    */