Closed ndm2 closed 11 months ago
All modified and coverable lines are covered by tests :white_check_mark:
Comparison is base (
44cdcc1
) 95.86% compared to head (b61880a
) 95.84%. Report is 2 commits behind head on master.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Reverts some of #593. The entity read from the DB will not hold uploaded file instances.
ps. I don't have time for it right now, but seeing how this slipped in, the tests are IMHO in dire need of de-mocking. There's so much mocking going on that could possibly hide all sorts of problems, it's wild.
Previously there was for example
$this->entity->field
being set and passed to expectations, but it was alwaysnull
, because it's all mocked, still the tests passed because there were no type hints on the arguments where the value would flow through.