[Tests-Only] Refactor etag tests

[phil-davis]

Description

Add test steps for theseEtagsShouldNotHaveChanged and theseEtagsShouldHaveChanged to do checks of a table of etags and report a message about all the etags that do not pass the test.

Reduce the number of example scenarios in the etag tests by using the new steps to check for etag changes in multiple elements.

Related Issue

How Has This Been Tested?

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:
• Changelog item, see TEMPLATE

[codecov]

Codecov Report

Merging #37980 into master will decrease coverage by 0.00%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master   #37980      +/-   ##
============================================
- Coverage     64.70%   64.70%   -0.01%     
  Complexity    19445    19445              
============================================
  Files          1286     1286              
  Lines         76032    76032              
  Branches       1336     1336              
============================================
- Hits          49196    49195       -1     
- Misses        26442    26443       +1     
  Partials        394      394              

Flag	Coverage Δ	Complexity Δ
#javascript	54.06% <ø> (ø)	0.00 <ø> (ø)
#phpunit	65.87% <ø> (-0.01%)	19445.00 <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ	Complexity Δ
apps/files_external/lib/Lib/Storage/SFTP.php	56.52% <0.00%> (-0.49%)	0.00% <0.00%> (ø%)

---

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 3c7db53...0bfbb3e. Read the comment docs.

[codecov]

Codecov Report

Merging #37980 into master will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master   #37980   +/-   ##
=========================================
  Coverage     64.70%   64.70%           
  Complexity    19445    19445           
=========================================
  Files          1286     1286           
  Lines         76032    76032           
  Branches       1336     1336           
=========================================
  Hits          49196    49196           
  Misses        26442    26442           
  Partials        394      394           

Flag	Coverage Δ	Complexity Δ
#javascript	54.06% <ø> (ø)	0.00 <ø> (ø)
#phpunit	65.87% <ø> (ø)	19445.00 <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out 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 3c7db53...d573ec8. Read the comment docs.