Don't directly depend on components to wire

Change API from magical side-effect require to exporting a function that
accepts the modules to wire together. The module now has no
dependencies, instead you pass in the components to wire up. This also
allows using a specific instance of e.g. stack-filter.

lib/buster-sinon.js

@@ -1,48 +1,45 @@
 ((typeof define === "function" && define.amd && function (m) {
-    define(["sinon", "buster-test", "stack-filter", "formatio"], m);
+    define(m);
 }) || (typeof module === "object" &&
        typeof require === "function" && function (m) {
-        module.exports = m(
-            require("sinon"),
-            require("buster-test"),
-            require("stack-filter"),
-            require("formatio")
-        );
+        module.exports = m();
     }) || function (m) {
-        m(this.sinon, this.buster, this.stackFilter, this.formatio);
+        this.busterSinon = m();
     }
-)(function (sinon, bt, stackFilter, formatio) {
-    stackFilter.filters.push("lib/sinon");
+)(function () {
+    return function (sinon, bt, stackFilter, formatio) {
+        stackFilter.filters.push("lib/sinon");
 
-    bt.testRunner.onCreate(function (runner) {
-        runner.on("test:setUp", function (test) {
-            var config = sinon.getConfig(sinon.config);
-            config.useFakeServer = false;
-            var sandbox = sinon.sandbox.create();
-            sandbox.inject(test.testCase);
+        bt.testRunner.onCreate(function (runner) {
+            runner.on("test:setUp", function (test) {
+                var config = sinon.getConfig(sinon.config);
+                config.useFakeServer = false;
+                var sandbox = sinon.sandbox.create();
+                sandbox.inject(test.testCase);
 
-            test.testCase.useFakeTimers = function () {
-                return sandbox.useFakeTimers.apply(sandbox, arguments);
-            };
+                test.testCase.useFakeTimers = function () {
+                    return sandbox.useFakeTimers.apply(sandbox, arguments);
+                };
 
-            test.testCase.useFakeServer = function () {
-                return sandbox.useFakeServer.apply(sandbox, arguments);
-            };
+                test.testCase.useFakeServer = function () {
+                    return sandbox.useFakeServer.apply(sandbox, arguments);
+                };
 
-            test.testCase.sandbox = sandbox;
-            var testFunc = test.func;
-        });
+                test.testCase.sandbox = sandbox;
+                var testFunc = test.func;
+            });
 
-        runner.on("test:tearDown", function (test) {
-            try {
-                test.testCase.sandbox.verifyAndRestore();
-            } catch (e) {
-                runner.assertionFailure(e);
-            }
-        });
+            runner.on("test:tearDown", function (test) {
+                try {
+                    test.testCase.sandbox.verifyAndRestore();
+                } catch (e) {
+                    runner.assertionFailure(e);
+                }
+            });
 
-        sinon.expectation.pass = function () {
-            runner.assertionPass();
-        };
-    });
+            sinon.expectation.pass = function () {
+                runner.assertionPass();
+            };
+        });
+    };
 });

package.json

@@ -35,13 +35,11 @@
     "scripts": {
         "test": "./run-tests"
     },
-    "dependencies": {
+    "devDependencies": {
+        "referee": "~0.11",
         "buster-test": "~0.7",
         "formatio": "~0.6",
         "sinon": "~1.4",
         "stack-filter": "~0.1"
-    },
-    "devDependencies": {
-        "referee": "~0.11"
     }
 }

test/buster-sinon-test.js

@@ -1,5 +1,10 @@
-require("../lib/buster-sinon");
 var buster = require("buster-test");
+require("../lib/buster-sinon")(
+    require("sinon"),
+    buster,
+    require("stack-filter"),
+    require("formatio")
+);
 var referee = require("referee");
 var assert = referee.assert;
 var refute = referee.refute;