fix: eslint

Author: atian25

app/controller/rss.js

@@ -5,6 +5,8 @@ const Controller = require('egg').Controller;
 const convert = require('data2xml')();
 
 function utf8ForXml(inputStr) {
+  // FIXME: no-control-regex
+  /* eslint-disable no-control-regex */
   return inputStr.replace(/[^\x09\x0A\x0D\x20-\xFF\x85\xA0-\uD7FF\uE000-\uFDCF\uFDE0-\uFFFD]/gm, '');
 }
 

app/controller/topic.js

@@ -1,6 +1,7 @@
 'use strict';
 
 const Controller = require('egg').Controller;
+const _ = require('lodash');
 
 class TopicController extends Controller {
 

app/extend/helper.js

@@ -1,15 +1,5 @@
-/* !
- * nodeclub - common/render_helpers.js
- * Copyright(c) 2013 fengmk2 <[email protected]>
- * MIT Licensed
- */
-
 'use strict';
 
-/**
- * Module dependencies.
- */
-
 const MarkdownIt = require('markdown-it');
 const validator = require('validator');
 const jsxss = require('xss');
@@ -54,7 +44,7 @@ md.renderer.rules.code_block = (tokens, idx /* , options */) => {
 };
 
 const myxss = new jsxss.FilterXSS({
-  onIgnoreTagAttr(tag, name, value, isWhiteAttr) {
+  onIgnoreTagAttr(tag, name, value) {
     // 让 prettyprint 可以工作
     if (tag === 'pre' && name === 'class') {
       return name + '="' + jsxss.escapeAttrValue(value) + '"';

app/router.js

@@ -4,11 +4,9 @@
  * @param {Egg.Application} app - egg application
  */
 module.exports = app => {
-  const { router, controller, config } = app;
+  const { router, controller } = app;
 
-  const {
-    site, sign, user, auth, topic,
-    reply, rss, search, page } = controller;
+  const { site, sign, user, topic, rss, search, page } = controller;
 
   // home page
   router.get('/', site.index);

package.json

@@ -16,6 +16,7 @@
     "egg-view-ejs": "^2.0.0",
     "loader": "^2.1.1",
     "loader-koa": "^2.0.0",
+    "lodash": "^4.17.5",
     "markdown-it": "^8.4.1",
     "uuid": "^3.2.1",
     "validator": "^9.4.1",

test/app/controller/site.test.js

@@ -1,13 +1,8 @@
 'use strict';
 
-const { app, assert } = require('egg-mock/bootstrap');
+const { app } = require('egg-mock/bootstrap');
 
 describe('test/app/controller/site.test.js', () => {
-  // it('should assert', async function () {
-  //   const pkg = require('../../../package.json');
-  //   assert(app.config.keys.startsWith(pkg.name));
-  // });
-
   it('should GET /sitemap.xml', async () => {
     await app.httpRequest()
       .get('/sitemap.xml')

test/app/controller/topic.test.js

@@ -1,6 +1,6 @@
 'use strict';
 
-const { app, assert } = require('egg-mock/bootstrap');
+const { app } = require('egg-mock/bootstrap');
 
 describe('test/app/controller/topic.test.js', () => {
   it('should GET /topic/:tid ok', async () => {