Regardless of any reasoning in the commit message my above comment is still valid `href="#"` is almost always going to be incorrect.

As far as the commit message goes it feels that the discussion would have been better on the dev list, rather than in a commit message.  I agree that 2 routes is generally wrong it seems that the issue has arisen during changes to the datastore that were incomplete (missed the non-js route) and so fixing that seems the most appropriate solution.

As far as front-end testing is concerned this is a separate issue which needs to be resolved.  Currently front-end test coverage is poor/non-existent due to various factors - currently it is only available for legacy templates.