Merge pull request #167 from codecrafters-io/andy/fix

Update writeFiles function to handle newline characters more accurately

Author: andy1li

internal/test_helpers/fixtures/ash/pipelines/pass

@@ -3,11 +3,11 @@ Debug = true
 [tester::#BR6] Running tests for Stage #BR6 (br6)
 [tester::#BR6] [setup] export PATH=/tmp/pear/orange/raspberry:$PATH
 [tester::#BR6] Running ./your_shell.sh
-[tester::#BR6] [setup] echo "strawberry apple\npear banana\nraspberry mango\nblueberry pineapple\norange grape" > "/tmp/foo/file-24"
+[tester::#BR6] [setup] echo -e "strawberry apple\npear banana\nraspberry mango\nblueberry pineapple\norange grape" > "/tmp/foo/file-24"
 [your-program] $ cat /tmp/foo/file-24 | wc
 [your-program]        5      10      78
 [tester::#BR6] ✓ Received expected response
-[tester::#BR6] [setup] echo "1. orange blueberry\n2. strawberry pineapple\n3. banana mango" > "/tmp/baz/file-3"
+[tester::#BR6] [setup] echo -e "1. orange blueberry\n2. strawberry pineapple\n3. banana mango" > "/tmp/baz/file-3"
 [your-program] $ tail -f /tmp/baz/file-3 | head -n 5
 [your-program] 1. orange blueberry
 [your-program] 2. strawberry pineapple
@@ -33,7 +33,7 @@ Debug = true
 
 [tester::#XK3] Running tests for Stage #XK3 (xk3)
 [tester::#XK3] [setup] export PATH=/tmp/orange/pineapple/orange:$PATH
-[tester::#XK3] [setup] echo "pineapple\nraspberry\nblueberry\ngrape\norange" > "/tmp/quz/file-26"
+[tester::#XK3] [setup] echo -e "pineapple\nraspberry\nblueberry\ngrape\norange" > "/tmp/quz/file-26"
 [tester::#XK3] Running ./your_shell.sh
 [your-program] $ cat /tmp/quz/file-26 | head -n 5 | wc
 [your-program]        5       5      43

internal/test_helpers/fixtures/bash/pipelines/pass

@@ -3,11 +3,11 @@ Debug = true
 [tester::#BR6] Running tests for Stage #BR6 (br6)
 [tester::#BR6] [setup] export PATH=/tmp/pear/orange/raspberry:$PATH
 [tester::#BR6] Running ./your_shell.sh
-[tester::#BR6] [setup] echo "strawberry apple\npear banana\nraspberry mango\nblueberry pineapple\norange grape" > "/tmp/foo/file-24"
+[tester::#BR6] [setup] echo -e "strawberry apple\npear banana\nraspberry mango\nblueberry pineapple\norange grape" > "/tmp/foo/file-24"
 [your-program] $ cat /tmp/foo/file-24 | wc
 [your-program]        5      10      78
 [tester::#BR6] ✓ Received expected response
-[tester::#BR6] [setup] echo "1. orange blueberry\n2. strawberry pineapple\n3. banana mango" > "/tmp/baz/file-3"
+[tester::#BR6] [setup] echo -e "1. orange blueberry\n2. strawberry pineapple\n3. banana mango" > "/tmp/baz/file-3"
 [your-program] $ tail -f /tmp/baz/file-3 | head -n 5
 [your-program] 1. orange blueberry
 [your-program] 2. strawberry pineapple
@@ -33,7 +33,7 @@ Debug = true
 
 [tester::#XK3] Running tests for Stage #XK3 (xk3)
 [tester::#XK3] [setup] export PATH=/tmp/orange/pineapple/orange:$PATH
-[tester::#XK3] [setup] echo "pineapple\nraspberry\nblueberry\ngrape\norange" > "/tmp/quz/file-26"
+[tester::#XK3] [setup] echo -e "pineapple\nraspberry\nblueberry\ngrape\norange" > "/tmp/quz/file-26"
 [tester::#XK3] Running ./your_shell.sh
 [your-program] $ cat /tmp/quz/file-26 | head -n 5 | wc
 [your-program]        5       5      43

internal/utils_fs.go

@@ -129,9 +129,17 @@ func writeFiles(paths []string, contents []string, logger *logger.Logger) error
 		logger.UpdateLastSecondaryPrefix("setup")
 
 		if strings.HasSuffix(content, "\n") {
-			logger.Infof("echo %q > %q", content[:len(content)-1], paths[i])
+			if strings.Count(content, "\n") == 1 { // Trailing newline only
+				logger.Infof("echo %q > %q", content[:len(content)-1], paths[i])
+			} else { // Newline(s) in the middle as well
+				logger.Infof("echo -e %q > %q", content[:len(content)-1], paths[i])
+			}
 		} else {
-			logger.Infof("echo -n %q > %q", content, paths[i])
+			if strings.Contains(content, "\n") {
+				logger.Infof("echo -ne %q > %q", content, paths[i])
+			} else {
+				logger.Infof("echo -n %q > %q", content, paths[i])
+			}
 		}
 
 		logger.ResetSecondaryPrefixes()