convert ProviderRepositoryTest to a kernel test

ayush.pandey → made their first commit to this issue’s fork.

Tried converting the basics to kernetTEST cases into the MR . Please review and mention the further changes to convert this fluidly.

It's core/modules/media/tests/src/Functional/ProviderRepositoryTest.php that should be converted, not the unit test. Unit tests are even quicker than kernel tests to run, so we should not be converting in that direction.

pooja_sharma → closed merge request !8494

Implemented ProviderRepositoryTest to a kernel test

tried to work with existing MR , however facing issue so created new MR

Please review MR 8495 , moved NR

You need to remove the old test too.

It looks promising, but at the moment the MR doesn't show me the old test so I can't tell for sure.

Removed Functional test case ProviderRepositoryTest.php as kernel test added.

Apart from it, nothing seems to be left.

Please review, moving to NR

Seems like a good conversion

@pooja_sharma this issue was tagged novice which is usually for new users. Based on your post history believe you can work on non novice issues going forward. Thanks!

Yeah sure @smustgrave, even I also keep it in mind, however later on realised it actually requires few minor changes

Rebased the MR with latest code, seems fine

Backported down to 10.3.x as a tests-only change.

Committed and pushed 3e3b49539d to 11.x and 429b5dc198 to 11.0.x and b55abf3d6e to 10.4.x and 05b9d0d02b to 10.3.x. Thanks!

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