From 862ea7d1d0226558f2750bec36da02492d1e516d Mon Sep 17 00:00:00 2001 From: evilebottnawi Date: Wed, 20 Oct 2021 16:31:16 +0300 Subject: [PATCH] fix: allow `interpolateName` works without options --- lib/interpolateName.js | 2 +- test/interpolateName.test.js | 9 ++++----- 2 files changed, 5 insertions(+), 6 deletions(-) diff --git a/lib/interpolateName.js b/lib/interpolateName.js index 6295de6..71999c2 100644 --- a/lib/interpolateName.js +++ b/lib/interpolateName.js @@ -3,7 +3,7 @@ const path = require("path"); const getHashDigest = require("./getHashDigest"); -function interpolateName(loaderContext, name, options) { +function interpolateName(loaderContext, name, options = {}) { let filename; const hasQuery = diff --git a/test/interpolateName.test.js b/test/interpolateName.test.js index 3e0bbe8..2e64164 100644 --- a/test/interpolateName.test.js +++ b/test/interpolateName.test.js @@ -263,12 +263,11 @@ describe("interpolateName()", () => { ], ]); - it("should return the same emoji for the same string", () => { - const args = [{}, "[emoji:5]", { content: "same_emoji" }]; - const result1 = loaderUtils.interpolateName.apply(loaderUtils, args); - const result2 = loaderUtils.interpolateName.apply(loaderUtils, args); + it("should work without options", () => { + const args = [{}, "foo/bar/[hash]"]; + const result = loaderUtils.interpolateName.apply(loaderUtils, args); - expect(result1).toBe(result2); + expect(result).toBe("foo/bar/[hash]"); }); describe("no loader context", () => {