chore: tweaks

Author: technote-space

__tests__/git-helper.test.ts

@@ -110,8 +110,6 @@ describe('GitHelper', () => {
 			await helper.checkout(workDir, context());
 
 			execCalledWith(mockExec, [
-				'rm -rdf \'.work\'',
-				'git init \'.\'',
 				'git remote add origin \'https://octocat:[email protected]/hello/world.git\' > /dev/null 2>&1 || :',
 				'git fetch --no-tags origin \'refs/heads/test-ref:refs/remotes/origin/test-ref\' || :',
 				'git checkout -qf test-sha',
@@ -124,8 +122,6 @@ describe('GitHelper', () => {
 			await helper.checkout(workDir, context({ref: 'refs/pull/123/merge'}));
 
 			execCalledWith(mockExec, [
-				'rm -rdf \'.work\'',
-				'git init \'.\'',
 				'git remote add origin \'https://octocat:[email protected]/hello/world.git\' > /dev/null 2>&1 || :',
 				'git fetch --no-tags origin \'refs/pull/123/merge:refs/pull/123/merge\' || :',
 				'git checkout -qf test-sha',
@@ -138,8 +134,6 @@ describe('GitHelper', () => {
 			await helper.checkout(workDir, context({ref: 'refs/tags/v1.2.3'}));
 
 			execCalledWith(mockExec, [
-				'rm -rdf \'.work\'',
-				'git init \'.\'',
 				'git remote add origin \'https://octocat:[email protected]/hello/world.git\' > /dev/null 2>&1 || :',
 				'git fetch --no-tags origin \'refs/tags/v1.2.3:refs/tags/v1.2.3\' || :',
 				'git checkout -qf test-sha',
@@ -179,6 +173,17 @@ describe('GitHelper', () => {
 
 			await helper.addOrigin(workDir, context());
 
+			execCalledWith(mockExec, [
+				'git remote add origin \'https://octocat:[email protected]/hello/world.git\' > /dev/null 2>&1 || :',
+			]);
+		});
+
+		it('should run git init and git remote add', async() => {
+			const mockExec = spyOnExec();
+			setExists([false, true]);
+
+			await helper.addOrigin(workDir, context());
+
 			execCalledWith(mockExec, [
 				`rm -rdf '${workDir}'`,
 				'git init \'.\'',
@@ -194,8 +199,6 @@ describe('GitHelper', () => {
 			await helper.fetchOrigin(workDir, context());
 
 			execCalledWith(mockExec, [
-				`rm -rdf '${workDir}'`,
-				'git init \'.\'',
 				'git remote add origin \'https://octocat:[email protected]/hello/world.git\' > /dev/null 2>&1 || :',
 				'git fetch origin || :',
 			]);
@@ -207,8 +210,6 @@ describe('GitHelper', () => {
 			await helper.fetchOrigin(workDir, context(), ['--no-tags']);
 
 			execCalledWith(mockExec, [
-				`rm -rdf '${workDir}'`,
-				'git init \'.\'',
 				'git remote add origin \'https://octocat:[email protected]/hello/world.git\' > /dev/null 2>&1 || :',
 				'git fetch --no-tags origin || :',
 			]);
@@ -220,8 +221,6 @@ describe('GitHelper', () => {
 			await helper.fetchOrigin(workDir, context(), undefined, ['+refs/pull/*/merge:refs/remotes/pull/*/merge', '+refs/heads/hoge:refs/remotes/origin/hoge']);
 
 			execCalledWith(mockExec, [
-				`rm -rdf '${workDir}'`,
-				'git init \'.\'',
 				'git remote add origin \'https://octocat:[email protected]/hello/world.git\' > /dev/null 2>&1 || :',
 				'git fetch origin \'+refs/pull/*/merge:refs/remotes/pull/*/merge\' \'+refs/heads/hoge:refs/remotes/origin/hoge\' || :',
 			]);

src/git-helper.ts

@@ -84,9 +84,14 @@ export default class GitHelper {
 
 	/**
 	 * @param {string} workDir work dir
+	 * @param {boolean} refresh refresh?
 	 * @return {Promise<void>} void
 	 */
-	private initialize = async(workDir: string): Promise<void> => {
+	private initialize = async(workDir: string, refresh = true): Promise<void> => {
+		if (isCloned(workDir) && !refresh) {
+			return;
+		}
+
 		if (fs.existsSync(workDir)) {
 			await this.runCommand(workDir, {command: 'rm', args: ['-rdf', workDir]});
 		}
@@ -106,7 +111,7 @@ export default class GitHelper {
 	 * @return {Promise<void>} void
 	 */
 	public addOrigin = async(workDir: string, context: Context): Promise<void> => {
-		await this.initialize(workDir);
+		await this.initialize(workDir, false);
 		await this.runCommand(workDir, {
 			command: 'git remote add',
 			args: ['origin', getGitUrlWithToken(context, this.token)],