fix(pluginutils): fix path normalisation for createFilter

packages/pluginutils/src/createFilter.ts

@@ -9,7 +9,7 @@ import normalizePath from './normalizePath';
 
 function getMatcherString(id: string, resolutionBase: string | false | null | undefined) {
   if (resolutionBase === false || isAbsolute(id) || id.startsWith('*')) {
-    return id;
+    return normalizePath(id);
   }
 
   // resolve('') is valid and will default to process.cwd()
@@ -20,7 +20,7 @@ function getMatcherString(id: string, resolutionBase: string | false | null | un
   // 1. the basePath has been normalized to use /
   // 2. the incoming glob (id) matcher, also uses /
   // otherwise Node will force backslash (\) on windows
-  return posix.join(basePath, id);
+  return posix.join(basePath, normalizePath(id));
 }
 
 const createFilter: CreateFilter = function createFilter(include?, exclude?, options?) {

packages/pluginutils/test/createFilter.ts

@@ -134,6 +134,34 @@ test('does not add current working directory when pattern is an absolute path',
   t.falsy(filter(resolve('..', 'c')));
 });
 
+test('normalizes path when pattern is an absolute path', (t) => {
+  const filterPosix = createFilter([`${resolve('.')}/*`]);
+  const filterWin = createFilter([`${resolve('.')}\\*`]);
+
+  t.truthy(filterPosix(resolve('a')));
+  t.truthy(filterWin(resolve('a')));
+});
+
+test('normalizes path when pattern starts with *', (t) => {
+  const filterPosix = createFilter([`**/a`]);
+  const filterWin = createFilter([`**\\a`]);
+
+  t.truthy(filterPosix(resolve('a')));
+  t.truthy(filterWin(resolve('a')));
+});
+
+test('normalizes path when pattern has resolution base', (t) => {
+  const filterPosix = createFilter([`test/*`], [], {
+    resolve: __dirname
+  });
+  const filterWin = createFilter([`test\\*`], [], {
+    resolve: __dirname
+  });
+
+  t.truthy(filterPosix(resolve('test/a')));
+  t.truthy(filterWin(resolve('test/a')));
+});
+
 test('does not add current working directory when pattern starts with a glob', (t) => {
   const filter = createFilter(['**/*']);
   t.truthy(filter(resolve('a')));