<p>This pull request converts the notes and note_comments test to use factories instead of fixtures.</p>

<p>There are a handful of occasions where the number of results in an assertion have changed, but this is just because e.g. two note comments are enough to test the functionality and creating additional comments is not testing anything different.</p>

<hr>

<h4>You can view, comment on, or merge this pull request online at:</h4>
<p>  <a href='https://github.com/openstreetmap/openstreetmap-website/pull/1312'>https://github.com/openstreetmap/openstreetmap-website/pull/1312</a></p>

<h4>Commit Summary</h4>
<ul>
  <li>Add factories for notes and note_comments</li>
  <li>Update note model tests to use factories.</li>
  <li>Fix bugs->notes in comment</li>
  <li>Update browse_controller_test to use note and note_comment factories.</li>
  <li>Update site_controller_test to use note and note_comment factories.</li>
  <li>Update notes_controller_test to use note and note_comment factories</li>
  <li>Remove the note and note_comments fixtures</li>
</ul>

<h4>File Changes</h4>
<ul>
  <li>
    <strong>M</strong>
    <a href="https://github.com/openstreetmap/openstreetmap-website/pull/1312/files#diff-0">app/controllers/notes_controller.rb</a>
    (2)
  </li>
  <li>
    <strong>M</strong>
    <a href="https://github.com/openstreetmap/openstreetmap-website/pull/1312/files#diff-1">test/controllers/browse_controller_test.rb</a>
    (23)
  </li>
  <li>
    <strong>M</strong>
    <a href="https://github.com/openstreetmap/openstreetmap-website/pull/1312/files#diff-2">test/controllers/notes_controller_test.rb</a>
    (256)
  </li>
  <li>
    <strong>M</strong>
    <a href="https://github.com/openstreetmap/openstreetmap-website/pull/1312/files#diff-3">test/controllers/site_controller_test.rb</a>
    (3)
  </li>
  <li>
    <strong>A</strong>
    <a href="https://github.com/openstreetmap/openstreetmap-website/pull/1312/files#diff-4">test/factories/note_comments.rb</a>
    (8)
  </li>
  <li>
    <strong>A</strong>
    <a href="https://github.com/openstreetmap/openstreetmap-website/pull/1312/files#diff-5">test/factories/notes.rb</a>
    (7)
  </li>
  <li>
    <strong>D</strong>
    <a href="https://github.com/openstreetmap/openstreetmap-website/pull/1312/files#diff-6">test/fixtures/note_comments.yml</a>
    (142)
  </li>
  <li>
    <strong>D</strong>
    <a href="https://github.com/openstreetmap/openstreetmap-website/pull/1312/files#diff-7">test/fixtures/notes.yml</a>
    (113)
  </li>
  <li>
    <strong>M</strong>
    <a href="https://github.com/openstreetmap/openstreetmap-website/pull/1312/files#diff-8">test/models/note_comment_test.rb</a>
    (10)
  </li>
  <li>
    <strong>M</strong>
    <a href="https://github.com/openstreetmap/openstreetmap-website/pull/1312/files#diff-9">test/models/note_test.rb</a>
    (32)
  </li>
  <li>
    <strong>M</strong>
    <a href="https://github.com/openstreetmap/openstreetmap-website/pull/1312/files#diff-10">test/test_helper.rb</a>
    (2)
  </li>
</ul>

<h4>Patch Links:</h4>
<ul>
  <li><a href='https://github.com/openstreetmap/openstreetmap-website/pull/1312.patch'>https://github.com/openstreetmap/openstreetmap-website/pull/1312.patch</a></li>
  <li><a href='https://github.com/openstreetmap/openstreetmap-website/pull/1312.diff'>https://github.com/openstreetmap/openstreetmap-website/pull/1312.diff</a></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/pull/1312">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/ABWnLePLZ-Dhptw4ijh2P59kSJpsV4RNks5qxLJpgaJpZM4KPrg2">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/ABWnLYUuQQi4FdDh515t96JYkkpO8FGJks5qxLJpgaJpZM4KPrg2.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/1312"></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":"DESCRIPTION","message":"Convert notes and note_comments tests to use factories (#1312)"}],"action":{"name":"View Pull Request","url":"https://github.com/openstreetmap/openstreetmap-website/pull/1312"}}}</script>