Page MenuHomePhabricator

Inject creation of identity on CompositePartMcrCRUDs for tests of CLUD and TransactionExecutor
Open, LowPublic12 Estimated Story Points

Description

Not prioritized. Since CLUD and TransactionExecutor is a generic facade on top of CRUD, it's enough that the CRUD test passes. Knowing that CLUD and TransactionExecutor pass on all other classes is enough to know that it indeed will work for everything.

In order to share code with all other CRUD tests we need to create identity here. Could probably be done using an abstract method or something.

See https://gerrit.wikimedia.org/r/c/mediawiki/extensions/WikispeechSpeechDataCollector/+/663864/5/tests/phpunit/integration/CRUD/Mcr/RecordingAnnotationsMcrCRUDTest.php#20