->willReturn(...) would make more sense here

Problem/Motivation

I've stumbled over the following inspection in the PHPStorm plugin "Php Inspections ​(EA Extended)​" called "promote ->willReturn*".

This points out PHPUnit MockObject ->expects()->will() which have $this->returnValue (and friends) as a stub and replaces them with ->expects()->willReturn() (and friends).

As seen in #3220128: [META] ->willReturn(...) would make more sense here (and friends) getting all of this committed in one go is way too much.

This sub-issue is about getting all ->willReturn(...) in, except ->willReturn() itself.

Remaining tasks

• Patch/MR
• Review
• Commit
• Rejoice