Laravel调用Facade::shouldReceive()模拟时报方法重复声明错误

这个错误是错误实现Laravel Facade导致的：

1. Laravel的Facade基类已经引入了Mockery的测试特性，自带shouldReceive()方法用于测试场景下的模拟调用。
2. 你把业务属性、非静态业务方法直接写在了自定义Facade类中，违反了Facade的设计约定：Facade类仅作为服务容器中绑定实例的静态代理入口，不承载任何实际业务逻辑。当Mockery尝试生成Facade的模拟类时，会和基类自带的shouldReceive方法产生冲突，触发方法重复声明的致命错误。

按以下步骤调整代码即可解决：

• 精简Facade类，移除所有业务属性、业务方法，仅保留必须的getFacadeAccessor()方法，修正后的Facade代码如下：

namespace App\Facades;

use Illuminate\Support\Facades\Facade;

class Twilio extends Facade
{
    protected static function getFacadeAccessor()
    {
        return 'twilio';
    }
}

• 新建独立的Twilio服务类，把原来写在Facade里的$adminCli、$http属性和AsAdmin()方法全部迁移到这个服务类中：

namespace App\Services;

use Illuminate\Http\Client\PendingRequest;
use Illuminate\Support\Facades\Http;
use Twilio\Rest\Client;

class TwilioService
{
    protected Client $adminCli;
    protected PendingRequest $http;

    public function AsAdmin()
    {
        $username = config('twilio.account_sid');
        $password = config('twilio.auth_token');

        $this->adminCli = new Client($username, $password);
        $this->http = Http::withBasicAuth($username, $password);

        return $this;
    }

    // 其余Twilio相关业务逻辑统一写在这个类中
}

• 调整服务提供者中的绑定逻辑，将容器标识twilio绑定到上述服务类的实例：

$this->app->bind('twilio', function () {
    return new \App\Services\TwilioService();
});

调整完成后原有测试代码不需要修改，重新运行测试就不会再触发该致命错误。

• 永远不要在Facade类中编写实例属性、非静态业务方法：Facade的静态调用会通过魔术方法自动转发到容器中绑定的实际服务实例，写在Facade类里的实例代码不会被正常调用流程执行，还会破坏框架内置的模拟、转发逻辑。
• 如果需要IDE对Facade静态方法的代码提示，只需要在Facade类头部通过PHPDoc注解声明方法类型即可，不要写入实际实现。

内容的提问来源于stack exchange，提问作者TylerMills