snapframework/snap — pull request #63 by nurpax (2012-12-26 19:20:25+00:00)
Title: Use () instead of AuthUser in forceLogin + Either return type consistency changes

As discussed on the snap mailing list, some return value changes to more consistently use Either in functions that may fail.

I originally planned to do only https://github.com/nurpax/snap/commit/6e5ee3538a10679257982f2e1226db7a8ef7c535 but realized that loginByRememberToken was inconsistent with the rest of the module and using Maybe instead of Either for its return value. Fixed this in https://github.com/nurpax/snap/commit/a7e1b7fb260baa240db5c8fc69f12deb6f2f0fc5.

Fixed the unit tests for the new types. Also ran the Auth tests and they still all passed. I did get some other failures like this:

```
ERROR: StatusCodeException (Status {statusCode = 500, statusMessage = "Internal Server Error"}) [("Content-Length","72"),("Date","Wed, 26 Dec 2012 19:12:02 GMT"),("Server","Snap/0.10.0"),("Content-Type","text/plain; charset=utf-8"),("X-Response-Body-Start","A web handler threw an exception. Details:\nTemplate \"bazpage\" not found.")]
internal/bazpage4: [Failed]
```

but I think I just ran the test suite incorrectly.