Add simple resource

Author: sergeysova

.eslintignore

@@ -1 +1,2 @@
 *.test.js
+examples

examples/resource.js

@@ -0,0 +1,20 @@
+const stringify = require('stringify-object')
+const chalk = require('chalk')
+
+const { createRest, flattenRoutes, printRoutes } = require('../dist')
+
+const ObjectController = {
+  read() {},
+  create() {},
+  update() {},
+  destroy() {},
+
+  beforeEach() {},
+  afterEach() {},
+}
+
+const routes = createRest(r => {
+  r.resource('demo', ObjectController)
+})
+
+printRoutes(routes, true)

lib/index.js

@@ -18,11 +18,11 @@ class Maker {
   }
 
   before(...list) {
-    this.ctx.before = this.ctx.before.concat(list)
+    this.ctx.before = this.ctx.before.concat(list.filter(ln => !!ln))
   }
 
   after(...list) {
-    this.ctx.after = this.ctx.after.concat(list)
+    this.ctx.after = this.ctx.after.concat(list.filter(ln => !!ln))
   }
 
   scope(name, creator) {
@@ -59,8 +59,9 @@ class Maker {
    * @param {string} method
    * @param {function[]} listeners
    */
-  method(method, listeners) {
+  method(method, _listeners) {
     let name = ''
+    let listeners = _listeners
     if (typeof listeners[0] === 'string') {
       name = listeners.splice(0, 1)[0]
     }
@@ -73,6 +74,11 @@ class Maker {
       throw new Error('Maybe you forget to add listener?')
     }
 
+    listeners = listeners.filter(e => typeof e === 'function')
+
+    // if added undefined listeners
+    if (listeners.length === 0) return
+
     if (name !== '') {
       let scoped = this.ctx.scoped[name]
       if (!scoped) {
@@ -126,6 +132,39 @@ class Maker {
   patch(...listeners) {
     this.method('PATCH', listeners)
   }
+
+  /**
+   *
+   * @param {string} name
+   * @param {object|function} controller
+   * @param {object} options
+   */
+  resource(name, controller, options = {}) {
+    if (!name || name.length === 0) {
+      throw new Error('Resource should be named')
+    }
+
+    if (typeof controller === 'undefined') {
+      return
+    }
+
+    const idName = `:${name}Id`
+    const list = [
+      ['get', 'read'],
+      ['post', 'create'],
+      ['put', 'update'],
+      ['delete', 'destroy'],
+    ]
+
+    this.scope(name, r => {
+      r.before(controller.beforeEach)
+      r.after(controller.afterEach)
+
+      list.forEach(([http, method]) => {
+        r[http](idName, controller[method])
+      })
+    })
+  }
 }
 
 /**

package.json

@@ -31,8 +31,7 @@
   "homepage": "https://github.com/atomixinteractions/createrest#readme",
   "dependencies": {
     "chalk": "^1.1.3",
-    "pluralize": "^3.1.0",
-    "ramda": "^0.23.0"
+    "pluralize": "^3.1.0"
   },
   "devDependencies": {
     "@ava/babel-preset-stage-4": "^1.0.0",

test/index.test.js

@@ -11,6 +11,13 @@ const destroy = () => {}
 const before = () => {}
 const after = () => {}
 
+const ObjectController = {
+  create() {},
+  read() {},
+  update() {},
+  destroy() {},
+}
+
 const make = (before = [], after = [], scoped = {}, local = {}) => ({
   before,
   after,
@@ -230,3 +237,36 @@ test('Fail if no listeners passed to method', t => {
   })
 })
 
+test('Simple resource with default options', t => {
+  t.deepEqual(
+    createRest(r => {
+      r.resource('unicorn', ObjectController)
+    }),
+    make([], [], {
+      unicorn: make([], [], {
+        ':unicornId': make([], [], {}, {
+          GET: [ObjectController.read],
+          POST: [ObjectController.create],
+          PUT: [ObjectController.update],
+          DELETE: [ObjectController.destroy],
+        }),
+      }),
+    }, {})
+  )
+})
+
+
+test('Fail resource with def opts', t => {
+  t.deepEqual(
+    createRest(r => {
+      r.resource('unicorn')
+    }),
+    make([], [], {}, {})
+  )
+
+  t.throws(() => {
+    createRest(r => {
+      r.resource()
+    })
+  })
+})