<p>Our test suite is heavily focussed on controller and model tests. Unfortunately this means that very little of the site is actually tested, that is, very few of the tests render the views, or check that links are working and pointing in the right place etc. The controller tests don't render views, and each controller method is directly tested, rather than using the routes.</p>
<p>This means that problems crop up even though all the tests pass - see e.g. <a href="https://github.com/openstreetmap/openstreetmap-website/commit/117f0e82260efba3c6862e3aefa4b015b062a109" class="commit-link"><tt>117f0e8</tt></a></p>
<p>The advice I've read over the years is to focus less on controller tests, and more on feature tests - where a web browser is used to render the html, click on links etc. I find this works well in my other projects.</p>
<p>I would suggest we do some or all of the following, but of course I'm open to suggestions:</p>
<ul>
<li>Add feature tests to catch all the problems in the commit mentioned above</li>
<li>Ensure that the signup flow works using feature tests</li>
<li>Consider replacing the non-API, non-XML/RSS controller tests as feature tests</li>
</ul>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/openstreetmap/openstreetmap-website/issues/1570">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/ABWnLSYL-fBYoWXGsLwhzgpoa-EFCg0Uks5sI6SIgaJpZM4OJSuQ">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/ABWnLT1SCNnc7K1---0K1_LCsAjMRZI6ks5sI6SIgaJpZM4OJSuQ.gif" width="1" /></p>
<div itemscope itemtype="http://schema.org/EmailMessage">
<div itemprop="action" itemscope itemtype="http://schema.org/ViewAction">
  <link itemprop="url" href="https://github.com/openstreetmap/openstreetmap-website/issues/1570"></link>
  <meta itemprop="name" content="View Issue"></meta>
</div>
<meta itemprop="description" content="View this Issue on GitHub"></meta>
</div>

<script type="application/json" data-scope="inboxmarkup">{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/openstreetmap/openstreetmap-website","title":"openstreetmap/openstreetmap-website","subtitle":"GitHub repository","main_image_url":"https://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/openstreetmap/openstreetmap-website"}},"updates":{"snippets":[{"icon":"DESCRIPTION","message":"Missing feature tests (#1570)"}],"action":{"name":"View Issue","url":"https://github.com/openstreetmap/openstreetmap-website/issues/1570"}}}</script>