AdminAuthorizationHelperTests.js 16 KB

123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346347348349350351352353354355356357358359360361362363364365366367368369370371372373374375376377378379380381382383384385386387388389390391392393394395396397398399400401402403404405406407408409410411412413414415416417418419420421422423424425426427428429430431432433434435436437438439440441442443444445446447448449450451452453454455456457458459460461462463464465466467468469470471472473474475476477478479
  1. const { expect } = require('chai')
  2. const SandboxedModule = require('sandboxed-module')
  3. const sinon = require('sinon')
  4. const MockRequest = require('../helpers/MockRequest')
  5. const MockResponse = require('../helpers/MockResponse')
  6. const modulePath =
  7. '../../../../app/src/Features/Helpers/AdminAuthorizationHelper'
  8. describe('AdminAuthorizationHelper', function () {
  9. beforeEach(function () {
  10. this.fireHook = sinon.stub().resolves([])
  11. this.settings = {
  12. adminPrivilegeAvailable: true,
  13. adminUrl: 'https://admin.overleaf.com',
  14. adminRolesEnabled: true,
  15. }
  16. this.AdminAuthorizationHelper = SandboxedModule.require(modulePath, {
  17. requires: {
  18. '@overleaf/settings': this.settings,
  19. '../../infrastructure/Modules': {
  20. promises: {
  21. hooks: {
  22. fire: this.fireHook,
  23. },
  24. },
  25. },
  26. },
  27. })
  28. })
  29. describe('getAdminCapabilities', function () {
  30. describe('when modules return capabilities', function () {
  31. let result
  32. const module1Capabilities = ['capability1', 'capability2']
  33. const module2Capabilities = ['capability2', 'capability3']
  34. beforeEach(async function () {
  35. this.fireHook.resolves([module1Capabilities, module2Capabilities])
  36. result = await this.AdminAuthorizationHelper.getAdminCapabilities({})
  37. })
  38. it('returns true for adminCapabilitiesAvailable', async function () {
  39. expect(result.adminCapabilitiesAvailable).to.be.true
  40. })
  41. it('returns a flattened array of the returned capabilities', function () {
  42. expect(result.adminCapabilities)
  43. .to.be.an('array')
  44. .that.includes(...module1Capabilities, ...module2Capabilities)
  45. })
  46. })
  47. describe('when no module returns capabilities', function () {
  48. let result
  49. beforeEach(async function () {
  50. result = await this.AdminAuthorizationHelper.getAdminCapabilities({})
  51. })
  52. it('returns false for adminCapabilitiesAvailable', function () {
  53. expect(result.adminCapabilitiesAvailable).to.be.false
  54. })
  55. it('returns an empty adminCapabilities array', function () {
  56. expect(result.adminCapabilities).to.be.an('array').that.is.empty
  57. })
  58. })
  59. })
  60. describe('useAdminCapabilities', function () {
  61. describe('when admin capabilities are not available', function () {
  62. describe('user is null', function () {
  63. beforeEach(async function () {
  64. this.req = new MockRequest()
  65. this.res = new MockResponse()
  66. this.next = sinon.stub()
  67. this.req.session = {
  68. user: null,
  69. }
  70. await this.AdminAuthorizationHelper.useAdminCapabilities(
  71. this.req,
  72. this.res,
  73. this.next
  74. )
  75. })
  76. it('does not define adminCapabilitiesAvailable on req', function () {
  77. expect(this.req).not.to.have.property('adminCapabilitiesAvailable')
  78. })
  79. it('defines adminCapabilities as an empty array on req', function () {
  80. expect(this.req).to.have.property('adminCapabilities')
  81. expect(this.req.adminCapabilities).to.be.an('array')
  82. expect(this.req.adminCapabilities).to.be.empty
  83. })
  84. })
  85. describe('user is not an admin', function () {
  86. beforeEach(async function () {
  87. this.req = new MockRequest()
  88. this.res = new MockResponse()
  89. this.next = sinon.stub()
  90. this.user = {
  91. isAdmin: false,
  92. }
  93. this.req.session = {
  94. user: this.user,
  95. }
  96. await this.AdminAuthorizationHelper.useAdminCapabilities(
  97. this.req,
  98. this.res,
  99. this.next
  100. )
  101. })
  102. it('does not define adminCapabilitiesAvailable on req', function () {
  103. expect(this.req).not.to.have.property('adminCapabilitiesAvailable')
  104. })
  105. it('defines adminCapabilities as an empty array on req', function () {
  106. expect(this.req).to.have.property('adminCapabilities')
  107. expect(this.req.adminCapabilities).to.be.an('array')
  108. expect(this.req.adminCapabilities).to.be.empty
  109. })
  110. })
  111. describe('user is an admin', function () {
  112. beforeEach(async function () {
  113. this.req = new MockRequest()
  114. this.res = new MockResponse()
  115. this.next = sinon.stub()
  116. this.user = {
  117. isAdmin: true,
  118. }
  119. this.req.session = {
  120. user: this.user,
  121. }
  122. await this.AdminAuthorizationHelper.useAdminCapabilities(
  123. this.req,
  124. this.res,
  125. this.next
  126. )
  127. })
  128. it('defines adminCapabilitiesAvailable as false on req', function () {
  129. expect(this.req).to.have.property('adminCapabilitiesAvailable', false)
  130. })
  131. it('defines adminCapabilities as an empty array', function () {
  132. expect(this.req).to.have.property('adminCapabilities')
  133. expect(this.req.adminCapabilities).to.be.an('array')
  134. expect(this.req.adminCapabilities).to.be.empty
  135. })
  136. })
  137. })
  138. describe('when admin capabilities are available', function () {
  139. beforeEach(function () {
  140. this.fireHook.resolves(['capability1', 'capability2'])
  141. })
  142. describe('user is not an admin', function () {
  143. beforeEach(async function () {
  144. this.req = new MockRequest()
  145. this.res = new MockResponse()
  146. this.next = sinon.stub()
  147. this.user = {
  148. isAdmin: false,
  149. }
  150. this.req.session = {
  151. user: this.user,
  152. }
  153. await this.AdminAuthorizationHelper.useAdminCapabilities(
  154. this.req,
  155. this.res,
  156. this.next
  157. )
  158. })
  159. it('does not define adminCapabilitiesAvailable on req', function () {
  160. expect(this.req).not.to.have.property('adminCapabilitiesAvailable')
  161. })
  162. it('defines adminCapabilities as an empty array on req', function () {
  163. expect(this.req).to.have.property('adminCapabilities')
  164. expect(this.req.adminCapabilities).to.be.an('array')
  165. expect(this.req.adminCapabilities).to.be.empty
  166. })
  167. })
  168. describe('user is an admin', function () {
  169. beforeEach(async function () {
  170. this.req = new MockRequest()
  171. this.res = new MockResponse()
  172. this.next = sinon.stub()
  173. this.user = {
  174. isAdmin: true,
  175. }
  176. this.req.session = {
  177. user: this.user,
  178. }
  179. await this.AdminAuthorizationHelper.useAdminCapabilities(
  180. this.req,
  181. this.res,
  182. this.next
  183. )
  184. })
  185. it('defines adminCapabilitiesAvailable as true on req', function () {
  186. expect(this.req).to.have.property('adminCapabilitiesAvailable', true)
  187. })
  188. it('defines adminCapabilities with the capabilities returned from modules', function () {
  189. expect(this.req).to.have.property('adminCapabilities')
  190. expect(this.req.adminCapabilities).to.be.an('array')
  191. expect(this.req.adminCapabilities).to.include('capability1')
  192. expect(this.req.adminCapabilities).to.include('capability2')
  193. })
  194. })
  195. })
  196. describe('when getting capabilities from modules throws an error', function () {
  197. beforeEach(async function () {
  198. this.fireHook.rejects(new Error('Module error'))
  199. this.req = new MockRequest()
  200. this.res = new MockResponse()
  201. this.next = sinon.stub()
  202. this.user = {
  203. isAdmin: true,
  204. }
  205. this.req.logger = {
  206. warn: sinon.stub(),
  207. }
  208. this.req.session = {
  209. user: this.user,
  210. }
  211. await this.AdminAuthorizationHelper.useAdminCapabilities(
  212. this.req,
  213. this.res,
  214. this.next
  215. )
  216. })
  217. it('logs the error', function () {
  218. expect(this.logger.warn).to.have.been.calledWith(
  219. sinon.match.has('err', sinon.match.instanceOf(Error))
  220. )
  221. })
  222. it('defines adminCapabilitiesAvailable as true on req', function () {
  223. expect(this.req).to.have.property('adminCapabilitiesAvailable', true)
  224. })
  225. it('defines adminCapabilities as an empty array', function () {
  226. expect(this.req).to.have.property('adminCapabilities')
  227. expect(this.req.adminCapabilities).to.be.an('array')
  228. expect(this.req.adminCapabilities).to.be.empty
  229. })
  230. })
  231. })
  232. describe('useHasAdminCapability', function () {
  233. it('adds hasAdminCapability to res.locals', function () {
  234. const req = new MockRequest()
  235. const res = new MockResponse()
  236. const next = sinon.stub()
  237. this.AdminAuthorizationHelper.useHasAdminCapability(req, res, next)
  238. expect(res.locals).to.have.property('hasAdminCapability')
  239. expect(res.locals.hasAdminCapability).to.be.a('function')
  240. })
  241. describe('when the user is not an admin', function () {
  242. describe('when req.adminCapabilitiesAvailable is true', function () {
  243. it('returns false for any capability', function () {
  244. const req = new MockRequest()
  245. const res = new MockResponse()
  246. const next = sinon.stub()
  247. req.adminCapabilitiesAvailable = true
  248. req.adminCapabilities = []
  249. req.session.user = { isAdmin: false }
  250. this.AdminAuthorizationHelper.useHasAdminCapability(req, res, next)
  251. expect(res.locals.hasAdminCapability('capability1')).to.be.false
  252. })
  253. })
  254. describe('when req.adminCapabilitiesAvailable is false', function () {
  255. it('returns false for any capability', function () {
  256. const req = new MockRequest()
  257. const res = new MockResponse()
  258. const next = sinon.stub()
  259. req.adminCapabilitiesAvailable = false
  260. req.adminCapabilities = []
  261. req.session.user = { isAdmin: false }
  262. this.AdminAuthorizationHelper.useHasAdminCapability(req, res, next)
  263. expect(res.locals.hasAdminCapability('capability1')).to.be.false
  264. })
  265. })
  266. describe('when req.adminCapabilitiesAvailable is undefined', function () {
  267. it('returns false for any capability', function () {
  268. const req = new MockRequest()
  269. const res = new MockResponse()
  270. const next = sinon.stub()
  271. req.session.user = { isAdmin: false }
  272. this.AdminAuthorizationHelper.useHasAdminCapability(req, res, next)
  273. expect(res.locals.hasAdminCapability('capability1')).to.be.false
  274. })
  275. })
  276. })
  277. describe('user is an admin', function () {
  278. describe('when req.adminCapabilitiesAvailable is false', function () {
  279. it('returns true for any capability', function () {
  280. const req = new MockRequest()
  281. const res = new MockResponse()
  282. const next = sinon.stub()
  283. req.session.user = { isAdmin: true }
  284. req.adminCapabilitiesAvailable = false
  285. this.AdminAuthorizationHelper.useHasAdminCapability(req, res, next)
  286. expect(res.locals.hasAdminCapability('capability1')).to.be.true
  287. })
  288. })
  289. describe('when req.adminCapabilitiesAvailable is undefined', function () {
  290. it('returns true for any capability', function () {
  291. const req = new MockRequest()
  292. const res = new MockResponse()
  293. const next = sinon.stub()
  294. req.session.user = { isAdmin: true }
  295. this.AdminAuthorizationHelper.useHasAdminCapability(req, res, next)
  296. expect(res.locals.hasAdminCapability('capability1')).to.be.true
  297. })
  298. })
  299. describe('when req.adminCapabilitiesAvailable is true', function () {
  300. let req, res, next
  301. beforeEach(function () {
  302. req = new MockRequest()
  303. res = new MockResponse()
  304. next = sinon.stub()
  305. req.session.user = { isAdmin: true }
  306. req.adminCapabilitiesAvailable = true
  307. req.adminCapabilities = ['capability1', 'capability2']
  308. this.AdminAuthorizationHelper.useHasAdminCapability(req, res, next)
  309. })
  310. it('returns true for a capability the user has', function () {
  311. expect(res.locals.hasAdminCapability('capability1')).to.be.true
  312. })
  313. it('returns false for a capability the user does not have', function () {
  314. expect(res.locals.hasAdminCapability('capability3')).to.be.false
  315. })
  316. })
  317. })
  318. })
  319. describe('hasAdminCapability', function () {
  320. describe('when user is not an admin', function () {
  321. it('returns false', function () {
  322. const req = {
  323. session: {
  324. user: { isAdmin: false },
  325. },
  326. }
  327. expect(
  328. this.AdminAuthorizationHelper.hasAdminCapability('capability')(req)
  329. ).to.be.false
  330. })
  331. })
  332. describe('when user is an admin', function () {
  333. describe('when adminCapabilitiesAvailable is falsey', function () {
  334. it('returns true', function () {
  335. const req = {
  336. session: {
  337. user: { isAdmin: true },
  338. },
  339. adminCapabilitiesAvailable: false,
  340. }
  341. expect(
  342. this.AdminAuthorizationHelper.hasAdminCapability('capability')(req)
  343. ).to.be.true
  344. })
  345. it('ignores the "requireAdminRoles" argument', function () {
  346. const req = {
  347. session: { user: { isAdmin: true } },
  348. adminCapabilitiesAvailable: false,
  349. }
  350. expect(
  351. this.AdminAuthorizationHelper.hasAdminCapability(
  352. 'capability',
  353. true
  354. )(req)
  355. ).to.be.true
  356. expect(
  357. this.AdminAuthorizationHelper.hasAdminCapability(
  358. 'capability',
  359. false
  360. )(req)
  361. ).to.be.true
  362. })
  363. })
  364. describe('when adminCapabilitiesAvailable is true', function () {
  365. describe('when user has the requested capability', function () {
  366. it('returns true', function () {
  367. const req = {
  368. session: { user: { isAdmin: true } },
  369. adminCapabilitiesAvailable: true,
  370. adminCapabilities: ['capability'],
  371. }
  372. expect(
  373. this.AdminAuthorizationHelper.hasAdminCapability('capability')(
  374. req
  375. )
  376. ).to.be.true
  377. })
  378. })
  379. describe('when user does not have the requested capability', function () {
  380. it('returns false', function () {
  381. const req = {
  382. session: { user: { isAdmin: true } },
  383. adminCapabilitiesAvailable: true,
  384. adminCapabilities: ['other-capability'],
  385. }
  386. expect(
  387. this.AdminAuthorizationHelper.hasAdminCapability('capability')(
  388. req
  389. )
  390. ).to.be.false
  391. })
  392. })
  393. })
  394. })
  395. describe('when admin roles are not enabled', function () {
  396. beforeEach(function () {
  397. this.settings.adminRolesEnabled = false
  398. })
  399. it('returns false even for admins', function () {
  400. const req = { session: { user: { isAdmin: true } } }
  401. expect(
  402. this.AdminAuthorizationHelper.hasAdminCapability('capability')(req)
  403. ).to.be.false
  404. expect(
  405. this.AdminAuthorizationHelper.hasAdminCapability(
  406. 'capability',
  407. true
  408. )(req)
  409. ).to.be.false
  410. })
  411. it('returns true when requireAdminRoles=false', function () {
  412. const req = { session: { user: { isAdmin: true } } }
  413. expect(
  414. this.AdminAuthorizationHelper.hasAdminCapability(
  415. 'capability',
  416. false
  417. )(req)
  418. ).to.be.true
  419. })
  420. })
  421. })
  422. })