Refactor check for routes instance

Author: sergeysova

lib/flatten.js

@@ -1,3 +1,5 @@
+import { isCreateRestInstance } from './index'
+
 /**
  * Recursive flatten
  * @private
@@ -54,5 +56,8 @@ function flattenize(routes, prefix, parent) {
  * )
  */
 export function flattenRoutes(routes) {
+  if (!isCreateRestInstance(routes)) {
+    throw new Error('You can flatten only createRest routes')
+  }
   return flattenize(routes, '', { before: [], after: [] })
 }

lib/index.js

@@ -1,3 +1,4 @@
+import { createRestInstanceSymbol } from './symbol'
 
 /**
  * @desc Simple object for `resource()`
@@ -26,7 +27,7 @@
  * }
  */
 
-export const isCreateRestInstance = Symbol('isCreateRestInstance')
+export const isCreateRestInstance = routing => routing[createRestInstanceSymbol] === true
 
 /**
  * @desc Routes object
@@ -62,7 +63,7 @@ export class Maker {
     Object.keys(this.ctx.scoped).forEach(name => {
       scoped[name] = this.ctx.scoped[name].build()
     })
-    return Object.assign({}, this.ctx, { scoped, [isCreateRestInstance]: true })
+    return Object.assign({}, this.ctx, { scoped, [createRestInstanceSymbol]: true })
   }
 
   /**

lib/symbol.js

@@ -0,0 +1 @@
+export const createRestInstanceSymbol = Symbol('createRestInstance')

test/flatten.test.js

@@ -96,3 +96,9 @@ avaTest('Creates local, scoped methods and methods in scope', test => {
     }
   )
 })
+
+avaTest('Fail if passed not routes', test => {
+  test.throws(() => {
+    flattenRoutes({})
+  })
+})

test/index.test.js

@@ -1,5 +1,6 @@
 import avaTest from 'ava'
-import { createRest, isCreateRestInstance } from '../lib'
+import { createRest } from '../lib'
+import { createRestInstanceSymbol } from '../lib/symbol'
 
 
 const get = () => {}
@@ -22,7 +23,7 @@ const make = (before = [], after = [], local = {}, scoped = {}) => ({
   after,
   local,
   scoped,
-  [isCreateRestInstance]: true,
+  [createRestInstanceSymbol]: true,
 })
 
 avaTest('Creates base structure', test => {