search

jenssegers / laravel-ab

A server-side A/B testing tool for Laravel.
288 stars 46 forks source link

Added support for Laravel 5 #7

Closed garygreen closed 9 years ago

garygreen commented 9 years ago

Might need a bit of tweaking but tested on L5 and L4.2, all working fine.

garygreen commented 9 years ago

@jenssegers I've updated the PR, all the tests should pass (hopefully!). I've refactored them so you don't need to set the config stuff (to be honest, you can remove Orchestra now as it's pretty much pointless).

The only test that is failing is testTrackWithoutExperiment I'm not even sure what that one is supose to be testing, but I think it's highlighted an issue with the track() code (should it not create experiments if they don't exist i.e. call checkExperiments() ?