Skip to content

Commit

Permalink
Make XML the default configuration format
Browse files Browse the repository at this point in the history
Signed-off-by: Luís Cobucci <[email protected]>
  • Loading branch information
lcobucci committed Feb 26, 2023
1 parent 1d1dc1d commit a5ec16c
Show file tree
Hide file tree
Showing 2 changed files with 12 additions and 10 deletions.
4 changes: 2 additions & 2 deletions src/Command/DetermineConfigurationFromFilesystem.php
Original file line number Diff line number Diff line change
Expand Up @@ -8,13 +8,13 @@
use Psl\Type;
use Roave\BackwardCompatibility\Configuration\Configuration;
use Roave\BackwardCompatibility\Configuration\ParseConfigurationFile;
use Roave\BackwardCompatibility\Configuration\ParseJsonConfigurationFile;
use Roave\BackwardCompatibility\Configuration\ParseXmlConfigurationFile;
use Symfony\Component\Console\Output\OutputInterface;

final class DetermineConfigurationFromFilesystem
{
public function __construct(
private readonly ParseConfigurationFile $parser = new ParseJsonConfigurationFile(),
private readonly ParseConfigurationFile $parser = new ParseXmlConfigurationFile(),
) {
}

Expand Down
18 changes: 10 additions & 8 deletions test/e2e/Command/AssertBackwardsCompatibleTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -30,12 +30,14 @@ final class AssertBackwardsCompatibleTest extends TestCase

JSON;

private const BASELINE_CONFIGURATION = <<<'JSON'
{
"baseline": ["#\\[BC\\] CHANGED: The parameter \\$a of TestArtifact\\\\TheClass\\#method()#"]
}

JSON;
private const BASELINE_CONFIGURATION = <<<'XML'
<?xml version="1.0" encoding="UTF-8" ?>
<roave-bc-check>
<baseline>
<ignoredRegex>#\[BC\] CHANGED: The parameter \$a of TestArtifact\\TheClass\#method.*#</ignoredRegex>
</baseline>
</roave-bc-check>
XML;

private const CLASS_VERSIONS = [
<<<'PHP'
Expand Down Expand Up @@ -273,15 +275,15 @@ public function testWillPickLatestTaggedVersionOnNoGivenFrom(): void

public function testWillAllowSpecifyingBaselineConfiguration(): void
{
File\write($this->sourcesRepository . '/.roave-backward-compatibility-check.json', self::BASELINE_CONFIGURATION);
File\write($this->sourcesRepository . '/.roave-backward-compatibility-check.xml', self::BASELINE_CONFIGURATION);

$output = Shell\execute(__DIR__ . '/../../../bin/roave-backward-compatibility-check', [
'--from=' . $this->versions[0],
'--to=' . $this->versions[1],
], $this->sourcesRepository, [], Shell\ErrorOutputBehavior::Append);

self::assertStringContainsString(
'.roave-backward-compatibility-check.json" as configuration file',
'.roave-backward-compatibility-check.xml" as configuration file',
$output,
);
}
Expand Down

0 comments on commit a5ec16c

Please sign in to comment.