Enhancement: Synchronize with ergebnis/php-package-template
#2719
Annotations
11 warnings
|
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
|
|
Run mutation tests with Xdebug and infection/infection:
src/Command/NormalizeCommand.php#L161
Escaped Mutant for Mutator "NotIdentical":
--- Original
+++ New
@@ @@
$io->writeError(\sprintf('<error>%s</error>', $exception->getMessage()));
return 1;
}
- if (null !== $indentFromExtra) {
+ if (null === $indentFromExtra) {
$indent = $indentFromExtra;
}
if (null !== $indentFromInput && null !== $indentFromExtra) {
|
|
Run mutation tests with Xdebug and infection/infection:
src/Command/NormalizeCommand.php#L166
Escaped Mutant for Mutator "LogicalAnd":
--- Original
+++ New
@@ @@
if (null !== $indentFromExtra) {
$indent = $indentFromExtra;
}
- if (null !== $indentFromInput && null !== $indentFromExtra) {
+ if (null !== $indentFromInput || null !== $indentFromExtra) {
$io->write('<warning>Configuration provided via options and composer extra. Using configuration from composer extra.</warning>');
}
if (false === $input->getOption('dry-run') && !\is_writable($composerFile)) {
|
|
Run mutation tests with Xdebug and infection/infection:
src/Command/NormalizeCommand.php#L173
Escaped Mutant for Mutator "FalseValue":
--- Original
+++ New
@@ @@
if (null !== $indentFromInput && null !== $indentFromExtra) {
$io->write('<warning>Configuration provided via options and composer extra. Using configuration from composer extra.</warning>');
}
- if (false === $input->getOption('dry-run') && !\is_writable($composerFile)) {
+ if (true === $input->getOption('dry-run') && !\is_writable($composerFile)) {
$io->writeError(\sprintf('<error>%s is not writable.</error>', $composerFile));
return 1;
}
|
|
Run mutation tests with Xdebug and infection/infection:
src/Command/NormalizeCommand.php#L173
Escaped Mutant for Mutator "Identical":
--- Original
+++ New
@@ @@
if (null !== $indentFromInput && null !== $indentFromExtra) {
$io->write('<warning>Configuration provided via options and composer extra. Using configuration from composer extra.</warning>');
}
- if (false === $input->getOption('dry-run') && !\is_writable($composerFile)) {
+ if (false !== $input->getOption('dry-run') && !\is_writable($composerFile)) {
$io->writeError(\sprintf('<error>%s is not writable.</error>', $composerFile));
return 1;
}
|
|
Run mutation tests with Xdebug and infection/infection:
src/Command/NormalizeCommand.php#L295
Escaped Mutant for Mutator "ArrayItemRemoval":
--- Original
+++ New
@@ @@
if (true === $input->getOption('diff') || true === $input->getOption('dry-run')) {
$io->writeError(\sprintf('<error>%s is not normalized.</error>', $composerFile));
$diff = $this->differ->diff($json->encoded(), $normalized->encoded());
- $io->write(['', '<fg=yellow>---------- begin diff ----------</>', self::formatDiff($diff), '<fg=yellow>----------- end diff -----------</>', '']);
+ $io->write(['<fg=yellow>---------- begin diff ----------</>', self::formatDiff($diff), '<fg=yellow>----------- end diff -----------</>', '']);
}
if (true === $input->getOption('dry-run')) {
return 1;
|
|
Run mutation tests with Xdebug and infection/infection:
src/Command/NormalizeCommand.php#L325
Escaped Mutant for Mutator "MethodCallRemoval":
--- Original
+++ New
@@ @@
if (true === $input->getOption('no-update-lock') || !$locker->isLocked()) {
return 0;
}
- $io->write('<info>Updating lock file.</info>');
+
$application = new Application();
$application->setAutoExit(false);
return self::updateLockerInWorkingDirectory($application, $input, $output, \dirname($composerFile));
|
|
Run mutation tests with Xdebug and infection/infection:
src/Command/NormalizeCommand.php#L369
Escaped Mutant for Mutator "CastInt":
--- Original
+++ New
@@ @@
if (null === $indentStyle) {
throw new \RuntimeException(\sprintf('When using the indent-size option, an indent style (one of "%s") needs to be specified using the indent-style option.', \implode('", "', \array_keys(Normalizer\Format\Indent::CHARACTERS))));
}
- if ((string) (int) $indentSize !== $indentSize || 1 > $indentSize) {
+ if ((string) $indentSize !== $indentSize || 1 > $indentSize) {
throw new \RuntimeException(\sprintf('Indent size needs to be an integer greater than 0, but "%s" is not.', $indentSize));
}
if (!\array_key_exists($indentStyle, Normalizer\Format\Indent::CHARACTERS)) {
|
|
Run mutation tests with Xdebug and infection/infection:
src/Command/NormalizeCommand.php#L448
Escaped Mutant for Mutator "GreaterThan":
--- Original
+++ New
@@ @@
if (!\is_int($indentSize)) {
throw new \RuntimeException(\sprintf('Indent size needs to be an integer, got %s instead.', \gettype($indentSize)));
}
- if (1 > $indentSize) {
+ if (1 >= $indentSize) {
throw new \RuntimeException(\sprintf('Indent size needs to be an integer greater than 0, but %d is not.', $indentSize));
}
$indentStyle = $configuration['indent-style'];
|
|
Run mutation tests with Xdebug and infection/infection:
src/Command/NormalizeCommand.php#L499
Escaped Mutant for Mutator "UnwrapArrayMap":
--- Original
+++ New
@@ @@
private static function formatDiff(string $diff) : string
{
$lines = \explode("\n", $diff);
- $formatted = \array_map(static function (string $line) : string {
- $replaced = \preg_replace(['/^(\\+.*)$/', '/^(-.*)$/'], ['<fg=green>$1</>', '<fg=red>$1</>'], $line);
- if (!\is_string($replaced)) {
- throw Exception\ShouldNotHappen::create();
- }
- return $replaced;
- }, $lines);
+ $formatted = $lines;
return \implode("\n", $formatted);
}
/**
|
|
Run mutation tests with Xdebug and infection/infection:
src/Command/NormalizeCommand.php#L501
Escaped Mutant for Mutator "ArrayItemRemoval":
--- Original
+++ New
@@ @@
{
$lines = \explode("\n", $diff);
$formatted = \array_map(static function (string $line) : string {
- $replaced = \preg_replace(['/^(\\+.*)$/', '/^(-.*)$/'], ['<fg=green>$1</>', '<fg=red>$1</>'], $line);
+ $replaced = \preg_replace(['/^(-.*)$/'], ['<fg=green>$1</>', '<fg=red>$1</>'], $line);
if (!\is_string($replaced)) {
throw Exception\ShouldNotHappen::create();
}
|
Loading