Refactor deletion tests in way_controller_test to use factories.

Refactor some way_controller tests to use factories.

Refactor some amf_controller tests to use factories.

Use factories for all factory associations.

Create an old_way_node factory, and start using it for tests.

Create an old_way factory, and start using it for tests.

Create an old_relation_member factory, and start using it for tests.

Create an old_relation factory, and start using it for tests.

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

Move list of containing relations to top of browse views

Moves the list of containing relations to above the list of
members for ways and relations - nodes have no equivalent list
so no change is needed there.

Fixes #794
Closes #1501

Convert the node_controller update tests to use factories.

Convert the duplicate_tags node controller test to use factories.

Localisation updates from https://translatewiki.net.

Credit the whole of UCL, rather than just the Bartlett / VR centre.

Localisation updates from https://translatewiki.net.

Localisation updates from https://translatewiki.net.

Update Potlatch 2 to 2.5-7-gbd74430 build

Remember user's choice of directions engine

Localisation updates from https://translatewiki.net.

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

Fix indentation in email template

The plain text templates had nice indentation in the source however that
got reflected in the output, which was visibly and needlessly indented.

Closes #1512

Localisation updates from https://translatewiki.net.

Use node and relation_member factories for testing containing_relations and containing_relation_members

Localisation updates from https://translatewiki.net.

Convert more node fixtures to factories

Add the closed trait to changeset factories.

Add factories for relations, relation_members, ways and way_nodes

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

Update mapzen directions credit link

Rather than pointing at a redirect, point at the target of the
redirect. Refs #1388

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

Fix tests that referred to node lat and lons

Format latitudes and longitudes as %7f in XML output

This matches the double formatting in XML from cgimap. Fixes #341.

Use node factories for browse_controller tests.

For deleted nodes created via factories, make sure the most recent old_node is also deleted

Use factories for nodes in browse_helper tests.

Add a with_history trait for node factories, to automatically create corresponding old_nodes

Localisation updates from https://translatewiki.net.

Localisation updates from https://translatewiki.net.

Fix typo

Closes #1502

Update Potlatch 2 to 2.5 build

Localisation updates from https://translatewiki.net.

Add explicit tests for lat/lon methods on node models

Use changeset factory for association in node factory

Set canonical port based on protocol

Remove various osm.org email addresses from example configuration

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

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

Make the publisher URL configurable, with no default

Add server_protocol configuration option

Use node and old_node factories for redaction model tests.

Use canonical-rails gem to generate canonical URLs

Create a node factory, and use it for most node model tests

Set server_url in the test environment

Comment out server_url in the default configuration

Add a canonical URL to the head

Use old_node factory for OldNode model tests.

The valid/invalid tests were simplified, since there is little to be
gained from inserting (invalid) nodes into the database and comparing
the node with another copy of itself.

Rework the changeset helper tests to use factories.

Use changeset factory for factory associations

One test needed updating since it assumed the comments were being
created on a particular changeset.

Create a changeset factory, and use it for model tests.

Remove the api_fixtures call from changeset model tests, since they do not need the fixtures.

Localisation updates from https://translatewiki.net.

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

Remove the link between the redaction owner and the user doing the redaction in tests.

Localisation updates from https://translatewiki.net.

Use redactions factory for old_way controller tests

Adjust rubocop threshold

Use redactions factory for old_node controller tests.

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

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.

Refactor the oauth integration test to avoid using user fixtures.

Use user factory for user_terms_seen integration test.

Use user factories for some changeset_controller tests.

Others are tightly interwoven with the fixtures and will be tackled seperately.

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.