Fix openFile method to handle error in acceptance tests

[dpakach]

Description

Modify the acceptance test method webUIFilesContext::openFile to handle errors that may occur while opening the file.

Related Issue

• Fixes <issue_link>

Motivation and Context

The method originally only assumes that the operation will be always successful. But sometimes it may fail and in that case the file is not opened. So the method needed modification to handle the situations when the user opens the file expecting to fail.

How Has This Been Tested?

🤖

Screenshots (if appropriate):

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:

Open tasks:

• Backport (if applicable set "backport-request" label and remove when the backport was done)

[dpakach]

Requires #35265 to be merged first.

[codecov]

Codecov Report

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

@@              Coverage Diff              @@
##             master   #35268       +/-   ##
=============================================
- Coverage     65.54%   48.87%   -16.67%     
=============================================
  Files          1218      109     -1109     
  Lines         70546    10560    -59986     
  Branches       1288     1288               
=============================================
- Hits          46236     5161    -41075     
+ Misses        23933     5022    -18911     
  Partials        377      377

Flag	Coverage Δ	Complexity Δ
#javascript	53.69% <ø> (ø)	0 <ø> (ø)	⬇️
#phpunit	38.17% <ø> (-28.73%)	0 <ø> (-18647)

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 1101 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 c0dc3cd...499573e. Read the comment docs.

[codecov]

Codecov Report

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

@@            Coverage Diff            @@
##             master   #35268   +/-   ##
=========================================
  Coverage     65.54%   65.54%           
  Complexity    18647    18647           
=========================================
  Files          1218     1218           
  Lines         70546    70546           
  Branches       1288     1288           
=========================================
  Hits          46236    46236           
  Misses        23933    23933           
  Partials        377      377

Flag	Coverage Δ	Complexity Δ
#javascript	53.69% <ø> (ø)	0 <ø> (ø)	⬇️
#phpunit	66.9% <ø> (ø)	18647 <ø> (ø)	⬇️

---

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 74ead9c...c9a689d. Read the comment docs.

[skshetry]

👍

[skshetry]

👍

[dpakach]

Backport on #35299