fix: remove duplicated default path

hexojs · Sep 16, 2019 · bfd8b0a · bfd8b0a
1 parent 67ef653
commit bfd8b0a
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 8 deletions.
diff --git a/lib/plugins/helper/full_url_for.js b/lib/plugins/helper/full_url_for.js
@@ -1,6 +1,6 @@
 'use strict';
 const { full_url_for } = require('hexo-util');
 
-module.exports = function(path = '/') {
+module.exports = function(path) {
   return full_url_for.call(this, path);
 };
diff --git a/test/scripts/helpers/full_url_for.js b/test/scripts/helpers/full_url_for.js
@@ -2,18 +2,16 @@
 
 describe('full_url_for', () => {
   const ctx = {
-    config: {}
+    config: { url: 'https://example.com' }
   };
 
   const fullUrlFor = require('../../../lib/plugins/helper/full_url_for').bind(ctx);
 
-  it('internal url', () => {
-    ctx.config.root = '/';
-    fullUrlFor('index.html').should.eql(ctx.config.url + '/index.html');
-    fullUrlFor('/').should.eql(ctx.config.url + '/');
-    fullUrlFor('/index.html').should.eql(ctx.config.url + '/index.html');
+  it('no path input', () => {
+    fullUrlFor().should.eql(ctx.config.url + '/');
+  });
 
-    ctx.config.root = '/blog/';
+  it('internal url', () => {
     fullUrlFor('index.html').should.eql(ctx.config.url + '/index.html');
     fullUrlFor('/').should.eql(ctx.config.url + '/');
     fullUrlFor('/index.html').should.eql(ctx.config.url + '/index.html');