Skip to content

Commit 3363d6d

Browse files
oliverkleeSam Tuke
authored andcommitted
[CLEANUP] Rename Tests/ to tests/ (#49)
This will bring the project closer to the default Symfony project structure.
1 parent 667a630 commit 3363d6d

File tree

7 files changed

+14
-13
lines changed

7 files changed

+14
-13
lines changed

.github/CONTRIBUTING.md

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -92,7 +92,7 @@ When running the integration tests, you will need to specify the database name
9292
and access credentials on the command line (in the same line):
9393

9494
```bash
95-
PHPLIST_DATABASE_NAME=phplist_test PHPLIST_DATABASE_USER=phplist PHPLIST_DATABASE_PASSWORD=batterystaple vendor/bin/phpunit -c Configuration/PHPUnit/phpunit.xml Tests/Integration/
95+
PHPLIST_DATABASE_NAME=phplist_test PHPLIST_DATABASE_USER=phplist PHPLIST_DATABASE_PASSWORD=batterystaple vendor/bin/phpunit -c Configuration/PHPUnit/phpunit.xml tests/Integration/
9696
```
9797

9898

@@ -108,7 +108,7 @@ We will only merge pull requests that follow the project's coding style.
108108
Please check your code with the provided PHP_CodeSniffer standard:
109109

110110
```bash
111-
vendor/bin/phpcs --standard=vendor/phplist/phplist4-core/Configuration/PhpCodeSniffer/ Classes/ Tests/
111+
vendor/bin/phpcs --standard=vendor/phplist/phplist4-core/Configuration/PhpCodeSniffer/ Classes/ tests/
112112
```
113113

114114
Please also check the code structure using PHPMD:
@@ -120,13 +120,13 @@ vendor/bin/phpmd Classes/ text vendor/phplist/phplist4-core/Configuration/PHPMD/
120120
And also please run the static code analysis:
121121

122122
```bash
123-
vendor/bin/phpstan analyse -l 5 Classes/ Tests/
123+
vendor/bin/phpstan analyse -l 5 Classes/ tests/
124124
```
125125

126126
You can also run all code style checks using one long line from a bash shell:
127127

128128
```bash
129-
find Classes/ Tests/ -name '*.php' -print0 | xargs -0 -n 1 -P 4 php -l && vendor/bin/phpstan analyse -l 5 Classes/ Tests/ && vendor/bin/phpmd Classes/ text vendor/phplist/phplist4-core/Configuration/PHPMD/rules.xml && vendor/bin/phpcs --standard=vendor/phplist/phplist4-core/Configuration/PhpCodeSniffer/ Classes/ Tests/
129+
find Classes/ tests/ -name '*.php' -print0 | xargs -0 -n 1 -P 4 php -l && vendor/bin/phpstan analyse -l 5 Classes/ tests/ && vendor/bin/phpmd Classes/ text vendor/phplist/phplist4-core/Configuration/PHPMD/rules.xml && vendor/bin/phpcs --standard=vendor/phplist/phplist4-core/Configuration/PhpCodeSniffer/ Classes/ tests/
130130
```
131131

132132
This will execute all tests except for the unit tests and the integration

.github/PULL_REQUEST_TEMPLATE.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ Are your changes covered with unit tests, and do they not break anything?
1212

1313
You can run the existing unit tests using this command:
1414

15-
vendor/bin/phpunit Tests/
15+
vendor/bin/phpunit tests/
1616

1717

1818
### Code style
@@ -22,7 +22,7 @@ style?
2222

2323
You can check for this using this command:
2424

25-
vendor/bin/phpcs --standard=PSR2 Classes/ Tests/
25+
vendor/bin/phpcs --standard=PSR2 Classes/ tests/
2626

2727

2828
### Other Information

.travis.yml

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -61,22 +61,22 @@ script:
6161
- >
6262
echo;
6363
echo "Linting all PHP files";
64-
find Classes/ Tests/ -name ''*.php'' -print0 | xargs -0 -n 1 -P 4 php -l;
64+
find Classes/ tests/ -name ''*.php'' -print0 | xargs -0 -n 1 -P 4 php -l;
6565
6666
- >
6767
echo;
6868
echo "Running the integration tests";
69-
vendor/bin/phpunit -c Configuration/PHPUnit/phpunit.xml Tests/Integration/;
69+
vendor/bin/phpunit -c Configuration/PHPUnit/phpunit.xml tests/Integration/;
7070
7171
- >
7272
echo;
7373
echo "Running the system tests";
74-
vendor/bin/phpunit -c Configuration/PHPUnit/phpunit.xml Tests/System/;
74+
vendor/bin/phpunit -c Configuration/PHPUnit/phpunit.xml tests/System/;
7575
7676
- >
7777
echo;
7878
echo "Running the static analysis";
79-
vendor/bin/phpstan analyse -l 5 Classes/ Tests/;
79+
vendor/bin/phpstan analyse -l 5 Classes/ tests/;
8080
8181
- >
8282
echo;
@@ -86,4 +86,4 @@ script:
8686
- >
8787
echo;
8888
echo "Running PHP_CodeSniffer";
89-
vendor/bin/phpcs --standard=vendor/phplist/phplist4-core/Configuration/PhpCodeSniffer/ Classes/ Tests/;
89+
vendor/bin/phpcs --standard=vendor/phplist/phplist4-core/Configuration/PhpCodeSniffer/ Classes/ tests/;

CHANGELOG.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,7 @@ This project adheres to [Semantic Versioning](https://semver.org/).
1010
- Integration tests for the dev and test environment (#44)
1111

1212
### Changed
13+
- Adopt more of the default Symfony project structure (#49)
1314

1415
### Deprecated
1516

composer.json

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,7 @@
4646
},
4747
"autoload-dev": {
4848
"psr-4": {
49-
"PhpList\\WebFrontend\\Tests\\": "Tests/"
49+
"PhpList\\WebFrontend\\Tests\\": "tests/"
5050
}
5151
},
5252
"scripts": {
@@ -81,7 +81,7 @@
8181
"symfony-bin-dir": "bin",
8282
"symfony-var-dir": "var",
8383
"symfony-web-dir": "web",
84-
"symfony-tests-dir": "Tests",
84+
"symfony-tests-dir": "tests",
8585
"phplist/phplist4-core": {
8686
"bundles": [],
8787
"routes": {}

0 commit comments

Comments
 (0)