Skip to content

Commit c1ea5aa

Browse files
serutserut
committed
lint fix
1 parent e5efb3c commit c1ea5aa

File tree

11 files changed

+101
-100
lines changed

11 files changed

+101
-100
lines changed

‎.eslintrc.json

+2-1
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,7 @@
1010
"sourceType": "module"
1111
},
1212
"rules": {
13+
"no-useless-escape": [0],
1314
"consistent-return": [0],
1415
"eqeqeq": [2, "smart"],
1516
"import/no-extraneous-dependencies": [0],
@@ -48,7 +49,7 @@
4849
"block-scoped-var": 2,
4950
"curly": [2, "multi-line"],
5051
"default-case": 2,
51-
"guard-for-in": 2,
52+
"guard-for-in": 0,
5253
"no-alert": 2,
5354
"no-caller": 2,
5455
"no-case-declarations": 2,

‎client/methods.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,7 @@ Meteor.sendCoverage = function (callback) {
4040
return callback(Meteor.getStats());
4141
}
4242

43-
// Send each property alone
43+
// Send each property alone
4444
for (var property in globalCoverage) {
4545
/* istanbul ignore else */
4646
if (globalCoverage.hasOwnProperty(property)) {

‎meteor-packages-coverage/server/handlers.js

+4-4
Original file line numberDiff line numberDiff line change
@@ -6,16 +6,16 @@ instrumentClientJs = function (params, req, res, next) {
66
if (Instrumenter.shallInstrumentClientScript(fileurl)) {
77
var path,
88
pathLabel;
9-
// Either a package
9+
// Either a package
1010
if (req.url.indexOf('/packages') === 0) {
1111
path = '../web.browser';
1212
pathLabel = path + fileurl;
1313
} else if (req.url.indexOf('/app') === 0) {
14-
// Or the app/app.js
14+
// Or the app/app.js
1515
path = '../web.browser';
1616
pathLabel = path + fileurl;
1717
} else {
18-
// Or a public file
18+
// Or a public file
1919
path = '../web.browser/app';
2020
pathLabel = path + fileurl;
2121
}
@@ -39,5 +39,5 @@ instrumentClientJs = function (params, req, res, next) {
3939
};
4040

4141
export default Handlers = {
42-
instrumentClientJs,
42+
instrumentClientJs
4343
};

‎meteor-packages-coverage/server/router.js

+12-12
Original file line numberDiff line numberDiff line change
@@ -17,19 +17,19 @@ const handleRequest = (method) => (path, cb) => {
1717
});
1818

1919
Promise.resolve()
20-
.then(() => new Promise(resolve => {
21-
bodyParser.urlencoded({ extended: false })(req, res, resolve);
22-
}))
23-
.then(() => new Promise(resolve => {
24-
bodyParser.json({ limit: '30mb' }).call(null, req, res, resolve);
25-
}))
26-
.then(() => cb(queryParams, req, res, next))
27-
.catch((e) => {
28-
console.log('Exception undandled:');
29-
console.log(e.stack);
20+
.then(() => new Promise(resolve => {
21+
bodyParser.urlencoded({ extended: false })(req, res, resolve);
22+
}))
23+
.then(() => new Promise(resolve => {
24+
bodyParser.json({ limit: '30mb' }).call(null, req, res, resolve);
25+
}))
26+
.then(() => cb(queryParams, req, res, next))
27+
.catch((e) => {
28+
console.log('Exception undandled:');
29+
console.log(e.stack);
3030

31-
next();
32-
});
31+
next();
32+
});
3333
});
3434
};
3535

‎package.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,7 @@ Package.onUse(function (api) {
4242

4343
Package.onTest(function (api) {
4444
api.use('ecmascript');
45-
api.use('lmieulet:[email protected]', 'server')
45+
api.use('lmieulet:[email protected]', 'server');
4646
api.use('http', 'client');
4747
api.use('webapp', 'server');
4848
api.use(['lmieulet:meteor-coverage']);

‎server/report/report-common.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ const Report = Npm.require('istanbul-lib-report');
66

77

88
export default ReportCommon = {
9-
/**
9+
/**
1010
* Alter fs to add a new method writer
1111
* Used by the istanbul-reports library
1212
* @param filepath

‎server/report/report-http.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -76,7 +76,7 @@ export default class {
7676
return context;
7777
}
7878

79-
// istanbul-reports expect to save HTML report to the file system and not over network
79+
// istanbul-reports expect to save HTML report to the file system and not over network
8080
alterFS(res) {
8181
res.close = function () {
8282
this.end();

‎server/report/report-remap.js

+10-10
Original file line numberDiff line numberDiff line change
@@ -44,16 +44,16 @@ export default class {
4444
ReportCommon.checkDirectory(this.remapPath);
4545

4646
let reports = {}, allReports = {
47-
'html': this.remapPath,
48-
'clover': this.getFilePath('clover.xml'),
49-
'cobertura': this.getFilePath('cobertura.xml'),
50-
'teamcity': this.getFilePath('teamcity.log'),
51-
'text-summary': this.getFilePath('summary.txt'),
52-
'text': this.getFilePath('report.txt'),
53-
'lcovonly': this.getFilePath('lcov.info'),
54-
'json-summary': this.getFilePath('summary.json'),
55-
'json': this.getFilePath('report.json')
56-
};
47+
'html': this.remapPath,
48+
'clover': this.getFilePath('clover.xml'),
49+
'cobertura': this.getFilePath('cobertura.xml'),
50+
'teamcity': this.getFilePath('teamcity.log'),
51+
'text-summary': this.getFilePath('summary.txt'),
52+
'text': this.getFilePath('report.txt'),
53+
'lcovonly': this.getFilePath('lcov.info'),
54+
'json-summary': this.getFilePath('summary.json'),
55+
'json': this.getFilePath('report.json')
56+
};
5757
Conf.remapFormat.forEach((type) => reports[type] = allReports[type]);
5858
this.remapWrapper(this.pathJSON, reports, this.options).await();
5959
this.res.end('{"type":"success"}');

‎server/report/report-service.js

+53-53
Original file line numberDiff line numberDiff line change
@@ -23,32 +23,32 @@ export default class {
2323
try {
2424
switch (type) {
2525
case 'remap':
26-
{
27-
let reportRemap = new ReportRemap(res, type, options);
28-
reportRemap.generate();
29-
break;
30-
}
26+
{
27+
let reportRemap = new ReportRemap(res, type, options);
28+
reportRemap.generate();
29+
break;
30+
}
3131
case 'lcovonly':
32-
{
33-
options = this.addFileToOptions(options, 'lcov.info');
34-
let istanbulFile1 = new IstanbulGenericReporter(res, type, options);
35-
istanbulFile1.generate();
36-
break;
37-
}
32+
{
33+
options = this.addFileToOptions(options, 'lcov.info');
34+
let istanbulFile1 = new IstanbulGenericReporter(res, type, options);
35+
istanbulFile1.generate();
36+
break;
37+
}
3838
case 'json':
39-
{
40-
options = this.addFileToOptions(options, 'summary.json');
41-
let istanbulFile2 = new IstanbulGenericReporter(res, type, options);
42-
istanbulFile2.generate();
43-
break;
44-
}
39+
{
40+
options = this.addFileToOptions(options, 'summary.json');
41+
let istanbulFile2 = new IstanbulGenericReporter(res, type, options);
42+
istanbulFile2.generate();
43+
break;
44+
}
4545
case 'coverage':
46-
{
47-
options = this.addFileToOptions(options, 'report.json');
48-
let reportCoverage = new ReportCoverage(res, options);
49-
reportCoverage.generate();
50-
break;
51-
}
46+
{
47+
options = this.addFileToOptions(options, 'report.json');
48+
let reportCoverage = new ReportCoverage(res, options);
49+
reportCoverage.generate();
50+
break;
51+
}
5252
/*case 'teamcity':
5353
{
5454
options = this.addFileToOptions(options, 'teamcity.log');
@@ -57,40 +57,40 @@ export default class {
5757
break;
5858
}*/
5959
case 'json-summary':
60-
{
61-
options = this.addFileToOptions(options, 'json_summary.json');
62-
let jsonSummary = new JsonSummary(res, type, options);
63-
jsonSummary.generate();
64-
break;
65-
}
60+
{
61+
options = this.addFileToOptions(options, 'json_summary.json');
62+
let jsonSummary = new JsonSummary(res, type, options);
63+
jsonSummary.generate();
64+
break;
65+
}
6666
case 'html':
67-
{
68-
options = Object.assign({}, {
69-
'prefix': '/coverage/'
70-
}, options);
71-
let html = new Html(res, options);
72-
html.generate();
73-
break;
74-
}
67+
{
68+
options = Object.assign({}, {
69+
'prefix': '/coverage/'
70+
}, options);
71+
let html = new Html(res, options);
72+
html.generate();
73+
break;
74+
}
7575
case 'text-summary':
76-
{
77-
options = this.addFileToOptions(options, 'summary.txt');
78-
let textSummary = new TextSummary(res, type, options);
79-
textSummary.generate();
80-
break;
81-
}
76+
{
77+
options = this.addFileToOptions(options, 'summary.txt');
78+
let textSummary = new TextSummary(res, type, options);
79+
textSummary.generate();
80+
break;
81+
}
8282
case 'http':
83-
{
84-
let http = new Http(res, options);
85-
http.generate();
86-
break;
87-
}
83+
{
84+
let http = new Http(res, options);
85+
http.generate();
86+
break;
87+
}
8888
default:
89-
{
90-
Log.error('Failed to export - this type is not implemented yet');
91-
res.writeHead(400);
92-
res.end('{"type":"This type [' + type + '] is not supported"}');
93-
}
89+
{
90+
Log.error('Failed to export - this type is not implemented yet');
91+
res.writeHead(400);
92+
res.end('{"type":"This type [' + type + '] is not supported"}');
93+
}
9494
}
9595
} catch (e) {
9696
Log.error('ReportService failed while creating report type [', type, ']');

‎server/router.js

+13-13
Original file line numberDiff line numberDiff line change
@@ -17,19 +17,19 @@ const handleRequest = (method) => (path, cb) => {
1717
});
1818

1919
Promise.resolve()
20-
.then(() => new Promise(resolve => {
21-
bodyParser.urlencoded({ extended: false })(req, res, resolve);
22-
}))
23-
.then(() => new Promise(resolve => {
24-
bodyParser.json({ limit: '30mb' }).call(null, req, res, resolve);
25-
}))
26-
.then(() => cb(queryParams, req, res, next))
27-
.catch((e) => {
28-
console.log('Exception undandled:');
29-
console.log(e.stack);
30-
31-
next();
32-
});
20+
.then(() => new Promise(resolve => {
21+
bodyParser.urlencoded({ extended: false })(req, res, resolve);
22+
}))
23+
.then(() => new Promise(resolve => {
24+
bodyParser.json({ limit: '30mb' }).call(null, req, res, resolve);
25+
}))
26+
.then(() => cb(queryParams, req, res, next))
27+
.catch((e) => {
28+
console.log('Exception undandled:');
29+
console.log(e.stack);
30+
31+
next();
32+
});
3333
});
3434
};
3535

‎server/services/coverage-data.js

+3-3
Original file line numberDiff line numberDiff line change
@@ -9,11 +9,11 @@ export default CoverageData = {
99
getReport: function (coverage) {
1010
// QUICKFIX FOR PACKAGES TESTS
1111
if (Meteor.isPackageTest) {
12-
if (Package["lmieulet:meteor-packages-coverage"] && Package["lmieulet:meteor-packages-coverage"].default && Package["lmieulet:meteor-packages-coverage"].default.CoverageData) {
12+
if (Package['lmieulet:meteor-packages-coverage'] && Package['lmieulet:meteor-packages-coverage'].default && Package['lmieulet:meteor-packages-coverage'].default.CoverageData) {
1313
// Ask for lmieulet:meteor-packages-coverage coverage report, as we are in package test
14-
return Package["lmieulet:meteor-packages-coverage"].default.CoverageData.getReport(coverage);
14+
return Package['lmieulet:meteor-packages-coverage'].default.CoverageData.getReport(coverage);
1515
}
16-
throw new Error("lmieulet:meteor-packages-coverage not found. Just add this server dependency in Package.onTest in your package.js")
16+
throw new Error('lmieulet:meteor-packages-coverage not found. Just add this server dependency in Package.onTest in your package.js');
1717
}
1818
// Used for meteor apps using babel
1919
return Coverage.createCoverageMap(coverage);

0 commit comments

Comments
 (0)