Why? Because we only have web tests for this class, and we have PHPUnit tested most of the other classes now. This is one of the remaining stragglers.
| Comment | File | Size | Author |
|---|---|---|---|
| #5 | interdiff-2191897-3.txt | 1.68 KB | damiankloip |
| #5 | 2191897-3.patch | 6.27 KB | damiankloip |
Comments
Comment #3
damiankloip commentedD8.serialization.EntityNormalizerTest.patch queued for re-testing.
Comment #4
dawehnerIt would be great to use @coversDefaultClass and @covers
I try to figure out why we cannot simply mock the ContentEntityInterface here? Oh right this bullshit transverable thing.
Comment #5
damiankloip commentedYep, it's that crap!
Comment #7
damiankloip commented5: 2191897-3.patch queued for re-testing.
Comment #9
damiankloip commented5: 2191897-3.patch queued for re-testing.
Comment #11
damiankloip commented5: 2191897-3.patch queued for re-testing.
Comment #12
damiankloip commentedComment #13
dawehnerThank you
Comment #14
alexpottCommitted 8f546e5 and pushed to 8.x. Thanks!