forked fromsymfony/symfony-docs
- Notifications
You must be signed in to change notification settings - Fork0
Commit5d4a3a4
committed
bugsymfony#3343 [Testing] Fix phpunit test dir paths (bicpi)
This PR was merged into the 2.3 branch.Discussion----------[Testing] Fix phpunit test dir paths| Q | A| ------------- | ---| Doc fix? | yes| New docs? | no| Applies to | all| Fixed tickets | -Maybe there could be further improvements: The paragraph that follows is a bit confusing. It says *...the following configuration adds the tests from the installed third-party bundles* and then the tests from AcmeDemoBundle are included with `<directory>../src/Acme/Bundle/*Bundle/Tests</directory>`. Mh, the AcmeDemoBundle is a 3rd party bundle that resides in `src`? Shouldn't it include a directory from `vendors` instead to demonstrate this? In addition, the AcmeDemoBundle tests are covered by the standard configuration, so there should be no need for an extra configuration.It continues with *To include other directories in the code coverage..* and then some dirs from the AcmeDemoBundle are excluded which is also covered by the standard configuration.What do you think, should I make some more updates here?Commits-------75bd1e5 Fixes after reviewd17a20f Further tweaks to the PHPUnit configuration description61cdfcc [Testing] Fix phpunit test dir paths1 file changed
+51
-28
lines changedLines changed: 51 additions & 28 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
21 | 21 |
| |
22 | 22 |
| |
23 | 23 |
| |
24 |
| - | |
| 24 | + | |
25 | 25 |
| |
26 | 26 |
| |
27 | 27 |
| |
| |||
771 | 771 |
| |
772 | 772 |
| |
773 | 773 |
| |
774 |
| - | |
775 |
| - | |
| 774 | + | |
| 775 | + | |
| 776 | + | |
776 | 777 |
| |
777 | 778 |
| |
778 | 779 |
| |
779 |
| - | |
| 780 | + | |
780 | 781 |
| |
781 | 782 |
| |
782 |
| - | |
783 |
| - | |
784 |
| - | |
785 |
| - | |
786 |
| - | |
| 783 | + | |
| 784 | + | |
| 785 | + | |
787 | 786 |
| |
788 | 787 |
| |
789 | 788 |
| |
790 |
| - | |
791 |
| - | |
792 |
| - | |
793 |
| - | |
794 |
| - | |
795 |
| - | |
796 |
| - | |
| 789 | + | |
| 790 | + | |
| 791 | + | |
| 792 | + | |
| 793 | + | |
| 794 | + | |
| 795 | + | |
| 796 | + | |
| 797 | + | |
| 798 | + | |
| 799 | + | |
| 800 | + | |
| 801 | + | |
| 802 | + | |
| 803 | + | |
| 804 | + | |
| 805 | + | |
| 806 | + | |
| 807 | + | |
| 808 | + | |
| 809 | + | |
| 810 | + | |
| 811 | + | |
| 812 | + | |
| 813 | + | |
| 814 | + | |
| 815 | + | |
| 816 | + | |
797 | 817 |
| |
798 | 818 |
| |
799 | 819 |
| |
800 | 820 |
| |
801 | 821 |
| |
802 | 822 |
| |
803 |
| - | |
804 |
| - | |
805 |
| - | |
806 |
| - | |
807 |
| - | |
808 |
| - | |
809 |
| - | |
810 |
| - | |
811 |
| - | |
812 |
| - | |
813 |
| - | |
814 |
| - | |
| 823 | + | |
| 824 | + | |
| 825 | + | |
| 826 | + | |
| 827 | + | |
| 828 | + | |
| 829 | + | |
| 830 | + | |
| 831 | + | |
| 832 | + | |
| 833 | + | |
| 834 | + | |
| 835 | + | |
| 836 | + | |
| 837 | + | |
815 | 838 |
| |
816 | 839 |
| |
817 | 840 |
| |
|
0 commit comments
Comments
(0)