mbland / url-pointers / 22 / 1

94%

master: 98%

push

travis-ci

RedisClient,RedirectDb: Change interface contracts

After starting to write the API router, I realized it would be better to
distinguish between forbidden or disallowed operations (HTTP 403 and
related) and internal errors (HTTP 500).

RedisClient now returns a Boolean if an operation was carried out
successfully, based on whether the result matched what was expected; and
throws Errors only when an operation failed.

RedirectDb now returns a rejected Promise containing a string on
forbidden/disallowed operations and an Error if a client operation
failed or an unexpected internal error occurred.

48 of 48 branches covered (100.0%)

Branch coverage included in aggregate %.

131 of 143 relevant lines covered (91.61%)

4.64 hits per line