add route tests

[yfractal]

Add some tests for morsel.

Relate issue: #15

[yfractal]

nobody notice this :( ...

[IainNZ]

The tests failed, so I think I forgot about it

[IainNZ]

You need to add a new commit with a test/REQUIRE file that has FactCheck in it

[yfractal]

oh, no i forget the requirement....

I forgot my local Meddle is in mast branch too :( ...

Meddle add params to immutable MeddleRequest in this pr .

but the Meddle's newest release v0.0.5 did not include these change...

And in Morsel, the params have been used, eg: https://github.com/JuliaWeb/Morsel.jl/blob/master/src/Morsel.jl#L197

Should we give the Meddle a new release? Or do something else?

[yfractal]

By the way. The pkg in my local.

 - Meddle                      0.0.5+             master
 - HttpCommon             0.0.10
 - HttpServer                 0.0.11

And the test works fine.

Morsel is listening on 8000...
route
get, post, post, put, delete, request
  get
  post, put, delete
5 facts verified.
Add route when the app is running
1 fact verified.
dynamic route
  the request will succes if the type is right
  can get the value by req.params[:id]
  regex route
5 facts verified.
namespace
1 fact verified.
0 facts verified.

[IainNZ]

OK so I need to tag a new version of Meddle, then this would work? I can do that - give me ~24 hours or so, I'm travelling right now

[yfractal]

Yeah, I think so. Tests will work if i check out to Meddle's master branch.
Thanks and take your time :)
Have a good travelling. :)

[IainNZ]

I tagged Meddle v0.0.6, and I've just restarted Travis

[IainNZ]

Green light :D

[yfractal]

yeah! That's cool!