feat: windows support (#36)

Supporting the correct slashings in the pathes.

Author: cspotcode

.appveyor.yml

@@ -0,0 +1,21 @@
+# Test against the latest version of this Node.js version
+environment:
+  nodejs_version: "8"
+
+# Install scripts. (runs after repo cloning)
+install:
+  # Get the latest stable version of Node.js or io.js
+  - ps: Install-Product node $env:nodejs_version
+  # install modules
+  - npm install
+
+# Post-install test scripts.
+test_script:
+  # Output useful info for debugging.
+  - node --version
+  - npm --version
+  # run tests
+  - npm test
+
+# Don't actually build.
+build: off

src/DeclarationIndex.ts

@@ -12,7 +12,7 @@ import { Namespace } from './resources/Namespace';
 import { Resource } from './resources/Resource';
 import { isExportableDeclaration } from './type-guards/TypescriptHeroGuards';
 import { TypescriptParser } from './TypescriptParser';
-import { normalizeFilename, normalizePathUri } from './utilities/PathHelpers';
+import { normalizeFilename, normalizePathUri, toPosix } from './utilities/PathHelpers';
 
 /**
  * Returns the name of the node folder. Is used as the library name for indexing.
@@ -308,7 +308,7 @@ export class DeclarationIndex {
                 break;
             }
             if (ex instanceof AllExport || ex instanceof NamedExport) {
-                const exported = '/' + relative(this.rootPath, normalize(join(resource.parsedPath.dir, ex.from)));
+                const exported = '/' + toPosix(relative(this.rootPath, normalize(join(resource.parsedPath.dir, ex.from))));
                 exportsResource = exported === resourcePath;
             }
         }

src/utilities/PathHelpers.ts

@@ -25,5 +25,16 @@ export function normalizePathUri(uri: string): string {
  * @returns {string} 
  */
 export function normalizeFilename(filepath: string): string {
-    return filepath.replace(/([.]d)?[.](t|j)sx?$/g, '');
+    return toPosix(filepath.replace(/([.]d)?[.](t|j)sx?$/g, ''));
+}
+
+/**
+ * On Windows, replaces all backslash delimeters with forward slashes.
+ * On other OSes, returns the path unmodified.
+ */
+export function toPosix(path: string): string {
+    if (platform() === 'win32') {
+        return path.replace(/\\/g, '/');
+    }
+    return path;
 }

test/declaration-index/DeclarationIndex.spec.ts

@@ -1,5 +1,5 @@
 import mockFs = require('mock-fs');
-import { join, resolve } from 'path';
+import { join, resolve } from '../testUtilities';
 
 import { DeclarationIndex } from '../../src/DeclarationIndex';
 import { ClassDeclaration } from '../../src/declarations';

test/declaration-index/specific-cases/body-parser/DeclarationIndex.body-parser.spec.ts

@@ -1,4 +1,4 @@
-import { join, resolve } from 'path';
+import { join, resolve } from '../../../testUtilities';
 
 import { DeclarationIndex } from '../../../../src/DeclarationIndex';
 import { TypescriptParser } from '../../../../src/TypescriptParser';

test/declaration-index/specific-cases/reindex-with-global-module/DeclarationIndex.reindex-with-global-module-export.spec.ts

@@ -1,5 +1,5 @@
 import { FileChanges } from '../../../../src';
-import { join, resolve } from 'path';
+import { join, resolve } from '../../../testUtilities';
 
 import { DeclarationIndex } from '../../../../src/DeclarationIndex';
 import { TypescriptParser } from '../../../../src/TypescriptParser';

test/testUtilities.ts

@@ -1,7 +1,18 @@
-import { resolve } from 'path';
+import * as Path from 'path';
+import { toPosix } from '../src/utilities/PathHelpers';
 
 export function getWorkspaceFile(pathFromWorkspace: string): string {
     return resolve(__dirname, '_workspace', pathFromWorkspace);
 }
 
+// Like path.resolve, but always replaces backslashes with forward slashes on Windows.
+export function resolve(...paths: string[]): string {
+    return toPosix(Path.resolve(...paths));
+}
+
+// Like path.join, but always replaces backslashes with forward slashes on Windows.
+export function join(...paths: string[]): string {
+    return toPosix(Path.join(...paths));
+}
+
 export const rootPath = resolve(__dirname, '_workspace');