v0.0.5 - order of middleware changed

middleware now runs like so:
first element in middleware[] gets run first, thus making it last on
flow back

last element in middleware[] therefore gets run last, thus making it
the first on flow back

rewrite compose to avoid having to .reverse() array

package.json

@@ -1,6 +1,6 @@
 {
   "name": "spirit",
-  "version": "0.0.4",
+  "version": "0.0.5",
   "description": "extensible web library for building applications & frameworks",
   "main": "index.js",
   "scripts": {

spec/core/core-spec.js

@@ -1,11 +1,11 @@
 const core = require("../../lib/core/core")
 
-describe("compose", () => {
+describe("core - compose", () => {
   const reduce = core.compose
 
   it("order", (done) => {
     const handler = (request) => {
-      expect(request.called).toBe(".cba")
+      expect(request.called).toBe(".abc")
       return Promise.resolve({ resp: "." })
     }
 
@@ -45,7 +45,7 @@ describe("compose", () => {
     const resp = route(mock_req)
     resp.then((response) => {
       expect(response).toEqual({
-        resp: ".123"
+        resp: ".321"
       })
       done()
     })

spec/spirit-spec.js

@@ -11,15 +11,15 @@ describe("spirit spec", () => {
     const middlewares = [
       (handler) => {
         return (input) => {
-          input += "c"
+          input += "b"
           return handler(input).then((result) => {
             return result + "1"
           })
         }
       },
       (handler) => {
         return (input) => {
-          input += "b"
+          input += "c"
           return handler(input).then((result) => {
             return result + "2"
           })
@@ -29,7 +29,7 @@ describe("spirit spec", () => {
 
     const reducer = spirit.compose(handler, middlewares)
     reducer("a").then((result) => {
-      expect(result).toBe("out12")
+      expect(result).toBe("out21")
       done()
     })
   })
@@ -63,19 +63,19 @@ describe("spirit spec", () => {
       throw "never gets run"
     }
     const middlewares = [
-      (handler) => {
-        return (input) => {
-          expect(input).toBe("ab")
-          return Promise.resolve("stop")
-        }
-      },
       (handler) => {
         return (input) => {
           input += "b"
           return handler(input).then((result) => {
             return result + "2"
           })
         }
+      },
+      (handler) => {
+        return (input) => {
+          expect(input).toBe("ab")
+          return Promise.resolve("stop")
+        }
       }
     ]
     const reducer = spirit.compose(handler, middlewares)

src/core/core.js

@@ -20,13 +20,22 @@ const compose = (handler, middleware) => {
     }
   }
 
+  let prev = wrap(handler)
+
+  for (let i = middleware.length - 1; i >= 0; i--) {
+    prev = wrap(middleware[i](prev))
+  }
+
+  return prev
+  /*
   return middleware.reduce((prev, curr) => {
     const r = curr(prev)
     if (typeof r !== "function") {
       throw new TypeError("Expected middleware to return a function that takes a request")
     }
     return wrap(r)
   }, wrap(handler))
+   */
 }
 
 module.exports = {

src/core/promise_utils.js

@@ -3,8 +3,8 @@
  */
 const Promise = require("bluebird")
 Promise.onPossiblyUnhandledRejection(function(e, promise) {
-  throw e;
-});
+  throw e
+})
 
 /**
  * checks if `p` is a promise, returning true or false