Move make_friend and remove_friend to friendships controller

Move friendship related tests to separate file

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

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

Remove unsued bootstrap components from Vendorfile

Remove arabic strings from en-GB locale

Add blob to frame-src in CSP for iD

Fixes #2582
Closes #2583

Localisation updates from https://translatewiki.net.

Update bundle

Ensure that urls are only valid if the entire string is a url

This replaces our homegrown regexps (that didn't quite work) with
ruby built-in regexps, and uses the termination anchor to ensure
that the entire string, not just the first part, is validated.

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

Remove custom table styling

This removes the custom table styling code, and move to using bootstrap table styling for almost all remaining tables.

Update leaflet plugins

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

Fix typo in comment

Closes #2576

Update leaflet to 1.6.0

Fix new rubocop warnings