add job create param caching to config file

Author: sanfilip

lib/jobs/create.js

@@ -183,6 +183,26 @@ function create(params, cb) {
 		}
 	}
 
+	if (params.machineType) projectConfig.setMachineType(params.project, params.machineType);
+	else params.machineType = projectConfig.getMachineType();
+
+	if (params.container) projectConfig.setContainer(params.project, params.container);
+	else params.container = projectConfig.getContainer();
+
+	if (params.command) projectConfig.setCommand(params.project, params.command);
+	else params.command = projectConfig.getCommand();
+
+	if (params.workspace) projectConfig.setWorkspace(params.project, params.workspace);
+	else params.workspace = projectConfig.getWorkspace();
+
+	if (params.name) projectConfig.setName(params.project, params.name);
+	else params.name = projectConfig.getName();
+
+	if (params.dataset) projectConfig.setDataset(params.project, params.dataset);
+	else params.dataset = projectConfig.getDataset();
+
+	if (!params.container) return ifCliPrintErrorOnly(new Error('Missing required parameter `container`'));
+
 	if (!params.workspace) params.workspace = cwd;
 
 	function ifCliPrintErrorOnly(err) {

lib/projectConfig.js

@@ -49,7 +49,6 @@ function setLastJobId(project, jobId) {
 	}
 }
 
-
 function removeLastJobId(jobId) {
 	if (jobId) {
 		read();
@@ -60,6 +59,108 @@ function removeLastJobId(jobId) {
 	}
 }
 
+function getContainer(project, container) {
+	if (!project && !container) {
+		read();
+		return config.container;
+	}
+	return container;
+}
+
+function setContainer(project, container) {
+	if (container) {
+		if ((!project || config.project === project) && config.container !== container) {
+			config.container = container;
+			write();
+		}
+	}
+}
+
+function getMachineType(project, machineType) {
+	if (!project && !machineType) {
+		read();
+		return config.machineType;
+	}
+	return machineType;
+}
+
+function setMachineType(project, machineType) {
+	if (machineType) {
+		if ((!project || config.project === project) && config.machineType !== machineType) {
+			config.machineType = machineType;
+			write();
+		}
+	}
+}
+
+function getCommand(project, command) {
+	if (!project && !command) {
+		read();
+		return config.command;
+	}
+	return command;
+}
+
+function setCommand(project, command) {
+	if (command) {
+		if ((!project || config.project === project) && config.command !== command) {
+			config.command = command;
+			write();
+		}
+	}
+}
+
+function getWorkspace(project, workspace) {
+	if (!project && !workspace) {
+		read();
+		return config.workspace;
+	}
+	return workspace;
+}
+
+function setWorkspace(project, workspace) {
+	if (workspace) {
+		if ((!project || config.project === project) && config.workspace !== workspace) {
+			config.workspace = workspace;
+			write();
+		}
+	}
+}
+
+function getDataset(project, dataset) {
+	if (!project && !dataset) {
+		read();
+		return config.dataset;
+	}
+	return dataset;
+}
+
+function setDataset(project, dataset) {
+	if (dataset) {
+		if ((!project || config.project === project) && config.dataset !== dataset) {
+			config.dataset = dataset;
+			write();
+		}
+	}
+}
+
+function getName(project, name) {
+	if (!project && !name) {
+		read();
+		return config.name;
+	}
+	return name;
+}
+
+function setName(project, name) {
+	if (name) {
+		if ((!project || config.project === project) && config.name !== name) {
+			config.name = name;
+			write();
+		}
+	}
+}
+
 function show() {
 	read();
 	return config;
@@ -70,5 +171,17 @@ module.exports = {
 	getLastJobId: getLastJobId,
 	setLastJobId: setLastJobId,
 	removeLastJobId: removeLastJobId,
+	getContainer: getContainer,
+	setContainer: setContainer,
+	getMachineType: getMachineType,
+	setMachineType: setMachineType,
+	getCommand: getCommand,
+	setCommand: setCommand,
+	getDataset: getDataset,
+	setDataset: setDataset,
+	getWorkspace: getWorkspace,
+	setWorkspace: setWorkspace,
+	getName: getName,
+	setName: setName,
 	show: show,
 };