chore: update nodejs typings to 8

Author: DimitarTachev

lib/common/file-system.ts

@@ -304,7 +304,7 @@ export class FileSystem implements IFileSystem {
 		}
 	}
 
-	public symlink(sourcePath: string, destinationPath: string, type?: string): void {
+	public symlink(sourcePath: string, destinationPath: string, type?: fs.symlink.Type): void {
 		fs.symlinkSync(sourcePath, destinationPath, type);
 	}
 

lib/common/helpers.ts

@@ -432,7 +432,7 @@ export function isPromise(candidateFuture: any): boolean {
 	return !!(candidateFuture && typeof (candidateFuture.then) === "function");
 }
 
-export async function attachAwaitDetach(eventName: string, eventEmitter: EventEmitter, eventHandler: Function, operation: Promise<any>) {
+export async function attachAwaitDetach(eventName: string, eventEmitter: EventEmitter, eventHandler: (...args: any[]) => void, operation: Promise<any>) {
 	eventEmitter.on(eventName, eventHandler);
 
 	try {

lib/common/mobile/ios/device/ios-device.ts

@@ -10,7 +10,7 @@ export class IOSDevice implements Mobile.IiOSDevice {
 	public deviceInfo: Mobile.IDeviceInfo;
 
 	private _socket: net.Socket;
-	private _deviceLogHandler: Function;
+	private _deviceLogHandler: (...args: any[]) => void;
 
 	constructor(private deviceActionInfo: IOSDeviceLib.IDeviceActionInfo,
 		private $injector: IInjector,

lib/common/prompter.ts

@@ -159,7 +159,7 @@ export class Prompter implements IPrompter {
 	private cleanListener(stream: NodeJS.WritableStream, eventName: string, listenerName: string): void {
 		const eventListeners: any[] = process.stdout.listeners(eventName);
 
-		const listenerFunction: Function = _.find(eventListeners, (func: any) => func.name === listenerName);
+		const listenerFunction: (...args: any[]) => void = _.find(eventListeners, (func: any) => func.name === listenerName);
 
 		if (listenerFunction) {
 			stream.removeListener(eventName, listenerFunction);

lib/common/services/process-service.ts

@@ -14,7 +14,7 @@ export class ProcessService implements IProcessService {
 		const callbackToString = callback.toString();
 
 		if (this._listeners.length === 0) {
-			_.each(ProcessService.PROCESS_EXIT_SIGNALS, (signal: string) => {
+			_.each(ProcessService.PROCESS_EXIT_SIGNALS, (signal: NodeJS.Signals) => {
 				process.on(signal, () => this.executeAllCallbacks.apply(this));
 			});
 		}

lib/common/test/unit-tests/process-service.ts

@@ -25,7 +25,7 @@ describe("Process service", () => {
 		$processService.attachToProcessExitSignals({}, emptyFunction);
 		$processService.attachToProcessExitSignals({}, emptyFunction);
 
-		_.each(processExitSignals, (signal: string) => {
+		_.each(processExitSignals, (signal: NodeJS.Signals) => {
 			// We need to search only for our listener because each exit signal have different listeners added to it.
 			const actualListeners = _.filter(process.listeners(signal), (listener: Function) => listener.toString().indexOf("executeAllCallbacks") >= 0);
 			assert.deepEqual(actualListeners.length, 1);

lib/services/analytics/analytics-service.ts

@@ -192,7 +192,7 @@ export class AnalyticsService extends AnalyticsServiceBase {
 		return new Promise<void>((resolve, reject) => {
 			if (broker && broker.connected) {
 				try {
-					broker.send(message, resolve);
+					broker.send(message, (error: Error) => resolve());
 				} catch (err) {
 					this.$logger.trace("Error while trying to send message to broker:", err);
 					resolve();

lib/services/debug-service.ts

@@ -111,7 +111,7 @@ export class DebugService extends EventEmitter implements IDebugService {
 
 		if (fullUrl) {
 			const parseQueryString = true;
-			const wsQueryParam = parse(fullUrl, parseQueryString).query.ws;
+			const wsQueryParam = <string>parse(fullUrl, parseQueryString).query.ws;
 			const hostPortSplit = wsQueryParam && wsQueryParam.split(":");
 			debugInfo.port = hostPortSplit && +hostPortSplit[1];
 		}

npm-shrinkwrap.json

@@ -95,7 +95,7 @@
     "@types/color": "3.0.0",
     "@types/lockfile": "1.0.0",
     "@types/lodash": "4.14.116",
-    "@types/node": "6.0.61",
+    "@types/node": "8.10.30",
     "@types/ora": "1.3.3",
     "@types/qr-image": "3.2.0",
     "@types/request": "0.0.45",

package.json

@@ -68,8 +68,8 @@ function makeInteractive() {
 }
 
 function makeNonInteractive() {
-	process.stdout.isTTY = false;
-	process.stdin.isTTY = false;
+	(<any>process.stdout).isTTY = false;
+	(<any>process.stdin).isTTY = false;
 }
 
 function reset() {

test/helpers/preview-command-helper.ts

@@ -18,7 +18,7 @@ const protocolVersion = "0.2.0";
 class TestSocket extends LiveSyncSocket {
 	public accomulatedData: Buffer[] = [];
 	public connect () {
-		//ignore
+		return this;
 	}
 
 	public writeAsync(data: Buffer | string, cb?: string | Function, encoding?: Function | string): Promise<Boolean> {

test/services/livesync/android-livesync-tool.ts

@@ -132,7 +132,7 @@ describe("platformEnvironmentRequirements ", () => {
 			assert.deepEqual(['Sync to Playground', 'Configure for Local Builds', 'Skip Step and Configure Manually'], promptForChoiceData[0].choices);
 		});
 		it("should skip env check when NS_SKIP_ENV_CHECK environment variable is passed", async() => {
-			process.env.NS_SKIP_ENV_CHECK = true;
+			(<any>process.env).NS_SKIP_ENV_CHECK = true;
 
 			const output = await platformEnvironmentRequirements.checkEnvironmentRequirements({ platform });
 
@@ -221,8 +221,8 @@ describe("platformEnvironmentRequirements ", () => {
 
 		describe("when console is non interactive", () => {
 			beforeEach(() => {
-				process.stdout.isTTY = false;
-				process.stdin.isTTY = false;
+				(<any>process).stdout.isTTY = false;
+				(<any>process.stdin).isTTY = false;
 				mockDoctorService({ canExecuteLocalBuild: false });
 			});