[dev] components app converted to PSR-4 (mostly) - want feedback

Ralf Lang lang at b1-systems.de
Sun Oct 18 07:28:23 UTC 2020


Am 17.10.2020 um 17:54 schrieb Ralf Lang:
> Update,
>
>
>> I'd like to merge this once I have fixed Horde_Test and components'
> test suite.
>
>>     Unit tests have not yet been touched and most likely are broken.
>>     This needs a PSR-4 aware Horde_Test to make work.
>> I think this should be improved until the qc command can run
> component's own unit tests before merging.
>
> I have fixed most unit tests [when run from the qc unit command]. One or
> two cases I have disabled as an intended failure won't happen due to new
> functionality. Components can now identify a component from almost any
> of its internal paths, not just from the base dir.
>
> Three tests remain failing. Two are version tests where the fixture
> expects a certain version and the code returns one bugfix (3rd digit)
> version later. I have not fully understood the intention of the test and
> if it expresses a real broken behaviour or just a mismatch with the fixture.
>
> One test fails because the composer.json file returned is exactly as
> expected from the fixture, but with today's date rather than some 2010
> date string. Either there is some broken "pretend" behaviour or the test
> is just wrong.
>
> Tests have been moved to the root of the test/ dir but have not yet been
> converted to namespaces. Also, AllTests.php & friends are not yet fixed.

Tests are now all namespaced. I also changed the Dependencies\Injector
and Dependencies Interface to be compatible both with the current
Horde_Injector and the upcoming Horde\Injector.

I changed the PR from WIP to ready for review/merge.

> I also did not yet unbundle the bundled dependencies. I think this must
> be done one by one, each time deciding if it's needed to be conservative
> or upgrade to recent versions.

-- 
Ralf Lang
Linux Consultant / Developer
Tel.: +49-170-6381563
Mail: lang at b1-systems.de
B1 Systems GmbH
Osterfeldstraße 7 / 85088 Vohburg / http://www.b1-systems.de
GF: Ralph Dehner / Unternehmenssitz: Vohburg / AG: Ingolstadt,HRB 3537



More information about the dev mailing list