search

aapatre / Automatic-Udemy-Course-Enroller-GET-PAID-UDEMY-COURSES-for-FREE

Do you want to LEARN NEW STUFF for FREE? Don't worry, with the power of web-scraping and automation, this script will find the necessary Udemy coupons & enroll you for PAID UDEMY COURSES, ABSOLUTELY FREE!
GNU General Public License v3.0
3.14k stars 540 forks source link

Introduce unit test and/or CI/CD #14

Closed latelylk closed 3 years ago

latelylk commented 4 years ago

Creating an issue based on the comment posted by @fakeid30 in #11.

This looks like good change. @aapatre can we introduce unit test and / or CI/CD so we know changes work before merging? https://www.lambdatest.com/blog/automated-testing-pipeline-with-gitlab-ci-cd-and-selenium/

aapatre commented 4 years ago

Lemme check

fakeid30 commented 4 years ago

https://dev.to/seleniumbase/running-browser-tests-from-github-workflows-with-seleniumbase-1oic

This is an easier way to do it using github action.

cullzie commented 4 years ago

Hey, I've created a PR which partially addresses this. Its more of a smoke test. Currently working with chrome

fakeid30 commented 3 years ago

@cullzie shall we close this ?

cullzie commented 3 years ago

@fakeid30 Ye I suppose we can if it covers what was originally wanted. We can just expand on the base that is there as the project moves forward