[DependencyInjection] Use more clear message when unused environment variables detected
This commit is contained in:
parent
17deeaf42e
commit
6dbdb1b750
@ -166,7 +166,7 @@ class PhpDumper extends Dumper
|
||||
}
|
||||
}
|
||||
if ($unusedEnvs) {
|
||||
throw new EnvParameterException($unusedEnvs);
|
||||
throw new EnvParameterException($unusedEnvs, null, 'Environment variables "%s" are never used. Please, check your container\'s configuration.');
|
||||
}
|
||||
|
||||
return $code;
|
||||
|
@ -18,8 +18,8 @@ namespace Symfony\Component\DependencyInjection\Exception;
|
||||
*/
|
||||
class EnvParameterException extends InvalidArgumentException
|
||||
{
|
||||
public function __construct(array $usedEnvs, \Exception $previous = null)
|
||||
public function __construct(array $envs, \Exception $previous = null, $message = 'Incompatible use of dynamic environment variables "%s" found in parameters.')
|
||||
{
|
||||
parent::__construct(sprintf('Incompatible use of dynamic environment variables "%s" found in parameters.', implode('", "', $usedEnvs)), 0, $previous);
|
||||
parent::__construct(sprintf($message, implode('", "', $envs)), 0, $previous);
|
||||
}
|
||||
}
|
||||
|
@ -302,7 +302,7 @@ class PhpDumperTest extends TestCase
|
||||
|
||||
/**
|
||||
* @expectedException \Symfony\Component\DependencyInjection\Exception\EnvParameterException
|
||||
* @expectedExceptionMessage Incompatible use of dynamic environment variables "FOO" found in parameters.
|
||||
* @expectedExceptionMessage Environment variables "FOO" are never used. Please, check your container's configuration.
|
||||
*/
|
||||
public function testUnusedEnvParameter()
|
||||
{
|
||||
|
Reference in New Issue
Block a user