CCExtractor / ccextractor

CCExtractor - Official version maintained by the core team
https://www.ccextractor.org
GNU General Public License v2.0
722 stars 427 forks source link

[FEATURE] Create unit test for rust code #1615

Closed IshanGrover2004 closed 3 months ago

IshanGrover2004 commented 5 months ago

In raising this pull request, I confirm the following (please check boxes):

My familiarity with the project is as follows (check one):


Creating Unit test cases for rust modules(decoder + hardsubx). Closes #911

IshanGrover2004 commented 5 months ago

The decoder moulde's unit test done @PunitLodha

PunitLodha commented 4 months ago

@IshanGrover2004 Can you also add a github CI check for running the unit tests

IshanGrover2004 commented 4 months ago

Sure @PunitLodha

ccextractor-bot commented 4 months ago

CCExtractor CI platform finished running the test files on linux. Below is a summary of the test results, when compared to test for commit f12f12b...:

Report Name Tests Passed
Broken 12/13
CEA-708 13/14
DVB 4/7
DVD 3/3
DVR-MS 2/2
General 23/27
Hauppage 3/3
MP4 3/3
NoCC 10/10
Options 84/86
Teletext 21/21
WTV 9/13
XDS 34/34

All tests passing on the master branch were passed completely.

NOTE: The following tests have been failing on the master branch as well as the PR:

Congratulations: Merging this PR would fix the following tests:


Check the result page for more info.

IshanGrover2004 commented 4 months ago

@PunitLodha @cfsmp3 You can proceed for merging. I have added CI + all modules unit test cases

PunitLodha commented 3 months ago

If you find time you can write the extra comments, otherwise it's fine

IshanGrover2004 commented 3 months ago

@PunitLodha Added the CI & fixed the warnings as well