commit 84505ea819d2cd8f2ca495f211c0dbd5f20d19ae
author Wyatt Allen <wyatta@google.com> Thu Aug 24 10:53:05 2017 -0700
committer Wyatt Allen <wyatta@google.com> Thu Aug 24 15:51:25 2017 -0700
tree a25a2c1712663ed72872799664a920e37dd07db9
parent f14482cbb57eae72442ff42f68b41973ce385735

Refactor gr-router functions into more testable component methods

Formerly, the most complex of gr-router's logic lived in functions
defined in the closure's scope rather than as methods on the component.
With this change, these functions are refactored into component methods
while preserving their semantics. This opens the door for dramatic
increases in test coverage in upcoming changes.

Summary of refactorings in this change:
- `startRouter` is refactored into a method rather than a plain function
  defined in the closure but is invoked in much the same manner. Whereas
  it formerly accepted the instance-scoped `_generateUrl` method as a
  parameter, it can now access the method via `this` and the parameter
  is removed.
- `restAPI` is promoted to be a Polymer property rather than a local to
  `startRouter`. In upcoming changes, this should be refactored to be
  defined in the usual way and accessed via `this.$`.
- `normalizeLegacyRouteParams` is refactored into a method and its
  behavior is encoded in unit tests.
- `redirectToLogin` is refactored into a method.
- `normalizePatchRangeParams` is refactored into an instance method and
  its behavior is partially encoded in unit tests. Whereas the method
  was formerly untested, one code path is left untested as it revealed
  a bug in the method, and will be patched in an upcoming change.
- Declaring new `app.params` now goes through the new (easily stub-able)
  `_setParams` method.
- Invoking a redirection now goes through the new (also easily
  stub-able) `_redirect` method.
- Behavior methods which were (somewhat hackily) invoked from their
  definitions inside the closure are now used like normal behavior
  methods.
- Tests are added for `_getPatchRangeExpression`.

This change increases gr-router test coverage from 18.9% to 26.4%.
Greater coverage increases are to come in future changes.

Change-Id: I86cb722e218a6ff599d7312d7efd71113f8d5ab4