search

livepeer / verification-classifier

Metrics-based Verification Classifier
MIT License
8 stars 7 forks source link

Run tests in CI #117

Closed yondonfu closed 4 years ago

yondonfu commented 4 years ago

Tests for this repo can currently be run via:

python3 -m pytest testing/tests

We should run these tests in CI and passing tests should be a prerequisite for merging a PR. One thing that will need to be considered is that some of the tests require a downloadable dataset. Perhaps the tests that require a downloadable dataset can be excluded from CI.

yondonfu commented 4 years ago

We should be able to add a test runner step here.

The tests also only need a small # of videos and don't need the full dataset mentioned in the OP. We should be able to upload a smaller dataset that only contains videos required for tests and that dataset can be downloaded in CI.

yondonfu commented 4 years ago

Closed by #124