diff --git a/services/spelling/.mocharc.json b/services/spelling/.mocharc.json new file mode 100644 index 0000000000..dc3280aa96 --- /dev/null +++ b/services/spelling/.mocharc.json @@ -0,0 +1,3 @@ +{ + "require": "test/setup.js" +} diff --git a/services/spelling/test/setup.js b/services/spelling/test/setup.js new file mode 100644 index 0000000000..9ebe599cea --- /dev/null +++ b/services/spelling/test/setup.js @@ -0,0 +1,21 @@ +const chai = require('chai') +const SandboxedModule = require('sandboxed-module') + +// Chai configuration +chai.should() + +// SandboxedModule configuration +SandboxedModule.configure({ + requires: { + 'logger-sharelatex': { + debug() {}, + log() {}, + info() {}, + warn() {}, + err() {}, + error() {}, + fatal() {} + } + }, + globals: { Buffer, JSON, console, process } +}) diff --git a/services/spelling/test/unit/js/ASpellTests.js b/services/spelling/test/unit/js/ASpellTests.js index 22798db6f2..9ea535dd3a 100644 --- a/services/spelling/test/unit/js/ASpellTests.js +++ b/services/spelling/test/unit/js/ASpellTests.js @@ -9,20 +9,13 @@ * DS102: Remove unnecessary code created because of implicit returns * Full docs: https://github.com/decaffeinate/decaffeinate/blob/master/docs/suggestions.md */ -const chai = require('chai') -const should = chai.should() +const { expect, assert } = require('chai') const SandboxedModule = require('sandboxed-module') -const { assert } = require('chai') describe('ASpell', function () { beforeEach(function () { return (this.ASpell = SandboxedModule.require('../../../app/js/ASpell', { requires: { - 'logger-sharelatex': { - log() {}, - info() {}, - err() {} - }, '@overleaf/metrics': { gauge() {}, inc() {} @@ -93,7 +86,7 @@ describe('ASpell', function () { }) return it('should return an error', function () { - return should.exist(this.error) + return expect(this.error).to.exist }) }) diff --git a/services/spelling/test/unit/js/ASpellWorkerTests.js b/services/spelling/test/unit/js/ASpellWorkerTests.js index 6fbd6d805e..aa85be41cc 100644 --- a/services/spelling/test/unit/js/ASpellWorkerTests.js +++ b/services/spelling/test/unit/js/ASpellWorkerTests.js @@ -3,8 +3,7 @@ no-undef */ const sinon = require('sinon') -const chai = require('chai') -const { expect } = chai +const { expect } = require('chai') const SandboxedModule = require('sandboxed-module') const EventEmitter = require('events') @@ -15,11 +14,6 @@ describe('ASpellWorker', function () { '../../../app/js/ASpellWorker', { requires: { - 'logger-sharelatex': { - log() {}, - info() {}, - err() {} - }, '@overleaf/metrics': { gauge() {}, inc() {} diff --git a/services/spelling/test/unit/js/LearnedWordsManagerTests.js b/services/spelling/test/unit/js/LearnedWordsManagerTests.js index 16aa763f2c..99249a2c39 100644 --- a/services/spelling/test/unit/js/LearnedWordsManagerTests.js +++ b/services/spelling/test/unit/js/LearnedWordsManagerTests.js @@ -1,12 +1,11 @@ const sinon = require('sinon') -const chai = require('chai') -const { expect } = chai +const { assert, expect } = require('chai') const SandboxedModule = require('sandboxed-module') const modulePath = require('path').join( __dirname, '../../../app/js/LearnedWordsManager' ) -const { assert } = require('chai') + describe('LearnedWordsManager', function () { beforeEach(function () { this.token = 'a6b3cd919ge' @@ -22,17 +21,9 @@ describe('LearnedWordsManager', function () { del: sinon.stub() } this.LearnedWordsManager = SandboxedModule.require(modulePath, { - globals: { - console: console - }, requires: { './mongodb': { db: this.db }, './MongoCache': this.cache, - 'logger-sharelatex': { - log() {}, - err() {}, - info() {} - }, '@overleaf/metrics': { timeAsyncMethod: sinon.stub(), inc: sinon.stub() diff --git a/services/spelling/test/unit/js/SpellingAPIManagerTests.js b/services/spelling/test/unit/js/SpellingAPIManagerTests.js index 76d592720f..8cf1d0c694 100644 --- a/services/spelling/test/unit/js/SpellingAPIManagerTests.js +++ b/services/spelling/test/unit/js/SpellingAPIManagerTests.js @@ -2,9 +2,7 @@ handle-callback-err */ const sinon = require('sinon') -const chai = require('chai') -const { expect } = chai -chai.should() +const { expect } = require('chai') const SandboxedModule = require('sandboxed-module') const modulePath = require('path').join( __dirname,