Move VideoConfigService tests near the service#38008
Conversation
5ff3b1f to
2472bbe
Compare
|
@farhan Just a question for more context, as per my understanding these tests are for testing videos with multiple or no transcripts translation retrieval. Why we need to move them into new class |
Actually, these are very old tests written before VideoConfigService existed. |
|
@farhan Thanks for the elaborating, why we are making a separate class to move these tests? As tests are already using service to reach these methods. |
We are moving |
2472bbe to
257a387
Compare
salman2013
left a comment
There was a problem hiding this comment.
LGTM please merge once tests are completed.
Move
VideoConfigServicetests near the appIn the test cases we are migrating we are actually testing the functionality of VideoConfigService.
It felt required while shifting
test_video.pytest case fromedx-platformtoxblocks-contribopenedx/xblocks-contrib#144