Update notes_controller_test to use note and note_comment factories

I've ported across the naming for each note from the old fixture names.
This helps in the mega-tests where multiple notes are tested at the
same time.

The number or results have changed in some tests, since there's little
point in creating e.g. 5 note comments when 2 is enough to test the
functionality.

Update site_controller_test to use note and note_comment factories.

Update browse_controller_test to use note and note_comment factories.

Fix bugs->notes in comment

Update note model tests to use factories.

The NoteTest.test_visible? has changed slightly, so that the second
test of .visible? is using a closed note, which is a more useful test
than the original.

Add factories for notes and note_comments

Update leaflet to 1.0.1

Patch classes on the locate control

This replaces the patching of the vendored source that we had been
doing before and which we always forget to apply after updating it.

Fixes #1306

Remove z-index override from leaflet controls on small screens

Fixes #1305

Patch leaflet-locationfilter for leaflet 1.x support

https://github.com/kajic/leaflet-locationfilter/issues/22
https://github.com/kajic/leaflet-locationfilter/pull/23

Update leaflet to 1.0.0

Localisation updates from https://translatewiki.net.

Extend donation drive banner end date to 2016-oct-31

Localisation updates from https://translatewiki.net.

Merge remote-tracking branch 'openstreetmap/pull/1298'

Fix rubocop warnings

Centre loading icon

Add links to new approved OSMF privacy policy and some rewording.

look for extra place tags in Nominatim search results

Administrative boundaries at municipality level are generally
better described by the place node (as town, village etc.).
Nominatim exports the place type in extratags if it was able
to merge place nodes with admin boundaries. Use this preferably
to create the descriptive term.

Remove SOTM 2016 banner, add Donation Drive 2016 Banner

Localisation updates from https://translatewiki.net.

Add thunderforest API key to embedded maps

Fixes #1293

Fix some rubocop warnings

Update to rails 4.2.7.1

Merge remote-tracking branch 'openstreetmap/pull/1285'

Fix new rubocop warnings

Rework R2 support for changes in sprockets

Update bundle

Add additional list items to ensure they are not selected

The other list tests already have assetions showing the different
lengths of lists.

Localisation updates from https://translatewiki.net.

Prioritize next upcoming event in banner rotation
(closes #1284)

Remove unneccessary fixture requirement.

Remove diary_entry and diary_comment fixtures, and refactor tests.

Note that you might need to empty the tables in your local test
database, since the removed fixtures may still linger there.

Add banner for SotM Latam 2016

Localisation updates from https://translatewiki.net.

Merge remote-tracking branch 'openstreetmap/pull/1279'

Add thunderforest API key to example configuration

Propagate API keys to minimaps in layer switcher

Hide thunderforest layers when no API key is available

Use an API key for thunderforest tile layers

Refactor tests to use factories instead of fixtures

Change the display_name of suspended user fixuture

The original name is reserved, which means the user was invalid,
and this causes problems when building objects using it via
associations.

Test DiaryComment body validation

Replaces unnecessary test.

Localisation updates from https://translatewiki.net.

Build a diary entry more succinctly

Assert there are no diary comments before creating one.

Update name of test

Rework DiaryEntry and DiaryComment model tests to use factories.

Since the database also contains fixtures from other tests, some
counts are dropped and instead tested for inclusion in the results.

Add FactoryGirl

Update offense counts in rubocop_todo.yml

Add xml builder files to rubocop.

Rubocop autofixes.

Add outer variable shadowing to rubocop_todo.yml

Use .present? instead of double negative to check if terms_agreed timestamp is set.

User.consider_pd is boolean, so no need for double negation.

Update schema

Localisation updates from https://translatewiki.net.

Set banner expiry correctly

Localisation updates from https://translatewiki.net.

Fix date for Gravatar migration

Improve code readability

Improve message when Gravater is disabled

Merge remote-tracking branch 'openstreetmap/pull/1036'

Fix rubocop warnings

Merge message when Gravatar status has changed with email confirmation

Merge message when Gravatar status has changed with email confirmation and make messages more verbose and friendly.

Localisation updates from https://translatewiki.net.

Set all cookies on the root path

Make banner cookies expire after one year

This avoids having lots of cookies stick around forever and
we shouldn't ever have a banner up for that long.

Merge remote-tracking branch 'openstreetmap/pull/1270'

Localisation updates from https://translatewiki.net.

Remove references to EWG meetings

Add stupid script to bulk check the gravatar status

Add tests for change of gravatar use depending on email

Support returning a HTTP status code for the http stubs for testing

Remove default enabling of gravatar, check on initial confirmation of e-mail address and on any changes afterward if
a gravatar exists and enable then if the user hasn't uploaded a picture.

Add test for formatting of multi-value wikidata links

Fix rubocop warnings

Merge remote-tracking branch 'openstreetmap/pull/788'

Pan to first Nominatim search result

Fixes #919
Closes #1041

Add site.key.table.entry.toilets to translations

Merge remote-tracking branch 'openstreetmap/pull/1004'

Fix html.dir for pl locale

Localisation updates from https://translatewiki.net.

Reorder auth_success handlers

Only treat auth_success as a possible login attempt if we're not
in the middle of validating a new user, or a change to a user.

Also validate the uniqueness of external auth credentials at the
rails level rather than just at the database level, and make sure
any errors are properly reported.

Fixes #1265

Blacklist all Google imagery with one regex

This expression catches 100% of Google imagery seen in imagery_used
tags with no false positives.

It won't stop someone who tries to load "https://www.google.com/maps" as
imagery, but that will fail to get imagery anyway.

INSTALL.md: Added inline code markdown.

Added inline code markdown in MacOSX install instructions, making it look like command line code.

Localisation updates from https://translatewiki.net.

Localisation updates from https://translatewiki.net.

Correct icon mappings for non OSRM routers

Add support for OSRM 5.0 API and new instruction types

Add support for the new API in OSRM 5.x including new icon sprites
for the new instruction types.

Localisation updates from https://translatewiki.net.

Localisation updates from https://translatewiki.net.

Add State of the Map Asia 2016

Localisation updates from https://translatewiki.net.

Link all objects into libpgosm.so

Allow multiple versions of DB functions to be built

Allows the pg_config location to be configured, along with
the target directory for the built code.

Revert "Don't pass the locale to the Mapzen routing engine"

This reverts commit 38180c56c9be9d99406c9088b60f5912c2648a04.

Localisation updates from https://translatewiki.net.

Localisation updates from https://translatewiki.net.

Localisation updates from https://translatewiki.net.