[tests-only] Adjust trashbinFilesFolders test setups

[phil-davis]

Description

PR #38541 adjusted the use of skeleton files. The files created in the Given steps need to be adjusted in some scenarios that use textfile2

Related Issue

• related to [QA] unexpected test failure  #38657

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

[sonarqubecloud]

Kudos, SonarCloud Quality Gate passed!

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
No Duplication information

[individual-it]

shouldn't And user "testtrashbin101" has deleted file "/textfile2.txt" have failed before this change?

[phil-davis]

shouldn't And user "testtrashbin101" has deleted file "/textfile2.txt" have failed before this change?

The file did not exist, so maybe the given step passes because actually the file does not exist at the end of the step. The code in WebDav.php userHasDeletedFile does:

		$this->theHTTPStatusCodeShouldBe(
			["204", "404"],
			"HTTP status code was not 204 or 404 while trying to $deleteText $fileOrFolder '$entry' for user '$user'"
		);

So it passes because the file does not exist (and never existed).

Maybe we should change that in a separate PR, so that we know when a Given step tries to delete a file that is already not existing? (I am trying that in #38660