Skip to content

Commit 336d7f4

Browse files
committed
tests: replace obsolete library mockery with rewiremock
1 parent 3256275 commit 336d7f4

7 files changed

Lines changed: 207 additions & 50 deletions

File tree

package-lock.json

Lines changed: 182 additions & 7 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -94,12 +94,12 @@
9494
"license-checker": "25.0.1",
9595
"load-grunt-tasks": "5.1.0",
9696
"lodash": "^4.17.21",
97-
"mockery": "2.1.0",
9897
"multer": "2.0.2",
9998
"nodemailer": "7.0.5",
10099
"npm-force-resolutions": "0.0.10",
101100
"postcss": "8.5.6",
102101
"postcss-minify": "^1.2.0",
102+
"rewiremock": "^3.14.6",
103103
"tinymce": "4.9.11"
104104
},
105105
"browserslist": [

spec/converter-spec.js

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44

55
describe('Template converter', function() {
66

7-
var mockery = require('mockery');
7+
var rewiremock = require('rewiremock/node');
88

99
var mockedBindingProvider = function(a, b) {
1010
// console.log("binding provider for", a, b);
@@ -18,9 +18,10 @@ describe('Template converter', function() {
1818
var _parseTemplate;
1919

2020
beforeAll(function() {
21-
mockery.registerMock('jquery', require('cheerio').load('<html />'));
22-
mockery.registerAllowables(['fs', '../src/js/converter/checkdefs.js', '../src/js/converter/declarations.js', '../src/js/converter/model.js', '../src/js/converter/parser.js', 'console', './utils.js', './domutils.js', 'console', '../node_modules/mensch', './lib/lexer', './lib/parser', './lib/stringify', './debug', 'jsep', './declarations.js', './cssparser.js', 'mensch/lib/parser.js', './lexer', './stylesheet.js', './model.js']);
23-
mockery.enable();
21+
var cheerioMock = require('cheerio').load('<html />');
22+
23+
rewiremock.enable();
24+
rewiremock('jquery').with(cheerioMock);
2425

2526
_parseTemplate = function(html) {
2627
var translateTemplate = require('../src/js/converter/parser.js');
@@ -299,8 +300,7 @@ describe('Template converter', function() {
299300
});
300301

301302
afterAll(function() {
302-
mockery.disable();
303-
mockery.deregisterAll();
303+
rewiremock.disable();
304304
});
305305

306306
});

spec/declarations-spec.js

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44

55
describe('Style declaration processor', function() {
66

7-
var mockery = require('mockery');
7+
var rewiremock = require('rewiremock/node');
88
var declarations;
99

1010
var templateUrlConverter = function(url) { return '.'+url; };
@@ -15,9 +15,11 @@ describe('Style declaration processor', function() {
1515
};
1616

1717
beforeAll(function() {
18-
mockery.registerMock('jquery', require('cheerio').load('<html />'));
19-
mockery.registerAllowables(['cheerio', '../src/js/converter/declarations.js', './utils.js', 'console', 'jsep', 'mensch/lib/parser.js', './debug', './lexer', './domutils.js']);
20-
mockery.enable();
18+
var cheerioMock = require('cheerio').load('<html />');
19+
20+
rewiremock.enable();
21+
rewiremock('jquery').with(cheerioMock);
22+
2123
declarations = require('../src/js/converter/declarations.js');
2224
});
2325

@@ -316,8 +318,7 @@ describe('Style declaration processor', function() {
316318
});
317319

318320
afterAll(function() {
319-
mockery.disable();
320-
mockery.deregisterAll();
321+
rewiremock.disable();
321322
});
322323

323324
});

spec/model-spec.js

Lines changed: 0 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -2,12 +2,6 @@
22
/* globals it:false, describe:false, expect:false */
33

44
describe('Model generator', function() {
5-
var mockery = require('mockery');
6-
7-
beforeAll(function() {
8-
mockery.enable();
9-
mockery.registerAllowables(['console', './domutils.js', './cssparser.js', 'mensch/lib/parser.js', './debug', './lexer', '../src/js/converter/utils.js', 'jsep']);
10-
});
115

126
/*
137
ensurePathAndGetBindValue: modelEnsurePathAndGetBindValue.bind(undefined, false),
@@ -33,9 +27,4 @@ describe('Model generator', function() {
3327

3428
});
3529

36-
afterAll(function() {
37-
mockery.disable();
38-
mockery.deregisterAll();
39-
});
40-
4130
});

0 commit comments

Comments
 (0)