Skip to content

Commit

Permalink
tests: only ignore external deprecation warnings
Browse files Browse the repository at this point in the history
and error out on internal ones
  • Loading branch information
lazka committed Jan 11, 2022
1 parent bbc5fe3 commit ca1a58c
Show file tree
Hide file tree
Showing 4 changed files with 4 additions and 4 deletions.
2 changes: 1 addition & 1 deletion phpunit.xml.dist
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@
<ini name="error_reporting" value="-1"/>
<server name="APP_ENV" value="test" force="true"/>
<server name="SHELL_VERBOSITY" value="-1"/>
<server name="SYMFONY_DEPRECATIONS_HELPER" value="weak"/>
<server name="SYMFONY_DEPRECATIONS_HELPER" value='max[direct]=0&amp;quiet[]=indirect'/>
<server name="KERNEL_CLASS" value="Dbp\Relay\TemplateBundle\Tests\Kernel"/>
</php>
<testsuites>
Expand Down
2 changes: 1 addition & 1 deletion src/Command/TestCommand.php
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ protected function configure()
$this->setDescription('Hey there!');
}

protected function execute(InputInterface $input, OutputInterface $output)
protected function execute(InputInterface $input, OutputInterface $output): int
{
$argument = $input->getArgument('argument');
$output->writeln($argument);
Expand Down
2 changes: 1 addition & 1 deletion src/DataPersister/PlaceDataPersister.php
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@ public function supports($data): bool
return $data instanceof Place;
}

public function persist($data)
public function persist($data): void
{
// TODO
}
Expand Down
2 changes: 1 addition & 1 deletion src/DependencyInjection/Configuration.php
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@

class Configuration implements ConfigurationInterface
{
public function getConfigTreeBuilder()
public function getConfigTreeBuilder(): TreeBuilder
{
$treeBuilder = new TreeBuilder('dbp_relay_template');

Expand Down

0 comments on commit ca1a58c

Please sign in to comment.