rename role to roleId

digidem · Aug 31, 2023 · 562f73d · 562f73d
1 parent d2e387c
commit 562f73d
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 3 deletions.
diff --git a/src/member-api.js b/src/member-api.js
@@ -25,19 +25,23 @@ export class MemberApi extends TypedEmitter {
    * @param {string} deviceId
    *
    * @param {Object} opts
-   * @param {string} opts.role
+   * @param {string} opts.roleId
    * @param {number} [opts.timeout]
    *
    * @returns {Promise<import('./generated/rpc.js').InviteResponse_Decision>}
    */
   async invite(deviceId, { timeout }) {
     const projectInfo = await this.#getProjectInfo()
 
-    return this.#rpc.invite(deviceId, {
+    const response = await this.#rpc.invite(deviceId, {
       projectKey: this.#projectKey,
       encryptionKeys: this.#encryptionKeys,
       projectInfo,
       timeout,
     })
+
+    // TODO: If response is ACCEPT, write to capabilities
+
+    return response
   }
 }
diff --git a/tests/member-api.js b/tests/member-api.js
@@ -26,7 +26,7 @@ test('Invite sends expected project-related details', async (t) => {
 
   r1.on('peers', async (peers) => {
     const response = await memberApi.invite(peers[0].id, {
-      role: 'member',
+      roleId: randomBytes(8).toString('hex'),
     })
 
     t.is(response, InviteResponse_Decision.ACCEPT)