Use redactions factory for old_node controller tests.

Use redaction factory for old_relation controller tests.

Fix typos in assertion messages.

Use redactions factory in redaction controller test.

Create a redactions factory and use for model tests.

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

Use a pending user for testing password resets.

Use users factory for user_controller tests.

Throw an exception if adding a note comment fails

Make search use HTTPS automatically

Currently users end up having a HTTP search function even if they
install it to their browsers from the HTTPS site. This is a unexpected
reduction in privacy.

Since search requests are private data they should be sent via HTTPS.

Closes #936

Build pending users via factory

Return maximum size of bounding box for note retrieval

Localisation updates from https://translatewiki.net.

Improve styling of flash messages on the front page

Show offline/readonly messages as normal flash messages

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

Add some whitespace

Use user factories for user_login integration tests.

The display_name logins were reworked to use downcase since the
factories supply users with an already-title-cased display_name.

The tests were also renamed to align better with the purpose of the
test rather than the name of the fixture used.

Use user factories for page_locale integration tests.

Use user factories for user_block integration tests.

Use user factories for user_roles integration tests.

Use user factory for user_creation test.

Use user factory for application_helper tests.

Use user factory for note_helper tests.

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

Improve trace#api_data tests

The api_data action doesn't actually take the username as an
arguement and should be modelled on api_read rather than the
actions that handle web views.

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

Remove redundant tests for deleted traces, and rearrange comments and factory calls for consistency.

Use user factory for user_roles_helper test

Create a super_user factory.

Create a fresh user when requesting non-existent traces.

Use user factory for message model tests.

Take the user from the node or changeset, rather than referring to the fixture.

Use user factory for note model test.

Use user factory for oauth_token model test.

Use user factory for user_preference model tests.

Use user factory for trace model tests.

Use user factory for trace_controller tests

Remove fake qqq locale

Localisation updates from https://translatewiki.net.

Do more preloading in browse controller methods

Fixes #1476

Fix accidental change in comments

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

Detect a timeout encapsulated in ActionView::Template::Error

Fixes #1476

Use user factory for swf_controller tests.

Use user factory for message_controller tests.

Use user factory for user_roles_controller tests.

Fix hiding of sidebar loading indicator

Fixes #1475

Remove fake qqq locale

Localisation updates from https://translatewiki.net.

Remove inline styles

Use users factory for user_preferences_controller tests.

Use users factory for user_blocks_controller tests.

Use users factory for redactions_controller tests.

Use local api to export map.osm

Closes #1282

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

Update HSTS to publish a max-age=0 to disable it

Silence remaining jshint warnings

Don't try and modify policy if we don't have one

Update Potlatch 2 to 2.4-46-gea38865 build

Fix some jshint warnings

Improve the content security policy

Remove inline javascript from page header

Remove inline javascript from id views

Correct name of zoom argument for Potlatch 2

Fixes #1471

Remove inline javascript from potlatch2 view

Remove inline javascript from potlatch view

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

Remove inline javascript from user views

Added new I18N string: notifier.friend_notification.hi

Included greeting string in changeset_comment_notification email

Revert "Update iD frame parameters if the parent URL changes"

This reverts commit a6fa8cc273017c2673d6d76dae7f5ac811c2027d.

Update iD frame parameters if the parent URL changes

Fixes #1468

Remove fake qqq locale

Localisation updates from https://translatewiki.net.

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

Externalize message about missing OAuth capabilities/permissions and make it less technical

Add support for Content-Security-Policy

Currently this is report only, and disabled unless a report URL has
been set in the application configuration.

Remove explicit context menu width

Firefox and Chromium at least seem to autosize correctly based
on the actual text in use for a given locale.

Rename 'clickable' to 'interactive' per leaflet 1.x change

Update to iD v2.1.3

Revert to nokogumbo 1.4.9

https://github.com/rubys/nokogumbo/issues/50

Update to rails 4.2.8

Blacklist VWorld imagery

Fixes #1464

User factories for browse, oauth_clients and site controller tests.

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

Remove fake qqq locale

Localisation updates from https://translatewiki.net.

User factory associations for all factories.

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.