Change to options.fileSystem for consistency

Refs dependents/node-filing-cabinet#68 (comment)

Readme.md

@@ -61,7 +61,7 @@ var deps2 = paperwork('styles.scss');
 Supported options:
 
 * `includeCore`: (default: true) set to `false` to exclude core Node dependencies from the list of dependencies.
-* `fs`: (default: undefined) set to an alternative `fs` implementation that will be used to read the file path.
+* `fileSystem`: (default: undefined) set to an alternative `fs` implementation that will be used to read the file path.
 * You may also pass detective-specific configuration like you would to `precinct(content, options)`.
 
 #### CLI

index.js

@@ -130,15 +130,17 @@ function assign(o1, o2) {
  * @param {String} filename
  * @param {Object} [options]
  * @param {Boolean} [options.includeCore=true] - Whether or not to include core modules in the dependency list
- * @param {Object} [options.fs=undefined] - An alternative fs implementation to use for reading the file path.
+ * @param {Object} [options.fileSystem=undefined] - An alternative fs implementation to use for reading the file path.
  * @return {String[]}
  */
 precinct.paperwork = function(filename, options) {
   options = assign({
     includeCore: true
   }, options || {});
 
-  var fileSystem = options.fs || fs;
+  // Note: released with options.fs but intended options.fileSystem for consistency in the community
+  // TODO: Remove options.fs in the next major version update
+  var fileSystem = options.fileSystem || options.fs || fs;
   var content =  fileSystem.readFileSync(filename, 'utf8');
   var ext = path.extname(filename);
   var type;

test/index.js

@@ -152,7 +152,7 @@ describe('node-precinct', function() {
   });
 
   describe('paperwork', function() {
-    it('uses fs from options if provided', function() {
+    it('uses fileSystem from options if provided', function() {
       var fsMock = {
         readFileSync: function(path, encoding) {
           assert.equal(path, '/foo.js');
@@ -161,7 +161,7 @@ describe('node-precinct', function() {
       };
 
       var options = {
-        fs: fsMock
+        fileSystem: fsMock
       };
       var results = precinct.paperwork('/foo.js', options);
       assert.equal(results.length, 1);