return unnecessary

Author: codemasher

src/Core/OAuth2CSRFTokenTrait.php

@@ -23,10 +23,10 @@ trait OAuth2CSRFTokenTrait{
 	/**
 	 * @param string|null $state
 	 *
-	 * @return \chillerlan\OAuth\Core\OAuth2Interface
+	 * @return void
 	 * @throws \chillerlan\OAuth\Core\ProviderException
 	 */
-	protected function checkState(string $state = null):OAuth2Interface{
+	protected function checkState(string $state = null):void{
 
 		if(empty($state) || !$this->storage->hasCSRFState($this->serviceName)){
 			throw new ProviderException('invalid state for '.$this->serviceName);
@@ -38,8 +38,6 @@ protected function checkState(string $state = null):OAuth2Interface{
 			throw new ProviderException('invalid CSRF state: '.$this->serviceName.' '.$state);
 		}
 
-		/** @noinspection PhpIncompatibleReturnTypeInspection */
-		return $this;
 	}
 
 	/**

tests/API/APITestAbstract.php

@@ -60,6 +60,11 @@ abstract class APITestAbstract extends TestCase{
 	 */
 	protected $dotEnv;
 
+	/**
+	 * @var float
+	 */
+	protected $requestDelay = 0.25;
+
 	protected function setUp(){
 		ini_set('date.timezone', 'Europe/Amsterdam');
 
@@ -80,7 +85,7 @@ protected function setUp(){
 			'ca_info'          => $this->CFG.'/cacert.pem',
 			'userAgent'        => 'chillerlanPhpOAuth/3.0.0 +https://github.com/chillerlan/php-oauth',
 			// testHTTPClient
-			'sleep'            => 0.25,
+			'sleep'            => $this->requestDelay,
 			// logger
 			'minLogLevel'      => 'debug',
 		];