Simplify exports.

jaredhanson · Feb 10, 2016 · 20846b4 · 20846b4
1 parent c9502ce
commit 20846b4
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 14 deletions.
diff --git a/lib/index.js b/lib/index.js
@@ -1,21 +1,11 @@
-/**
- * Module dependencies.
- */
+// Load modules.
 var Strategy = require('./strategy')
   , InternalOAuthError = require('./errors/internaloautherror');
 
 
-/**
- * Expose `Strategy` directly from package.
- */
+// Expose Strategy.
 exports = module.exports = Strategy;
 
-/**
- * Export constructors.
- */
+// Exports.
 exports.Strategy = Strategy;
-
-/**
- * Export errors.
- */
 exports.InternalOAuthError = InternalOAuthError;
diff --git a/test/package.test.js b/test/package.test.js
@@ -2,11 +2,15 @@ var strategy = require('..');
 
 describe('passport-oauth', function() {
 
-  it('should export Strategy constructor directly from package', function() {
+  it('should export Strategy constructor as module', function() {
     expect(strategy).to.be.a('function');
     expect(strategy).to.equal(strategy.Strategy);
   });
 
+  it('should export Strategy constructor', function() {
+    expect(strategy.Strategy).to.be.a('function');
+  });
+
   it('should export Error constructors', function() {
     expect(strategy.InternalOAuthError).to.be.a('function');
   });