Guard against a possible string being pulled from the query string and coerce to a Regular Expression

Author: jhamlet

lib/util/coerce-to-reg-exp.js

@@ -0,0 +1,12 @@
+var regexp;
+
+module.exports = function coerceToRegExp (stringOrRegExp) {
+  var result = stringOrRegExp;
+
+  if (typeof result === 'string') {
+    eval('regexp = ' + stringOrRegExp);
+    result = regexp;
+  }
+
+  return result;
+};

lib/util/title-case.js

@@ -1,10 +1,11 @@
 var R = require('ramda');
+var coerceToRegExp = require('./coerce-to-reg-exp');
 var uppercaseFirst = require('./upper-case-first');
 
 var DEFAULT = /[:\w-]/;
 
 module.exports = R.curry(function titleCase (delim, text) {
-    var words = text.split(delim || DEFAULT);
+    var words = text.split(coerceToRegExp(delim) || DEFAULT);
     return words.
         map(uppercaseFirst).
         join('');

test/unit/utility/title-case.js

@@ -19,12 +19,18 @@ describe('svg-react-loader/lib/util/title-case', () => {
             delim: /[:-]/g,
             text: 'foo-bar-baz',
             result: 'FooBarBaz'
+        },
+        {
+            delim: '/[:-]/g',
+            text: 'foo-bar-baz',
+            result: 'FooBarBaz'
         }
     ];
 
     expectations.
     forEach((spec) => {
-        const description = `titleCase(/${spec.delim.source}/, '${spec.text}') should equal '${spec.result}'`;
+        const type = spec.delim.constructor.name;
+        const description = `titleCase(${spec.delim} as ${type}, '${spec.text}') should equal '${spec.result}'`;
 
         it(description, () => {
             titleCase(spec.delim, spec.text).