Fix PHPStan L2 error in traits

+++ b/core/modules/rest/tests/src/Functional/BasicAuthResourceTestTrait.php
@@ -23,7 +23,7 @@ trait BasicAuthResourceTestTrait {
-        'Authorization' => 'Basic ' . base64_encode($this->account->name->value . ':' . $this->account->passRaw),
+        'Authorization' => 'Basic ' . base64_encode($this->account->getAccountName()->value . ':' . $this->account->passRaw),

I think this is the cause of all the broken tests. getAccountName returns a string, so value isn't needed.

The patch also doesn't apply to 10.1.x anymore, fixed the issues but couldn't get the baseline to regenerate properly. I was getting a lot of new warnings added to the baseline

Self addressed the feedback in #14 and regenerated the baseline. Lets see how the tests go.

Believe I tested this one write

I applied the patch
Changed the level to 2
Staged everything
./core/scripts/dev/commit-code-check.sh --cached
Got no errors.

Let me know if I missed a step for these level 2 stuff.

Should this be postponed on 📌 Fix PHPStan L1 errors "Call to method getDefinitions()/getSortedDefinitions() on an unknown class Drupal\Core\Plugin\CategorizingPluginManagerTrait." Needs work , or should that part of the change here be removed, given it interferes with the fix there?

Oh good call. Yeah I think we should remove it from this issue. Thats only one part of this patch, so we can just remove it and regenerate the baseline

Looks like the problem isn't fixed yet.

Is there a better solution to it other than adding an @phpstan-ignore-next-line?

Manually applied parts of #15 that weren't already resolved in HEAD and pushed up a new MR. Hiding patches.

Changes look good, the only thing I noticed missing from the original patch were the changes to BasicAuthResourceTestTrait::getAuthenticationRequestOptions. Should they be included in this?

Other that that it looks good and is ready to be RTBC, just checking in on that one first.

Good spotting, I'm not sure why that one didn't make it initially. I generated the baseline at level 2, then updated that method and regenerated it. This removed a further 96 errors from the baseline, so I think that's worth it.

Seems feedback has been addressed.