fixed Windows issues

Author: Christoph Lühr

src/update-ts-references.js

@@ -183,16 +183,11 @@ const getReferencesFromDependencies = (
         .sort((refA, refB) => (refA.path > refB.path ? 1 : -1));
 };
 
-const ensureLeadingDotSlash=(p)=> {
-    if (!p || path.isAbsolute(p) || p.startsWith('./') || p.startsWith('../')) {
+const ensureLeadingDotSlash = (p) => {
+    if (p == null || path.isAbsolute(p) || p.startsWith('.' + path.sep) || p.startsWith('..' + path.sep)) {
         return p;
     }
-
-    if (p.length === 0) {
-        return '.'
-    }
-
-    return './' + p;
+    return p.length === 0 ? '.' : '.' + path.sep + p;
 }
 
 const ensurePosixPathStyle = (reference) => ({
@@ -454,4 +449,4 @@ const execute = async ({
     return changesCount;
 };
 
-module.exports = { execute, defaultOptions };
+module.exports = { execute, defaultOptions, ensureLeadingDotSlash };

tests/ensureLeadingDotSlash.test.js

@@ -0,0 +1,37 @@
+const path = require('path');
+const { ensureLeadingDotSlash } = require('../src/update-ts-references');
+
+test('returns null as-is', () => {
+    expect(ensureLeadingDotSlash(null)).toBe(null);
+});
+
+test('returns undefined as-is', () => {
+    expect(ensureLeadingDotSlash(undefined)).toBe(undefined);
+});
+
+test('returns . for empty string', () => {
+    expect(ensureLeadingDotSlash('')).toBe('.');
+});
+
+test('returns absolute path as-is', () => {
+    const p = path.resolve('/some/absolute/path');
+    expect(ensureLeadingDotSlash(p)).toBe(p);
+});
+
+test('returns path already starting with .' + path.sep + ' as-is', () => {
+    const p = '.' + path.sep + 'foo';
+    expect(ensureLeadingDotSlash(p)).toBe(p);
+});
+
+test('returns path already starting with ..' + path.sep + ' as-is', () => {
+    const p = '..' + path.sep + 'foo';
+    expect(ensureLeadingDotSlash(p)).toBe(p);
+});
+
+test('prepends .' + path.sep + ' to a plain relative path', () => {
+    expect(ensureLeadingDotSlash('foo')).toBe('.' + path.sep + 'foo');
+});
+
+test('prepends .' + path.sep + ' to a nested relative path', () => {
+    expect(ensureLeadingDotSlash('foo' + path.sep + 'bar')).toBe('.' + path.sep + 'foo' + path.sep + 'bar');
+});