Extend SharingApi test context to test uppercase to lowercase conversion

[micbar]

Description

Extend Acceptance Test Suite

Related Issue

• Needed for Add acceptance test for EE Ticket #3241 guests#325

Motivation and Context

We need to test the stability of the SharingAPI even if we feed it with uppercase/lowercase differences to the correctn UID

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)

[codecov]

Codecov Report

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

@@            Coverage Diff            @@
##             master   #35156   +/-   ##
=========================================
  Coverage     65.48%   65.48%           
  Complexity    18636    18636           
=========================================
  Files          1217     1217           
  Lines         70536    70536           
  Branches       1296     1296           
=========================================
  Hits          46194    46194           
  Misses        23965    23965           
  Partials        377      377

Flag	Coverage Δ	Complexity Δ
#javascript	53.5% <ø> (ø)	0 <ø> (ø)	⬇️
#phpunit	66.87% <ø> (ø)	18636 <ø> (ø)	⬇️

---

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 cc5f317...c3b664e. Read the comment docs.

[codecov]

Codecov Report

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

@@            Coverage Diff            @@
##             master   #35156   +/-   ##
=========================================
  Coverage     65.48%   65.48%           
  Complexity    18636    18636           
=========================================
  Files          1217     1217           
  Lines         70536    70536           
  Branches       1296     1296           
=========================================
  Hits          46194    46194           
  Misses        23965    23965           
  Partials        377      377

Flag	Coverage Δ	Complexity Δ
#javascript	53.5% <ø> (ø)	0 <ø> (ø)	⬇️
#phpunit	66.87% <ø> (ø)	18636 <ø> (ø)	⬇️

---

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 cc5f317...c3b664e. Read the comment docs.

[individual-it]

in owncloud/guests#325 you don't use that new step and I don't understand why creating a new function and not using userHasSharedFileWithUserUsingTheSharingApi() ?

[micbar]

@individual-it

1. I do not use it in the guest pr because the new method is not in daily-stable-10-qa.
   I intend to use it.

2. The other method throws an error if you use it with an uppercase UserID and the shareAPI returns the UserID converted to lowercase. We need a method which accepts the conversion of the UserID.

[individual-it]

so we could use \strtolower every time. meaning put it into userHasSharedFileWithUserUsingTheSharingApi() as it will not make a difference for the other tests and we can have one function less

[individual-it]

closing in favour of #35342