Skip to content

Commit

Permalink
Remove “paratest:run” command
Browse files Browse the repository at this point in the history
  • Loading branch information
alexislefebvre committed Feb 9, 2020
1 parent 6024b27 commit a09fb5c
Show file tree
Hide file tree
Showing 13 changed files with 9 additions and 252 deletions.
9 changes: 7 additions & 2 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,6 +1,11 @@
# Changelog

## 3.0 [TBA]
## 4.0

### Removed
* Drop support of `paratest:run`

## 3.0
This new major version introduces a number of breaking changes; see the [upgrade guide](UPGRADE-3.0.md) for more details.

### Added
Expand All @@ -14,7 +19,7 @@ This new major version introduces a number of breaking changes; see the [upgrade
### Removed
* Removed fixtures loading in favor of https://github.com/liip/LiipTestFixturesBundle

## 2.0 [TBA]
## 2.0
This new major version introduces a number of breaking changes; see the [upgrade guide](UPGRADE-2.0.md) for more details.

### Added
Expand Down
1 change: 0 additions & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,6 @@ Documentation
* [Caveats](doc/caveats.md)

To run tests in parallel:
* [Paratest](doc/paratest.md)
* [Fastest](doc/fastest.md)

[Travis Master]: https://travis-ci.org/liip/LiipFunctionalTestBundle
Expand Down
3 changes: 1 addition & 2 deletions composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,6 @@
"symfony/framework-bundle": "4.3.0"
},
"suggest": {
"brianium/paratest": "Required when using paratest to parallelize tests",
"liip/test-fixtures-bundle": "Efficient loading of Doctrine fixtures in functional test-cases for Symfony applications"
},
"autoload": {
Expand All @@ -51,7 +50,7 @@
},
"extra": {
"branch-alias": {
"dev-master": "3.x-dev"
"dev-master": "4.x-dev"
}
}
}
2 changes: 1 addition & 1 deletion doc/installation.md
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ Installation
following command to download the latest stable version of this bundle:

```bash
$ composer require --dev liip/functional-test-bundle:^3.0.0
$ composer require --dev liip/functional-test-bundle:^4.0.0
```

This command requires you to have Composer installed globally, as explained
Expand Down
65 changes: 0 additions & 65 deletions doc/paratest.md

This file was deleted.

89 changes: 0 additions & 89 deletions src/Command/RunParatestCommand.php

This file was deleted.

11 changes: 0 additions & 11 deletions src/DependencyInjection/Configuration.php
Original file line number Diff line number Diff line change
Expand Up @@ -58,17 +58,6 @@ public function getConfigTreeBuilder(): TreeBuilder
->end()
->end()
->end()
->arrayNode('paratest')
->addDefaultsIfNotSet()
->children()
->scalarNode('process')
->defaultValue(5)
->end()
->scalarNode('phpunit')
->defaultValue('./bin/phpunit')
->end()
->end()
->end()
->end()
;

Expand Down
1 change: 0 additions & 1 deletion src/DependencyInjection/LiipFunctionalTestExtension.php
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,6 @@ public function load(array $configs, ContainerBuilder $container): void
$config = $this->processConfiguration(new Configuration(), $configs);

$loader = new XmlFileLoader($container, new FileLocator(__DIR__.'/../Resources/config'));
$loader->load('commands.xml');
$loader->load('functional_test.xml');

if (interface_exists('Symfony\Component\Validator\Validator\ValidatorInterface')) {
Expand Down
11 changes: 0 additions & 11 deletions src/Resources/config/commands.xml

This file was deleted.

3 changes: 0 additions & 3 deletions tests/AppConfig/config.yml
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,3 @@ liip_functional_test:
authentication:
username: "foobar"
password: "12341234"
paratest:
process: 3
phpunit: 'vendor/bin/phpunit'
62 changes: 0 additions & 62 deletions tests/Command/ParatestCommandTest.php

This file was deleted.

2 changes: 0 additions & 2 deletions tests/DependencyInjection/ConfigurationConfigTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -44,8 +44,6 @@ public function parametersProvider(): array
['query.max_query_count', 1],
['authentication.username', 'foobar'],
['authentication.password', '12341234'],
['paratest.process', 3],
['paratest.phpunit', 'vendor/bin/phpunit'],
];
}
}
2 changes: 0 additions & 2 deletions tests/DependencyInjection/ConfigurationTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -60,8 +60,6 @@ public function parametersProvider(): array
['query.max_query_count', null],
['authentication.username', ''],
['authentication.password', ''],
['paratest.process', 5],
['paratest.phpunit', './bin/phpunit'],
];
}
}

0 comments on commit a09fb5c

Please sign in to comment.