fix(interpolateName): don't interpolated hashType without hash or contenthash (#140)

Author: evilebottnawi

lib/interpolateName.js

@@ -91,7 +91,7 @@ function interpolateName(loaderContext, name, options) {
       // `hash` and `contenthash` are same in `loader-utils` context
       // let's keep `hash` for backward compatibility
       .replace(
-        /\[(?:([^:\]]+):)?(?:hash||contenthash)(?::([a-z]+\d*))?(?::(\d+))?\]/gi,
+        /\[(?:([^:\]]+):)?(?:hash|contenthash)(?::([a-z]+\d*))?(?::(\d+))?\]/gi,
         (all, hashType, digestType, maxLength) =>
           getHashDigest(content, hashType, digestType, parseInt(maxLength, 10))
       )

test/interpolateName.test.js

@@ -124,6 +124,13 @@ describe('interpolateName()', () => {
       'test content',
       'modal.1n8osQznuT8jOAwdzg_n.css',
     ],
+    // Should not interpret without `hash` or `contenthash`
+    [
+      '/lib/components/modal/modal.css',
+      '[name].[md5::base64:20].[ext]',
+      'test content',
+      'modal.[md5::base64:20].css',
+    ],
   ].forEach((test) => {
     it('should interpolate ' + test[0] + ' ' + test[1], () => {
       const interpolatedName = loaderUtils.interpolateName(