#44: adds test cases

Author: kaelzhang

bin/rm.sh

@@ -389,6 +389,8 @@ remove(){
         read answer
         if [[ ${answer:0:1} =~ [yY] ]]; then
           [[ $(ls -A "$file") ]] && {
+            debug "$LINENO: $file: Directory not empty: $(ls -A "$file")"
+
             echo "$COMMAND: $file: Directory not empty"
 
             return 1

package.json

@@ -6,7 +6,9 @@
     "safe-rm": "./bin/rm.sh"
   },
   "scripts": {
-    "test": "ava --timeout=10s --verbose",
+    "test:rm": "ava --timeout=10s --verbose test/safe-rm.test.js",
+    "test:as": "ava --timeout=10s --verbose test/safe-rm-as.test.js",
+    "test": "npm run test:rm && npm run test:as",
     "test:dev": "NODE_DEBUG=safe-rm npm run test",
     "test:debug": "SAFE_RM_DEBUG=1 npm run test:dev",
     "test:mock-linux": "SAFE_RM_DEBUG_LINUX=1 npm run test",

test/cases.js

@@ -272,4 +272,46 @@ module.exports = (
     t.true(result.stderr.includes('protected'), 'stderr should include "protected"')
     t.true(await pathExists(filepath), 'file should not be removed')
   })
+
+  !is_as(type) && test(`#44: recursively and interactively`, async t => {
+    const {
+      createDir,
+      createFile,
+      runRm,
+      pathExists
+    } = t.context
+
+    const name = 'foo'
+
+    const dir = await createDir({
+      name
+    })
+
+    const fileA = await createFile({
+      name: 'a',
+      under: dir
+    })
+
+    const fileB = await createFile({
+      name: 'b',
+      under: dir
+    })
+
+    const result = await runRm(['-ri', dir], {
+      input: ['y', 'y', 'y', 'y']
+    })
+
+    t.is(result.stderr, '', 'stderr should be empty')
+    t.is(result.code, 0, 'exit code should be 0')
+    t.is(
+      result.stdout,
+      `examine files in directory ${dir}? y
+remove ${fileA}? y
+remove ${fileB}? y
+remove ${dir}? y
+`,
+      'stdout does not match'
+    )
+    // t.false(await pathExists(dir), 'directory should be removed')
+  })
 }

test/helper.js

@@ -58,13 +58,13 @@ const generateContextMethods = (
 
   // Helper function to run rm commands
   function runRm (args, {
-    input = '',
+    input = [],
     command = rm_command,
     env: arg_env = {}
   } = {}) {
-    return new Promise(resolve => {
+    return new Promise((resolve, reject) => {
       const env = {
-        // ...process.env,
+        ...process.env,
         ...{
           SAFE_RM_TRASH: t.context.trash_path
         },
@@ -80,21 +80,33 @@ const generateContextMethods = (
 
       child.stdout.on('data', data => {
         stdout += data.toString()
+
+        if (/\?\s*$/.test(stdout)) {
+          if (!child.stdin) {
+            reject(new Error('Child process does not support stdin'))
+            return
+          }
+
+          const this_input = input.shift()
+
+          if (!this_input) {
+            reject(new Error('Need more input'))
+            return
+          }
+
+          child.stdin.write(`${this_input}\n`)
+          stdout += `${this_input}\n`
+
+          if (input.length === 0) {
+            child.stdin.end()
+          }
+        }
       })
 
       child.stderr.on('data', data => {
         stderr += data.toString()
       })
 
-      if (input) {
-        if (!child.stdin) {
-          throw new Error('Child process does not support stdin')
-        }
-
-        child.stdin.write(input)
-        child.stdin.end()
-      }
-
       child.on('close', code => {
         const resolved = {
           code,
@@ -105,6 +117,7 @@ const generateContextMethods = (
         log(command, 'result:', resolved)
 
         resolve(resolved)
+        child.kill()
       })
     })
   }