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

Add Ascend/Descend when routing with GraphHopper

Remove fake qqq locale

Localisation updates from https://translatewiki.net.

Fixed quote style to please rubocop

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.

merging master into html-mail-fixes

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

A better way to contrain the width of the message box whilst also allowing it to shrink on narrow displays

Setting a hard "width" it very compatible, but won't squeeze on narrow screens. Setting max-width works well, except in Outlook and Hotmail. We resort to old skool table tricks: a table with width=100%, with 3 columns, only oneof which, the middle one, had content, and a fixed with. This works.

Two more "nowrap" instructions removed to accommodate narrower displays

Remove nowrap on footer text, to accommodate narrow screens

Set font on title

Remove invalid YML

Closes #1434

Make the mails squeezable on narrow screens

Only add traces for POST requests

Only create diary entries for POST requests

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

Fixed alignment & underlining bugs in Outlook / WinMail

Render message properly on error

Only send messages for POST requests

Remove fake qqq locale

Localisation updates from https://translatewiki.net.

Avoid squished-up grey borders around the message on narrow screens

Even out the widths of the white bars around the message box

Better alt text for the logo

This is what clients that block images will display, so shorter is better.

Use the same fonts for the footer text as for the rest of the mail

Fix title alignment in Outlook/WindowsMail

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.

Removed italic styling on message body

Contributor BushmanK pointed out quite rightly that italic isn't appropriate for a full body of text.

Force the font on every <p> tag

Without this Outlook and Windows Mail don't honour the font set at the top level. Note that this change predates all the screenshots I uploaded to the wiki on 2017-02-02, I'd just forgotten to commit them.

Tweak cell width for proper centering on narrow screens

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.