Test and fix for allowing services with only a setup method (#285)

src/application.js

@@ -46,7 +46,8 @@ export default {
 
     // Run the provider functions to register the service
     this.providers.forEach(provider =>
-      provider.call(this, location, protoService, options));
+      provider.call(this, location, protoService, options)
+    );
 
     // If we ran setup already, set this service up explicitly
     if (this._isSetup && typeof protoService.setup === 'function') {
@@ -75,10 +76,11 @@ export default {
       });
 
     const hasMethod = methods => methods.some(name =>
-      (service && typeof service[name] === 'function'));
+      (service && typeof service[name] === 'function')
+    );
 
     // Check for service (any object with at least one service method)
-    if(hasMethod(['handle', 'set']) || !hasMethod(this.methods)) {
+    if(hasMethod(['handle', 'set']) || !hasMethod(this.methods.concat('setup'))) {
       return this._super.apply(this, arguments);
     }
 

test/application.test.js

@@ -326,6 +326,20 @@ describe('Feathers application', () => {
     assert.equal(otherApp.mixins.length, 4);
   });
 
+  it('initializes a service with only a setup method (#285)', done => {
+    const app = feathers();
+
+    app.use('/setup-only', {
+      setup(_app, path) {
+        assert.equal(_app, app);
+        assert.equal(path, 'setup-only');
+        done();
+      }
+    });
+
+    app.setup();
+  });
+
   it('Event punching happens after normalization (#150)', done => {
     const todoService = {
       create(data) {