v init: create a .gitignore in existing git repo if it does not exist yet (#10488)

cmd/tools/vcreate.v

@@ -109,14 +109,14 @@ fn (c &Create) create_git_repo(dir string) {
 			cerror('Unable to create git repo')
 			exit(4)
 		}
-		if !os.exists('$dir/.gitignore') {
+	}
-			mut fl := os.create('$dir/.gitignore') or {
+	if !os.exists('$dir/.gitignore') {
-				// We don't really need a .gitignore, it's just a nice-to-have
+		mut fl := os.create('$dir/.gitignore') or {
-				return
+			// We don't really need a .gitignore, it's just a nice-to-have
-			}
+			return
-			fl.write_string(gen_gitignore(c.name)) or { panic(err) }
-			fl.close()
 		}
+		fl.write_string(gen_gitignore(c.name)) or { panic(err) }
+		fl.close()
 	}
 }
 

cmd/tools/vcreate_test.v

@@ -2,15 +2,7 @@ import os
 
 const test_path = 'vcreate_test'
 
-fn test_v_init() ? {
+fn init_and_check() ? {
-	dir := os.join_path(os.temp_dir(), test_path)
-	os.rmdir_all(dir) or {}
-	os.mkdir(dir) ?
-	defer {
-		os.rmdir_all(dir) or {}
-	}
-	os.chdir(dir)
-
 	vexe := os.getenv('VEXE')
 	os.execute_or_panic('$vexe init')
 
@@ -45,3 +37,43 @@ fn test_v_init() ? {
 		'',
 	].join('\n')
 }
+
+fn test_v_init() ? {
+	dir := os.join_path(os.temp_dir(), test_path)
+	os.rmdir_all(dir) or {}
+	os.mkdir(dir) ?
+	defer {
+		os.rmdir_all(dir) or {}
+	}
+	os.chdir(dir)
+
+	init_and_check() ?
+}
+
+fn test_v_init_in_git_dir() ? {
+	dir := os.join_path(os.temp_dir(), test_path)
+	os.rmdir_all(dir) or {}
+	os.execute_or_panic('git init $dir')
+	defer {
+		os.rmdir_all(dir) or {}
+	}
+	os.chdir(dir)
+
+	init_and_check() ?
+}
+
+fn test_v_init_no_overwrite_gitignore() ? {
+	dir := os.join_path(os.temp_dir(), test_path)
+	os.rmdir_all(dir) or {}
+	os.mkdir(dir) or {}
+	os.write_file('$dir/.gitignore', 'blah') ?
+	defer {
+		os.rmdir_all(dir) or {}
+	}
+	os.chdir(dir)
+
+	vexe := os.getenv('VEXE')
+	os.execute_or_panic('$vexe init')
+
+	assert os.read_file('.gitignore') ? == 'blah'
+}