Merge pull request #5238 from NativeScript/vladimirov/temp-service

fix: temporary files created by CLI are not deleted in some cases

Author: rosen-vladimirov

lib/bootstrap.ts

@@ -236,3 +236,4 @@ $injector.require("ipService", "./services/ip-service");
 $injector.require("jsonFileSettingsService", "./common/services/json-file-settings-service");
 $injector.require("markingModeService", "./services/marking-mode-service");
 $injector.require("metadataFilteringService", "./services/metadata-filtering-service");
+$injector.require("tempService", "./services/temp-service");

lib/commands/plugin/build-plugin.ts

@@ -1,7 +1,7 @@
 import { EOL } from "os";
 import * as path from "path";
 import * as constants from "../../constants";
-import * as temp from "temp";
+
 export class BuildPluginCommand implements ICommand {
 	public allowedParameters: ICommandParameter[] = [];
 	public pluginProjectPath: string;
@@ -10,7 +10,8 @@ export class BuildPluginCommand implements ICommand {
 		private $errors: IErrors,
 		private $logger: ILogger,
 		private $fs: IFileSystem,
-		private $options: IOptions) {
+		private $options: IOptions,
+		private $tempService: ITempService) {
 
 		this.pluginProjectPath = path.resolve(this.$options.path || ".");
 	}
@@ -29,8 +30,7 @@ export class BuildPluginCommand implements ICommand {
 			}
 		}
 
-		temp.track();
-		const tempAndroidProject = temp.mkdirSync("android-project");
+		const tempAndroidProject = await this.$tempService.mkdirSync("android-project");
 
 		const options: IPluginBuildOptions = {
 			aarOutputDir: platformsAndroidPath,

lib/common/mobile/android/android-device-file-system.ts

@@ -1,6 +1,5 @@
 import * as path from "path";
 import * as semver from "semver";
-import * as temp from "temp";
 import { AndroidDeviceHashService } from "./android-device-hash-service";
 import { executeActionByChunks } from "../../helpers";
 import { DEFAULT_CHUNK_SIZE } from '../../constants';
@@ -12,6 +11,7 @@ export class AndroidDeviceFileSystem implements Mobile.IDeviceFileSystem {
 		private $fs: IFileSystem,
 		private $logger: ILogger,
 		private $mobileHelper: Mobile.IMobileHelper,
+		private $tempService: ITempService,
 		private $injector: IInjector) { }
 
 	public async listFiles(devicePath: string, appIdentifier?: string): Promise<any> {
@@ -27,8 +27,7 @@ export class AndroidDeviceFileSystem implements Mobile.IDeviceFileSystem {
 		const stdout = !outputPath;
 
 		if (stdout) {
-			temp.track();
-			outputPath = temp.path({ prefix: "sync", suffix: ".tmp" });
+			outputPath = await this.$tempService.path({ prefix: "sync", suffix: ".tmp" });
 		}
 
 		await this.adb.executeCommand(["pull", deviceFilePath, outputPath]);
@@ -134,7 +133,7 @@ export class AndroidDeviceFileSystem implements Mobile.IDeviceFileSystem {
 	}
 
 	public async createFileOnDevice(deviceFilePath: string, fileContent: string): Promise<void> {
-		const hostTmpDir = this.getTempDir();
+		const hostTmpDir = await this.$tempService.mkdirSync("application-");
 		const commandsFileHostPath = path.join(hostTmpDir, "temp.commands.file");
 		this.$fs.writeFile(commandsFileHostPath, fileContent);
 
@@ -159,9 +158,4 @@ export class AndroidDeviceFileSystem implements Mobile.IDeviceFileSystem {
 
 		return this._deviceHashServices[appIdentifier];
 	}
-
-	private getTempDir(): string {
-		temp.track();
-		return temp.mkdirSync("application-");
-	}
 }

lib/common/mobile/android/android-device-hash-service.ts

@@ -1,5 +1,4 @@
 import * as path from "path";
-import * as temp from "temp";
 import { cache } from "../../decorators";
 import { executeActionByChunks } from "../../helpers";
 import { DEFAULT_CHUNK_SIZE, LiveSyncPaths } from "../../constants";
@@ -10,7 +9,8 @@ export class AndroidDeviceHashService implements Mobile.IAndroidDeviceHashServic
 	constructor(private adb: Mobile.IDeviceAndroidDebugBridge,
 		private appIdentifier: string,
 		private $fs: IFileSystem,
-		private $mobileHelper: Mobile.IMobileHelper) {
+		private $mobileHelper: Mobile.IMobileHelper,
+		private $tempService: ITempService) {
 	}
 
 	@cache()
@@ -34,8 +34,9 @@ export class AndroidDeviceHashService implements Mobile.IAndroidDeviceHashServic
 	}
 
 	public async uploadHashFileToDevice(data: IStringDictionary): Promise<void> {
-		this.$fs.writeJson(this.hashFileLocalPath, data);
-		await this.adb.pushFile(this.hashFileLocalPath, this.hashFileDevicePath);
+		const hashFileLocalPath = await this.getHashFileLocalPath();
+		this.$fs.writeJson(hashFileLocalPath, data);
+		await this.adb.pushFile(hashFileLocalPath, this.hashFileDevicePath);
 	}
 
 	public async updateHashes(localToDevicePaths: Mobile.ILocalToDevicePathData[]): Promise<void> {
@@ -86,20 +87,20 @@ export class AndroidDeviceHashService implements Mobile.IAndroidDeviceHashServic
 	}
 
 	@cache()
-	private get hashFileLocalPath(): string {
-		return path.join(this.tempDir, AndroidDeviceHashService.HASH_FILE_NAME);
+	private async getHashFileLocalPath(): Promise<string> {
+		return path.join(await this.getTempDir(), AndroidDeviceHashService.HASH_FILE_NAME);
 	}
 
 	@cache()
-	private get tempDir(): string {
-		temp.track();
-		return temp.mkdirSync(`android-device-hash-service-${this.appIdentifier}`);
+	private getTempDir(): Promise<string> {
+		return this.$tempService.mkdirSync(`android-device-hash-service-${this.appIdentifier}`);
 	}
 
 	private async downloadHashFileFromDevice(): Promise<string> {
-		if (!this.$fs.exists(this.hashFileLocalPath)) {
-			await this.adb.executeCommand(["pull", this.hashFileDevicePath, this.tempDir]);
+		const hashFileLocalPath = await this.getHashFileLocalPath();
+		if (!this.$fs.exists(hashFileLocalPath)) {
+			await this.adb.executeCommand(["pull", this.hashFileDevicePath, await this.getTempDir()]);
 		}
-		return this.hashFileLocalPath;
+		return hashFileLocalPath;
 	}
 }

lib/common/mobile/ios/simulator/ios-simulator-application-manager.ts

@@ -4,7 +4,6 @@ import { hook, getPidFromiOSSimulatorLogs } from "../../../helpers";
 import { cache } from "../../../decorators";
 import { IOS_LOG_PREDICATE } from "../../../constants";
 import * as path from "path";
-import * as temp from "temp";
 import * as log4js from "log4js";
 
 export class IOSSimulatorApplicationManager extends ApplicationManagerBase {
@@ -16,6 +15,7 @@ export class IOSSimulatorApplicationManager extends ApplicationManagerBase {
 		private $options: IOptions,
 		private $fs: IFileSystem,
 		protected $deviceLogProvider: Mobile.IDeviceLogProvider,
+		private $tempService: ITempService,
 		$logger: ILogger,
 		$hooksService: IHooksService) {
 		super($logger, $hooksService, $deviceLogProvider);
@@ -28,8 +28,7 @@ export class IOSSimulatorApplicationManager extends ApplicationManagerBase {
 	@hook('install')
 	public async installApplication(packageFilePath: string): Promise<void> {
 		if (this.$fs.exists(packageFilePath) && path.extname(packageFilePath) === ".zip") {
-			temp.track();
-			const dir = temp.mkdirSync("simulatorPackage");
+			const dir = await this.$tempService.mkdirSync("simulatorPackage");
 			await this.$fs.unzip(packageFilePath, dir);
 			const app = _.find(this.$fs.readDirectory(dir), directory => path.extname(directory) === ".app");
 			if (app) {

lib/common/mobile/mobile-helper.ts

@@ -1,13 +1,13 @@
 import * as helpers from "../helpers";
 import * as shell from "shelljs";
-import * as temp from "temp";
 
 export class MobileHelper implements Mobile.IMobileHelper {
 	private static DEVICE_PATH_SEPARATOR = "/";
 
 	constructor(private $errors: IErrors,
 		private $fs: IFileSystem,
-		private $devicePlatformsConstants: Mobile.IDevicePlatformsConstants) { }
+		private $devicePlatformsConstants: Mobile.IDevicePlatformsConstants,
+		private $tempService: ITempService) { }
 
 	public get platformNames(): string[] {
 		return [this.$devicePlatformsConstants.iOS, this.$devicePlatformsConstants.Android];
@@ -58,8 +58,7 @@ export class MobileHelper implements Mobile.IMobileHelper {
 	}
 
 	public async getDeviceFileContent(device: Mobile.IDevice, deviceFilePath: string, projectData: IProjectData): Promise<string> {
-		temp.track();
-		const uniqueFilePath = temp.path({ suffix: ".tmp" });
+		const uniqueFilePath = await this.$tempService.path({ suffix: ".tmp" });
 		const platform = device.deviceInfo.platform.toLowerCase();
 		try {
 			await device.fileSystem.getFile(deviceFilePath, projectData.projectIdentifiers[platform], uniqueFilePath);

lib/common/test/unit-tests/mobile/android-device-file-system.ts

@@ -8,6 +8,7 @@ import { DevicePlatformsConstants } from "../../../mobile/device-platforms-const
 import * as path from "path";
 import { assert } from "chai";
 import { LiveSyncPaths } from "../../../constants";
+import { TempServiceStub } from "../../../../../test/stubs";
 
 const myTestAppIdentifier = "org.nativescript.myApp";
 let isAdbPushExecuted = false;
@@ -64,7 +65,7 @@ function createTestInjector(): IInjector {
 	injector.register("errors", Errors);
 	injector.register("devicePlatformsConstants", DevicePlatformsConstants);
 	injector.register("projectFilesManager", {});
-
+	injector.register("tempService", TempServiceStub);
 	return injector;
 }
 

lib/common/test/unit-tests/mobile/ios-simulator-discovery.ts

@@ -5,7 +5,7 @@ import { assert } from "chai";
 import { DeviceDiscoveryEventNames, CONNECTED_STATUS } from "../../../constants";
 import { DevicePlatformsConstants } from "../../../mobile/device-platforms-constants";
 import { ErrorsStub, CommonLoggerStub, HooksServiceStub, LockServiceStub } from "../stubs";
-import { FileSystemStub, ChildProcessStub } from "../../../../../test/stubs";
+import { FileSystemStub, ChildProcessStub, TempServiceStub } from "../../../../../test/stubs";
 import { DeviceConnectionType } from "../../../../constants";
 
 let currentlyRunningSimulators: Mobile.IiSimDevice[];
@@ -45,7 +45,7 @@ function createTestInjector(): IInjector {
 	injector.register("options", {});
 	injector.register("hooksService", HooksServiceStub);
 	injector.register("logger", CommonLoggerStub);
-
+	injector.register("tempService", TempServiceStub);
 	return injector;
 }
 

lib/common/test/unit-tests/mobile/project-files-manager.ts

@@ -14,6 +14,7 @@ import { ProjectFilesProviderBase } from "../../../services/project-files-provid
 
 import temp = require("temp");
 import { LiveSyncPaths } from "../../../constants";
+import { TempServiceStub } from "../../../../../test/stubs";
 temp.track();
 
 const testedApplicationIdentifier = "com.telerik.myApp";
@@ -49,6 +50,7 @@ function createTestInjector(): IInjector {
 	});
 	testInjector.register("logger", Logger);
 	testInjector.register("config", {});
+	testInjector.register("tempService", TempServiceStub);
 	return testInjector;
 }
 

lib/definitions/ios.d.ts

@@ -30,8 +30,8 @@ declare global {
 	}
 
 	interface IExportOptionsPlistService {
-		createDevelopmentExportOptionsPlist(archivePath: string, projectData: IProjectData, buildConfig: IBuildConfig): IExportOptionsPlistOutput;
-		createDistributionExportOptionsPlist(projectRoot: string, projectData: IProjectData, buildConfig: IBuildConfig): IExportOptionsPlistOutput;
+		createDevelopmentExportOptionsPlist(archivePath: string, projectData: IProjectData, buildConfig: IBuildConfig): Promise<IExportOptionsPlistOutput>;
+		createDistributionExportOptionsPlist(projectRoot: string, projectData: IProjectData, buildConfig: IBuildConfig): Promise<IExportOptionsPlistOutput>;
 	}
 
 	interface IExportOptionsPlistOutput {