PHPCSStandards / PHP_CodeSniffer / 14007039587
78%
master: 79%

LAST BUILD BRANCH: fix/interface-indentation
DEFAULT BRANCH: master
Ran
Jobs 8
Files 263
Run time 1min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
coverage: 77.547%. Remained the same
14007039587

push

github

jrfnl 
PropertyTypeHandlingTest: split data provider

The `PropertyTypeHandlingTest` class tests the setting of property values from the ruleset as well as via inline `// phpcs:set ...`  annotations.

As both ways of setting the property value should be supported in the same manner, the tests used one data provider for both tests and the fixtures (`PropertyTypeHandlingTest.xml` ruleset and the `Fixtures/PropertyTypeHandlingInline.inc` file containing the inline annotations) mirrored the exact same test cases.

There is one feature, however, which is only supported when setting properties via the ruleset, not when setting them via inline annotations: array property extending.

Until now, the tests for that were also mirrored in the inline annotation tests, even though the feature isn't supported for inline annotations.

This commit now splits the data provider in two:
* One data provider for the tests which apply for both (= most tests).
* One specifically for extending array properties.

That second data provider will now only be used by the `testTypeHandlingWhenSetViaRuleset()` test method.

This should make it more straight-forward to add additional tests for property extending in the future.

---

In case anyone is wondering: As `phpcs:set` is only supposed to be used in tests, I don't think adding support for the `extend` option to inline properties is necessary.

19303 of 24892 relevant lines covered (77.55%)

76.75 hits per line

Jobs
ID Job ID Ran Files Coverage
1 cbf-os-ubuntu-latest-ubuntu-latest-php-8.1-custom-ini-true - 14007039587.1 263
1.89
 GitHub Action Run
2 os-ubuntu-latest-php-8.1-custom-ini-true - 14007039587.2 263
76.09
 GitHub Action Run
3 cbf-os-ubuntu-latest-ubuntu-latest-php-7.2-custom-ini-false - 14007039587.3 263
0.81
 GitHub Action Run
4 os-ubuntu-latest-php-7.2-custom-ini-false - 14007039587.4 263
73.88
 GitHub Action Run
5 os-windows-latest-php-7.2-custom-ini-false - 14007039587.5 263
2.08
 GitHub Action Run
6 os-windows-latest-php-8.4-custom-ini-false - 14007039587.6 263
2.62
 GitHub Action Run
7 os-ubuntu-latest-php-8.4-custom-ini-false - 14007039587.7 263
76.06
 GitHub Action Run
8 cbf-os-ubuntu-latest-ubuntu-latest-php-8.4-custom-ini-false - 14007039587.8 263
1.89
 GitHub Action Run
Source Files on build 14007039587