fix(#135): filter was executed before it was loaded (#175)

* fix(#135): filter was executed before it was loaded

* test: unit test enchancement

index.js

@@ -34,7 +34,7 @@ const renderer = new Renderer(hexo);
 renderer.disableNunjucks = Boolean(hexo.config.markdown.disableNunjucks);
 
 function render(data, options) {
-  return renderer.parser.render(data.text);
+  return renderer.render(data, options);
 }
 
 hexo.extend.renderer.register('md', 'html', render, true);

lib/renderer.js

@@ -10,6 +10,7 @@ class Renderer {
    * @param {*} hexo context of hexo
    */
   constructor(hexo) {
+    this.hexo = hexo;
 
     let { markdown } = hexo.config;
 
@@ -44,7 +45,11 @@ class Renderer {
     if (anchors) {
       this.parser.use(require('./anchors'), anchors);
     }
-    hexo.execFilterSync('markdown-it:renderer', this.parser, { context: this });
+  }
+
+  render(data, options) {
+    this.hexo.execFilterSync('markdown-it:renderer', this.parser, { context: this });
+    return this.parser.render(data.text);
   }
 }
 

test/index.js

@@ -256,21 +256,57 @@ describe('Hexo Renderer Markdown-it', () => {
     it('default', () => {
 
       const renderer = new Renderer(hexo);
-      const result = renderer.parser.render('[foo](javascript:bar)');
+      const result = renderer.render({ text: '[foo](javascript:bar)' });
 
       result.should.equal('<p>[foo](javascript:bar)</p>\n');
     });
 
     it('enable unsafe link', () => {
-      hexo.extend.filter.register('markdown-it:renderer', md => {
+      const filter = md => {
         md.validateLink = function() { return true; };
-      });
+      };
+      hexo.extend.filter.register('markdown-it:renderer', filter);
 
       const renderer = new Renderer(hexo);
-      const result = renderer.parser.render('[foo](javascript:bar)');
+      const result = renderer.render({ text: '[foo](javascript:bar)' });
+
+      hexo.extend.filter.unregister('markdown-it:renderer', filter);
+
+      result.should.equal('<p><a href="javascript:bar">foo</a></p>\n');
+    });
+
+    it('should execute loaded scripts', async () => {
+      const renderer = new Renderer(hexo);
+
+      hexo.env.init = true;
+      await hexo.init();
+
+      const result = renderer.render({ text: '[foo](javascript:bar)' });
 
       result.should.equal('<p><a href="javascript:bar">foo</a></p>\n');
     });
+
+    it('should be called in render', () => {
+      const iterates = 3;
+      const spy = {
+        called: 0,
+        call() {
+          this.called++;
+        }
+      };
+
+      const filter = md => spy.call(md);
+      hexo.extend.filter.register('markdown-it:renderer', filter);
+
+      const renderer = new Renderer(hexo);
+      for (let i = 0; i < iterates; i++) {
+        renderer.render({ text: '' });
+      }
+
+      hexo.extend.filter.unregister('markdown-it:renderer', filter);
+
+      spy.called.should.equal(iterates);
+    });
   });
 
   describe('nunjucks', () => {

test/scripts/enable_unsafe_link.js

@@ -0,0 +1,6 @@
+'use strict';
+
+// eslint-disable-next-line no-undef
+hexo.extend.filter.register('markdown-it:renderer', md => {
+  md.validateLink = function() { return true; };
+});