Misleading error code used

[britsta]

Readiness checklist

• I added/updated tests.
• I ensured that the PR title is good enough for the changelog.
• I labeled the PR.
• I self-reviewed the PR.

Description

The invalidAcceptParameters error code should not be used if the accept parameters when trying to accept a RequestItem are valid but the RequestItem cannot be accepted for other reasons. Otherwise, the error code gives the impression that the recipient can change the fact that the RequestItem is not acceptable by using other accept parameters, which is not the case.

The invalidRequestItem error code may be appropriate instead if the RequestItem contains errors that should have been prevented on the sender side anyway. In other cases, more specific error messages could be used.

[codecov]

Codecov Report

Attention: Patch coverage is 60.00000% with 2 lines in your changes missing coverage. Please review.

Files with missing lines	Patch %	Lines
...rship/TransferFileOwnershipRequestItemProcessor.ts	60.00%	2 Missing ⚠️

Files with missing lines	Coverage Δ
...rship/TransferFileOwnershipRequestItemProcessor.ts	88.23% <60.00%> (ø)

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[jkoenig134]

great! :)