feat(permissions): A2 — гигиена реестра (lint-тест) + ясные метки
Тест permissions-registry: каждый ключ из requirePermission/perm('…') в backend
есть в registry (ловит опечатки/дрейф; perm() падал на старте, сырой
requirePermission — нет). Заодно логирует ключи реестра, не используемые в
requirePermission (информативно — часть гейтится на клиенте через /me).
Метки theory.access/simulations.access переформулированы: «… доступен роли»
(видимость конкретного контента — по классам в «Доступ · контент»).
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This commit is contained in:
@@ -144,14 +144,14 @@ const PERMISSIONS = {
|
||||
},
|
||||
'theory.access': {
|
||||
role: 'student', roles: ['student', 'free_student'], default: 1,
|
||||
label: 'Доступ к теории',
|
||||
desc: 'Просматривать теоретические курсы и уроки',
|
||||
label: 'Раздел теории доступен роли',
|
||||
desc: 'Включает раздел теории для роли. Какие именно курсы видны — настраивается по классам в «Доступ · контент»',
|
||||
requireConfirmOff: true,
|
||||
},
|
||||
'simulations.access': {
|
||||
role: 'student', roles: ['student', 'free_student'], default: 1,
|
||||
label: 'Доступ к симуляциям',
|
||||
desc: 'Открывать лабораторию с физическими, химическими и биологическими симуляциями',
|
||||
label: 'Лаборатория доступна роли',
|
||||
desc: 'Включает раздел лаборатории для роли. Какие именно симуляции видны — настраивается по классам в «Доступ · контент»',
|
||||
requireConfirmOff: true,
|
||||
},
|
||||
'simulations.quiz': {
|
||||
|
||||
@@ -0,0 +1,55 @@
|
||||
'use strict';
|
||||
/**
|
||||
* Гигиена реестра прав (A2): каждый ключ, переданный в requirePermission('…') /
|
||||
* perm('…') в коде backend/src, ДОЛЖЕН существовать в registry.js. Ловит опечатки
|
||||
* и дрейф (раньше perm() падал на старте, но «сырой» requirePermission — нет).
|
||||
* Дополнительно: список ключей реестра, не используемых в requirePermission (это
|
||||
* НЕ ошибка — многие студенческие ключи гейтятся на клиенте через /me — просто
|
||||
* выводим для обзора).
|
||||
*/
|
||||
const { describe, it } = require('node:test');
|
||||
const assert = require('node:assert/strict');
|
||||
const fs = require('node:fs');
|
||||
const path = require('node:path');
|
||||
const registry = require('../src/permissions/registry');
|
||||
|
||||
const SRC = path.join(__dirname, '..', 'src');
|
||||
const USE_RE = /\b(?:requirePermission|perm)\(\s*['"]([\w.]+)['"]/g;
|
||||
|
||||
function walk(dir, out = []) {
|
||||
for (const e of fs.readdirSync(dir, { withFileTypes: true })) {
|
||||
const p = path.join(dir, e.name);
|
||||
if (e.isDirectory()) walk(p, out);
|
||||
else if (e.isFile() && e.name.endsWith('.js')) out.push(p);
|
||||
}
|
||||
return out;
|
||||
}
|
||||
|
||||
describe('permissions registry hygiene', () => {
|
||||
const files = walk(SRC);
|
||||
const used = new Map(); // key -> first file where seen
|
||||
for (const f of files) {
|
||||
const txt = fs.readFileSync(f, 'utf8');
|
||||
let m;
|
||||
while ((m = USE_RE.exec(txt)) !== null) {
|
||||
if (!used.has(m[1])) used.set(m[1], path.relative(SRC, f));
|
||||
}
|
||||
}
|
||||
|
||||
it('все ключи requirePermission/perm(...) в коде есть в реестре', () => {
|
||||
const unknown = [...used.entries()].filter(([k]) => !registry.isKnown(k));
|
||||
assert.equal(unknown.length, 0,
|
||||
'неизвестные ключи прав (опечатка / не добавлены в registry): ' +
|
||||
unknown.map(([k, f]) => `${k} (${f})`).join(', '));
|
||||
});
|
||||
|
||||
it('обзор: ключи реестра, не используемые в requirePermission (информативно, не ошибка)', () => {
|
||||
const unused = registry.listKeys().filter(k => !used.has(k));
|
||||
// Не падаем: часть прав enforce-ится на клиенте через /permissions/me.
|
||||
if (unused.length) {
|
||||
// eslint-disable-next-line no-console
|
||||
console.log('[perm-hygiene] не используются в requirePermission (ок, могут гейтиться на клиенте):', unused.join(', '));
|
||||
}
|
||||
assert.ok(Array.isArray(unused));
|
||||
});
|
||||
});
|
||||
Reference in New Issue
Block a user