test: improve coverage (#5638)

* test: improve coverage

* fix

* fix

* chore: skipLibCheck temporarily

* fix

Author: D-Sketon

lib/hexo/router.ts

@@ -8,12 +8,6 @@ interface Data {
   modified: boolean;
 }
 
-declare module 'stream' {
-  export default class _Stream extends Stream {
-    readable: boolean;
-  }
-}
-
 class RouteStream extends Readable {
   public _data: any;
   public _ended: boolean;
@@ -58,7 +52,7 @@ class RouteStream extends Readable {
     this._ended = true;
 
     data().then(data => {
-      if (data instanceof Stream && data.readable) {
+      if (data instanceof Stream && (data as Stream.Readable).readable) {
         data.on('data', d => {
           this.push(d);
         });

test/scripts/console/generate.ts

@@ -12,7 +12,8 @@ type OriginalReturn = ReturnType<typeof generateConsole>;
 describe('generate', () => {
   let hexo: Hexo, generate: (...args: OriginalParams) => OriginalReturn;
 
-  beforeEach(async () => {
+  beforeEach(async function() {
+    this.timeout(5000);
     hexo = new Hexo(join(__dirname, 'generate_test'), {silent: true});
     generate = generateConsole.bind(hexo);
 
@@ -179,7 +180,8 @@ describe('generate', () => {
     result.should.eql(content);
 
     // Stop watching
-    await hexo.unwatch();
+    hexo.unwatch();
+    await BluebirdPromise.delay(300);
   });
 
   it('deploy', async () => {
@@ -204,14 +206,15 @@ describe('generate', () => {
       writeFile(join(hexo.theme_dir, 'source', 'b.txt'), 'b'),
       writeFile(join(hexo.theme_dir, 'source', 'c.njk'), 'c')
     ]);
+    await BluebirdPromise.delay(300);
     await generate();
 
     // Update source file
     await BluebirdPromise.all([
       writeFile(join(hexo.theme_dir, 'source', 'b.txt'), 'bb'),
       writeFile(join(hexo.theme_dir, 'source', 'c.njk'), 'cc')
     ]);
-
+    await BluebirdPromise.delay(300);
     // Generate again
     await generate();
 

test/scripts/helpers/gravatar.ts

@@ -6,7 +6,7 @@ describe('gravatar', () => {
     return crypto.createHash('md5').update(str).digest('hex');
   }
 
-  const gravatar = gravatarHelper as any;
+  const gravatar = gravatarHelper;
 
   const email = 'abc@abc.com';
   const hash = md5(email);

test/scripts/hexo/hexo.ts

@@ -153,6 +153,45 @@ describe('Hexo', () => {
 
   it('load() - theme', async () => await testLoad(join(hexo.theme_dir, 'source')));
 
+
+  it('load() - load database', async () => {
+    hexo._dbLoaded = false;
+    const dbPath = hexo.database.options.path;
+
+    const fixture = {
+      meta: {
+        version: 1,
+        warehouse: require('warehouse').version
+      },
+      models: {
+        PostTag: [
+          { _id: 'cuid111111111111111111113', post_id: 'cuid111111111111111111111', tag_id: 'cuid111111111111111111112' }
+        ],
+        Tag: [
+          { _id: 'cuid111111111111111111112', name: 'foo' }
+        ],
+        Post: [
+          { _id: 'cuid111111111111111111111', source: 'test', slug: 'test' }
+        ]
+      }
+    };
+    await writeFile(dbPath, JSON.stringify(fixture));
+    await hexo.load();
+    // check Model
+    hexo.model('PostTag').toArray({lean: true}).length.should.eql(fixture.models.PostTag.length);
+    hexo.model('Tag').toArray({lean: true}).length.should.eql(fixture.models.Tag.length);
+    hexo.model('Post').toArray({lean: true}).length.should.eql(fixture.models.Post.length);
+    hexo._binaryRelationIndex.post_tag.keyIndex.size.should.eql(1);
+    hexo._binaryRelationIndex.post_tag.valueIndex.size.should.eql(1);
+    await unlink(dbPath);
+    // clean up
+    await hexo.model('PostTag').removeById('cuid111111111111111111113');
+    await hexo.model('Tag').removeById('cuid111111111111111111112');
+    await hexo.model('Post').removeById('cuid111111111111111111111');
+    hexo._binaryRelationIndex.post_tag.keyIndex.clear();
+    hexo._binaryRelationIndex.post_tag.valueIndex.clear();
+  });
+
   // Issue #3964
   it('load() - merge theme config - deep clone', async () => {
     const hexo = new Hexo(__dirname, { silent: true });

test/scripts/hexo/load_plugins.ts

@@ -4,6 +4,7 @@ import Hexo from '../../../lib/hexo';
 import loadPlugins from '../../../lib/hexo/load_plugins';
 import BluebirdPromise from 'bluebird';
 import chai from 'chai';
+import { spy } from 'sinon';
 const should = chai.should();
 
 describe('Load plugins', () => {
@@ -93,6 +94,20 @@ describe('Load plugins', () => {
     });
   });
 
+  it('fail to load plugins', () => {
+    const logSpy = spy();
+    hexo.log.error = logSpy;
+    const name = 'hexo-plugin-test';
+    const path = join(hexo.plugin_dir, name, 'index.js');
+    return BluebirdPromise.all([
+      createPackageFile(name),
+      writeFile(path, 'throw new Error("test")')
+    ]).then(() => loadPlugins(hexo)).then(() => {
+      logSpy.args[0][1].should.contains('Plugin load failed: %s');
+      logSpy.args[0][2].should.contains('hexo-plugin-test');
+    });
+  });
+
   it('load async plugins', () => {
     const name = 'hexo-async-plugin-test';
     const path = join(hexo.plugin_dir, name, 'index.js');
@@ -217,6 +232,20 @@ describe('Load plugins', () => {
     return unlink(path);
   });
 
+  it('fail to load scripts', async () => {
+    const logSpy = spy();
+    hexo.log.error = logSpy;
+    const path = join(hexo.script_dir, 'test.js');
+
+    writeFile(path, 'throw new Error("test")');
+    await loadPlugins(hexo);
+
+    logSpy.args[0][1].should.contains('Script load failed: %s');
+    logSpy.args[0][2].should.contains('test.js');
+    return unlink(path);
+  });
+
+
   it('load theme scripts', () => {
     const path = join(hexo.theme_script_dir, 'test.js');
 

test/scripts/hexo/post.ts

@@ -666,7 +666,7 @@ describe('Post', () => {
     ].join('\n'));
 
     const path = join(hexo.source_dir, '_posts', 'fooo.md');
-    const data = await post.create({
+    await post.create({
       title: 'fooo',
       layout: 'draft',
       tags: customTags,
@@ -1486,6 +1486,16 @@ describe('Post', () => {
     data.content.should.contains('1');
     data.content.should.contains('22222');
 
+    content = '{{ 1 }} \n `{% custom %}` 22222  `{% endcustom }`';
+    data = await post.render('', {
+      content,
+      engine: 'markdown'
+    });
+    data.content.should.contains('1');
+    data.content.should.contains('{% custom %}');
+    data.content.should.contains('22222');
+    data.content.should.contains('{% endcustom }');
+
     // lost two characters
     content = '{{ 1 }} \n `{#` \n 22222';
     data = await post.render('', {

test/scripts/processors/post.ts

@@ -1,6 +1,6 @@
 
 import { join } from 'path';
-import { mkdirs, rmdir, unlink, writeFile } from 'hexo-fs';
+import { exists, mkdirs, rmdir, unlink, writeFile } from 'hexo-fs';
 import BluebirdPromise from 'bluebird';
 import defaultConfig from '../../../lib/hexo/default_config';
 import Hexo from '../../../lib/hexo';
@@ -358,6 +358,49 @@ describe('post', () => {
     ]);
   });
 
+  it('post - type: create - post_asset_folder enabled without asset', async () => {
+    hexo.config.post_asset_folder = true;
+
+    const fooPath = join(hexo.source_dir, '_posts', 'foo');
+    if (await exists(fooPath)) {
+      await rmdir(fooPath);
+    }
+
+    const body = [
+      'title: "Hello world"',
+      'date: 2006-01-02 15:04:05',
+      'updated: 2014-12-13 01:02:03',
+      '---',
+      'The quick brown fox jumps over the lazy dog'
+    ].join('\n');
+
+    const file = newFile({
+      path: 'foo.html',
+      published: true,
+      type: 'create',
+      renderable: true
+    });
+
+    await writeFile(file.source, body);
+    await process(file);
+    const post = Post.findOne({ source: file.path });
+
+    post.title.should.eql('Hello world');
+    post.date.format(dateFormat).should.eql('2006-01-02 15:04:05');
+    post.updated.format(dateFormat).should.eql('2014-12-13 01:02:03');
+    post._content.should.eql('The quick brown fox jumps over the lazy dog');
+    post.source.should.eql(file.path);
+    post.raw.should.eql(body);
+    post.slug.should.eql('foo');
+    post.published.should.be.true;
+
+    hexo.config.post_asset_folder = false;
+    return BluebirdPromise.all([
+      post.remove(),
+      unlink(file.source)
+    ]);
+  });
+
   it('post - type: update', async () => {
     const body = [
       'title: "New world"',

tsconfig.json

@@ -5,6 +5,7 @@
     "sourceMap": true,
     "outDir": "dist",
     "declaration": true,
+    "skipLibCheck": true,
     "esModuleInterop": true,
     "types": [
       "node",