[stable10] Fix drone webUIMasterKeyType so it runs the tests

[phil-davis]

Description

I noticed that the last job in drone, webUIMasterKeyType, ran and exited with success but never actually executed the expected acceptance tests

Somewhere in backports or... the line:

      TEST_SUITE: webui

is now:

      TEST_SUITE: selenium

and that did not get updated for the sstable10-only webUIMasterKeyType encryption suite.

1. fixup the drone job parameters for webUIMasterKeyType
2. move the other encryption suite webUIUserKeysType down the end also, so they are close together (easier to find)
3. comment all the encryption suites with "This suite is part of the encryption app in later core versions" (consistent with the way it has been done for the user_management app)

Note: this is stable10 only. encryption is in a separate repo for core master. So no forward-back-port needed.

Motivation and Context

Actually run tests in CI ;)

How Has This Been Tested?

Observe the run of CI

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Database schema changes (next release will require increase of minor version instead of patch)
• Breaking change (fix or feature that would cause existing functionality to change)
• Technical debt
• Tests only (no source changes)

Checklist:

• Code changes
• Unit tests added
• Acceptance tests added
• Documentation ticket raised:

[phil-davis]

https://drone.owncloud.com/owncloud/core/18826/1105

2 scenarios (2 passed)
19 steps (19 passed)
6m22.96s (21.19Mb)

[individual-it]

Don't we need?

CONFIGURE_ENCRYPTION: true
ENCRYPTION_TYPE: masterkey

[phil-davis]

Don't we need?

CONFIGURE_ENCRYPTION: true
ENCRYPTION_TYPE: masterkey

That is already in the job parameters, further down.

[phil-davis]

Note: webUIUserKeysType runs fine without setting CONFIGURE_ENCRYPTION or ENCRYPTION_TYPE
The feature file sets up the encryption type etc.

[codecov]

Codecov Report

Merging #35659 into stable10 will decrease coverage by 18.49%.
The diff coverage is n/a.

@@              Coverage Diff               @@
##             stable10   #35659      +/-   ##
==============================================
- Coverage       64.02%   45.52%   -18.5%     
==============================================
  Files            1294      116    -1178     
  Lines           77280    11554   -65726     
  Branches         1301     1301              
==============================================
- Hits            49475     5260   -44215     
+ Misses          27420     5909   -21511     
  Partials          385      385

Flag	Coverage Δ	Complexity Δ
#javascript	53.85% <ø> (ø)	0 <ø> (ø)	⬇️
#phpunit	30.81% <ø> (-34.29%)	0 <ø> (-20225)

Impacted Files	Coverage Δ	Complexity Δ
lib/private/Files/Storage/DAV.php	59.45% <0%> (-21.64%)	0% <0%> (ø)
apps/updatenotification/templates/admin.php
lib/private/Encryption/Keys/Storage.php
lib/private/App/CodeChecker/NodeVisitor.php
lib/private/RedisFactory.php
apps/dav/lib/Avatars/AvatarNode.php
...s/dav/appinfo/Migrations/Version20170202213905.php
apps/dav/lib/Upload/ChunkLocationProvider.php
apps/files/lib/AppInfo/Application.php
apps/systemtags/list.php
... and 1169 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6b49720...7a4e050. Read the comment docs.

[codecov]

Codecov Report

Merging #35659 into stable10 will not change coverage.
The diff coverage is n/a.

@@             Coverage Diff             @@
##             stable10   #35659   +/-   ##
===========================================
  Coverage       64.83%   64.83%           
  Complexity      20225    20225           
===========================================
  Files            1294     1294           
  Lines           77315    77315           
  Branches         1301     1301           
===========================================
  Hits            50126    50126           
  Misses          26804    26804           
  Partials          385      385

Flag	Coverage Δ	Complexity Δ
#javascript	53.85% <ø> (ø)	0 <ø> (ø)	⬇️
#phpunit	65.99% <ø> (ø)	20225 <ø> (ø)	⬇️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3dd5b2b...da3dd93. Read the comment docs.

[phil-davis]

General CI is improved now. I rebased, let's see if we get a good result.