Rename friend to friendship in notifier

Followup to #2269

Move make_friend and remove_friend to friendships controller

Merge remote-tracking branch 'upstream/pull/2639'

Merge remote-tracking branch 'upstream/pull/2638'

Merge remote-tracking branch 'upstream/pull/2637'

Merge remote-tracking branch 'upstream/pull/2640'

Move friendship related tests to separate file

Convert lib files to model concerns

Remove some unused scripts

Merge pull request #2624 from francois2metz/diary-entry

Add link to my diary in the diary index

Use rails 6.0 framework defaults

Keep the old autoloader and utf-8 forms as they were, for now.

Remove the framework defaults initializer for 5.2

We're already using these defaults, as per 6a25ee9ec962bc10997a4c62a6246873130d41a1
so we don't need to keep this file

Remove ineffective query cache disable initializer

This doesn't have any effect, since there is no middleware by that name.

Localisation updates from https://translatewiki.net.

Update bundle

Localisation updates from https://translatewiki.net.

Update bundle

Update to rails 6.0.3.1

Localisation updates from https://translatewiki.net.

Merge remote-tracking branch 'upstream/pull/2626'

Remove unused test helper

Refactor user controller tests

Because we don't have direct access to the session and cookies, some
of the tests need more setup e.g. by moving through the signup process
to get the session tokens into the right place for the confirmation
steps to work.

Refactor api traces_controller tests

Don't try and resize images that aren't resizable

Attach resized images to notification emails

Fixes #2625

Fix new rubocop warnings

Update bundle

Add link to my diary in the diary index.

Refactor the link to create a new entry diary.

Localisation updates from https://translatewiki.net.

Refactor users_controller_test and user_preferences_controller_test

Fix issue with iD embedded document sometimes showing scrollbars

Localisation updates from https://translatewiki.net.

Update to rails 6.0.3

Refactor notes_controller_test

Add test for browsing a note closed by a deleted user

Merge remote-tracking branch 'upstream/pull/2613'

More controller test refactoring

Look at all note comments to find the close event

Fixes #2612

Refactor permissions_controller_test

Refactor versions_controller_test

Refactor tracepoints_controller_test

Remove unused function

Refactor map_controller_test

Switch oauth integration tests to use new shared helper

Refactor changeset comments API controller tests

Add helper methods for making OAuth signed requests

Refactor the AMF controller API tests

Localisation updates from https://translatewiki.net.

Localisation updates from https://translatewiki.net.

Update bundle

Localisation updates from https://translatewiki.net.

Merge remote-tracking branch 'upstream/pull/2605'

Refactor changesets_controller_test

Introduces some route names which need to be unique from the non-api versions

Refactor more controller tests

Merge remote-tracking branch 'upstream/pull/2601'

Refactor traces controller tests

Move document-level styling for iD to its stylesheet

Fix issue where iD would not take the full height of its iframe

Refactor more controller tests

Localisation updates from https://translatewiki.net.

Refactor more controller tests

Refactor more controller tests

Update to iD v2.17.3

Fix deprecation warning

Update bundle

Update vagrant provision script for switch to selenium

Merge remote-tracking branch 'upstream/pull/2598'

Merge remote-tracking branch 'upstream/pull/2600'

Refactor some more controller tests

Merge remote-tracking branch 'upstream/pull/2599'

Remove tempfile patch for trace data

Effectively reverts c0d2ad40c30e5a0837b6012d7b9067d69ce41dd0

This patch is no longer required, since we only use send_data in
combination with Tempfile.read and that all works fine.

Switch to using selenium for system tests

Merge remote-tracking branch 'upstream/pull/2596'

Merge remote-tracking branch 'upstream/pull/2597'

Use Open3.capture2 instead of backticks, to avoid command line injection risks

In this situation, trace_name can be trivially checked as legitimate, but this
removes any lingering risks from interpolating into a command line instead of
passing parameters explicitly.

Refs #2229

Redirect to the error page for non-existent issues

Send meta information with error pages

Refactor site controller tests to inherit from ActionDispatch::IntegrationTest

This is the modern way of writing controller tests, since it uses the
rails middleware (among other things) and leads to more realistic tests.

Refs #2563

Remove last traces of konacha

Localisation updates from https://translatewiki.net.

Fix new rubocop warnings

Enable new cops by default

Update bundle

Localisation updates from https://translatewiki.net.

Cache OpenID provider names on a per-locale basis

Fixes #2591

Localise formatting of point count for traces

Fixes #2592

Update bundle

Merge remote-tracking branch 'upstream/pull/2431'

Fix rubocop-minitest warnings

Sort cops

Fix erblint

Merge pull request #2547 from bezdna/mobile-layout

Trigger small screen layout based on the width of the screen, rather than the width of the menus.

Merge remote-tracking branch 'upstream/pull/2579'

Merge remote-tracking branch 'upstream/pull/2590'

Add rubocop-minitest

Localisation updates from https://translatewiki.net.

Add tile.openstreetmap.org to security policy

Update leaflet.osm plugin

Update leaflet.osm plugin