We are missing tests for some classes.

We need to add test coverage for the methods in the EntityResource, the Routes and the ResourceManager classes.

I suggest adding Kernel testing to make sure that the individual and the collection methods are returning as expected under different scenarios.
I suggest adding Unit testing to make sure that the routes are created for the different bundles and entity types.
I suggest adding Unit testing to make sure that the resource config for the different bundles are available and discovered.

Comments

e0ipso created an issue. See original summary.

e0ipso’s picture

Title: [BUGFIX] Add test coverage for EntityResource » [BUGFIX] Add test coverage for EntityResource, Routes and ResourceManager
Issue summary: View changes
e0ipso’s picture

e0ipso’s picture

Assigned: Unassigned » e0ipso
e0ipso’s picture

Status: Active » Needs review
StatusFileSize
new19.81 KB

Work in progress. Testing is hard.

e0ipso’s picture

Status: Needs review » Needs work
e0ipso’s picture

Status: Needs work » Needs review
StatusFileSize
new17.51 KB

Ready for review. I removed the cruft from the other patch.

  • e0ipso committed b9a95d4 on 8.x-1.x
    Issue #2726265 by e0ipso: [BUGFIX] Add test coverage for EntityResource...
e0ipso’s picture

Status: Needs review » Fixed

Tests merged.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.