Wrapped paths in quotation marks because they can contain a space tha…

…t will break the command.
cksource · Apr 26, 2018 · be9485d · be9485d
1 parent 7c2a2ee
commit be9485d
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/lib/commands/bootstrap.js b/lib/commands/bootstrap.js
@@ -32,8 +32,8 @@ module.exports = {
 			promise = Promise.resolve();
 		} else {
 			const command = [
-				`git clone --progress ${ data.repository.url } ${ destinationPath }`,
-				`cd ${ destinationPath }`,
+				`git clone --progress "${ data.repository.url }" "${ destinationPath }"`,
+				`cd "${ destinationPath }"`,
 				`git checkout --quiet ${ data.repository.branch }`
 			].join( ' && ' );
 

diff --git a/tests/commands/bootstrap.js b/tests/commands/bootstrap.js
@@ -88,9 +88,9 @@ describe( 'commands/bootstrap', () => {
 
 					// Clone the repository.
 					expect( cloneCommand[ 0 ] )
-						.to.equal( 'git clone --progress git@github.com/organization/test-package.git packages/test-package' );
+						.to.equal( 'git clone --progress "git@github.com/organization/test-package.git" "packages/test-package"' );
 					// Change the directory to cloned package.
-					expect( cloneCommand[ 1 ] ).to.equal( 'cd packages/test-package' );
+					expect( cloneCommand[ 1 ] ).to.equal( 'cd "packages/test-package"' );
 					// And check out to proper branch.
 					expect( cloneCommand[ 2 ] ).to.equal( 'git checkout --quiet master' );