adapt-authoring-roles 1.4.0 → 1.5.0

package/.github/workflows/standardjs.yml

@@ -1,4 +1,4 @@
-name: Standard.js formatting check
+name: Lint
 on: push
 jobs:
   default:
@@ -9,4 +9,4 @@ jobs:
         with:
           node-version: 'lts/*'
       - run: npm install
-      - run: npx standard
+      - run: npx standard

package/lib/RolesModule.js

@@ -8,7 +8,6 @@ class RolesModule extends AbstractApiModule {
   /** @override */
   async init () {
     await super.init()
-    this.cache.isEnabled = true
     try {
       await this.initConfigRoles()
 
@@ -17,6 +16,7 @@ class RolesModule extends AbstractApiModule {
     } catch (e) {
       this.log('error', e)
     }
+    this.cache.isEnabled = true
     const [authlocal, users] = await this.app.waitForModule('auth-local', 'users')
     authlocal.registerHook.tap(this.onUpdateRoles.bind(this))
     users.requestHook.tap(this.onUpdateRoles.bind(this))
@@ -96,10 +96,9 @@ class RolesModule extends AbstractApiModule {
 
   /** @override */
   async setValues () {
-    /** @ignore */ this.root = 'roles'
+    await super.setValues()
     /** @ignore */ this.schemaName = 'role'
     /** @ignore */ this.collectionName = 'roles'
-    this.useDefaultRouteConfig()
   }
 
   /**

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "adapt-authoring-roles",
-  "version": "1.4.0",
+  "version": "1.5.0",
   "description": "Module for managing user roles",
   "homepage": "https://github.com/adapt-security/adapt-authoring-roles",
   "license": "GPL-3.0",
@@ -44,7 +44,7 @@
     ]
   },
   "dependencies": {
-    "adapt-authoring-api": "^2.0.0"
+    "adapt-authoring-api": "^3.0.0"
   },
   "peerDependencies": {
     "adapt-authoring-auth": "^2.0.0",

package/routes.json

@@ -0,0 +1,3 @@
+{
+  "root": "roles"
+}

package/tests/RolesModule.spec.js

@@ -13,10 +13,8 @@ import assert from 'node:assert/strict'
 /** Build a minimal RolesModule-like instance with sensible stub defaults */
 function createInstance (overrides) {
   const instance = {
-    root: undefined,
     schemaName: undefined,
     collectionName: undefined,
-    useDefaultRouteConfig: mock.fn(),
     app: {
       waitForModule: mock.fn(async () => ({})),
       errors: {
@@ -46,10 +44,8 @@ function createInstance (overrides) {
 // ── Method references (copied from source for isolated testing) ─────
 
 async function setValues () {
-  this.root = 'roles'
   this.schemaName = 'role'
   this.collectionName = 'roles'
-  this.useDefaultRouteConfig()
 }
 
 async function getScopesForRole (_id) {
@@ -171,12 +167,6 @@ describe('RolesModule', () => {
   // ── setValues ──────────────────────────────────────────────────────
 
   describe('setValues', () => {
-    it('should set root to "roles"', async () => {
-      const inst = createInstance()
-      await setValues.call(inst)
-      assert.equal(inst.root, 'roles')
-    })
-
     it('should set schemaName to "role"', async () => {
       const inst = createInstance()
       await setValues.call(inst)
@@ -188,12 +178,6 @@ describe('RolesModule', () => {
       await setValues.call(inst)
       assert.equal(inst.collectionName, 'roles')
     })
-
-    it('should call useDefaultRouteConfig', async () => {
-      const inst = createInstance()
-      await setValues.call(inst)
-      assert.equal(inst.useDefaultRouteConfig.mock.callCount(), 1)
-    })
   })
 
   // ── getScopesForRole ───────────────────────────────────────────────