<p>Merged but I realised after I'd done so that you weren't using the factory to create the objects (I had misread the create call before) so I fixed that in <a href="https://github.com/openstreetmap/openstreetmap-website/commit/9c926c6c71e0f387fddb24b010a57923b79125ad" class="commit-link"><tt>9c926c6</tt></a>.</p>
<p>I also dropped the use of translation lookups in the tests in <a href="https://github.com/openstreetmap/openstreetmap-website/commit/9b52542e521a0df1de742d9915770dcf8a07272e" class="commit-link"><tt>9b52542</tt></a> in favour of straight string matches which is more consistent with what we do elsewhere.</p>

<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/pull/1445#issuecomment-280914293">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/ABWnLQBV16MoOE0QTiemRUabs9sAhAnCks5reC2bgaJpZM4MFL3S">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/ABWnLRPergSK7mgN9Ygqj02BhcEo5eLnks5reC2bgaJpZM4MFL3S.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/pull/1445#issuecomment-280914293"></link>
  <meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request 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":"PERSON","message":"@tomhughes in #1445: Merged but I realised after I'd done so that you weren't using the factory to create the objects (I had misread the create call before) so I fixed that in 9c926c6c71e0f387fddb24b010a57923b79125ad.\r\n\r\nI also dropped the use of translation lookups in the tests in 9b52542e521a0df1de742d9915770dcf8a07272e in favour of straight string matches which is more consistent with what we do elsewhere."}],"action":{"name":"View Pull Request","url":"https://github.com/openstreetmap/openstreetmap-website/pull/1445#issuecomment-280914293"}}}</script>