Keep the current indentation of json files

Since our other clients use 2 spaces for indentation in json files we should use 2 spaces too. Since some users are using only the CLI we should not change their current indentation.

Author: TsvetanMilanov

file-system.ts

@@ -9,6 +9,9 @@ import * as shelljs from "shelljs";
 
 @injector.register("fs")
 export class FileSystem implements IFileSystem {
+	private static DEFAULT_INDENTATION_CHARACTER = "\t";
+	private static JSON_OBJECT_REGEXP = new RegExp(`{\\r*\\n*(\\W*)"`, "m");
+
 	constructor(private $injector: IInjector) { }
 
 	//TODO: try 'archiver' module for zipping
@@ -269,7 +272,11 @@ export class FileSystem implements IFileSystem {
 		return future;
 	}
 
-	public writeJson(filename: string, data: any, space: string = "\t", encoding?: string): IFuture<void> {
+	public writeJson(filename: string, data: any, space?: string, encoding?: string): IFuture<void> {
+		if (!space) {
+			space = this.getIndentationCharacter(filename).wait();
+		}
+
 		return this.writeFile(filename, JSON.stringify(data, null, space), encoding);
 	}
 
@@ -523,4 +530,23 @@ export class FileSystem implements IFileSystem {
 			}
 		}).future<void>()();
 	}
+
+	private getIndentationCharacter(filePath: string): IFuture<string> {
+		return ((): string => {
+			if (!this.exists(filePath).wait()) {
+				return FileSystem.DEFAULT_INDENTATION_CHARACTER;
+			}
+
+			let fileContent = this.readText(filePath).wait().trim();
+			let matches = fileContent.match(FileSystem.JSON_OBJECT_REGEXP);
+
+			if (!matches || !matches[1]) {
+				return FileSystem.DEFAULT_INDENTATION_CHARACTER;
+			}
+
+			let indentation = matches[1];
+
+			return indentation[0] === " " ? indentation : FileSystem.DEFAULT_INDENTATION_CHARACTER;
+		}).future<string>()();
+	}
 }

services/user-settings-service.ts

@@ -41,7 +41,7 @@ export class UserSettingsServiceBase implements IUserSettingsService {
 					this.userSettingsData[propertyName] = data[propertyName];
 				});
 
-			this.$fs.writeJson(this.userSettingsFilePath, this.userSettingsData, "\t").wait();
+			this.$fs.writeJson(this.userSettingsFilePath, this.userSettingsData).wait();
 		}).future<void>()();
 	}
 

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

@@ -146,6 +146,7 @@ describe("Android device file system tests", () => {
 				isDirectory: () => false,
 				isFile: () => true
 			});
+			fs.readText = () => Future.fromResult("");
 
 			let androidDeviceFileSystem = createAndroidDeviceFileSystem(injector);
 			androidDeviceFileSystem.transferFile = (localPath: string, devicePath: string) => {
@@ -173,6 +174,7 @@ describe("Android device file system tests", () => {
 				isDirectory: () => false,
 				isFile: () => true
 			});
+			fs.readText = () => Future.fromResult("");
 
 			let androidDeviceFileSystem = createAndroidDeviceFileSystem(injector);
 			let transferedFilesOnDevice: string[] = [];
@@ -206,6 +208,7 @@ describe("Android device file system tests", () => {
 				isDirectory: () => false,
 				isFile: () => true
 			});
+			fs.readText = () => Future.fromResult("");
 
 			let androidDeviceFileSystem = createAndroidDeviceFileSystem(injector);
 			androidDeviceFileSystem.transferFile = (localPath: string, devicePath: string) => {
@@ -233,6 +236,7 @@ describe("Android device file system tests", () => {
 				isDirectory: () => false,
 				isFile: () => true
 			});
+			fs.readText = () => Future.fromResult("");
 
 			let androidDeviceFileSystem = createAndroidDeviceFileSystem(injector);
 			androidDeviceFileSystem.transferFile = (localPath: string, devicePath: string) => {