- First commit to issue fork.
- 🇮🇳India mohit_aghera Rajkot
I believe this issue might not be required now.
Since this issue was added to increase the test coverage on file_field_presave function.
Since we have entire EntityReference mechanism happening insideFileFieldItemList
, I believe this might not be required.There is another class
EntityReferenceFieldItemList
which is responsible for loading the referenced entities has enough checks.Considering that we might not need to add additional test case coverage.
For testing purposes, I’ve added an additional test to ensure that things goes well.
This test is passing on local. - 🇺🇸United States smustgrave
PMNMI may be the right status but can the issue summary be updated with why this isn't needed?
- 🇮🇳India mohit_aghera Rajkot
Ahh, apologies for the confusion.
I mean there won't be any errors now since it is handled in D8 core properly.We can merge the test as it is passing on the local.
I've triggered the job again and will update the issue once it is green.