Remove a few unnecessary attributes from moderator users

Make all factory-generated users 'normal'

The use of create(:normal_user) makes the tests needlessly harder to
understand and reason about, particularly when we start using
associations in the factories.

Fix rubocop warning

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

Index oauth_tokens and client_applications by user_id

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

Convert notes_controller tests to user factories.

Revert changes where the user is a query parameter

Refactor api_controller tests to avoid users fixtures.

Refactor amf_controller tests to avoid users fixtures

In all honesty I wouldn't be surprised if there are some semi-hidden
dependencies on the user fixtures here, given the number of magic numbers
all over the place. But this commit takes care of the explicit
dependencies for now.

Pass the user object, rather than the id, for sessions.

This simplifies the code slightly.

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

Enable context menu on mouse down with no shift pressed

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

Add a couple of notes to the vagrant instructions

Use created at for RSS format with note#show

This matches other formats and is also what the tests expect.

Remove fake qqq locale

Localisation updates from https://translatewiki.net.

Refactor diary entry tests to use User factory

Use simple string matches for user_blocks_helper tests

Testing against the translations is not something we do anywhere
else is and is dangerously close to testing the helper against
itself which is a bit pointless.

Use the factory to create user blocks

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

Make rupocop happy and correct placement of include

Improve wording and add test

Fix shift key handling in contextmenu

Small improvement to block status display

Merge branch 'contextmenu'

Fix rubocop warnings

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

Make test for a "zero hour" block more robust

Apparently some fool things the colon is part of the protocol

Actually use user_block

Add functionality to return a specific message for zero hour blocks

Preserve protocol in standard links

Preserve protocol in short links

Initialise locale before looking up user blocked error

Implement the OWG credit policy for hosting partners.

See https://operations.osmfoundation.org/policies/hosting/

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

Add line segment to each steps of GraphHopper routing

Fix latLng from GraphHopper engine

latLng is an array and not a latLng object

Remove fake qqq locale

Localisation updates from https://translatewiki.net.

Allow a POST with no arguments to trace#edit to fetch the form

Remove U+200C unicode "zero width non-joiner" from html.dir

Remove fake qqq locale

Localisation updates from https://translatewiki.net.

Allow context menu to add notes without panning the map

Update Leaflet.contextmenu to the 1.2.1 release

Make context menu entries translateable

Improve context menu initialisation to avoid namespace pollution

Merge remote-tracking branch 'openstreetmap/pull/1030' into next

Remove invalid YML

Closes #1434

Only add traces for POST requests

Only create diary entries for POST requests

Test that GET on message#new can't send a message

Render message properly on error

Only send messages for POST requests

Remove fake qqq locale

Localisation updates from https://translatewiki.net.

Don't display a box at all if the note event doesn't include a comment body

Before this it just displays en empty box, which looks wrong.

Only record a visible object if it exists

Fixes #1430

Update to iD v2.1.2

Update to iD v2.1.1

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

Remove fake qqq locale

Localisation updates from https://translatewiki.net.

Link GPX fixture files instead of stubbing

Cleanup double comment markers

Make TraceController#list sort by id instead of timestamp

The effect is much the same but id is guaranteed to give a stable
sort if two traces have the same timestamp.

Merge remote-tracking branch 'openstreetmap/pull/1347' into master

Fix rubocop warnings

Use .first in place of [0]

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

Update bundle

Update rubocop

Update to iD v2.1.0

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

Convert the test_users_nearby to user factories.

Use factory for user language tests

Refactor the friend_users test, and remove the tests which duplicate the (renamed) test_friends_with above.

Convert test_friend_with to use factories, and use alice/bob/charlie for easier understanding.

User factory for uniqueness tests

Convert class method tests to use User factory.

Also improve the tests by being explicit about which results should
be returned, rather than just hoping that the fixtures cover all
possibilities.

Add status traits to user factory.

Add moderator_user and administrator_user factories.

Add a user_role factory

Naming the association in the model makes the relationship easier
to express in the factory.

Add with_home_location trait for user factories.

Create a simple user factory, and convert some basic tests.

Reduce maximum changeset size to 10k changes

Ten thousand changes ought to be enough for any normal mapping activity.

Automatic edits and imports are controlled by scripts anyway so it doesn't
make a difference to them as they just have to adapt the limit.

The reason for my suggestion is that large changesets are becoming
increasingly difficult to handle due to frequent timeouts when trying
to load and process them.

The changeset size limit is returned by the API in the "capabilities" request
so client software that honours that information will automatically pick up
the new limit.

Closes #1259

Remove fake qqq locale

Localisation updates from https://translatewiki.net.

That new message in the HTML version of the mail should also be added to the plain text version of the mail

Duh.

Use each_with_object and make the code pithier

Renamed `apply_inline_css` to `style_message` as suggested

Set border=0 on that img

IE9 shows a big nasty blue border on linked images if you don't specify border=0

Convert client applications fixtures into a factory

Closes #1415

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

Load user fixtures for ChangesetComment model tests

Localisation updates from https://translatewiki.net.

Left-align logo