Ignore require calls and logging calls in string literals mutator

Author: DiegoRBaquero

example-module/src/killed-dep.js

@@ -0,0 +1 @@
+module.exports = 'hello'

example-module/src/killed.js

@@ -1,3 +1,5 @@
+const string = require('./killed-dep')
+
 module.exports = {
   deletion () {
     return true
@@ -47,7 +49,7 @@ module.exports = {
   },
   stringLiterals: {
     hello () {
-      return 'hello'
+      return string
     },
     empty () {
       return ''

example-module/src/survived.js

@@ -1,7 +1,7 @@
 function other (a = 0) {
   for (let i = 0; false; ) {
   }
-  console.log()
+  console.log('')
   a++
   a--
 }

src/mutators/stringLiteralsMutator.js

@@ -13,11 +13,27 @@ const lineDiff = require('../util/lineDiff')
 module.exports = async function stringLiteralsMutator ({mutodeInstance, filePath, lines, queue, ast}) {
   debug('Running string literals mutator on %s', filePath)
 
-  walk.simple(ast, {
-    StringLiteral (node) {
+  walk.ancestor(ast, {
+    StringLiteral (node, state, ancestors) {
       const line = node.loc.start.line
       const lineContent = lines[line - 1]
 
+      if (ancestors.length >= 2) {
+        const ancestor = ancestors[ancestors.length - 2]
+        if (ancestor.type && ancestor.type === 'CallExpression' && ancestor.callee) {
+          if (ancestor.callee.type === 'MemberExpression' && ancestor.callee.object.name === 'console') return
+          if (ancestor.callee.name) {
+            switch (ancestor.callee.name) {
+              case 'require':
+              case 'debug':
+                return
+              default:
+                break
+            }
+          }
+        }
+      }
+
       if (node.value.length !== 0) {
         const mutantLineContent = lineContent.substr(0, node.loc.start.column) +
           node.extra.raw.replace(node.value, '') +

test/exampleModule.js

@@ -12,7 +12,7 @@ if (process.env.MUTODE_CONCURRENCY) opts.concurrency = process.env.MUTODE_CONCUR
 
 test.serial('Exmaple module - killed', async t => {
   const testOpts = Object.assign({
-    paths: 'src/killed.js'
+    paths: ['src/killed.js', 'src/killed-dep.js']
   }, opts)
   let mutode = new Mutode(testOpts)
   await mutode.run()