Fix horrible process.binding() atrocity

fs.js

@@ -1,11 +1 @@
-// eeeeeevvvvviiiiiiillllll
-// more evil than monkey-patching the native builtin?
-// Not sure.
-
-var mod = require("module")
-var pre = '(function (exports, require, module, __filename, __dirname) { '
-var post = '});'
-var src = pre + process.binding('natives').fs + post
-var vm = require('vm')
-var fn = vm.runInThisContext(src)
-fn(exports, require, module, __filename, __dirname)
+module.exports = Object.create(require('fs'))

graceful-fs.js

@@ -1,5 +1,3 @@
-// Monkey-patching the fs module.
-// It's ugly, but there is simply no other way to do this.
 var fs = module.exports = require('./fs.js')
 
 var assert = require('assert')

test/module-evil-hackery.js

@@ -0,0 +1,10 @@
+var test = require('tap').test
+var realfs = require('fs')
+var fs = require('../')
+
+test('module assignment should not leak', function (t) {
+    t.plan(1)
+
+    fs.abc = 3
+    t.equal(realfs.abc, undefined)
+})

test/stats.js

@@ -0,0 +1,12 @@
+var test = require('tap').test
+var fs = require('fs')
+var gfs = require('../graceful-fs.js')
+
+test('graceful fs uses same stats constructor as fs', function (t) {
+  t.equal(gfs.Stats, fs.Stats, 'should reference the same constructor')
+  t.ok(fs.statSync(__filename) instanceof fs.Stats,
+    'should be instance of fs.Stats')
+  t.ok(gfs.statSync(__filename) instanceof fs.Stats,
+    'should be instance of fs.Stats')
+  t.end()
+})