stefanpenner/ember-app-kit — pull request #621 by bantic (2014-05-07 21:10:09+00:00)
Title: Revert "Remove unnecessary router reopening"

This reverts commit 46040476a75b4470acff131f39a1391f039af422.

As it turns out, `App.setupForTesting`, even though it calls `App.Router.reopen` to set `location: 'none'`, does not affect the routing behavior of the tests. I'm not sure if this is due to the way EAK interacts with the resolver to create its router, or if it is a more ember-y bug. I'm going to look into a PR for ember to ensure that `App.setupForTesting` does change the router either way, but in the meantime this commit should probably be reverted.

If you check out master of EAK and run the tests, specifically test 2 ("Acceptances - Helper"), you'll see the URL changes from http://localhost:8000/tests?testNumber=2 to http://localhost:8000/tests?testNumber=2#/helper-test

![eak-router](https://cloud.githubusercontent.com/assets/2023/2908824/ee1b1f22-d62b-11e3-8eeb-af9779a1fa41.gif)